[10] (S) RFR 8184361: AOT lib at jdk/lib/libjava.base-coop.so seems to override -XX:AOTLibrary=

dean.long at oracle.com dean.long at oracle.com
Wed Nov 29 02:19:16 UTC 2017


On 11/28/17 5:55 PM, Vladimir Kozlov wrote:

> http://cr.openjdk.java.net/~kvn/8184361/webrev.00/
> https://bugs.openjdk.java.net/browse/JDK-8184361
>
> Change priority in which AOT libraries are loaded.
>
> First, load libraries specified by AOTLibrary. And then, well known 
> libraries from JAVA installation directory. Don't load a library if an 
> other library with the same name is already loaded.
>
> It will allow do quick experiments with modified libraries.
>
> Manual testing and AOT jtreg tests.
>
> Thanks,
> Vladimir

It looks OK, but it imposes an undocumented? naming convention that must 
be followed.  If have use dir1/foo.so and dir2/foo.so, then only first 
will get loaded, so I need to use unique file names.  And if I 
(accidentally) name it lib<module>.so, then the system version won't get 
loaded, so users must know the naming convention to avoid conflicts.  If 
users need to following a naming convention to avoid conflicts, then it 
might make sense to deprecate AOTLibrary and instead have something more 
convenient for multiple libraries, like a AOTLibraryPath search path option.

I wonder if we should always print the warning, instead of putting it 
under PrintAOT.

dl


More information about the hotspot-compiler-dev mailing list