RFR (S): 8150630: Add logging for ParScanThreadState merge phase
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Feb 25 15:21:31 UTC 2016
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?
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list