RFR (7xS): 8178148: Log more detailed information about scan rs phase

Thomas Schatzl thomas.schatzl at oracle.com
Mon Jun 12 11:34:38 UTC 2017


Hi all,

  sorry for another round of reviews: Erik asked me to add a gtest test
for the linked subitems, both for the (existing) set() and (new) add()
methods.

Webrev: http://cr.openjdk.java.net/~tschatzl/8178148/webrev.2_to_3/ (di
ff): http://cr.openjdk.java.net/~tschatzl/8178148/webrev.3/ (full)

Testing:
local testing, jprt

Thanks,
  Thomas

On Tue, 2017-05-23 at 12:15 -0700, sangheon wrote:
> Hi Thomas,
> 
> On 05/23/2017 05:50 AM, Thomas Schatzl wrote:
> > 
> > Hi all,
> > 
> >    unfortunately, for support of some code there is need for one
> > more
> > public method in the g1gcphasetimes class.
> > 
> > Webrev:
> > http://cr.openjdk.java.net/~tschatzl/8178148/webrev.1_to_2/ (diff)
> > http://cr.openjdk.java.net/~tschatzl/8178148/webrev.2/ (full)
> Webrev.2 still looks good to me.
> 
> Thanks,
> Sangheon
> 
> 
> > 
> > 
> > Sorry for the issue.
> > 
> > Thanks,
> >    Thomas
> > 



More information about the hotspot-gc-dev mailing list