RFR: JDK-8313764: Offer JVM HS functionality to shared lib load operations done by the JDK codebase [v2]
Matthias Baesken
mbaesken at openjdk.org
Mon Aug 28 07:05:13 UTC 2023
On Mon, 28 Aug 2023 02:26:30 GMT, David Holmes <dholmes at openjdk.org> wrote:
> 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.
Hi David, so would you prefer to call into 'os::dll_load' (probably you meant that) instead ? That would indeed make the coding smaller and less 'messy' .
However 'os::dll_load' has no flags parameter but uses a default, have to check if this might be a problem ...
-------------
PR Comment: https://git.openjdk.org/jdk/pull/15264#issuecomment-1695142975
More information about the security-dev
mailing list