Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jdk17+ should give a better error message for failed library loads #17285

Closed
pshipton opened this issue Apr 26, 2023 · 0 comments · Fixed by #17374
Closed

jdk17+ should give a better error message for failed library loads #17285

pshipton opened this issue Apr 26, 2023 · 0 comments · Fixed by #17374

Comments

@pshipton
Copy link
Member

pshipton commented Apr 26, 2023

The way native libraries are loaded changed in jdk17 to use JVM_LoadLibrary() rather than OpenJ9 internal natives. The error message thrown from JVM_LoadLibrary() is generic (Failed to load library \"%s\") and doesn't provide any information about why the load failed. It should include the message from j9error_last_error_message() after the call to to j9sl_open_shared_library() in order to give more detailed failure information.

pshipton added a commit to pshipton/openj9 that referenced this issue May 10, 2023
JVM_LoadLibrary() is used to load JNI libraries from jdk17.

Issue eclipse-openj9#17285

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
pshipton added a commit to pshipton/openj9 that referenced this issue May 10, 2023
JVM_LoadLibrary() is used to load JNI libraries from jdk17.

Issue eclipse-openj9#17285

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
pshipton added a commit to pshipton/openj9 that referenced this issue May 10, 2023
JVM_LoadLibrary() is used to load JNI libraries from jdk17.

Issue eclipse-openj9#17285

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
midronij pushed a commit to midronij/openj9 that referenced this issue May 24, 2023
JVM_LoadLibrary() is used to load JNI libraries from jdk17.

Issue eclipse-openj9#17285

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
midronij pushed a commit to midronij/openj9 that referenced this issue Jun 1, 2023
JVM_LoadLibrary() is used to load JNI libraries from jdk17.

Issue eclipse-openj9#17285

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant