RFR: 8033764: Remove the usage of StarTask from BufferingOopClosure
thomas.schatzl at oracle.com
Mon Feb 10 12:15:41 UTC 2014
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:
More information about the hotspot-gc-dev