RFR (S/M): 8159440: Move marking of promoted objects during initial mark into the concurrent phas

Thomas Schatzl thomas.schatzl at oracle.com
Tue Dec 4 12:27:27 UTC 2018


Hi all,

On Tue, 2018-12-04 at 11:08 +0100, Thomas Schatzl wrote:
> Hi,
> 
> On Mon, 2018-12-03 at 23:38 -0500, Kim Barrett wrote:
> > > On Dec 3, 2018, at 9:44 AM, Thomas Schatzl <
> > > thomas.schatzl at oracle.com> wrote:
> > > http://cr.openjdk.java.net/~tschatzl/8159440/webrev.0_to_1/
> > > (diff)
> > > http://cr.openjdk.java.net/~tschatzl/8159440/webrev.1/ (full)
> > 
> > 
[...]
> > -------------------------------------------------------------------
> > -----------
> > src/hotspot/share/gc/g1/g1ConcurrentMark.cpp
> >  267   _root_regions = NEW_C_HEAP_ARRAY(HeapRegion*, max_regions,
> > mtGC);
> > 
> > Consider asserting _root_regions == NULL first.
> > 
> > -------------------------------------------------------------------
> > -----------
> 
> Added.
> 
> http://cr.openjdk.java.net/~tschatzl/8159440/webrev.1_to_2/ (diff)
> http://cr.openjdk.java.net/~tschatzl/8159440/webrev.2/ (full)

I updated the webrevs because the above assert showed that there had
been a memory leak (oops!).

Sorry,
  Thomas





More information about the hotspot-gc-dev mailing list