RFR: 8033764: Remove the usage of StarTask from BufferingOopClosure
Thomas Schatzl
thomas.schatzl at oracle.com
Mon Feb 10 12:15:41 UTC 2014
Hi Stefan,
On Mon, 2014-02-10 at 11:53 +0100, Stefan Karlsson wrote:
> Hi Thomas,
>
> On 2014-02-10 11:23, Thomas Schatzl wrote:
> > Hi Stefan,
> >
> > looks okay.
> >
> > Some comments:
> > - copyright dates for jni.cpp and g1RemSet.cpp still read 2013
> > - I would prefer the use of OopOrNarrowOopStar although it is just a
> > void*
> > - could you add the comment about why StarTask is not used here, i.e.
> > that oops embedded in code are not aligned, preventing us to use
> > StarTask here? I.e. the comment from
> > G1CollectedHeap::g1_process_strong_roots().
>
> Thanks for reviewing.
>
> I've addressed all your comments:
> http://cr.openjdk.java.net/~stefank/8033764/webrev.02.delta/
> http://cr.openjdk.java.net/~stefank/8033764/webrev.02/
>
Looks good,
Thomas
More information about the hotspot-gc-dev
mailing list