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

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

Reply via email to