RFR (S): 8131319: Move G1Allocator::_summary_bytes_used back to G1CollectedHeap

Thomas Schatzl thomas.schatzl at oracle.com
Thu Jul 23 09:18:25 UTC 2015


Hi Tom,

On Fri, 2015-07-17 at 13:20 -0400, Tom Benson wrote:
> Hi Thomas,
> 
> On 7/17/2015 11:55 AM, Thomas Schatzl wrote:
> > Hi,
> [..]
> > The problem with immediate updates would obviously be concurrency, e.g.
> > if multiple threads were allocating into an archive region(s). Or
> > another allocator doing its updates directly like for pinned regions.
> >
> > Not sure about how MT safe the current archive allocator is actually.
> It is designed for single threaded use from the VM thread, not general 
> allocation.
> 
> Anyway,  your change for 8131319 still looks good to me.  Tnx,
> Tom

  thanks for your review :)

Thomas



More information about the hotspot-gc-dev mailing list