Review Request JDK-8164512: Replace ClassLoader use of finalizer with phantom reference to unload native library
David Holmes
david.holmes at oracle.com
Tue Oct 10 03:27:54 UTC 2017
On 10/10/2017 6:17 AM, mandy chung wrote:
> David, Peter, Alan
>
> The VM has a fast path to search the symbol from libjava.so first for
> bootstrap loader. That was the case I mostly concern about performance
> and it's not impacted by this change. Also I consulted with Claes on
> the performance impact. I took your suggestion and made
> systemNativeLibraries and nativeLibraries volatile.
>
> Updated webrev:
> http://cr.openjdk.java.net/~mchung/jdk10/webrevs/8164512/webrev.04
Looks good.
Thanks,
David
-----
> thanks
> Mandy
>
> On 10/9/17 1:20 AM, Alan Bateman wrote:
>> Alternatively change nativeLibraries and systemNativeLibraries to
>> volatile so the synchronization is only needed to initialize them.
>> Otherwise this version (webrev.03) looks good to me.
>>
>> -Alan
>
More information about the core-libs-dev
mailing list