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