RFR (M): 8153503: Move remset scan iteration claim to remset local data structure

Thomas Schatzl thomas.schatzl at oracle.com
Thu Apr 14 13:55:08 UTC 2016


Hi Mikael,

On Thu, 2016-04-14 at 15:11 +0200, Mikael Gerdin wrote:
> Hi Thomas,
> 
> On 2016-04-14 11:56, Thomas Schatzl wrote:
> > Hi,
[...]
> >    new webrevs at
> > http://cr.openjdk.java.net/~tschatzl/8153503/webrev.0_to_1/ (diff)
> > http://cr.openjdk.java.net/~tschatzl/8153503/webrev.1/ (full)
> 
> This looks fine to me.
> 
> The indentation is a bit fishy in g1RemSet.cpp
>   272   if (_scan_state->set_iter_complete(region_idx)) {
>   273      // Scan the strong code root list attached to the current
> region
>   274      scan_strong_code_roots(r);
> 
> I don't need to see a new webrev for that, though :)
> 

  fixed :) Thanks for your review.

Thomas



More information about the hotspot-gc-dev mailing list