On Wed, 16 Apr 2025 09:51:49 GMT, Magnus Ihse Bursie wrote:
> `locale -a`
C
POSIX
en_US.8859-15
en_US.IBM-858
en_US.ISO8859-1
en_US
I don't know if UTF-8 can be installed. If so, we should also document that as
requirement for AIX build machines.
-
PR Comment: https://git.openj
On Mon, 14 Apr 2025 12:53:35 GMT, Magnus Ihse Bursie wrote:
>> Most of the JDK code base has been transitioned to UTF-8, but not all. This
>> has recently become an acute problem, since our mixing of iso-8859-1 and
>> utf-8 in properties files confused the version of `sed` that is shipped with
On Fri, 11 Apr 2025 07:11:19 GMT, Matthias Baesken wrote:
>> JLI_ReportErrorMessageSys is not used any more on Windows, so we can remove
>> it and make it UNIX-only (there a still a few remaining usages).
>
> Matthias Baesken has updated the pull request incrementally with one
> additional comm
On Fri, 4 Apr 2025 08:10:34 GMT, Thomas Schatzl wrote:
>> Hi all,
>>
>> please review this change that implements (currently Draft) JEP: G1:
>> Improve Application Throughput with a More Efficient Write-Barrier.
>>
>> The reason for posting this early is that this is a large change, and the
On Fri, 21 Mar 2025 16:19:38 GMT, Joachim Kern wrote:
>> After "JDK-8339480: Build static-jdk image with a statically linked
>> launcher" AIX was not able to build the new target. Therefore with
>> "JDK-8345590 AIX 'make all' fails after JDK-8339480" the new target was
>> disabled again.
>>
>
On Fri, 21 Mar 2025 11:44:49 GMT, Joachim Kern wrote:
>> After "JDK-8339480: Build static-jdk image with a statically linked
>> launcher" AIX was not able to build the new target. Therefore with
>> "JDK-8345590 AIX 'make all' fails after JDK-8339480" the new target was
>> disabled again.
>>
>
On Tue, 4 Mar 2025 09:57:56 GMT, Thomas Schatzl wrote:
>> Hi all,
>>
>> please review this change that implements (currently Draft) JEP: G1:
>> Improve Application Throughput with a More Efficient Write-Barrier.
>>
>> The reason for posting this early is that this is a large change, and the
On Sun, 23 Feb 2025 18:53:33 GMT, Thomas Schatzl wrote:
> Hi all,
>
> please review this change that implements (currently Draft) JEP: G1:
> Improve Application Throughput with a More Efficient Write-Barrier.
>
> The reason for posting this early is that this is a large change, and the JEP
pository.
> The backport will help prevent frequent tier1 test timeouts in our CI testing.
>
> The commit being backported was authored by Richard Reingruber on 22 Jan 2025
> and was reviewed by Christoph Langer, Matthias Baesken and Martin Doerr.
>
> Thanks, Richa
On Wed, 15 Jan 2025 14:14:37 GMT, Richard Reingruber wrote:
> This PR reverts the fix from
> [JDK-8339166](https://bugs.openjdk.org/browse/JDK-8339166) because it
> increases the runtime of the test a lot.
> Instead a full gc is requested via the whitebox api. This solves the issues
> (see bug
On Sat, 28 Dec 2024 18:43:30 GMT, Martin Doerr wrote:
> [JDK-8317545](https://bugs.openjdk.org/browse/JDK-8317545) introduced code
> for supporting 4 Byte aligned doubles:
> https://github.com/openjdk/jdk/blob/60e0730a3ba26180d0eb2cd278e389c3e70fec5f/src/java.base/share/classes/jdk
On Sat, 28 Dec 2024 18:43:30 GMT, Martin Doerr wrote:
> [JDK-8317545](https://bugs.openjdk.org/browse/JDK-8317545) introduced code
> for supporting 4 Byte aligned doubles:
> https://github.com/openjdk/jdk/blob/60e0730a3ba26180d0eb2cd278e389c3e70fec5f/src/java.base/share/classes/jdk
On Mon, 13 Jan 2025 13:58:43 GMT, Matthias Baesken wrote:
> looks okay, guess you have to adjust the COPYRIGHT year now to 2025 .
Only if I make more modifications. No files were changed in 2025. The update
script says "No files were changed".
But, thanks for the review!
-
PR Comm
On Thu, 9 Jan 2025 11:37:17 GMT, Joachim Kern wrote:
>> The test java/lang/ProcessHandle/InfoTest.java still fails sporadically on
>> AIX. The test exclusion was removed through
>> [JDK-8211847](https://bugs.openjdk.org/browse/JDK-8211847) under the
>> assumption the problem was gone. But it t
On Thu, 9 Jan 2025 11:37:17 GMT, Joachim Kern wrote:
>> The test java/lang/ProcessHandle/InfoTest.java still fails sporadically on
>> AIX. The test exclusion was removed through
>> [JDK-8211847](https://bugs.openjdk.org/browse/JDK-8211847) under the
>> assumption the problem was gone. But it t
On Wed, 8 Jan 2025 11:53:43 GMT, Joachim Kern wrote:
> The test java/lang/ProcessHandle/InfoTest.java still fails sporadically on
> AIX. The test exclusion was removed through
> [JDK-8211847](https://bugs.openjdk.org/browse/JDK-8211847) under the
> assumption the problem was gone. But it turne
[JDK-8317545](https://bugs.openjdk.org/browse/JDK-8317545) introduced code for
supporting 4 Byte aligned doubles:
https://github.com/openjdk/jdk/blob/60e0730a3ba26180d0eb2cd278e389c3e70fec5f/src/java.base/share/classes/jdk/internal/foreign/abi/ppc64/aix/AixPPC64Linker.java#L59
We should test this
On Mon, 2 Dec 2024 13:10:54 GMT, Per Minborg wrote:
>> This PR proposes to rewrite the `StringSupport::chunkedStrlen*` methods,
>> fixing a bug in the `short_strlen` variant for odd offsets (`offset % 2 !=
>> 0`).
>>
>> This PR also improves performance on modern hardware, as there is no need
On Mon, 25 Nov 2024 14:42:14 GMT, Per Minborg wrote:
> This PR proposes to fix some tests that fails on big-endian platforms.
Thanks for fixing it!
-
PR Comment: https://git.openjdk.org/jdk/pull/22362#issuecomment-2500784507
On Mon, 25 Nov 2024 14:42:14 GMT, Per Minborg wrote:
> This PR proposes to fix some tests that fails on big-endian platforms.
This fixes the test failures on AIX.
-
Marked as reviewed by mdoerr (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22362#pullrequestreview-2458985
On Thu, 7 Nov 2024 19:09:46 GMT, Per Minborg wrote:
> This PR fixes some regressions in the FFM benchmarks introduced by
> https://bugs.openjdk.org/browse/JDK-8332744.
LGTM.
-
Marked as reviewed by mdoerr (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/21963#pullrequestre
On Tue, 15 Oct 2024 07:57:46 GMT, Per Minborg wrote:
> Are you able to run the `SegmentBulkMismatch` benchmark on the various BE
> platforms?
Thanks for the hint. I've run it with the old implementation before
[JDK-8339531](https://bugs.openjdk.org/browse/JDK-8339531) but with the new
micro b
On Mon, 30 Sep 2024 09:19:21 GMT, Per Minborg wrote:
> This PR suggests fixing a problem on big-endian platforms.
This version uses little endian accesses on big endian machines. This may
defeat the purpose of
[JDK-8339531](https://bugs.openjdk.org/browse/JDK-8339531) for those platforms.
Acc
On Mon, 30 Sep 2024 09:57:11 GMT, Amit Kumar wrote:
>> This PR suggests fixing a problem on big-endian platforms.
>
> @minborg , I am still getting error on s390x (big endian):
>
> STDOUT:
> test TestMismatch.random(): failure [197ms]
> java.lang.AssertionError: expected [1] but found [0]
>
On Thu, 19 Sep 2024 07:40:43 GMT, Matthias Baesken wrote:
> Windows Server 2025 will be releases in a few months.
> The OS detection code of the JVM/JDK should recognize the new Windows server
> 2025 version.
> (currently Windows server 2022 is printed , that is wrong)
>
> The build numbers of
On Mon, 9 Sep 2024 15:30:47 GMT, Maurizio Cimadamore
wrote:
>> Maurizio Cimadamore has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Drop spurious change
>
> src/java.base/share/classes/java/nio/MappedMemoryUtils.java line 128:
>
>> 126:
On Mon, 9 Sep 2024 12:57:17 GMT, Maurizio Cimadamore
wrote:
> The new test added by https://github.com/openjdk/jdk/pull/20854 fails
> spuriously.
> While JNI lookup is now moved into the static initializer of the
> `MappedMemoryUtils` class, this class might only get initialized while in the
On Fri, 6 Sep 2024 17:51:15 GMT, Jorn Vernee wrote:
>> As discussed in the JBS issue:
>>
>> FFM upcall stubs embed a `Method*` of the target method in the stub. This
>> `Method*` is read from the `LambdaForm::vmentry` field associated with the
>> target method handle at the time when the upcal
On Wed, 4 Sep 2024 16:45:38 GMT, Amit Kumar wrote:
>> I looked into it, but couldn't find out. But I remove the `save_return_pc` &
>> `restore_return_pc` and everything seems fine. So maybe we can remove it.
>
> Tier1 test are fine with/without "saving & restoring" return_pc;
I found it:
https
On Wed, 4 Sep 2024 13:14:55 GMT, Jorn Vernee wrote:
>> As discussed in the JBS issue:
>>
>> FFM upcall stubs embed a `Method*` of the target method in the stub. This
>> `Method*` is read from the `LambdaForm::vmentry` field associated with the
>> target method handle at the time when the upcal
On Tue, 6 Aug 2024 17:26:55 GMT, Jorn Vernee wrote:
> As discussed in the JBS issue:
>
> FFM upcall stubs embed a `Method*` of the target method in the stub. This
> `Method*` is read from the `LambdaForm::vmentry` field associated with the
> target method handle at the time when the upcall stu
On Tue, 6 Aug 2024 17:26:55 GMT, Jorn Vernee wrote:
> As discussed in the JBS issue:
>
> FFM upcall stubs embed a `Method*` of the target method in the stub. This
> `Method*` is read from the `LambdaForm::vmentry` field associated with the
> target method handle at the time when the upcall stu
On Fri, 9 Aug 2024 12:44:45 GMT, Jorn Vernee wrote:
> I think that's a good compromise. (Although I wish the C++ code was just as
> fast, as it's much nicer)
Agreed.
> I'm not sure what other decorator would apply.
I think `ON_STRONG_OOP_REF` and `IS_NOT_NULL` could also be used. But, I guess
On Tue, 6 Aug 2024 17:26:55 GMT, Jorn Vernee wrote:
> As discussed in the JBS issue:
>
> FFM upcall stubs embed a `Method*` of the target method in the stub. This
> `Method*` is read from the `LambdaForm::vmentry` field associated with the
> target method handle at the time when the upcall stu
On Tue, 3 Sep 2024 07:26:53 GMT, Matthias Baesken wrote:
>> We get a couple of warnings as errors on AIX because of unused variables or
>> functions , for example :
>> /priv/jenkins/client-home/workspace/openjdk-jdk-dev-aix_ppc64-opt/jdk/src/java.base/unix/native/libjava/ProcessHandleImpl_unix.c
On Mon, 2 Sep 2024 13:25:51 GMT, Matthias Baesken wrote:
>> We get a couple of warnings as errors on AIX because of unused variables or
>> functions , for example :
>> /priv/jenkins/client-home/workspace/openjdk-jdk-dev-aix_ppc64-opt/jdk/src/java.base/unix/native/libjava/ProcessHandleImpl_unix.c
On Thu, 29 Aug 2024 13:55:25 GMT, Matthias Baesken wrote:
> We see HiddenClassUnloading.java failing on the ppc64 based platforms. On AIX
> it seems to fail always; Linux ppc64le sometimes.
> Failure output :
> java.lang.RuntimeException: unloadedClassCount is zero
> at HiddenClassUnloading.main
On Tue, 25 Jun 2024 13:47:39 GMT, Adam Sotona wrote:
>> After JDK-8294960 is java.lang.invoke.ClassSpecializer using lamdas for code
>> generation and unfortunately it causes StackOverflow on BigEndian platforms.
>>
>> This patch converts all lambdas in ClassSpecializer into anonymous inner
>>
On Tue, 25 Jun 2024 13:47:39 GMT, Adam Sotona wrote:
>> After JDK-8294960 is java.lang.invoke.ClassSpecializer using lamdas for code
>> generation and unfortunately it causes StackOverflow on BigEndian platforms.
>>
>> This patch converts all lambdas in ClassSpecializer into anonymous inner
>>
On Mon, 24 Jun 2024 09:40:14 GMT, Christoph Langer wrote:
> Hi all,
>
> This pull request contains a backport of
> [JDK-8222884](https://bugs.openjdk.org/browse/JDK-8222884), commit
> [bd046d9b](https://github.com/openjdk/jdk/commit/bd046d9b9e79e4eea89c72af358961ef6e98e660)
> from the [openjd
On Tue, 28 May 2024 14:01:44 GMT, Martin Doerr wrote:
> Fix obvious typo in micro benchmark.
This pull request has now been integrated.
Changeset: 9ac8d05a
Author: Martin Doerr
URL:
https://git.openjdk.org/jdk/commit/9ac8d05a2567fbf65b944660739e5f8ad1fc2020
Stats: 8 lines i
On Tue, 28 May 2024 14:01:44 GMT, Martin Doerr wrote:
> Fix obvious typo in micro benchmark.
Thanks!
-
PR Comment: https://git.openjdk.org/jdk/pull/19427#issuecomment-2136008570
On Tue, 28 May 2024 14:01:44 GMT, Martin Doerr wrote:
> Fix obvious typo in micro benchmark.
Thanks for the review!
-
PR Comment: https://git.openjdk.org/jdk/pull/19427#issuecomment-2135446758
Fix obvious typo in micro benchmark.
-
Commit messages:
- 8332228: TypePollution.java: Unrecognized VM option 'UseSecondarySuperCache'
Changes: https://git.openjdk.org/jdk/pull/19427/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19427&range=00
Issue: https://bugs.openjd
On Fri, 24 May 2024 07:24:13 GMT, Matthias Baesken wrote:
>> When building with ubsan enabled (--enable-uban) on Linux x86_64 and doing
>> jtreg tests afterwards I run into this error :
>>
>> /jdk/src/java.base/unix/native/libjava/ProcessImpl_md.c:562:5: runtime
>> error: null pointer passed a
On Wed, 22 May 2024 14:47:43 GMT, Yudi Zheng wrote:
>> Moving array construction within BigInteger.implMultiplyToLen intrinsic
>> candidate to its caller simplifies the intrinsic implementation in JIT
>> compiler.
>
> Yudi Zheng has updated the pull request incrementally with one additional
>
On Tue, 7 May 2024 08:08:05 GMT, Joachim Kern wrote:
>> Since ~ end of March, after
>> [JDK-8329131](https://bugs.openjdk.org/browse/JDK-8329131),
>> tools/launcher/JliLaunchTest.java fails on AIX. Failure is :
>>
>> stdout: [];
>> stderr: [Error: could not find libjava.so
>> Error: Could no
On Tue, 23 Apr 2024 07:51:28 GMT, Matthias Baesken wrote:
> In the hashN usages of readCen from zip_util.c we see a lot of signed integer
> overflows.
> For example in the java/util jtreg tests those are easily reproducable when
> compiling with -ftrapv (clang/gcc toolchains).
> While those ove
On Wed, 17 Apr 2024 11:20:24 GMT, Suchismith Roy 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 incremen
On Wed, 17 Apr 2024 11:20:24 GMT, Suchismith Roy 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 incremen
On Mon, 15 Apr 2024 16:10:38 GMT, Suchismith Roy 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 incremen
On Mon, 15 Apr 2024 16:10:38 GMT, Suchismith Roy 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 incremen
On Mon, 15 Apr 2024 16:07:14 GMT, Suchismith Roy wrote:
>> test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java
>> line 71:
>>
>>> 69: }
>>> 70: }
>>> 71: }
>>
>> I think files typically have a newline at the end.
>
> In ClassLoader i didnt see it. P
On Mon, 8 Apr 2024 10:27:40 GMT, Martin Doerr wrote:
>> Suchismith Roy has updated the pull request incrementally with three
>> additional commits since the last revision:
>>
>> - update tests
>> - update tests
>> - update tests
>
> src
On Mon, 15 Apr 2024 13:23:22 GMT, Suchismith Roy 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 incremen
On Wed, 10 Apr 2024 16:46:30 GMT, Suchismith Roy 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 incremen
On Wed, 10 Apr 2024 18:07:33 GMT, Suchismith Roy wrote:
> > BUILD_JDK_JTREG_OUTPUT_DIR
>
> I agree. How can i fetch this path ? I tried
> System.getEnv("BUILD_JDK_JTREG_OUTPUT_DIR") But it doesn't seem to work.
Hmm. Good question. Maybe it can get extracted from the library path. I noticed
th
On Wed, 10 Apr 2024 16:33:53 GMT, Suchismith Roy wrote:
> I have referred to the lib directory under jdk and not jtreg anymore.
> Wouldn't that be fine ?
I don't think writing to the jdk is a good design. People may even want to
write protect it.
-
PR Comment: https://git.openjdk
On Wed, 10 Apr 2024 11:06:21 GMT, Suchismith Roy 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 incremen
On Tue, 9 Apr 2024 10:01:59 GMT, Suchismith Roy wrote:
>> test/jdk/java/lang/RuntimeTests/loadLibrary/aix/LoadAIXLibraryFromArchiveObject.java
>> line 37:
>>
>>> 35: File awtSharedObjectPath = new File("/test/lib/libawt.so");
>>> 36: File awtArchivePath = new File("/test/lib/lib
On Mon, 8 Apr 2024 18:44:51 GMT, Suchismith Roy 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 increment
On Fri, 5 Apr 2024 18:14:36 GMT, Suchismith Roy 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 increment
On Fri, 5 Apr 2024 18:14:36 GMT, Suchismith Roy 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 increment
On Wed, 3 Apr 2024 12:32:21 GMT, Suchismith Roy 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 increment
On Wed, 3 Apr 2024 12:32:21 GMT, Suchismith Roy 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 increment
On Mon, 1 Apr 2024 17:07:47 GMT, Suchismith Roy 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 increment
On Mon, 1 Apr 2024 17:07:47 GMT, Suchismith Roy 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 increment
On Mon, 1 Apr 2024 18:31:52 GMT, Mandy Chung wrote:
> > Now, I'm getting "java.lang.UnsatisfiedLinkError: no libname.a in
> > java.library.path" when trying `System.loadLibrary("libname.a")` even
> > though the file exists in the library path. Is this intended?
>
> To load a library file, you
On Mon, 1 Apr 2024 17:07:47 GMT, Suchismith Roy 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 increment
On Wed, 27 Mar 2024 17:23:50 GMT, Suchismith Roy 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 with a n
On Wed, 27 Mar 2024 17:01:30 GMT, Suchismith Roy wrote:
>>> So we should keep the mapAlternativeName for atleast .so to .a
>>> mapping(without any members mentioned).
>>
>> "libname.so(member_name)" is not a valid library name. No reason why
>> `System.load` has to support it.
>
>> > So we s
On Wed, 27 Mar 2024 08:23:53 GMT, Suchismith Roy wrote:
>> 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.
>
> @mlchung The first name
On Mon, 25 Mar 2024 09:46:50 GMT, Suchismith Roy 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 incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy 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 incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy 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 incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy 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 incremen
On Mon, 18 Mar 2024 17:43:45 GMT, Suchismith Roy 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 incremen
On Thu, 21 Mar 2024 15:22:22 GMT, Jaikiran Pai wrote:
>> Hello @suchismith1993, I haven't fully caught up on the necessity of this
>> change. The current proposed change, renames files in the underlying
>> filesystem in the code flow of `System.loadLibrary()` and I don't think is
>> the right
On Tue, 19 Mar 2024 10:01:31 GMT, Suchismith Roy wrote:
>>> > In AIX, we have an usecase where shared libraries have certain member
>>> > objects to be referred to. E.g libclang.a(shr_64.o) .
>>>
>>> Would you happen to know any official documentation which explains that AIX
>>> syntax?
>>>
>
On Tue, 19 Mar 2024 14:59:19 GMT, Goetz Lindenmaier wrote:
> …rve enough space for 2097152KB object heap
>
> I would like to fix this as the two related issues mentioned in the JBS bug.
> We see it currently in most GHA runs.
LGTM.
-
Marked as reviewed by mdoerr (Reviewer).
PR Re
On Tue, 30 Jan 2024 14:02:41 GMT, Matthias Baesken wrote:
>>> Yes there is a nice define in the AIX header
>>
>> *sigh* On linux, they go to some lengths to avoid this, using a __REDEFINE
>> mechanism. Oh well.
>>
>> Anyway, I think this particular can be resolved by not including the
>> sta
On Thu, 25 Jan 2024 09:15:38 GMT, Matthias Baesken wrote:
>> According to the sysinfo manpage (
>> https://man7.org/linux/man-pages/man2/sysinfo.2.html ) the memory and swap
>> related entries in the struct sysinfo are given as multiples of mem_unit
>> bytes.
>> "In the above structure, sizes
On Fri, 29 Dec 2023 13:44:27 GMT, Christoph Langer wrote:
> In the review of the PR for JDK-8322417 it was noted that a fully qualified
> class name "java.util.Arrays" is unnecessary but it was forgotten to clean it
> up prior to integration.
LGTM.
-
Marked as reviewed by mdoerr
On Mon, 4 Dec 2023 21:25:41 GMT, Martin Doerr wrote:
> I'd like to clean this up: Use the canonical layouts which are available in
> JDK22. Use try-with-resources for `Arena.ofConfined()`.
This pull request has now been integrated.
Changeset: a1fe16b5
Author:Martin
On Mon, 4 Dec 2023 21:25:41 GMT, Martin Doerr wrote:
> I'd like to clean this up: Use the canonical layouts which are available in
> JDK22. Use try-with-resources for `Arena.ofConfined()`.
Thanks for the reviews! The Pre-submit test errors are unrelated.
-
PR Com
I'd like to clean this up: Use the canonical layouts which are available in
JDK22. Use try-with-resources for `Arena.ofConfined()`.
-
Commit messages:
- 8321300: Cleanup TestHFA
Changes: https://git.openjdk.org/jdk/pull/16959/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=
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
>> j
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/enablenativeaccess/TestEnabl
On Tue, 21 Nov 2023 19:30:30 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Tue, 21 Nov 2023 17:54:08 GMT, suchismith1993 wrote:
>> src/java.base/aix/native/libsyslookup/syslookup.c line 30:
>>
>>> 28: #include
>>> 29: #include
>>> 30: #include
>>
>> Are string.h and stdlib.h needed? I can't see them in the comments below.
>
> string.h is needed for strlen. Let m
On Tue, 21 Nov 2023 17:49:43 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Tue, 21 Nov 2023 13:01:50 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Tue, 21 Nov 2023 13:01:50 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Tue, 21 Nov 2023 11:52:23 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Tue, 21 Nov 2023 11:21:40 GMT, suchismith1993 wrote:
>> The math library in AIX specifically, is a static archive. Doing a -lm wont
>> suffice, because when the symbols are looked up using dlsym or accessing
>> native code through Java, it will lead to failures.
>> Hence we had to come up wi
On Fri, 17 Nov 2023 12:45:32 GMT, suchismith1993 wrote:
> And I still don't understand if this is the list of symbols that are required
> by our tests, or the complete list of symbols that FFI `defaultLookup`
> returns to user applications. If it is the latter, then this is sort-of okay
> as a
On Mon, 13 Nov 2023 11:47:52 GMT, suchismith1993 wrote:
>>> There is not generic way of generating this. It needs a manual intervention
>>> and symbols are to be added on a need basis in future. Looks like a list to
>>> be maintained. I had tried adding comments to explain the list, but that
>
On Wed, 15 Nov 2023 08:18:31 GMT, Per Minborg wrote:
> This PR proposes to fix a failing test on big endian architectures like AIX.
Thanks for fixing it so quickly! Looks correct and makes the test happy on AIX.
-
Marked as reviewed by mdoerr (Reviewer).
PR Review: https://git.ope
On Thu, 9 Nov 2023 09:22:45 GMT, suchismith1993 wrote:
> 1. use pragma directive to handle alignment.
>
> JBS Issue: [JDK-8318175](https://bugs.openjdk.org/browse/JDK-8318175)
Looks good. I had proposed this solution here:
https://github.com/openjdk/jdk/pull/16179#discussion_r1360709308
-
On Mon, 13 Nov 2023 12:51:36 GMT, Jorn Vernee wrote:
>> Add the ability to pass heap segments to native code. This requires using
>> `Linker.Option.critical(true)` as a linker option. It has the same
>> limitations as normal critical calls, namely: upcalls into Java are not
>> allowed, and the
1 - 100 of 309 matches
Mail list logo
