RFR: JDK-8319516 - Native library suffix impact on the library loading in AIX- Java Class Loader [v5]

[Mandy Chung]

On Tue, 26 Mar 2024 20:59:54 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

>> Suchismith Roy has updated the pull request incrementally with four additional commits since the last revision:
>> 
>>  - coding style
>>  - set false
>>  - restore fil
>>  -  remove member check code
>
> src/java.base/aix/classes/jdk/internal/loader/ClassLoaderHelper.java line 68:
> 
>> 66:             int dotIndex = name.lastIndexOf('.');
>> 67:             String memberName = name.substring(openBracketIndex,dotIndex);
>> 68:             //Reconstruct <libname>.so(<member_name>) as <libname>.a(<member_name>)
> 
> Do we really need to support libname.so(member)? Isn't it always libname.a(member)?

I think `mapAlternativeName` isn't needed at all.   If `loadLibraryOnlyIfPresent` returns false, `System.load("libname.a(member)")` should be passed to dlopen directly.   @suchismith1993 can verify it.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17945#discussion_r1540117924