RFR: 8327054: DiagnosticCommand Compiler.perfmap does not log on output() [v2]

Thomas Stuefe stuefe at openjdk.org
Sat Jul 20 10:52:31 UTC 2024


On Fri, 19 Jul 2024 20:17:46 GMT, Sonia Zaldana Calles <szaldana at openjdk.org> wrote:

>> Hi all, 
>> 
>> This is a small patch to address [8327054](https://bugs.openjdk.org/browse/JDK-8327054) making `CodeCache::write_perf_map` aware of which output stream errors and warning message should be going to.
>> 
>> Testing: 
>> - [x] Added test case passes. 
>> 
>> Thanks, 
>> Sonia
>
> Sonia Zaldana Calles has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Ensuring test case deletes file in case of exception

src/hotspot/share/code/codeCache.hpp line 226:

> 224:   static void print_summary(outputStream* st, bool detailed = true); // Prints a summary of the code cache usage
> 225:   static void log_state(outputStream* st);
> 226:   LINUX_ONLY(static void write_perf_map(const char* filename, outputStream* st);)

Please add a comment about what the stream `st` is supposed to be.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/20257#discussion_r1685374253


More information about the serviceability-dev mailing list