Re: RFR: 8266310: deadlock while loading the JNI code [v3]

2021-05-21 Thread Jorn Vernee
On Fri, 21 May 2021 15:39:33 GMT, Aleksei Voitylov wrote: >> src/java.base/share/classes/jdk/internal/loader/NativeLibraries.java line >> 262: >> >>> 260: } finally { >>> 261: releaseNativeLibraryLock(name); >>> 262: } >> >> The new locking scheme looks incorrect t

Re: RFR: 8266310: deadlock while loading the JNI code [v3]

2021-05-21 Thread Aleksei Voitylov
On Wed, 19 May 2021 18:47:52 GMT, Jorn Vernee wrote: >> Aleksei Voitylov has updated the pull request incrementally with one >> additional commit since the last revision: >> >> fix trailing whitespace > > src/java.base/share/classes/jdk/internal/loader/NativeLibraries.java line 262: > >> 260

Re: RFR: 8266310: deadlock while loading the JNI code [v3]

2021-05-20 Thread Aleksei Voitylov
On Wed, 19 May 2021 16:29:33 GMT, Aleksei Voitylov wrote: >> Please review this PR which fixes the deadlock in ClassLoader between the >> two lock objects - a lock object associated with the class being loaded, and >> the ClassLoader.loadedLibraryNames hash map, locked during the native >> li

Re: RFR: 8266310: deadlock while loading the JNI code [v3]

2021-05-19 Thread Jorn Vernee
On Wed, 19 May 2021 16:29:33 GMT, Aleksei Voitylov wrote: >> Please review this PR which fixes the deadlock in ClassLoader between the >> two lock objects - a lock object associated with the class being loaded, and >> the ClassLoader.loadedLibraryNames hash map, locked during the native >> li

Re: RFR: 8266310: deadlock while loading the JNI code [v3]

2021-05-19 Thread Aleksei Voitylov
> Please review this PR which fixes the deadlock in ClassLoader between the two > lock objects - a lock object associated with the class being loaded, and the > ClassLoader.loadedLibraryNames hash map, locked during the native library > load operation. > > Problem being fixed: > > The initial