RFR: JDK-8320005 : Allow loading of shared objects with .a extension on AIX [v22]
Suchismith Roy
sroy at openjdk.org
Thu Feb 15 17:53:01 UTC 2024
On Thu, 15 Feb 2024 08:15:16 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> Hi,
>
> some remarks:
>
> * there is no need for a copy for the first call to dll_load_library. Just hand in the string 1:1.
> * I would only do the *.a fallback loading if the error indicates that the *.so file had not been there. So, only if EACCESS or ENOENT; in all other cases I would not do the fallback. E.g. if the *.so file cannot be loaded due to a header mismatch. See https://www.ibm.com/docs/en/aix/7.1?topic=l-load-loadandinit-subroutines
> * Please use os::strdup.
> * Please assert that the replacement string is smaller than the original string (which it should be, *.so is longer than *.a, but this is insurance against anyone changing the code in the future)
>
> Thank you, Thomas
Sure working on them.
May i know why we are using the load routine in the 2nd point ? . Currently we do a *.a fallback only when dlopen fails. Does load function save some steps here ?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/16604#issuecomment-1946756694
More information about the serviceability-dev
mailing list