[8u] Request for Approval for backport of 8180048: Interned string and symbol table leak memory during parallel unlinking

[Rob McKenna]

Apologies, I had overlooked that.

Approved

    -Rob

On 18/05/17 09:16, Thomas Schatzl wrote:
> Hi,
> 
> On Thu, 2017-05-18 at 19:39 +0100, Rob McKenna wrote:
> > If the patch requires changes it will need a separate review on 8.
> > 
> 
>   the email contained the webrev and review threads with the necessary
> Reviewer approvals for both 8u and 9 already.
> 
> See below.
> 
> If you need more, please tell me.
> 
> Thanks,
>   Thomas
> 
> >     -Rob
> > 
> > On 18/05/17 07:35, Thomas Schatzl wrote:
> > > 
> > > Hi all,
> > > 
> > >   could you please approve the backport of JDK-8180048 to 8u-dev?
> > > 
> > > This fixes a serious memory leak in G1.
> > > 
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8180048
> > > Webrev: http://cr.openjdk.java.net/~tschatzl/8180048-8u/webrev/
> > > Review thread (8u): http://mail.openjdk.java.net/pipermail/hotspot-
> > > gc-dev/2017-May/019872.html
> 
> ^^^^ here
> 
> > > Original webrev
> > > (9): http://cr.openjdk.java.net/~tschatzl/8180048/webrev.2/
> > > Review thread (9): http://mail.openjdk.java.net/pipermail/hotspot-
> > > gc-dev/2017-May/019855.html
> > > 
> > > Patch from 9 does not apply cleanly because of cleanup done in the
> > > meantime (split of symbolTable.?pp into
> > > symbolTable/stringTable.?pp, and removal of some code in 9).
> > > 
> > > Thanks,
> > >   Thomas
> > >