RFR: 8234440: ZGC: Print relocation information on info level

erik.osterlund at oracle.com erik.osterlund at oracle.com
Wed Jan 22 09:04:33 UTC 2020


Hi Per,

Looks good.

/Erik

On 1/21/20 1:56 PM, Per Liden wrote:
> I got some off-line comments from Stefan, updated webrev:
>
> Diff: http://cr.openjdk.java.net/~pliden/8234440/webrev.1-diff
> Full: http://cr.openjdk.java.net/~pliden/8234440/webrev.1
>
> /Per
>
> On 1/21/20 11:18 AM, Per Liden wrote:
>> When using -Xlog:gc*, I now and then find that I miss basic 
>> relocation information, since it's currently printed at the debug 
>> level on the relocation set selector. I think we should leave the 
>> current logging as is, since that's still useful when debugging the 
>> relocation set selector itself. However, I think we should propagate 
>> some of the high level information and print it on the info level.
>>
>> Here's an example of what the output looks like with this patch:
>>
>> [...]
>> [68.926s][info][gc,reloc    ] GC(6) Small Pages: 529 / 1058M(93%), 
>> Empty: 350M(31%), Compacting: 450M(40%)->20M(2%)
>> [68.926s][info][gc,reloc    ] GC(6) Medium Pages: 2 / 64M(6%), Empty: 
>> 0M(0%), Compacting: 64M(6%)->32M(3%)
>> [68.926s][info][gc,reloc    ] GC(6) Large Pages: 2 / 12M(1%), Empty: 
>> 6M(1%), Compacting: 0M(0%)->0M(0%)
>> [68.926s][info][gc,reloc    ] GC(6) Relocation: Successful
>> [...]
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8234440
>> Webrev: http://cr.openjdk.java.net/~pliden/8234440/webrev.0
>>
>> /Per



More information about the hotspot-gc-dev mailing list