On Wed, 8 Nov 2023 08:29:52 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

> On AIX the test 
> test/hotspot/jtreg/serviceability/jvmti/RedefineClasses/RedefineLeakThrowable.java
>  runs into this error:
> 
> java.lang.RuntimeException: java.lang.OutOfMemoryError: Metaspace
> at 
> jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.invocationHelper(JavacTaskImpl.java:168)
> at 
> jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.doCall(JavacTaskImpl.java:100)
> at 
> jdk.compiler/com.sun.tools.javac.api.JavacTaskImpl.call(JavacTaskImpl.java:94)
> at 
> jdk.test.lib.compiler.InMemoryJavaCompiler.compile(InMemoryJavaCompiler.java:188)
> at RedefineClassHelper.redefineClass(RedefineClassHelper.java:50)
> at RedefineLeakThrowable.main(RedefineLeakThrowable.java:64)
> at 
> java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
> at java.base/java.lang.reflect.Method.invoke(Method.java:580)
> at 
> com.sun.javatest.regtest.agent.MainWrapper$MainTask.run(MainWrapper.java:138)
> at java.base/java.lang.Thread.run(Thread.java:1570)
> Caused by: java.lang.OutOfMemoryError: Metaspace
> at java.base/java.time.Duration.<clinit>(Duration.java:149)
> at java.base/java.time.temporal.ChronoUnit.<clinit>(ChronoUnit.java:83)
> at java.base/java.time.temporal.ChronoField.<clinit>(ChronoField.java:124)
> at java.base/java.time.LocalDate.of(LocalDate.java:272)
> at java.base/java.time.LocalDate.<clinit>(LocalDate.java:147)
> at java.base/java.time.LocalDateTime.<clinit>(LocalDateTime.java:145)
> at jdk.zipfs/jdk.nio.zipfs.ZipUtils.dosToJavaTime(ZipUtils.java:216)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystem$Entry.readCEN(ZipFileSystem.java:2816)
> at jdk.zipfs/jdk.nio.zipfs.ZipFileSystem$Entry.<init>(ZipFileSystem.java:2778)
> at jdk.zipfs/jdk.nio.zipfs.ZipFileSystem.getEntry(ZipFileSystem.java:1941)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystem.newInputStream(ZipFileSystem.java:859)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystem.isMultiReleaseJar(ZipFileSystem.java:1444)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystem.initializeReleaseVersion(ZipFileSystem.java:1416)
> at jdk.zipfs/jdk.nio.zipfs.ZipFileSystem.<init>(ZipFileSystem.java:191)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystemProvider.getZipFileSystem(ZipFileSystemProvider.java:125)
> at 
> jdk.zipfs/jdk.nio.zipfs.ZipFileSystemProvider.newFileSystem(ZipFileSystemProvider.java:120)
> at 
> jdk.compiler/com.sun.tools.javac.file.JavacFileManager$ArchiveContainer.<init>(JavacFileManager.java:566)
> at 
> jdk.compiler/com.sun.tools.javac.file.JavacFileManager.getContainer(JavacFileManager.java:329)
> at jdk.compiler/com.sun.tools.javac.file.JavacFileManager.pathsA...

I just want it confirmed that the proposed test changes will exhibit a leak on 
all platforms on an unpatched VM. If that can't be ascertained then it may be 
better to only change for AIX.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/16553#issuecomment-1805134932

Reply via email to