On Wed, 23 Aug 2023 15:18:03 GMT, Matthias Baesken <mbaes...@openjdk.org> wrote:

>> Currently there is a number of functionality that would be interesting to 
>> have for shared lib load operations in the JDK C code.
>> Some examples :
>> Events::log_dll_message for hs-err files reporting
>> JFR event NativeLibraryLoad
>> There is the need to update the shared lib Cache on AIX ( see 
>> LoadedLibraries::reload() , see also 
>> https://bugs.openjdk.org/browse/JDK-8314152 ),
>> this is currently not fully in sync with libs loaded form jdk c-libs and 
>> sometimes reports outdated information
>> 
>> Offer an interface (e.g. jvm.cpp) to support this.
>
> Matthias Baesken has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   windows aarch64 build issues

Sorry but looking at the hotspot part of this I do not like the code in jvm.cpp 
at all - it is  far too messy. I expected to see a simple interface to 
os::dlopen which then handles all the platform specific issues.

I'm also somewhat dubious about having all the JDK code use a VM interface for 
this, The usual coupling to the VM is to just provide native implementations of 
core library methods, not to have the VM provide a general purpose utility 
interface like dynamic library loading.

-------------

PR Comment: https://git.openjdk.org/jdk/pull/15264#issuecomment-1694907802

Reply via email to