RFR: JDK-8133206 - Higher native memory usage caused by upgrade to zlib 1.2.8
Alex Kashchenko
akashche at redhat.com
Mon Mar 14 23:37:29 UTC 2016
Hi,
On 11/10/2015 10:44 AM, Alex Kashchenko wrote:
> Please review the following fix to JDK-8133206:
>
> - bug: https://bugs.openjdk.java.net/browse/JDK-8133206
> - webrev for jdk9:
> http://cr.openjdk.java.net/~akasko/jdk9/8133206/webrev.00/
>
> This fix is a forward-port of the patch to jdk7u created by Nikolay
> Gorshkov:
>
> - webrev for jdk7u:
> http://cr.openjdk.java.net/~nikgor/8133206/jdk7u-dev/webrev.01/
> - jdk7u review thread with the issue details:
> http://mail.openjdk.java.net/pipermail/jdk7u-dev/2015-October/010420.html
> - the same thread on core-libs-dev:
> http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-October/035884.html
>
>
> Forward-port to jdk9 is almost the same as original patch (with changed
> paths), one additional inflate(...) call is changed in zip_util.c .
>
> jdk9 with this patch passes all the regression tests from java/util/jar,
> java/util/zip, tools/jar and tools/pack200.
>
> I created a jtreg reproducer that works with jdk7u and jdk9:
>
> - web link: http://cr.openjdk.java.net/~akasko/jdk9/8133206/reproducer/
> - download:
> http://cr.openjdk.java.net/~akasko/jdk9/8133206/reproducer.zip
>
> This reproducer is Linux-only, slow (takes more than a minute) and
> requires Valgrind and zlib-debuginfo (if system zlib is used). It runs
> java.util.zip.Inflater under the Memcheck multiple times and parses the
> output to compare the number of zlib-specific memory allocations. I am
> not sure whether such reproducer should be included into the jdk
> regression tests tree, so publishing it as a standalone jtreg test (not
> a webrev).
>
> Please let me know if I need to create a new bug for jdk9 (in addition
> to JDK-8133206).
This is a follow-up on a positive response to this patch:
http://mail.openjdk.java.net/pipermail/jdk7u-dev/2016-March/010488.html
Please review and sponsor the updated change:
- updated webrev:
http://cr.openjdk.java.net/~akasko/jdk9/8133206/webrev.01/
- issue for jdk9: https://bugs.openjdk.java.net/browse/JDK-8151848
Change tested with java/util/zip and tools/pack200 tests and a
reproducer linked above on RHEL 7.2.
--
-Alex
More information about the jdk9-dev
mailing list