RFR: JDK-8150367: Add back information about the number of GC workers

Bengt Rutisson bengt.rutisson at oracle.com
Mon Feb 22 13:29:25 UTC 2016



On 2016-02-22 14:25, Stefan Johansson wrote:
>
> On 2016-02-22 14:18, Bengt Rutisson wrote:
>>
>> Forgot to add the webrev link:
>>
>> http://cr.openjdk.java.net/~brutisso/8150367/webrev.00/
>>
> Looks good,

Thanks, Stefan!

Bengt

>
> StefanJ
>
>> Thanks Thomas for notifying me about it :)
>>
>> Bengt
>>
>> On 2016-02-22 13:46, Bengt Rutisson wrote:
>>>
>>> Hi everyone,
>>>
>>> Could I have a couple of reviews for this change?
>>>
>>> https://bugs.openjdk.java.net/browse/JDK-8150367
>>>
>>> Before the changes to use the unified logging framework for the GC 
>>> logging the number of GC worker threads was logged for each G1 
>>> evacuation pause. It was printed as part of the "Parallel Time" line:
>>>
>>> [Parallel Time: 5,5 ms, GC Workers: 23]
>>>
>>> This information was lost in the conversion to the unified logging 
>>> framework. This is relevant information and should be added back.
>>>
>>> The proposed patch adds this back as a separate log line. That looks 
>>> like this:
>>>
>>> [1,647s][info][gc,task     ] GC(36) GC Workers: 23
>>>
>>> The logging is done in AbstractWorkGang which is shared between G1 
>>> and CMS (ParNew). So, this information is now also logged for CMS.
>>>
>>> Thanks,
>>> Bengt
>>>
>>>
>>
>



More information about the hotspot-gc-dev mailing list