Re: RFR: 8352084: Add more test code in TestSetupAOT.java [v3]

2025-03-15 Thread Ioi Lam
> I modified TestSetupAOT.java to exercise more functionalities in the JDK so > that we can have a more substantial AOT cache when running tests with > `AOT_JDK=true`. E.g: > > > make test JTREG=AOT_JDK=true \ > TEST=open/test/jdk/java/util/TimeZone/ListTimeZones.java > > > Before: the ge

Re: RFR: 8351542: LIBMANAGEMENT_OPTIMIZATION remove special optimization settings

2025-03-15 Thread Erik Joelsson
On Mon, 10 Mar 2025 15:38:45 GMT, Matthias Baesken wrote: > On Linux there are some special settings for LIBMANAGEMENT_OPTIMIZATION that > are most likely not needed any more and could be removed. make/modules/java.management/Lib.gmk line 33: > 31: ## Build libmanagement > 32: > #

Re: RFR: 8351309: test/hotspot/jtreg/runtime/posixSig/TestPosixSig.java fails on static-jdk

2025-03-15 Thread Magnus Ihse Bursie
On Wed, 5 Mar 2025 22:59:06 GMT, Jiangli Zhou wrote: > Please review this PR that excludes `libjsig` from being statically linked > with `static-jdk` `java` launcher by default. Please see details in > https://bugs.openjdk.org/browse/JDK-8351309 description and comments. Thanks It seems we agr

Re: RFR: 8351542: LIBMANAGEMENT_OPTIMIZATION remove special optimization settings [v3]

2025-03-15 Thread Matthias Baesken
On Tue, 11 Mar 2025 15:59:09 GMT, Erik Joelsson wrote: > I think what Kevin is after is having this explanation made clear in the bug > and PR description so that it's made clear what the change is and intends to > do. I changed the description in the JBS issue a bit. - PR Commen

RFR: 8351842: Test issues on Windows with combination of --enable-linkable-runtime and --with-external-symbols-in-bundles=public

2025-03-15 Thread Christoph Langer
Alternative approach to #24012 This keeps the current handling of *.pdb vs *.stripped.pdb which allows debugging at the cost of a little hack in jlink. - Commit messages: - JDK-8351842 Changes: https://git.openjdk.org/jdk/pull/24057/files Webrev: https://webrevs.openjdk.org/?rep

Re: RFR: 8351440: Link with -reproducible on macOS

2025-03-15 Thread Erik Joelsson
On Fri, 7 Mar 2025 20:46:32 GMT, Mikael Vidstedt wrote: > Background (from JBS): > > Recent versions of Xcode, most likely starting with Xcode 15, do not produce > deterministic binaries/libraries out of the box. In particular, the UUID data > in the LC_UUID load command is not stable. > > Af

Re: RFR: 8351322: Parameterize link option for pthreads [v2]

2025-03-15 Thread Erik Joelsson
On Sat, 8 Mar 2025 13:39:44 GMT, snake66 wrote: >> Replace hardcoded instances of `-lpthread` with `$(LIBPTHREAD)`, so that >> it's possible to parameterize this for platforms that use different flags >> for enabling posix threads. >> >> This work is a continuation of the work done by Greg Lew

Re: RFR: 8351322: Parameterize link option for pthreads

2025-03-15 Thread snake66
On Thu, 6 Mar 2025 12:46:25 GMT, David Holmes wrote: > Abstracting this out seems reasonable to me, though I should say I thought we > already used `-pthread` rather than `-lpthread`. I noticed there were a few places that used `-pthread` by default. I left these alone in this PR. ---

Re: RFR: JDK-8348598 : Update Libpng to 1.6.47 [v3]

2025-03-15 Thread Harshitha Onkar
On Fri, 14 Mar 2025 02:02:58 GMT, Sergey Bylokhov wrote: >> make/modules/java.desktop/lib/ClientLibraries.gmk line 249: >> >>> 247: DISABLED_WARNINGS_clang_dgif_lib.c := sign-compare, \ >>> 248: DISABLED_WARNINGS_clang_gzwrite.c := format-nonliteral, \ >>> 249: DISABLED_WARNING

Re: RFR: 8351322: Parameterize link option for pthreads

2025-03-15 Thread Magnus Ihse Bursie
On Thu, 6 Mar 2025 10:39:27 GMT, snake66 wrote: > Replace hardcoded instances of `-lpthread` with `$(LIBPTHREAD)`, so that it's > possible to parameterize this for platforms that use different flags for > enabling posix threads. > > This work is a continuation of the work done by Greg Lewis in

Re: RFR: 8345169: Implement JEP 503: Remove the 32-bit x86 Port [v2]

2025-03-15 Thread Alan Bateman
On Mon, 10 Mar 2025 09:49:44 GMT, Aleksey Shipilev wrote: >> This PR implements JEP 503: Remove the 32-bit x86 Port. >> >> The JEP is proposed to target 25, we would not integrate until JEP is ready. >> Reviews are appreciated meanwhile. >> >> This is only the removal of obvious 32-bit x86 par

Re: RFR: 8352084: Add more test code in TestSetupAOT.java [v2]

2025-03-15 Thread Ioi Lam
On Fri, 14 Mar 2025 23:43:53 GMT, Erik Joelsson wrote: >> Ioi Lam has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Review comments from @erikj79 and @lmesnik > > make/test/BuildTestSetupAOT.gmk line 62: > >> 60: JavacBenchAp

Re: RFR: 8352044: Add --with-import-jvms to configure

2025-03-15 Thread Aleksey Shipilev
On Fri, 14 Mar 2025 17:57:31 GMT, Magnus Ihse Bursie wrote: > No, it doesn't -- only libjvm.so. .jsa files complicate the situation. Hm. Aha. Importing CDS archives without regenerating them is futile, IIRC: they would never load properly, CDS would get disabled, and we would just carry dead w

Re: RFR: 8345169: Implement JEP 503: Remove the 32-bit x86 Port

2025-03-15 Thread Coleen Phillimore
On Tue, 4 Mar 2025 16:52:16 GMT, Aleksey Shipilev wrote: > This PR implements JEP 503: Remove the 32-bit x86 Port. > > The JEP is proposed to target 25, we would not integrate until JEP is ready. > Reviews are appreciated meanwhile. > > This is only the removal of obvious 32-bit x86 parts, mos

Re: RFR: 8311227: Add .editorconfig so IDEs would pick up the common settings automatically: indent, trim trailing whitespace

2025-03-15 Thread Erik Joelsson
On Mon, 10 Mar 2025 11:03:30 GMT, Kim Barrett wrote: > > As for the coding style (indentation), I followed the official style guides > > to define this. > > For C++ we have > > https://github.com/openjdk/jdk/blob/master/doc/hotspot-style.md#whitespace > > which states: > > > Indentation levels

Re: RFR: 8351322: Parameterize link option for pthreads

2025-03-15 Thread Antonio Vieiro
On Thu, 6 Mar 2025 10:39:27 GMT, snake66 wrote: > Replace hardcoded instances of `-lpthread` with `$(LIBPTHREAD)`, so that it's > possible to parameterize this for platforms that use different flags for > enabling posix threads. > > This work is a continuation of the work done by Greg Lewis in

Re: RFR: 8352044: Add --with-import-jvms to configure

2025-03-15 Thread Magnus Ihse Bursie
On Fri, 14 Mar 2025 16:32:39 GMT, Aleksey Shipilev wrote: > This change copies `libjvm.so` _and_ sibling `.jsa` files, right? No, it doesn't -- only libjvm.so. .jsa files complicate the situation. Hm. Maybe we should point to an import JVM directory, and copy it with all its files, and not jus

Re: RFR: 8351309: test/hotspot/jtreg/runtime/posixSig/TestPosixSig.java fails on static-jdk [v2]

2025-03-15 Thread Jiangli Zhou
> Please review this PR that excludes `libjsig` from being statically linked > with `static-jdk` `java` launcher by default. Please see details in > https://bugs.openjdk.org/browse/JDK-8351309 description and comments. Thanks Jiangli Zhou has updated the pull request incrementally with two addit

Re: RFR: 8352015: LIBVERIFY_OPTIMIZATION remove special optimization settings

2025-03-15 Thread Magnus Ihse Bursie
On Fri, 14 Mar 2025 11:15:26 GMT, Matthias Baesken wrote: > On Linux there are some special settings for LIBVERIFY_OPTIMIZATION that are > most likely not needed any more and could be removed. > The removal (on Linux) brings the lib optimization level de facto from LOW to > HIGH. What testing

Re: RFR: 8351842: Test issues on Windows with combination of --enable-linkable-runtime and --with-external-symbols-in-bundles=public [v2]

2025-03-15 Thread Christoph Langer
On Fri, 14 Mar 2025 11:48:59 GMT, Magnus Ihse Bursie wrote: > Given all of this, is there any possibility at all for > --enable-linkable-runtime to handle both full and stripped pdb files, or do > we need to accept that this is an inherent limitation for the linkable > runtime? Yes, I think t