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

Thomas Schatzl thomas.schatzl at oracle.com
Thu May 18 19:16:08 UTC 2017


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
> > 


More information about the jdk8u-dev mailing list