[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: JDK-8315706: com/sun/tools/attach/warnings/DynamicLoadWarningTest.java real fix for failure
From:       Joachim Kern <jkern () openjdk ! org>
Date:       2023-11-27 15:22:29
Message-ID: EtFelHoT0wA8T2U3p-Mh4ZLziq-4ZM6CHCmRyEk5nPk=.11af29d2-6541-4421-a5d4-8c05834f46aa () github ! com
[Download RAW message or body]

On Fri, 15 Sep 2023 07:22:32 GMT, Joachim Kern <jkern@openjdk.org> wrote:

> > After push of [JDK-8307478](https://bugs.openjdk.org/browse/JDK-8307478) , the \
> > following test started to fail on AIX : \
> > com/sun/tools/attach/warnings/DynamicLoadWarningTest.java; The problem was \
> > described in [JDK-8309549](https://bugs.openjdk.org/browse/JDK-8309549) with a \
> > first try of a fix. A second fix via \
> > [JDK-8310191](https://bugs.openjdk.org/browse/JDK-8310191) was necessary. Both \
> > fixes just disable the specific subtest on AIX, without correction of the root \
> > cause. The root cause is, that dlopen() on AIX returns different handles every \
> > time, even if you load a library twice. There is no official AIX API available to \
> > get this information on a different way. My proposal is, to use the stat64x API \
> > with the fields st_device and st_inode. After a dlopen() the stat64x() API is \
> > called additionally to get this information which is then stored parallel to the \
> > library handle in the jvmtiAgent. For AIX we then can compare these values \
> > instead of the library handle and get the same functionality as on linux.
> 
> Joachim Kern has updated the pull request incrementally with one additional commit \
> since the last revision: 
> adopt types

If you have time, please call me for a short discussion.

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

PR Comment: https://git.openjdk.org/jdk/pull/15583#issuecomment-1828040131


[prev in list] [next in list] [prev in thread] [next in thread]