RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread

2023-02-07 Thread Varada M
1. test/jdk/jni/nullCaller/NullCallerTest.java 2. test/jdk/java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java 3. test/hotspot/jtreg/runtime/jni/CalleeSavedRegisters/FPRegs.java The above tests were blocked on AIX [@require os.family != "aix"] because these tests are failing to call

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread

2023-02-08 Thread Varada M
On Wed, 8 Feb 2023 11:03:35 GMT, David Holmes wrote: > I hate to see the code duplication, but we don't have a sharing mechanism for > the native parts of tests so that can't be helped. > > Changes look good. > > Please update the Oracle copyright lines so that the second year is 2023 if > ne

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread

2023-02-08 Thread Varada M
On Wed, 8 Feb 2023 11:37:14 GMT, Alan Bateman wrote: >> 1. test/jdk/jni/nullCaller/NullCallerTest.java >> 2. test/jdk/java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java >> 3. test/hotspot/jtreg/runtime/jni/CalleeSavedRegisters/FPRegs.java >> >> The above tests were blocked on AIX [

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v2]

2023-02-08 Thread Varada M
i/daemonDestroy/TestDaemonDestroy.java { PR : > [12006](https://github.com/openjdk/jdk/pull/12006) } > 5. test/lib-test/jdk/test/lib/process/TestNativeProcessBuilder.java > > The reported issue : [8300139](https://bugs.openjdk.org/browse/JDK-8300139l) Varada M has updated the pull requ

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v2]

2023-02-09 Thread Varada M
On Wed, 8 Feb 2023 21:30:29 GMT, David Holmes wrote: > It just occurred to me that we may also need to update the test makefiles to > link to libpthread for these modified tests? Running test 'jtreg:test/hotspot/jtreg/runtime/jni/CalleeSavedRegisters/FPRegs.java' Passed: runtime/jni/CalleeSav

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v3]

2023-02-09 Thread Varada M
i/daemonDestroy/TestDaemonDestroy.java { PR : > [12006](https://github.com/openjdk/jdk/pull/12006) } > 5. test/lib-test/jdk/test/lib/process/TestNativeProcessBuilder.java > > The reported issue : [8300139](https://bugs.openjdk.org/browse/JDK-8300139l) Varada M has updated the pull requ

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v2]

2023-02-09 Thread Varada M
On Wed, 8 Feb 2023 15:53:52 GMT, Tyler Steele wrote: >> Varada M has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fixes > > test/hotspot/jtreg/runtime/jni/CalleeSavedRegisters/FPRegs.java line 2: > >

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v4]

2023-02-09 Thread Varada M
i/daemonDestroy/TestDaemonDestroy.java { PR : > [12006](https://github.com/openjdk/jdk/pull/12006) } > 5. test/lib-test/jdk/test/lib/process/TestNativeProcessBuilder.java > > The reported issue : [8300139](https://bugs.openjdk.org/browse/JDK-8300139l) Varada M has updated the pull requ

Re: RFR: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread [v4]

2023-02-09 Thread Varada M
On Thu, 9 Feb 2023 11:30:11 GMT, Thomas Stuefe wrote: >> Varada M has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Fixes > > test/jdk/jni/nullCaller/exeNullCallerTest.cpp line 159: > >> 157: } &

Integrated: 8300139 : [AIX] Use pthreads to avoid JNI_createVM call from primordial thread

2023-02-13 Thread Varada M
On Tue, 31 Jan 2023 05:24:35 GMT, Varada M wrote: > 1. test/jdk/jni/nullCaller/NullCallerTest.java > 2. test/jdk/java/lang/reflect/exeCallerAccessTest/CallerAccessTest.java > 3. test/hotspot/jtreg/runtime/jni/CalleeSavedRegisters/FPRegs.java > > The above tests were

RFR: 8304320: java.base does not export jdk.internal.org.objectweb.asm to module jdk.jshell

2023-03-16 Thread Varada M
jdk/test/jdk/tools/jimage/VerifyJimage.java is failing because the module java.base is not exporting jdk.internal.org.objectweb.asm to module jdk.jshell. Reported Issue : [JDK-8304320](https://bugs.openjdk.org/browse/JDK-8304320) - Commit messages: - exporting jdk.internal.org.obj

Re: RFR: 8304320: java.base does not export jdk.internal.org.objectweb.asm to module jdk.jshell

2023-03-16 Thread Varada M
On Thu, 16 Mar 2023 08:43:46 GMT, Alan Bateman wrote: > jshell was migrated to new the class file API in JDK-8294974 so it shouldn't > have any references to ASM anymore. Would it be possible to check if you have > a mismatch between your build and the test? Hi @AlanBateman. The changes from J

Re: RFR: 8304320: java.base does not export jdk.internal.org.objectweb.asm to module jdk.jshell

2023-03-17 Thread Varada M
On Thu, 16 Mar 2023 08:43:46 GMT, Alan Bateman wrote: >> jdk/test/jdk/tools/jimage/VerifyJimage.java is failing because the module >> java.base is not exporting jdk.internal.org.objectweb.asm to module >> jdk.jshell. >> >> Reported Issue : [JDK-8304320](https://bugs.openjdk.org/browse/JDK-830

Withdrawn: 8304320: java.base does not export jdk.internal.org.objectweb.asm to module jdk.jshell

2023-03-17 Thread Varada M
On Thu, 16 Mar 2023 08:14:25 GMT, Varada M wrote: > jdk/test/jdk/tools/jimage/VerifyJimage.java is failing because the module > java.base is not exporting jdk.internal.org.objectweb.asm to module > jdk.jshell. > > Reported Issue : [JDK-8304320](https://bugs.openjdk.org/bro

RFR: JDK-8252802: java launcher should set MALLOCOPTIONS and LDR_CNTRL in order to use 64KB pages on AIX

2024-02-18 Thread Varada M
DeCapo Benchmark Results (3 runs) : Before : = DaCapo 9.12 h2 PASSED in 281402 msec = = DaCapo 9.12 h2 PASSED in 269818 msec = = DaCapo 9.12 h2 PASSED in 279279 msec = After: = DaCapo 9.12 h2 PASSED in 279192 msec = = DaCapo 9.12 h2 PASSED in 269769 msec ==

Re: RFR: JDK-8252802: java launcher should set MALLOCOPTIONS and LDR_CNTRL in order to use 64KB pages on AIX

2024-02-20 Thread Varada M
On Tue, 20 Feb 2024 07:45:59 GMT, Thomas Stuefe wrote: >> DeCapo Benchmark Results (3 runs) : >> >> Before : >> = DaCapo 9.12 h2 PASSED in 281402 msec = >> = DaCapo 9.12 h2 PASSED in 269818 msec = >> = DaCapo 9.12 h2 PASSED in 279279 msec = >> >> After: >> = DaCa

Withdrawn: JDK-8252802: java launcher should set MALLOCOPTIONS and LDR_CNTRL in order to use 64KB pages on AIX

2024-02-20 Thread Varada M
On Mon, 19 Feb 2024 05:52:22 GMT, Varada M wrote: > DeCapo Benchmark Results (3 runs) : > > Before : > = DaCapo 9.12 h2 PASSED in 281402 msec = > = DaCapo 9.12 h2 PASSED in 269818 msec = > = DaCapo 9.12 h2 PASSED in 279279 msec = > > After

RFR: JDK-8320309: AIX: pthreads created by foreign test library don't work as expected

2023-11-22 Thread Varada M
Following test fails due to missing pthread attributes on AIX : java/foreign/TestUpcallAsync.java java/foreign/stackwalk/TestAsyncStackWalk.java java/foreign/loaderLookup/TestLoaderLookupJNI.java java/foreign/enablenativeaccess/TestEnableNativeAccessJarManifest.java java/foreign/enablenativeaccess/

Re: RFR: JDK-8320309: AIX: pthreads created by foreign test library don't work as expected

2023-11-22 Thread Varada M
On Tue, 21 Nov 2023 14:24:47 GMT, Martin Doerr wrote: >> Following test fails due to missing pthread attributes on AIX : >> java/foreign/TestUpcallAsync.java >> java/foreign/stackwalk/TestAsyncStackWalk.java >> java/foreign/loaderLookup/TestLoaderLookupJNI.java >> java/foreign/enablenativeaccess/

Re: RFR: JDK-8320309: AIX: pthreads created by foreign test library don't work as expected

2023-11-22 Thread Varada M
On Tue, 21 Nov 2023 10:09:04 GMT, Varada M wrote: > Following test fails due to missing pthread attributes on AIX : > java/foreign/TestUpcallAsync.java > java/foreign/stackwalk/TestAsyncStackWalk.java > java/foreign/loaderLookup/TestLoaderLookupJNI.java > java/foreign/ena

Integrated: JDK-8320309: AIX: pthreads created by foreign test library don't work as expected

2023-11-22 Thread Varada M
On Tue, 21 Nov 2023 10:09:04 GMT, Varada M wrote: > Following test fails due to missing pthread attributes on AIX : > java/foreign/TestUpcallAsync.java > java/foreign/stackwalk/TestAsyncStackWalk.java > java/foreign/loaderLookup/TestLoaderLookupJNI.java > java/foreign/ena