RFR (S): 8155810: Bound the number of root region scan threads to the number of survivor regions

Thomas Schatzl thomas.schatzl at oracle.com
Tue May 10 14:49:36 UTC 2016


Hi,

On Tue, 2016-05-10 at 16:37 +0200, Stefan Johansson wrote:
> 
> On 2016-05-10 13:30, Thomas Schatzl wrote:
> > 
> > Hi all,
> > 
> >    due to recent changes I had to do a minor one-line change:
> > 
> > http://cr.openjdk.java.net/~tschatzl/8155810/webrev.1/
> > 
> > in g1ConcurrentMark.cpp,
> > 
> >   312 uint G1CMRootRegions::num_root_regions() const {
> >   313   return _young_list->survivor_length();
> >   314 }
> > 
> > had to change to:
> > 
> >   297 uint G1CMRootRegions::num_root_regions() const {
> >   298   return (uint)_survivors->regions()->length();
> >   299 }
> > 
> > Please have a look asap.
> Looks good,

  thanks for your review.

Thomas



More information about the hotspot-gc-dev mailing list