RFR: 8323795: jcmd Compiler.codecache counts total sizes of used/free [v2]

[Tobias Hartmann]

On Wed, 17 Jan 2024 03:02:27 GMT, Yi Yang <yyang at openjdk.org> wrote:

>> CodeHeap 'non-profiled nmethods': size=118592Kb used=29Kb max_used=29Kb free=118562Kb
>>  bounds [0x00007fbe84622000, 0x00007fbe84892000, 0x00007fbe8b9f2000]
>> CodeHeap 'profiled nmethods': size=118588Kb used=80Kb max_used=80Kb free=118507Kb
>>  bounds [0x00007fbe7c9f2000, 0x00007fbe7cc62000, 0x00007fbe83dc1000]
>> CodeHeap 'non-nmethods': size=8580Kb used=1258Kb max_used=1834Kb free=7321Kb
>>  bounds [0x00007fbe83dc1000, 0x00007fbe84031000, 0x00007fbe84622000]
>>  total_blobs=474 nmethods=87 adapters=293
>>  compilation: enabled
>>               stopped_count=0, restarted_count=0
>>  full_count=0
>> 
>> 
>> It's better to accumulates total size of used/free/size, for example
>> 
>> -SegmentedCodeCache
>> CodeCache: size=245760Kb used=1366Kb max_used=1943Kb free=244393Kb
>>  bounds [0x00007fdcc89f2000, 0x00007fdcc8c62000, 0x00007fdcd79f2000]
>>  total_blobs=474, nmethods=87, adapters=293
>>  stopped_count=0, restarted_count=0, full_count=0 
>>  compilation=enabled
>> 
>> 
>> 
>> +SegmentedCodeCache
>> CodeHeap 'non-profiled nmethods': size=118592Kb used=29Kb max_used=29Kb free=118562Kb
>>  bounds [0x00007f89c8622000, 0x00007f89c8892000, 0x00007f89cf9f2000]
>> CodeHeap 'profiled nmethods': size=118588Kb used=80Kb max_used=80Kb free=118507Kb
>>  bounds [0x00007f89c09f2000, 0x00007f89c0c62000, 0x00007f89c7dc1000]
>> CodeHeap 'non-nmethods': size=8580Kb used=1258Kb max_used=1834Kb free=7321Kb
>>  bounds [0x00007f89c7dc1000, 0x00007f89c8031000, 0x00007f89c8622000]
>> CodeCache: size=245760Kb, used=1367Kb, max_used=1943Kb, free=244390Kb
>>  total_blobs=474, nmethods=87, adapters=293
>>  stopped_count=0, restarted_count=0, full_count=0 
>>  compilation=enabled
>
> Yi Yang has updated the pull request incrementally with one additional commit since the last revision:
> 
>   new output && fix test

The title is confusing. Should it be something like "jcmd Compiler.codecache should print total size of code cache"?

src/hotspot/share/code/codeCache.cpp line 1802:

> 1800:               "enabled" : Arguments::mode() == Arguments::_int ?
> 1801:               "disabled (interpreter mode)" :
> 1802:               "disabled (not enough contiguous free space left)");

Why did you change the order of the `compilation=` and the `stopped_count=` output?

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

PR Comment: https://git.openjdk.org/jdk/pull/17445#issuecomment-1895153379
PR Review Comment: https://git.openjdk.org/jdk/pull/17445#discussion_r1454744516