RFR (S): 8150630: Add logging for ParScanThreadState merge phase
Thomas Schatzl
thomas.schatzl at oracle.com
Thu Feb 25 16:23:28 UTC 2016
Hi Jon,
On Thu, 2016-02-25 at 07:59 -0800, Jon Masamitsu wrote:
>
> 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.
Done. I updated the webrev in-place (and reran the test).
Thanks for the review,
Thomas
More information about the hotspot-gc-dev
mailing list