RFR: 8254723: add diagnostic command to write Linux perf map file [v2]

Nick Gasson ngasson at openjdk.java.net
Wed Oct 21 09:13:10 UTC 2020


On Wed, 21 Oct 2020 04:34:44 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:

>> Nick Gasson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Update for review comments
>
> src/hotspot/share/code/codeCache.cpp line 1562:
> 
>> 1560: }
>> 1561: 
>> 1562: void CodeCache::write_perf_map(outputStream* st) {
> 
> Should this code (and changes in header files) be included for Linux only? (`#ifdef LINUX` like the change in diagnosticCommand.cpp)

I'm not sure, I didn't want to add too much `#ifdef` mess. The code will compile on other platforms, it just won't be called. Better to add `#ifdef`s around all of it?

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

PR: https://git.openjdk.java.net/jdk/pull/760


More information about the serviceability-dev mailing list