RFR: JDK-8153186: Convert TraceGCTaskThread to use unified logging

Bengt Rutisson bengt.rutisson at oracle.com
Thu Mar 31 12:59:45 UTC 2016


Hi Thomas,

Thanks for looking at this!

On 2016-03-31 14:50, Thomas Schatzl wrote:
> Hi,
>
> On Thu, 2016-03-31 at 14:23 +0200, Bengt Rutisson wrote:
>> Hi everyone,
>>
>> Could I have a couple of reviews for this change?
>>
>> http://cr.openjdk.java.net/~brutisso/8153186/webrev.00/
>> https://bugs.openjdk.java.net/browse/JDK-8153186
>>
>> The logging guarded by TraceGCTaskThread should be converted to use
>> the
>> unified logging framework.
>    - these log messages seem to be gc specific, so maybe add the gc-id
> decoration.

This logging is all done at initilization, before we start a GC so I 
don't think there is a point in adding the GC id decoration.

>
>    - this change simply removes the flag from the available options.
> Shouldn't it be added to the table we have that translates old flags to
> new logging somewhere? (I do not have an opinion about this, just
> asking).

There are some internal table for such conversion. I can look in to 
adding it there. For the public documentation we have selected a subset 
of all flags to document. I don't think this flag is used widely enough 
to qualify for that list.

Thanks,
Bengt

>
> Thanks,
>    Thomas
>




More information about the hotspot-gc-dev mailing list