RFR (M) 8075210: Refactor strong root processing in order to allow G1 to evolve separately from GenCollectedHeap
Thomas Schatzl
thomas.schatzl at oracle.com
Wed Mar 18 12:32:56 UTC 2015
Hi again,
On Wed, 2015-03-18 at 13:28 +0100, Thomas Schatzl wrote:
> Hi Mikael,
>
> On Tue, 2015-03-17 at 17:00 +0100, Mikael Gerdin wrote:
> > On 2015-03-17 15:26, Thomas Schatzl wrote:
> > > Hi Mikael,
> [...]
> > >
> > > - I think the instantiation of G1RootsProcessor in
> > > G1CollectedHeap::verify() should be scoped, i.e. it and the call to
> > > process_all_roots() enclosed with braces.
> >
> > Fixed.
> >
>
> Wrong indentation of the &blobsl parameter. I do not need to re-review
> that.
>
> [...]
> >
> > Here's an incremental webrev incorporating your suggestions:
> > http://cr.openjdk.java.net/~mgerdin/8075210/thomas-comments/webrev/
> > It's based on
> > http://cr.openjdk.java.net/~mgerdin/8075210/done_with_threads/webrev/
> > which incorporates Eric's renaming request.
> >
> > New full webrev at:
> > http://cr.openjdk.java.net/~mgerdin/8075210/full2/webrev/
>
> Looks good.
Some last-minute comment if it is not still too late:
- G1CollectedHeap::set_par_threads(uint t) can be removed. It only calls
the inherited method anyway.
Thanks,
Thomas
More information about the hotspot-gc-dev
mailing list