RFR (S): 8150630: Add logging for ParScanThreadState merge phase
Jon Masamitsu
jon.masamitsu at oracle.com
Thu Feb 25 15:59:49 UTC 2016
On 2/25/2016 7:21 AM, Thomas Schatzl wrote:
> Hi Jon,
>
> On Thu, 2016-02-25 at 07:09 -0800, Jon Masamitsu wrote:
>> http://cr.openjdk.java.net/~tschatzl/8150630/webrev/src/share/vm/gc/g
>> 1/g1GCPhaseTimes.cpp.frames.html
>>
>> 416 print_stats(Indents[2], "Merge PSS",
>> _recorded_merge_pss_time_ms);
>>
>> I would appreciate a label easier for me to understand such as
>>
>> "Merge parallel threads data"
>>
>> or
>>
>> "Merge parallel threads data (PSS)"
>>
>> Otherwise, looks good.
>>
> thanks for the review. Would "Merge per-thread state" be okay with
> you?
That would be excellent. I don't need a new webrev.
Reviewed.
Jon
>
> Thanks,
> Thomas
More information about the hotspot-gc-dev
mailing list