Request for Review (s) - 8149343: assert(rp->num_q() == no_of_gc_workers) failed: sanity

Thomas Schatzl thomas.schatzl at oracle.com
Thu Mar 17 08:59:30 UTC 2016


Hi Jon,

On Wed, 2016-03-16 at 15:42 -0700, Jon Masamitsu wrote:
> The change to re-initialize _next_id passed testing so is now
> the proposed fix.
> 
> Delta from original fix:
> http://cr.openjdk.java.net/~jmasa/8149343/webrev_delta.00_01/
> 
> Full fix version 01
> http://cr.openjdk.java.net/~jmasa/8149343/webrev.01/
> 
> Thanks.

  looks good.

Please maybe fix the indentation of the closing braces in line 716 and
781 before pushing while you are working in this area. These are pre
-existing bugs, so it's up to you.

Thanks,
  Thomas



More information about the hotspot-gc-dev mailing list