RFR: JDK-8319516 - Native library suffix impact on the library loading in AIX- Java Class Loader [v8]
Mandy Chung
mchung at openjdk.org
Mon Apr 1 18:30:04 UTC 2024
On Mon, 1 Apr 2024 17:07:47 GMT, Suchismith Roy <sroy at openjdk.org> wrote:
>> Allow support for both .a and .so files in AIX.
>> If .so file is not found, allow fallback to .a extension.
>> JBS Issue: [JDK-8319516](https://bugs.openjdk.org/browse/JDK-8319516)
>
> Suchismith Roy has updated the pull request incrementally with one additional commit since the last revision:
>
> spaces
I adjust the comment which also answer your question. Please add a AIX-only test to verify this behavior.
src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 2:
> 1: /*
> 2: * Copyright (c) 2012, 2024, Oracle and/or its affiliates. All rights reserved.
Suggestion:
* Copyright (c) 2024, Oracle and/or its affiliates. All rights reserved.
New file. Copyright header start year should be 2024.
src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 38:
> 36: /**
> 37: * Returns true if loading a native library only if
> 38: * it's present on the file system.
Suggestion:
* Shared objects may be loaded from .a archive object on AIX and .so may not exist.
* This method returns false so that loading of shared library continues if libname.so is not present.
src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 48:
> 46: * file may be located at the alternate location.
> 47: * For most platforms, this behavior is not supported and returns null.
> 48: */
Suggestion:
/**
* AIX implementation of JVM_LoadLibrary handles the alternate path name mapping.
* If loading of the given library name with ".so" suffix fails, it will attempt to load the library of
* the same name with ".a" suffix as the alternate name.
*
* This method simply returns null. It could implement the alternate name converting ".so" with ".a" suffix but redundant.
-------------
PR Review: https://git.openjdk.org/jdk/pull/17945#pullrequestreview-1971830272
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1546680293
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1546662648
PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1546679104
More information about the core-libs-dev
mailing list