Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v2]

2022-07-14 Thread Roger Riggs
On Thu, 14 Jul 2022 12:54:43 GMT, Jaikiran Pai wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Cleanup waiting for library unload > > test/jdk/java/lang/ClassLoader/loadLibraryUnload/libloadLibraryUnload.c line > 4

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v2]

2022-07-14 Thread Roger Riggs
On Wed, 13 Jul 2022 20:39:53 GMT, Mandy Chung wrote: >> Roger Riggs has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Cleanup waiting for library unload > > test/jdk/java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnload.java line > 16

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v2]

2022-07-14 Thread Jaikiran Pai
On Wed, 13 Jul 2022 14:06:00 GMT, Roger Riggs wrote: >> The test `java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnloadTest.java` >> Fails intermittently when expected output from a subprocess is not found. >> >> I suspect a race between the Cleaner that is going to call JNI_OnUnload (in >>

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v2]

2022-07-13 Thread Mandy Chung
On Wed, 13 Jul 2022 14:06:00 GMT, Roger Riggs wrote: >> The test `java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnloadTest.java` >> Fails intermittently when expected output from a subprocess is not found. >> >> I suspect a race between the Cleaner that is going to call JNI_OnUnload (in >>

Re: RFR: 8289919: [test] LoadLibraryUnloadTest.java failed with "Failed to unload native library" [v2]

2022-07-13 Thread Roger Riggs
> The test `java/lang/ClassLoader/loadLibraryUnload/LoadLibraryUnloadTest.java` > Fails intermittently when expected output from a subprocess is not found. > > I suspect a race between the Cleaner that is going to call JNI_OnUnload (in > NativeLibraries.java:377) when the ClassLoader is no longer