From tpushkin at openjdk.org Mon Sep 1 07:42:11 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 07:42:11 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources Message-ID: Backport of [JDK-8274893](https://bugs.openjdk.org/browse/JDK-8274893) for parity with Oracle's JDK 8u461. The first commit is a cherry-pick of the backport to JDK 11 with conflicts unresolved, the second resolves the conflicts, the rest resolve issues uncaught by git. Resolved: - `jdk/src/share/classes/com/sun/java/swing/plaf/gtk/Metacity.java` - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant - Missing JDK-8041679: `StringBuffer` is used instead of `StringBuilder` - `jdk/src/share/classes/javax/swing/text/html/HTMLEditorKit.java` - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant - `jdk/src/share/classes/com/sun/media/sound/StandardMidiFileWriter.java` - Missing JDK-8143909: no `Objects.requireNonNull(...)` in this version - `jdk/src/solaris/classes/sun/awt/motif/MFontConfiguration.java` - Missing JDK-8054834: imports from `sun.font` package are still needed in this file - `jdk/src/share/classes/com/sun/imageio/plugins/png/PNGImageReader.java` - Missing JDK-8080835: use `IOUtils.readAllBytes(InputStream is)` instead of the missing `InputStream.readAllBytes()` - Missing JDK-7196163: try-with-resources cannot be used with pre-existing variables ? fell back to try-finally in such cases Testing: GitHub CI, `jdk_imageio` test group locally (headless linux/arm64) ------------- Commit messages: - Use IOUtils.readAllBytes(...) - Fall back to try-finally where pre-existing variables are used - Bring back required imports - Resolve conflicts - 8274893: Update java.desktop classes to use try-with-resources Changes: https://git.openjdk.org/jdk8u-dev/pull/686/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=686&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8274893 Stats: 205 lines in 23 files changed: 32 ins; 79 del; 94 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/686.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/686/head:pull/686 PR: https://git.openjdk.org/jdk8u-dev/pull/686 From tpushkin at openjdk.org Mon Sep 1 07:50:10 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 07:50:10 GMT Subject: [jdk8u-dev] RFR: 8347911: Limit the length of inflated text chunks Message-ID: <-2zwBZ1FTUa13C6m980zm5KlZvlGKXsXv03jX4h1IA0=.0a4f66da-8a44-4ded-a65d-3ed400a8a2cf@github.com> Backport of [JDK-8347911](https://bugs.openjdk.org/browse/JDK-8347911) for parity with Oracle's JDK 8u461. Conflicts: - Copyright years - Missing JDK-8139206: use `IOUtils.readNBytes(...)` instead of the missing `InputStream.readNBytes(...)` Testing: GitHub CI, jdk_imageio test group locally (headless linux/arm64) ------------- Depends on: https://git.openjdk.org/jdk8u-dev/pull/686 Commit messages: - 8347911: Limit the length of inflated text chunks Changes: https://git.openjdk.org/jdk8u-dev/pull/687/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=687&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8347911 Stats: 3 lines in 1 file changed: 1 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/687.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/687/head:pull/687 PR: https://git.openjdk.org/jdk8u-dev/pull/687 From sgehwolf at openjdk.org Mon Sep 1 08:22:52 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Mon, 1 Sep 2025 08:22:52 GMT Subject: [jdk8u-dev] Integrated: 8361212: Remove AffirmTrust root CAs In-Reply-To: References: Message-ID: On Wed, 27 Aug 2025 12:55:57 GMT, Severin Gehwolf wrote: > Backport of JDK-8361212 to remove expired certificates. The patch is almost clean as compared to the JDK 11 version (modulo path suffeling). > > Testing: > > Passed: sun/security/lib/cacerts/VerifyCACerts.java > FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#amazonrootca1 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#amazonrootca2 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#amazonrootca3 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#amazonrootca4 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#buypassclass2ca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#buypassclass3ca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#certainlyroote1 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#certainlyrootr1 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#certignarootca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#comodoeccca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#comodorsaca > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#digicerttlseccrootg5 > FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#digicerttlsrsarootg5 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#emsigneccrootcag3 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#emsignrootcag1 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcaec1 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#globalsigne46 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#globalsigneccrootcar4 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#globalsignr46 > Passed: security/infra/java/security/cert/CertPathValidator/certification/CAIntero... This pull request has now been integrated. Changeset: 3f2b6ffe Author: Severin Gehwolf URL: https://git.openjdk.org/jdk8u-dev/commit/3f2b6ffee4b2a077f5fc8e00d58bce906a80cca0 Stats: 503 lines in 12 files changed: 0 ins; 491 del; 12 mod 8361212: Remove AffirmTrust root CAs Reviewed-by: andrew Backport-of: 5173435bfd0db5db0113a6209f2f827b16598596 ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/684 From aturbanov at openjdk.org Mon Sep 1 10:42:48 2025 From: aturbanov at openjdk.org (Andrey Turbanov) Date: Mon, 1 Sep 2025 10:42:48 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources In-Reply-To: References: Message-ID: <59KlsnLyi4hp_QlWRNRE9nKnFTAvXm_4qvXuCb61id4=.e48d03bb-f635-42ba-8315-39ad58651a39@github.com> On Mon, 1 Sep 2025 07:36:51 GMT, Timofei Pushkin wrote: > Backport of [JDK-8274893](https://bugs.openjdk.org/browse/JDK-8274893) for parity with Oracle's JDK 8u461. > > The first commit is a cherry-pick of the backport to JDK 11 with conflicts unresolved, the second resolves the conflicts, the rest resolve issues uncaught by git. > > Resolved: > - `jdk/src/share/classes/com/sun/java/swing/plaf/gtk/Metacity.java` > - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant > - Missing JDK-8041679: `StringBuffer` is used instead of `StringBuilder` > - `jdk/src/share/classes/javax/swing/text/html/HTMLEditorKit.java` > - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant > - `jdk/src/share/classes/com/sun/media/sound/StandardMidiFileWriter.java` > - Missing JDK-8143909: no `Objects.requireNonNull(...)` in this version > - `jdk/src/solaris/classes/sun/awt/motif/MFontConfiguration.java` > - Missing JDK-8054834: imports from `sun.font` package are still needed in this file > - `jdk/src/share/classes/com/sun/imageio/plugins/png/PNGImageReader.java` > - Missing JDK-8080835: use `IOUtils.readAllBytes(InputStream is)` instead of the missing `InputStream.readAllBytes()` > - Missing JDK-7196163: try-with-resources cannot be used with pre-existing variables ? fell back to try-finally in such cases > > Testing: GitHub CI, `jdk_imageio` test group locally (headless linux/arm64) jdk/src/share/classes/com/sun/media/sound/JARSoundbankReader.java line 91: > 89: if (stream == null) > 90: return null; > 91: try { There should be try-with-resources ------------- PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/686#discussion_r2313598761 From tpushkin at openjdk.org Mon Sep 1 11:02:50 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 11:02:50 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources In-Reply-To: <59KlsnLyi4hp_QlWRNRE9nKnFTAvXm_4qvXuCb61id4=.e48d03bb-f635-42ba-8315-39ad58651a39@github.com> References: <59KlsnLyi4hp_QlWRNRE9nKnFTAvXm_4qvXuCb61id4=.e48d03bb-f635-42ba-8315-39ad58651a39@github.com> Message-ID: On Mon, 1 Sep 2025 10:40:38 GMT, Andrey Turbanov wrote: >> Backport of [JDK-8274893](https://bugs.openjdk.org/browse/JDK-8274893) for parity with Oracle's JDK 8u461. >> >> The first commit is a cherry-pick of the backport to JDK 11 with conflicts unresolved, the second resolves the conflicts, the rest resolve issues uncaught by git. >> >> Resolved: >> - `jdk/src/share/classes/com/sun/java/swing/plaf/gtk/Metacity.java` >> - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant >> - Missing JDK-8041679: `StringBuffer` is used instead of `StringBuilder` >> - `jdk/src/share/classes/javax/swing/text/html/HTMLEditorKit.java` >> - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant >> - `jdk/src/share/classes/com/sun/media/sound/StandardMidiFileWriter.java` >> - Missing JDK-8143909: no `Objects.requireNonNull(...)` in this version >> - `jdk/src/solaris/classes/sun/awt/motif/MFontConfiguration.java` >> - Missing JDK-8054834: imports from `sun.font` package are still needed in this file >> - `jdk/src/share/classes/com/sun/imageio/plugins/png/PNGImageReader.java` >> - Missing JDK-8080835: use `IOUtils.readAllBytes(InputStream is)` instead of the missing `InputStream.readAllBytes()` >> - Missing JDK-7196163: try-with-resources cannot be used with pre-existing variables ? fell back to try-finally in such cases >> >> Testing: GitHub CI, `jdk_imageio` test group locally (headless linux/arm64) > > jdk/src/share/classes/com/sun/media/sound/JARSoundbankReader.java line 91: > >> 89: if (stream == null) >> 90: return null; >> 91: try { > > There should be try-with-resources There was in the original change but in this JDK try-with-resources on an existing variable is not possible (no JDK-7196163), so I fell back to try-finally. This is not the only place where I've done this, see [832c3c3](https://github.com/openjdk/jdk8u-dev/pull/686/commits/832c3c3bb4c132b025e7d3043b40be0d2d025a95). Alternatives are to either create a copy variable for try-with-resources (which also must be used inside it to prevent a compiler warning) or move all interactions with the variable inside try-with-resources ? both diverge code from the original change and the pre-existing code so I decided against them. ------------- PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/686#discussion_r2313639165 From aturbanov at openjdk.org Mon Sep 1 11:53:48 2025 From: aturbanov at openjdk.org (Andrey Turbanov) Date: Mon, 1 Sep 2025 11:53:48 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources In-Reply-To: References: <59KlsnLyi4hp_QlWRNRE9nKnFTAvXm_4qvXuCb61id4=.e48d03bb-f635-42ba-8315-39ad58651a39@github.com> Message-ID: On Mon, 1 Sep 2025 11:00:23 GMT, Timofei Pushkin wrote: >> jdk/src/share/classes/com/sun/media/sound/JARSoundbankReader.java line 91: >> >>> 89: if (stream == null) >>> 90: return null; >>> 91: try { >> >> There should be try-with-resources > > There was in the original change but in this JDK try-with-resources on an existing variable is not possible (no JDK-7196163), so I fell back to try-finally. This is not the only place where I've done this, see [832c3c3](https://github.com/openjdk/jdk8u-dev/pull/686/commits/832c3c3bb4c132b025e7d3043b40be0d2d025a95). > > Alternatives are to either create a copy variable for try-with-resources (which also must be used inside it to prevent a compiler warning) or move all interactions with the variable inside try-with-resources ? both diverge code from the original change and the pre-existing code so I decided against them. Shouldn't we revert formatting changes here, then? ------------- PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/686#discussion_r2313743255 From tpushkin at openjdk.org Mon Sep 1 12:54:33 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 12:54:33 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources [v2] In-Reply-To: References: Message-ID: > Backport of [JDK-8274893](https://bugs.openjdk.org/browse/JDK-8274893) for parity with Oracle's JDK 8u461. > > The first commit is a cherry-pick of the backport to JDK 11 with conflicts unresolved, the second resolves the conflicts, the rest resolve issues uncaught by git. > > Resolved: > - `jdk/src/share/classes/com/sun/java/swing/plaf/gtk/Metacity.java` > - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant > - Missing JDK-8041679: `StringBuffer` is used instead of `StringBuilder` > - `jdk/src/share/classes/javax/swing/text/html/HTMLEditorKit.java` > - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant > - `jdk/src/share/classes/com/sun/media/sound/StandardMidiFileWriter.java` > - Missing JDK-8143909: no `Objects.requireNonNull(...)` in this version > - `jdk/src/solaris/classes/sun/awt/motif/MFontConfiguration.java` > - Missing JDK-8054834: imports from `sun.font` package are still needed in this file > - `jdk/src/share/classes/com/sun/imageio/plugins/png/PNGImageReader.java` > - Missing JDK-8080835: use `IOUtils.readAllBytes(InputStream is)` instead of the missing `InputStream.readAllBytes()` > - Missing JDK-7196163: try-with-resources cannot be used with pre-existing variables ? fell back to try-finally in such cases > > Testing: GitHub CI, `jdk_imageio` test group locally (headless linux/arm64) Timofei Pushkin has updated the pull request incrementally with one additional commit since the last revision: Bring back old formatting in JARSoundbankReader ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/686/files - new: https://git.openjdk.org/jdk8u-dev/pull/686/files/605f1e88..3fd18e4f Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=686&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=686&range=00-01 Stats: 5 lines in 1 file changed: 3 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/686.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/686/head:pull/686 PR: https://git.openjdk.org/jdk8u-dev/pull/686 From tpushkin at openjdk.org Mon Sep 1 12:54:33 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 12:54:33 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources [v2] In-Reply-To: References: <59KlsnLyi4hp_QlWRNRE9nKnFTAvXm_4qvXuCb61id4=.e48d03bb-f635-42ba-8315-39ad58651a39@github.com> Message-ID: <5NwRHMhOEAw11NFUW9mWvEThvNF7uHgUeqdQD1PQK-s=.2351e95e-95f6-465a-a296-9c2bedf02981@github.com> On Mon, 1 Sep 2025 11:51:18 GMT, Andrey Turbanov wrote: >> There was in the original change but in this JDK try-with-resources on an existing variable is not possible (no JDK-7196163), so I fell back to try-finally. This is not the only place where I've done this, see [832c3c3](https://github.com/openjdk/jdk8u-dev/pull/686/commits/832c3c3bb4c132b025e7d3043b40be0d2d025a95). >> >> Alternatives are to either create a copy variable for try-with-resources (which also must be used inside it to prevent a compiler warning) or move all interactions with the variable inside try-with-resources ? both diverge code from the original change and the pre-existing code so I decided against them. > > Shouldn't we revert formatting changes here, then? I wasn't sure what would be the best here: keeping the existing formatting to changing it to be closer to the backported change ? and went with the latter. Since I don't really have a strong preference, reverted to the old one now as you suggest. ------------- PR Review Comment: https://git.openjdk.org/jdk8u-dev/pull/686#discussion_r2313878810 From evergizova at openjdk.org Mon Sep 1 13:07:32 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Mon, 1 Sep 2025 13:07:32 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. > > GHA testing revealed no regressions, additionally tested manually with J2Ddemo. Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'master' into 8297088 - Backport 01b213899cb8124860441fa26c9652b4a4bff896 ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/677/files - new: https://git.openjdk.org/jdk8u-dev/pull/677/files/7536819c..6f31652b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=677&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=677&range=00-01 Stats: 1740 lines in 97 files changed: 279 ins; 1087 del; 374 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/677.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/677/head:pull/677 PR: https://git.openjdk.org/jdk8u-dev/pull/677 From tpushkin at openjdk.org Mon Sep 1 14:48:49 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 14:48:49 GMT Subject: [jdk8u-dev] RFR: 8274893: Update java.desktop classes to use try-with-resources [v2] In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 12:54:33 GMT, Timofei Pushkin wrote: >> Backport of [JDK-8274893](https://bugs.openjdk.org/browse/JDK-8274893) for parity with Oracle's JDK 8u461. >> >> The first commit is a cherry-pick of the backport to JDK 11 with conflicts unresolved, the second resolves the conflicts, the rest resolve issues uncaught by git. >> >> Resolved: >> - `jdk/src/share/classes/com/sun/java/swing/plaf/gtk/Metacity.java` >> - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant >> - Missing JDK-8041679: `StringBuffer` is used instead of `StringBuilder` >> - `jdk/src/share/classes/javax/swing/text/html/HTMLEditorKit.java` >> - Missing JDK-8271456: `"ISO-8859-1"` string is used instead of `ISO_8859_1` enum constant >> - `jdk/src/share/classes/com/sun/media/sound/StandardMidiFileWriter.java` >> - Missing JDK-8143909: no `Objects.requireNonNull(...)` in this version >> - `jdk/src/solaris/classes/sun/awt/motif/MFontConfiguration.java` >> - Missing JDK-8054834: imports from `sun.font` package are still needed in this file >> - `jdk/src/share/classes/com/sun/imageio/plugins/png/PNGImageReader.java` >> - Missing JDK-8080835: use `IOUtils.readAllBytes(InputStream is)` instead of the missing `InputStream.readAllBytes()` >> - Missing JDK-7196163: try-with-resources cannot be used with pre-existing variables ? fell back to try-finally in such cases >> >> Testing: GitHub CI, `jdk_imageio` test group locally (headless linux/arm64) > > Timofei Pushkin has updated the pull request incrementally with one additional commit since the last revision: > > Bring back old formatting in JARSoundbankReader I believe test failures are not related ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/686#issuecomment-3242631971 From tpushkin at openjdk.org Mon Sep 1 14:49:48 2025 From: tpushkin at openjdk.org (Timofei Pushkin) Date: Mon, 1 Sep 2025 14:49:48 GMT Subject: [jdk8u-dev] RFR: 8347911: Limit the length of inflated text chunks In-Reply-To: <-2zwBZ1FTUa13C6m980zm5KlZvlGKXsXv03jX4h1IA0=.0a4f66da-8a44-4ded-a65d-3ed400a8a2cf@github.com> References: <-2zwBZ1FTUa13C6m980zm5KlZvlGKXsXv03jX4h1IA0=.0a4f66da-8a44-4ded-a65d-3ed400a8a2cf@github.com> Message-ID: On Mon, 1 Sep 2025 07:44:51 GMT, Timofei Pushkin wrote: > Backport of [JDK-8347911](https://bugs.openjdk.org/browse/JDK-8347911) for parity with Oracle's JDK 8u461. > > Conflicts: > - Copyright years > - Missing JDK-8139206: use `IOUtils.readNBytes(...)` instead of the missing `InputStream.readNBytes(...)` > > Testing: GitHub CI, jdk_imageio test group locally (headless linux/arm64) I believe the test failures are not related ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/687#issuecomment-3242633226 From gnu.andrew at redhat.com Mon Sep 1 15:46:41 2025 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Mon, 1 Sep 2025 16:46:41 +0100 Subject: [RAMPDOWN] 8u472 Now in Rampdown Stage Message-ID: 8u472 is now in rampdown for release in October 2025. jdk8u-dev is CLOSED for commits until openjdk8u482 is added to the OpenJDK bug database and https://bugs.openjdk.org/browse/JDK-8366574 is integrated to begin the 8u482 release cycle. For critical fixes (i.e. regressions or urgent fixes like tzdata & cacerts updates) for 8u472, please file a PR against https://github.com/openjdk/jdk8u and use jdk8u-critical-request to obtain approval to push (automatic with the /approval request command [0]). [0] https://wiki.openjdk.org/display/SKARA/Pull+Request+Commands#PullRequestCommands-/approval Thanks, -- Andrew :) Pronouns: he / him or they / them Red Hat, Inc. (http://www.redhat.com) PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 Please contact via e-mail, not proprietary chat networks -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 228 bytes Desc: not available URL: From evergizova at openjdk.org Mon Sep 1 15:59:51 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Mon, 1 Sep 2025 15:59:51 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Fri, 29 Aug 2025 22:40:44 GMT, Andrew John Hughes wrote: > Patch itself looks ok. Could you re-run the Windows x86_64 build? It seems to be failing in an odd way (failing to build FreeType) that I can't see is related to the change. @gnu-andrew, thanks for the review. It looks like the problem with Windows builds were fixed by https://github.com/openjdk/jdk8u-dev/commit/41e6bf7d1f6a0b576c46c1f00e2c4cbc89a48a96 All builds have passed successfully after the merge, and the remaining test failures don't seem to be related to the backport: - security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca - gc/concurrentMarkSweep/SystemGCOnForegroundCollector.java - runtime/7158988/FieldMonitor.java ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3242813045 From andrew at openjdk.org Mon Sep 1 16:23:04 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Mon, 1 Sep 2025 16:23:04 GMT Subject: [jdk8u-dev] RFR: 8366574: Bump update version of OpenJDK: 8u482 Message-ID: Rampdown for 8u472 [has begun](https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020424.html). 8u-dev needs to transition to 8u482. PR will also need to wait on `openjdk8u482` being added to the bug database. ------------- Commit messages: - 8366574: Bump update version of OpenJDK: 8u482 Changes: https://git.openjdk.org/jdk8u-dev/pull/688/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=688&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8366574 Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/688.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/688/head:pull/688 PR: https://git.openjdk.org/jdk8u-dev/pull/688 From sgehwolf at openjdk.org Mon Sep 1 16:53:48 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Mon, 1 Sep 2025 16:53:48 GMT Subject: [jdk8u-dev] RFR: 8366574: Bump update version of OpenJDK: 8u482 In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 16:19:27 GMT, Andrew John Hughes wrote: > Rampdown for 8u472 [has begun](https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020424.html). 8u-dev needs to transition to 8u482. > > PR will also need to wait on `openjdk8u482` being added to the bug database. LGTM ------------- Marked as reviewed by sgehwolf (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/688#pullrequestreview-3174214951 From ktakakuri at openjdk.org Tue Sep 2 09:16:32 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Tue, 2 Sep 2025 09:16:32 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v6] In-Reply-To: References: Message-ID: > This is a backport of JDK-8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. > > Applying the JDK-8154043 fix as is will result in a regression of JDK-8182577. > The fix of JDK-8182577 adds an interface for JDK10, therefore this fix cannot be backported simply for JDK8u. > So, I propose to judge the buttonModel is an instance of DefaultButtonModel. > > Testing: > java/awt > javax/swing > ButtonGroupLayoutTraversalTest.java > bug8033699.java > DefaultButtonModelCrashTest.java > on Windows x86_64 Kazuhisa Takakuri has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision: - Merge branch 'openjdk:master' into 8154043 - Merge branch 'master' into 8154043 - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f - Fix the copyright years - Merge branch 'openjdk:master' into 8154043 - Chmod ButtonGroupLayoutTraversalTest - Chmod DefaultButtonModelCrashTest - Fix the copyright year - Backport f3abf05b31893b9a066a436e2c4a587f8899ab10 - ... and 1 more: https://git.openjdk.org/jdk8u-dev/compare/62ac9384...df933477 ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/285/files - new: https://git.openjdk.org/jdk8u-dev/pull/285/files/004551a0..df933477 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=285&range=05 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=285&range=04-05 Stats: 1740 lines in 97 files changed: 279 ins; 1087 del; 374 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/285.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/285/head:pull/285 PR: https://git.openjdk.org/jdk8u-dev/pull/285 From shade at openjdk.org Tue Sep 2 12:20:50 2025 From: shade at openjdk.org (Aleksey Shipilev) Date: Tue, 2 Sep 2025 12:20:50 GMT Subject: [jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash In-Reply-To: References: Message-ID: On Thu, 14 Aug 2025 10:25:41 GMT, Kerem Kat wrote: > Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). > > The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. > > I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. > > For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. > > The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. This looks fine. Like the backports to other releases, this looks like a nice safety net. But yes, @gnu-andrew or @jerboaa would need to approve it for JDK 8. ------------- Marked as reviewed by shade (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/674#pullrequestreview-3176289185 From sgehwolf at openjdk.org Tue Sep 2 12:26:49 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 2 Sep 2025 12:26:49 GMT Subject: [jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash In-Reply-To: References: Message-ID: On Thu, 14 Aug 2025 10:25:41 GMT, Kerem Kat wrote: > Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). > > The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. > > I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. > > For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. > > The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. jdk8u-dev is currently [closed for pushes](https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020424.html). I'll approve once JDK-8366574 is in. In the meantime please merge latest master to avoid the windows build failure in CI. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/674#issuecomment-3245101421 From krk at openjdk.org Tue Sep 2 15:34:08 2025 From: krk at openjdk.org (Kerem Kat) Date: Tue, 2 Sep 2025 15:34:08 GMT Subject: [jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash [v2] In-Reply-To: References: Message-ID: > Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). > > The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. > > I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. > > For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. > > The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. Kerem Kat has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: - Merge remote-tracking branch 'upstream/master' into JDK-8212155 - Fix test for JDK 8 - Backport 64ec8b3e5c8a8d44c92591710d73b833f13c1500 ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/674/files - new: https://git.openjdk.org/jdk8u-dev/pull/674/files/6dac15dc..bb97234c Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=674&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=674&range=00-01 Stats: 1741 lines in 98 files changed: 279 ins; 1087 del; 375 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/674.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/674/head:pull/674 PR: https://git.openjdk.org/jdk8u-dev/pull/674 From serb at openjdk.org Tue Sep 2 22:14:52 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Tue, 2 Sep 2025 22:14:52 GMT Subject: [jdk8u-dev] RFR: 8345358: Some DLL Files are missing Windows Properties [v3] In-Reply-To: <-fy54zhS5ozkNT-AEjzfN4A97IzGt1IHuYXVzEpBWS0=.0641651e-e085-4e95-9835-b2495e469d5d@github.com> References: <-fy54zhS5ozkNT-AEjzfN4A97IzGt1IHuYXVzEpBWS0=.0641651e-e085-4e95-9835-b2495e469d5d@github.com> Message-ID: On Wed, 20 Aug 2025 16:29:55 GMT, Alexey Bakhtin wrote: >> Add missing properties for the j2gss.dll and sspi_bridge.dll files > > Alexey Bakhtin has updated the pull request incrementally with one additional commit since the last revision: > > Add CheckWindowsProperty test Marked as reviewed by serb (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk8u-dev/pull/611#pullrequestreview-3178291736 From sgehwolf at openjdk.org Wed Sep 3 08:27:54 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Wed, 3 Sep 2025 08:27:54 GMT Subject: [jdk8u-dev] RFR: 8366574: Bump update version of OpenJDK: 8u482 In-Reply-To: References: Message-ID: <0G6MQ5LodM7tUa9zIp-S12gFiX19K3lJ0zQ3k_Xfe18=.7f7830d9-e373-451b-b833-b56c97fd5e3d@github.com> On Mon, 1 Sep 2025 16:19:27 GMT, Andrew John Hughes wrote: > Rampdown for 8u472 [has begun](https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020424.html). 8u-dev needs to transition to 8u482. > > PR will also need to wait on `openjdk8u482` being added to the bug database. According to my info the version is now there. Please apply for approval. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/688#issuecomment-3248202187 From andrew at openjdk.org Wed Sep 3 08:57:45 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 3 Sep 2025 08:57:45 GMT Subject: [jdk8u-dev] RFR: 8366574: Bump update version of OpenJDK: 8u482 In-Reply-To: <0G6MQ5LodM7tUa9zIp-S12gFiX19K3lJ0zQ3k_Xfe18=.7f7830d9-e373-451b-b833-b56c97fd5e3d@github.com> References: <0G6MQ5LodM7tUa9zIp-S12gFiX19K3lJ0zQ3k_Xfe18=.7f7830d9-e373-451b-b833-b56c97fd5e3d@github.com> Message-ID: On Wed, 3 Sep 2025 08:24:32 GMT, Severin Gehwolf wrote: > According to my info the version is now there. Please apply for approval. Confirmed on the bug. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/688#issuecomment-3248311190 From krk at openjdk.org Wed Sep 3 09:35:50 2025 From: krk at openjdk.org (Kerem Kat) Date: Wed, 3 Sep 2025 09:35:50 GMT Subject: [jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash [v2] In-Reply-To: References: Message-ID: <_r5Ams5cmb4dWTD1MqSnCkqpGkQwbi5nES63iCmx6y4=.f55cb998-e537-4f62-b292-45dcb0ed54c4@github.com> On Tue, 2 Sep 2025 15:34:08 GMT, Kerem Kat wrote: >> Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). >> >> The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. >> >> I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. >> >> For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. >> >> The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. > > Kerem Kat has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into JDK-8212155 > - Fix test for JDK 8 > - Backport 64ec8b3e5c8a8d44c92591710d73b833f13c1500 After merge from master, the windows build error is gone, thanks for the suggestion. New failures seem unrelated to the current PR: ### CAInterop There are `CAInterop.java` failures in different architectures, similar to https://github.com/openjdk/jdk8u-dev/pull/673. The CA roots in question are `actalisauthenticationrootca` and in some cases `entrustrootcag4`. ### macOS x64 jdk/tier1 multiple failures with "Target VM failed to initialize", similar to the closed https://bugs.openjdk.org/browse/JDK-8286193 ### GC segfault Segfault in `CompactibleFreeListSpace::block_size` on Linux x86 hotspot/tier1 gc/6581734/Test6581734.java is tracked in [JDK-8332853](https://bugs.openjdk.org/browse/JDK-8332853). ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/674#issuecomment-3248467005 From andrew at openjdk.org Wed Sep 3 10:51:52 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 3 Sep 2025 10:51:52 GMT Subject: [jdk8u-dev] Integrated: 8366574: Bump update version of OpenJDK: 8u482 In-Reply-To: References: Message-ID: <6d8ii8NL57Z-_V2BSIB1d61l8Mg9taS_u5b3qUVceZQ=.c73390f8-6d7f-4aff-821b-ab4d409d2454@github.com> On Mon, 1 Sep 2025 16:19:27 GMT, Andrew John Hughes wrote: > Rampdown for 8u472 [has begun](https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020424.html). 8u-dev needs to transition to 8u482. > > PR will also need to wait on `openjdk8u482` being added to the bug database. This pull request has now been integrated. Changeset: c0a36d8f Author: Andrew John Hughes URL: https://git.openjdk.org/jdk8u-dev/commit/c0a36d8fbe59b6fcd878cf14fed2c28c1c30c6eb Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod 8366574: Bump update version of OpenJDK: 8u482 Reviewed-by: sgehwolf ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/688 From duke at openjdk.org Wed Sep 3 12:10:47 2025 From: duke at openjdk.org (duke) Date: Wed, 3 Sep 2025 12:10:47 GMT Subject: [jdk8u-dev] RFR: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash [v2] In-Reply-To: References: Message-ID: On Tue, 2 Sep 2025 15:34:08 GMT, Kerem Kat wrote: >> Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). >> >> The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. >> >> I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. >> >> For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. >> >> The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. > > Kerem Kat has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: > > - Merge remote-tracking branch 'upstream/master' into JDK-8212155 > - Fix test for JDK 8 > - Backport 64ec8b3e5c8a8d44c92591710d73b833f13c1500 @krk Your change (at version bb97234c5ad464aa2597eac33c7922d0d3513c32) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/674#issuecomment-3248977129 From krk at openjdk.org Wed Sep 3 12:15:51 2025 From: krk at openjdk.org (Kerem Kat) Date: Wed, 3 Sep 2025 12:15:51 GMT Subject: [jdk8u-dev] Integrated: 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash In-Reply-To: References: Message-ID: On Thu, 14 Aug 2025 10:25:41 GMT, Kerem Kat wrote: > Backport of [JDK-8212155](https://bugs.openjdk.org/browse/JDK-8212155). > > The patch did not apply cleanly due to a merge conflict. The original code `JvmtiThreadState::state_for(JavaThread::current())` had already been refactored to `JavaThread::current()->jvmti_thread_state()` in this branch. > > I have also removed the `@requires vm.jvmti` tag from the test, as it is not supported by the jtreg version in JDK 11. > > For JDK 8, `DynamicCodeGeneratedTest.sh` was added as the runner, which compiles the library first. > > The `DynamicCodeGeneratedTest` was confirmed to sometimes crash without this patch and passes with it. This pull request has now been integrated. Changeset: 942d6d06 Author: Kerem Kat Committer: Aleksey Shipilev URL: https://git.openjdk.org/jdk8u-dev/commit/942d6d06140b6bfc6fc1daff2f4883e84049b09a Stats: 201 lines in 4 files changed: 194 ins; 0 del; 7 mod 8212155: Race condition when posting dynamic_code_generated event leads to JVM crash Reviewed-by: phh, shade Backport-of: 64ec8b3e5c8a8d44c92591710d73b833f13c1500 ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/674 From gnu.andrew at redhat.com Wed Sep 3 15:03:20 2025 From: gnu.andrew at redhat.com (Andrew Hughes) Date: Wed, 3 Sep 2025 16:03:20 +0100 Subject: [RAMPDOWN] 8u472 Now in Rampdown Stage In-Reply-To: References: Message-ID: On 16:46 Mon 01 Sep , Andrew Hughes wrote: > 8u472 is now in rampdown for release in October 2025. > > jdk8u-dev is CLOSED for commits until openjdk8u482 is added to the > OpenJDK bug database and https://bugs.openjdk.org/browse/JDK-8366574 > is integrated to begin the 8u482 release cycle. > > For critical fixes (i.e. regressions or urgent fixes like tzdata & > cacerts updates) for 8u472, please file a PR against > https://github.com/openjdk/jdk8u and use jdk8u-critical-request to > obtain approval to push (automatic with the /approval request command > [0]). > > [0] https://wiki.openjdk.org/display/SKARA/Pull+Request+Commands#PullRequestCommands-/approval > > Thanks, > -- > Andrew :) > Pronouns: he / him or they / them > Red Hat, Inc. (http://www.redhat.com) > > PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) > Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 > > Please contact via e-mail, not proprietary chat networks jdk8u-dev is now open again. Thanks, -- Andrew :) Pronouns: he / him or they / them Red Hat, Inc. (http://www.redhat.com) PGP Key: ed25519/0xCFDA0F9B35964222 (hkp://keys.gnupg.net) Fingerprint = 5132 579D D154 0ED2 3E04 C5A0 CFDA 0F9B 3596 4222 Please contact via e-mail, not proprietary chat networks -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 228 bytes Desc: not available URL: From jjiang at openjdk.org Wed Sep 3 15:40:33 2025 From: jjiang at openjdk.org (John Jiang) Date: Wed, 3 Sep 2025 15:40:33 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent Message-ID: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. This backport is not clean, because most files don't exist in jdk8u, and the copyright years in the following files are different between the repos. - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java ------------- Commit messages: - Backport 4c9eaddaef83c6ba30e27ae3e0d16caeeec206cb Changes: https://git.openjdk.org/jdk8u-dev/pull/689/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=689&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8364597 Stats: 10 lines in 7 files changed: 0 ins; 3 del; 7 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/689.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/689/head:pull/689 PR: https://git.openjdk.org/jdk8u-dev/pull/689 From andrew at openjdk.org Wed Sep 3 15:57:49 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 3 Sep 2025 15:57:49 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 13:07:32 GMT, Ekaterina Vergizova wrote: >> I'd like to backport it for parity with Oracle 8u. >> 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. >> I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. >> >> GHA testing revealed no regressions, additionally tested manually with J2Ddemo. > > Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into 8297088 > - Backport 01b213899cb8124860441fa26c9652b4a4bff896 Issues with Windows build resolved. > > Patch itself looks ok. Could you re-run the Windows x86_64 build? It seems to be failing in an odd way (failing to build FreeType) that I can't see is related to the change. > > @gnu-andrew, thanks for the review. It looks like the problem with Windows builds were fixed by [41e6bf7](https://github.com/openjdk/jdk8u-dev/commit/41e6bf7d1f6a0b576c46c1f00e2c4cbc89a48a96) > > All builds have passed successfully after the merge, and the remaining test failures don't seem to be related to the backport: > > * security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca > > * gc/concurrentMarkSweep/SystemGCOnForegroundCollector.java > > * runtime/7158988/FieldMonitor.java Thanks for re-testing. It's good to hear that this fixed the build as the relationship between the two wasn't clear from the build failure. 8u482 is open now, so please apply for approval to get this in. ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/677#pullrequestreview-3181305840 PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3249837571 From ktakakuri at openjdk.org Thu Sep 4 12:47:55 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Thu, 4 Sep 2025 12:47:55 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v6] In-Reply-To: References: Message-ID: On Tue, 2 Sep 2025 09:16:32 GMT, Kazuhisa Takakuri wrote: >> This is a backport of JDK-8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. >> >> Applying the JDK-8154043 fix as is will result in a regression of JDK-8182577. >> The fix of JDK-8182577 adds an interface for JDK10, therefore this fix cannot be backported simply for JDK8u. >> So, I propose to judge the buttonModel is an instance of DefaultButtonModel. >> >> Testing: >> java/awt >> javax/swing >> ButtonGroupLayoutTraversalTest.java >> bug8033699.java >> DefaultButtonModelCrashTest.java >> on Windows x86_64 > > Kazuhisa Takakuri has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision: > > - Merge branch 'openjdk:master' into 8154043 > - Merge branch 'master' into 8154043 > - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f > - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f > - Fix the copyright years > - Merge branch 'openjdk:master' into 8154043 > - Chmod ButtonGroupLayoutTraversalTest > - Chmod DefaultButtonModelCrashTest > - Fix the copyright year > - Backport f3abf05b31893b9a066a436e2c4a587f8899ab10 > - ... and 1 more: https://git.openjdk.org/jdk8u-dev/compare/ece02b57...df933477 I have merged the latest master branch and confirmed that there are no problems after running :jdk_awt and :jdk_swing tests on Windows. @jerboaa I previously received feedback from another maintainer on this PR. However, there hasn't been any further response for some time, and I'm wondering if there's an opportunity for it to be reviewed. JDK-8154043's bug is impacting several users' applications. If your schedule permits, would it be possible for you to take a look at this PR? Thanks. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3253529762 From andrew at openjdk.org Thu Sep 4 14:45:50 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 14:45:50 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent In-Reply-To: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: <0JrOxSb-8gcgN0Bdo0IHMh9AmuEA26APxYiHfpoGQZc=.148f492b-593a-4ac1-a61f-41ecd3f2b1b7@github.com> On Wed, 3 Sep 2025 15:35:39 GMT, John Jiang wrote: > Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. > This backport is not clean, because most files don't exist in jdk8u, > and the copyright years in the following files are different between the repos. > - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java > - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java It looks like you are not running the GitHub Actions tests on your fork. While I don't expect any regression from this change, as it only alters comments, you should enable this for future backports at https://github.com/johnshajiang/jdk8u-dev/actions ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/689#issuecomment-3254069842 From andrew at openjdk.org Thu Sep 4 17:09:46 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 17:09:46 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent In-Reply-To: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: <0W4rotd1ZUKdrpjUuMBoVA8WLoZLdnY_BWmt57J4ee0=.d6dce0a4-187a-47d5-b8d5-a39df44bedc4@github.com> On Wed, 3 Sep 2025 15:35:39 GMT, John Jiang wrote: > Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. > This backport is not clean, because most files don't exist in jdk8u, > and the copyright years in the following files are different between the repos. > - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java > - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java Rather than treat this as a backport, it seems to make more sense to ensure that all occurrences of 'THL A29' are removed from the code base for each release. As such, along with sanity checking the patch itself, I compared the list of files covered by the patch with those in the repository containing 'THL A29': ~~~ $ grep -r 'THL A29' * 2>&1 | tee /tmp/thl.8u $ diffstat -p0 /tmp/8364597.17u > /tmp/diffstat.8u $ for file in $(cut -d : -f 1 /tmp/thl.8u ); do if ! grep -q ${file} /tmp/diffstat.8u ; then echo "${file} missing"; fi; done ~~~ With this, I confirmed that this change covers all cases. The list was actually so short for this one, that it was clear from observation as well. ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/689#pullrequestreview-3186257457 From serb at openjdk.org Thu Sep 4 17:42:00 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Thu, 4 Sep 2025 17:42:00 GMT Subject: [jdk8u-dev] RFR: 8157758: JDK9 does not compile on Linux with GCC 6.1 because left-shifting a negative number has undefined behavior Message-ID: Hi all, This pull request contains a backport of commit [4abb5e4a](https://github.com/openjdk/jdk/commit/4abb5e4aafe05871a07d510f002a38298fe4dbb2) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. The commit being backported was authored by Alex Henrie on 23 Jun 2016 and was reviewed by Kim Barrett and Max Ockner. Thanks! ------------- Commit messages: - Backport 4abb5e4aafe05871a07d510f002a38298fe4dbb2 Changes: https://git.openjdk.org/jdk8u-dev/pull/682/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=682&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8157758 Stats: 2 lines in 2 files changed: 0 ins; 0 del; 2 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/682.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/682/head:pull/682 PR: https://git.openjdk.org/jdk8u-dev/pull/682 From andrew at openjdk.org Thu Sep 4 21:44:15 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 21:44:15 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v6] In-Reply-To: References: Message-ID: On Tue, 2 Sep 2025 09:16:32 GMT, Kazuhisa Takakuri wrote: >> This is a backport of JDK-8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. >> >> Applying the JDK-8154043 fix as is will result in a regression of JDK-8182577. >> The fix of JDK-8182577 adds an interface for JDK10, therefore this fix cannot be backported simply for JDK8u. >> So, I propose to judge the buttonModel is an instance of DefaultButtonModel. >> >> Testing: >> java/awt >> javax/swing >> ButtonGroupLayoutTraversalTest.java >> bug8033699.java >> DefaultButtonModelCrashTest.java >> on Windows x86_64 > > Kazuhisa Takakuri has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains 11 additional commits since the last revision: > > - Merge branch 'openjdk:master' into 8154043 > - Merge branch 'master' into 8154043 > - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f > - Backport 8d81ec63b2bafc431cbb8572a3e45e76580ab46f > - Fix the copyright years > - Merge branch 'openjdk:master' into 8154043 > - Chmod ButtonGroupLayoutTraversalTest > - Chmod DefaultButtonModelCrashTest > - Fix the copyright year > - Backport f3abf05b31893b9a066a436e2c4a587f8899ab10 > - ... and 1 more: https://git.openjdk.org/jdk8u-dev/compare/71fe4a29...df933477 Good to go. Please apply for approval. ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/285#pullrequestreview-3187333120 From andrew at openjdk.org Thu Sep 4 21:44:29 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 21:44:29 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v3] In-Reply-To: References: Message-ID: <4QBeXvV3hhaKzayUcmL9yf4uwDrGfjSx0CbkfiBHk7g=.01b09ef3-c018-46e6-8901-11172bcb92ec@github.com> On Thu, 13 Feb 2025 23:12:54 GMT, Andrew John Hughes wrote: >> Kazuhisa Takakuri has updated the pull request incrementally with one additional commit since the last revision: >> >> Fix the copyright years > > The associated CSR [JDK-8182695](https://bugs.openjdk.org/browse/JDK-8182695) is for adding the `getModel()` method in [JDK-8182577](https://bugs.openjdk.org/browse/JDK-8182577) but this patch does not add that method (and should not). It is just bringing over the test case and some minor code fixes. > @gnu-andrew This issue is impacting several users' applications. In conjunction with the related backport of 8074883, I would appreciate it if this could be integrated until the October 2025 release. We understand you are busy, but would you be able to review this? Sorry, I thought this had been abandoned. I'm having a look again now, but have to reacquaint myself with it again. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3255799933 From andrew at openjdk.org Thu Sep 4 21:44:36 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 21:44:36 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v3] In-Reply-To: <1zEG61X9kwvXkm75J_ZTz-aOSRdrE9FwEhSDacyiICE=.0d97fea5-08f3-4572-9c96-7f33a7e07997@github.com> References: <1zEG61X9kwvXkm75J_ZTz-aOSRdrE9FwEhSDacyiICE=.0d97fea5-08f3-4572-9c96-7f33a7e07997@github.com> Message-ID: <5mHP7qNy4caccu5ggKtfjUhlhTh19TAk2Ef1jRKXum4=.3b418171-1aca-403e-965d-80aa7c7de0df@github.com> On Fri, 6 Jun 2025 07:02:21 GMT, Kazuhisa Takakuri wrote: > > we should also include the correction to the indentation of group.getElements() and presumably the instanceof test? > > Thanks for pointing that out. I've corrected the indentations. What does "the instanceof test" mean? I think I just meant this in the 8182577 11u patch: ~~~ - if (member.isVisible() && member.isDisplayable() && - member.isEnabled() && member.isFocusable()) { + if (member instanceof JToggleButton && + member.isVisible() && member.isDisplayable() && + member.isEnabled() && member.isFocusable()) { ~~~ Of course, it doesn't have the `instanceof` test in the 8u version, so sorry for the confusion. Anyway, I compared the 11u version of `LayoutFocusTraversalPolicy.java` with your current patched 8u version and it looks good, so you may have just sorted this one out from the start. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3255823238 From andrew at openjdk.org Thu Sep 4 21:44:42 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 4 Sep 2025 21:44:42 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v4] In-Reply-To: References: Message-ID: On Fri, 22 Aug 2025 21:13:28 GMT, Paul Hohensee wrote: > In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. > > Also, please enable GHA on this repo. Good catch. 8226892 was already backported with a later copyright date on this file. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3255826326 From serb at openjdk.org Fri Sep 5 03:37:48 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Fri, 5 Sep 2025 03:37:48 GMT Subject: [jdk8u-dev] RFR: 8157758: JDK9 does not compile on Linux with GCC 6.1 because left-shifting a negative number has undefined behavior [v2] In-Reply-To: References: Message-ID: > Hi all, > > This pull request contains a backport of commit [4abb5e4a](https://github.com/openjdk/jdk/commit/4abb5e4aafe05871a07d510f002a38298fe4dbb2) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit being backported was authored by Alex Henrie on 23 Jun 2016 and was reviewed by Kim Barrett and Max Ockner. > > Thanks! Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'openjdk:master' into backport-mrserb-4abb5e4a-master - Backport 4abb5e4aafe05871a07d510f002a38298fe4dbb2 ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/682/files - new: https://git.openjdk.org/jdk8u-dev/pull/682/files/a4ea7dfb..ba79b13c Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=682&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=682&range=00-01 Stats: 1943 lines in 103 files changed: 473 ins; 1087 del; 383 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/682.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/682/head:pull/682 PR: https://git.openjdk.org/jdk8u-dev/pull/682 From ktakakuri at openjdk.org Fri Sep 5 15:02:28 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Fri, 5 Sep 2025 15:02:28 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v4] In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 21:39:04 GMT, Andrew John Hughes wrote: >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > > Good catch. 8226892 was already backported with a later copyright date on this file. @gnu-andrew Thank you for your review. I'm planning to reopen and work on https://github.com/openjdk/jdk8u-dev/pull/212 related to this PR soon, so I'd appreciate it if you could review that as well when the time comes. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3258662383 From ktakakuri at openjdk.org Fri Sep 5 15:15:23 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Fri, 5 Sep 2025 15:15:23 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v4] In-Reply-To: References: Message-ID: On Thu, 4 Sep 2025 21:39:04 GMT, Andrew John Hughes wrote: >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > > Good catch. 8226892 was already backported with a later copyright date on this file. @gnu-andrew JDK-8154043 has already been labeled with jdk8u-fix-no, and I can't submit the request again. I want to resubmit it. How should I proceed? ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3258715994 From jkratochvil at openjdk.org Sat Sep 6 20:16:53 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Sat, 6 Sep 2025 20:16:53 GMT Subject: [jdk8u-dev] RFR: 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag Message-ID: <9OU59-FHZ2M70rEbLCoTEySTkJO7Js9yIF8f_UuGNlo=.344b0a1f-836b-433b-9d4e-b56bad4b5f23@github.com> It is needed in jdk8 as a prerequisite for JDK-6986863. A testcase is included. It is not a clean backport - source files are moved, include files angle vs. quote change, copyright years update. This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. ------------- Commit messages: - 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag Changes: https://git.openjdk.org/jdk8u-dev/pull/690/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=690&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8263622 Stats: 133 lines in 2 files changed: 88 ins; 35 del; 10 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/690.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/690/head:pull/690 PR: https://git.openjdk.org/jdk8u-dev/pull/690 From jkratochvil at openjdk.org Sat Sep 6 20:29:52 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Sat, 6 Sep 2025 20:29:52 GMT Subject: [jdk8u-dev] RFR: 6986863: ProfileDeferralMgr throwing ConcurrentModificationException Message-ID: It is needed in jdk8 as a prerequisite for JDK-8347377. A testcase is included. It is not a clean backport - source files are moved, copyright years update, original patch uses extra function getStandardProfileInputStream(), Anton Voznia hasn't implemented it by himself, he borrowed it from another patch https://bugs.openjdk.org/browse/JDK-8039271 and changed it a bit according to the requirements. He didn't include entire fix because it's redundant. This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. It requires https://github.com/openjdk/jdk8u-dev/pull/690 as a prerequisite. ------------- Depends on: https://git.openjdk.org/jdk8u-dev/pull/690 Commit messages: - 6986863: ProfileDeferralMgr throwing ConcurrentModificationException Changes: https://git.openjdk.org/jdk8u-dev/pull/691/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=691&range=00 Issue: https://bugs.openjdk.org/browse/JDK-6986863 Stats: 387 lines in 7 files changed: 204 ins; 115 del; 68 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/691.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/691/head:pull/691 PR: https://git.openjdk.org/jdk8u-dev/pull/691 From jkratochvil at openjdk.org Sat Sep 6 20:37:00 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Sat, 6 Sep 2025 20:37:00 GMT Subject: [jdk8u-dev] RFR: 8347377: Add validation checks for ICC_Profile header fields Message-ID: ICC_Profile header is validated too late leading later to unexpected exceptions. A testcase is included. It is not a clean backport - it has to change return switch expressions to their old format, also source files are moved, copyright years updated. This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. It requires https://github.com/openjdk/jdk8u-dev/pull/691 as a prerequisite. ------------- Depends on: https://git.openjdk.org/jdk8u-dev/pull/691 Commit messages: - 8347377: Add validation checks for ICC_Profile header fields Changes: https://git.openjdk.org/jdk8u-dev/pull/692/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=692&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8347377 Stats: 347 lines in 4 files changed: 336 ins; 2 del; 9 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/692.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/692/head:pull/692 PR: https://git.openjdk.org/jdk8u-dev/pull/692 From jkratochvil at openjdk.org Sat Sep 6 20:44:04 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Sat, 6 Sep 2025 20:44:04 GMT Subject: [jdk8u-dev] RFR: 8346465: Add a check in setData() to restrict the update of Built-In ICC_Profiles Message-ID: Built-in singleton ICC_Profiles could be modified using setData(). Throw an exception for such an attempt. A testcase is included. It is not a clean backport - it has to change return switch expressions to their old format and there are also other 2 minor conflicts. This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. It requires https://github.com/openjdk/jdk8u-dev/pull/692 as a prerequisite. ------------- Depends on: https://git.openjdk.org/jdk8u-dev/pull/692 Commit messages: - 8346465: Add a check in setData() to restrict the update of Built-In ICC_Profiles Changes: https://git.openjdk.org/jdk8u-dev/pull/693/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=693&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8346465 Stats: 271 lines in 7 files changed: 257 ins; 1 del; 13 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/693.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/693/head:pull/693 PR: https://git.openjdk.org/jdk8u-dev/pull/693 From andrew at openjdk.org Sun Sep 7 14:58:19 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Sun, 7 Sep 2025 14:58:19 GMT Subject: [jdk8u-dev] RFR: 8345358: Some DLL Files are missing Windows Properties In-Reply-To: References: Message-ID: On Mon, 25 Aug 2025 17:10:32 GMT, Alexey Bakhtin wrote: >> keep open > >> LGTM. @mrserb Could you please confirm before ramp down phase on August 29? > > @mrserb could you please approve the review? Without it, I can not make a fix request. Thank you @alexeybakhtin this is just waiting for an approval request. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/611#issuecomment-3263831240 From andrew at openjdk.org Sun Sep 7 15:05:20 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Sun, 7 Sep 2025 15:05:20 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v4] In-Reply-To: References: Message-ID: <1M2JGbEUiySp4wo0UrlEWVCIY1YVzRbm3qptP4FHwcg=.9bca9004-dd6f-41aa-82a6-869e10edf6eb@github.com> On Thu, 4 Sep 2025 21:39:04 GMT, Andrew John Hughes wrote: >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > >> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >> >> Also, please enable GHA on this repo. > > Good catch. 8226892 was already backported with a later copyright date on this file. > @gnu-andrew JDK-8154043 has already been labeled with jdk8u-fix-no, and I can't submit the request again. I want to resubmit it. How should I proceed? I've approved manually on the bugs. This should be good to go. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3263835116 From serb at openjdk.org Sun Sep 7 23:28:12 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Sun, 7 Sep 2025 23:28:12 GMT Subject: [jdk8u-dev] RFR: 8157758: JDK9 does not compile on Linux with GCC 6.1 because left-shifting a negative number has undefined behavior [v2] In-Reply-To: References: Message-ID: On Fri, 5 Sep 2025 03:37:48 GMT, Sergey Bylokhov wrote: >> Hi all, >> >> This pull request contains a backport of commit [4abb5e4a](https://github.com/openjdk/jdk/commit/4abb5e4aafe05871a07d510f002a38298fe4dbb2) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. >> >> The commit being backported was authored by Alex Henrie on 23 Jun 2016 and was reviewed by Kim Barrett and Max Ockner. >> >> Thanks! > > Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'openjdk:master' into backport-mrserb-4abb5e4a-master > - Backport 4abb5e4aafe05871a07d510f002a38298fe4dbb2 The issue is similar to https://github.com/openjdk/jdk11u-dev/pull/3042 ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/682#issuecomment-3264145860 From duke at openjdk.org Mon Sep 8 04:51:21 2025 From: duke at openjdk.org (Min Choi) Date: Mon, 8 Sep 2025 04:51:21 GMT Subject: [jdk8u-dev] RFR: 8345149: javax/sound/midi/Soundbanks/GetSoundBankSecurityException/GetSoundBankSecurityException.java failes with FileSystemException Message-ID: <4c3EWMImHibADVPzaAf7AItE4-rIVPHozAx9AvZLQkI=.860400ce-6565-41fe-9beb-add6835fbaa6@github.com> Added javax/sound/midi/Soundbanks/GetSoundBankSecurityException/GetSoundBankSecurityException.java to ProblemList.txt to exclude on Windows Machines. ------------- Commit messages: - Fix Root Cause (8178401) for JDK-8345149 - Add GetSoundBankSecurityException.java to ProblemList.txt (JDK-8345149) Changes: https://git.openjdk.org/jdk8u-dev/pull/613/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=613&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8345149 Stats: 2 lines in 1 file changed: 2 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/613.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/613/head:pull/613 PR: https://git.openjdk.org/jdk8u-dev/pull/613 From ktakakuri at openjdk.org Mon Sep 8 07:30:22 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Mon, 8 Sep 2025 07:30:22 GMT Subject: [jdk8u-dev] RFR: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. [v4] In-Reply-To: <1M2JGbEUiySp4wo0UrlEWVCIY1YVzRbm3qptP4FHwcg=.9bca9004-dd6f-41aa-82a6-869e10edf6eb@github.com> References: <1M2JGbEUiySp4wo0UrlEWVCIY1YVzRbm3qptP4FHwcg=.9bca9004-dd6f-41aa-82a6-869e10edf6eb@github.com> Message-ID: On Sun, 7 Sep 2025 15:02:24 GMT, Andrew John Hughes wrote: >>> In bug8033699.java, line 2, leave the end copyright date at 2019. Other than that, lgtm. >>> >>> Also, please enable GHA on this repo. >> >> Good catch. 8226892 was already backported with a later copyright date on this file. > >> @gnu-andrew JDK-8154043 has already been labeled with jdk8u-fix-no, and I can't submit the request again. I want to resubmit it. How should I proceed? > > I've approved manually on the bugs. This should be good to go. @gnu-andrew Thank you for your prompt support. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/285#issuecomment-3264957690 From ktakakuri at openjdk.org Mon Sep 8 07:30:23 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Mon, 8 Sep 2025 07:30:23 GMT Subject: [jdk8u-dev] Integrated: 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. In-Reply-To: References: Message-ID: On Wed, 15 Mar 2023 05:47:54 GMT, Kazuhisa Takakuri wrote: > This is a backport of JDK-8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. > > Applying the JDK-8154043 fix as is will result in a regression of JDK-8182577. > The fix of JDK-8182577 adds an interface for JDK10, therefore this fix cannot be backported simply for JDK8u. > So, I propose to judge the buttonModel is an instance of DefaultButtonModel. > > Testing: > java/awt > javax/swing > ButtonGroupLayoutTraversalTest.java > bug8033699.java > DefaultButtonModelCrashTest.java > on Windows x86_64 This pull request has now been integrated. Changeset: 86ecd5bb Author: Kazuhisa Takakuri URL: https://git.openjdk.org/jdk8u-dev/commit/86ecd5bb248026cc056f21c7368e5b322a244d44 Stats: 304 lines in 5 files changed: 270 ins; 15 del; 19 mod 8154043: Fields not reachable anymore by tab-key, because of new tabbing behaviour of radio button groups. 8182577: Exception when Tab key moves focus to a JCheckbox with a custom ButtonModel Reviewed-by: phh, andrew Backport-of: 8d81ec63b2bafc431cbb8572a3e45e76580ab46f ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/285 From jkratochvil at openjdk.org Mon Sep 8 14:18:03 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Mon, 8 Sep 2025 14:18:03 GMT Subject: [jdk8u-dev] RFR: 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag [v2] In-Reply-To: <9OU59-FHZ2M70rEbLCoTEySTkJO7Js9yIF8f_UuGNlo=.344b0a1f-836b-433b-9d4e-b56bad4b5f23@github.com> References: <9OU59-FHZ2M70rEbLCoTEySTkJO7Js9yIF8f_UuGNlo=.344b0a1f-836b-433b-9d4e-b56bad4b5f23@github.com> Message-ID: > It is needed in jdk8 as a prerequisite for JDK-6986863. > A testcase is included. > It is not a clean backport - source files are moved, include files angle vs. quote change, copyright years update. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. Jan Kratochvil has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag Reviewed-by: azvegint ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/690/files - new: https://git.openjdk.org/jdk8u-dev/pull/690/files/50ad5e5e..1fade413 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=690&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=690&range=00-01 Stats: 2 lines in 1 file changed: 1 ins; 1 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/690.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/690/head:pull/690 PR: https://git.openjdk.org/jdk8u-dev/pull/690 From jjiang at openjdk.org Mon Sep 8 14:41:30 2025 From: jjiang at openjdk.org (John Jiang) Date: Mon, 8 Sep 2025 14:41:30 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent [v2] In-Reply-To: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: > Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. > This backport is not clean, because most files don't exist in jdk8u, > and the copyright years in the following files are different between the repos. > - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java > - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java John Jiang has updated the pull request incrementally with one additional commit since the last revision: Trigger CI ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/689/files - new: https://git.openjdk.org/jdk8u-dev/pull/689/files/d25937c3..17c4ba3a Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=689&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=689&range=00-01 Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/689.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/689/head:pull/689 PR: https://git.openjdk.org/jdk8u-dev/pull/689 From sgehwolf at redhat.com Mon Sep 8 16:26:04 2025 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 08 Sep 2025 18:26:04 +0200 Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan Message-ID: I hereby nominate Sendao Yan (syan [1]) to JDK 8 Updates Project[2] committer. Sendao has so far authored 21 PRs [3] that were integrated into the JDK 8 Updates Project. Sendao intends to contribute more. Most of these PRs fix test bugs or make the test more robust. Some of those fixes were non-trivial. Making him committer would help speed up getting those fixes into the JDK 8 update trees faster without the help of a sponsor. Sendao worked as compiler test engineer for several years and works at Hygon now. They will continue to sustains JDK 8 for the next few years. Votes are due by 23:59 CEST on Monday, September 22nd, 2025. Only current JDK 8 Updates Project Committers [4] are eligible to vote on this nomination. Votes must be cast in the open by replying to this mailing list. For Lazy Consensus voting instructions see [5]. Thanks, Severin [1] https://openjdk.org/census#syan [2] https://openjdk.org/projects/jdk8u/ [3] https://github.com/openjdk/jdk8u-dev/issues?q=is%3Apr%20author%3AsendaoYan%20label%3Aintegrated [4] https://openjdk.org/census#jdk8u [5] https://openjdk.org/projects/#committer-vote From serb at openjdk.org Mon Sep 8 16:34:05 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Mon, 8 Sep 2025 16:34:05 GMT Subject: [jdk8u-dev] RFR: 8312518: [macos13] setFullScreenWindow() shows black screen on macOS 13 & above Message-ID: <5u5R1gq0ZcLvJ8y7wI6ci9Q_i9sqFQu-AY2Pn3OLAM4=.f69fe961-48ab-4869-9567-0231e9b524a5@github.com> Hi all, This pull request contains a backport of commit [37123ee7](https://github.com/openjdk/jdk11u-dev/commit/37123ee7a5058ec81f2ef85a68fe9ae43b0ba216) from the [openjdk/jdk11u-dev](https://git.openjdk.org/jdk11u-dev) repository. The commit being backported was authored by Sergey Bylokhov on 27 Aug 2025 and had no reviewers. Thanks! ------------- Commit messages: - Backport 37123ee7a5058ec81f2ef85a68fe9ae43b0ba216 Changes: https://git.openjdk.org/jdk8u-dev/pull/694/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=694&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8312518 Stats: 91 lines in 2 files changed: 90 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/694.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/694/head:pull/694 PR: https://git.openjdk.org/jdk8u-dev/pull/694 From hohensee at amazon.com Mon Sep 8 17:15:52 2025 From: hohensee at amazon.com (Hohensee, Paul) Date: Mon, 8 Sep 2025 17:15:52 +0000 Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan Message-ID: Vote: yes ?On 9/8/25, 9:26 AM, "jdk8u-dev on behalf of Severin Gehwolf" on behalf of sgehwolf at redhat.com > wrote: I hereby nominate Sendao Yan (syan [1]) to JDK 8 Updates Project[2] committer. Sendao has so far authored 21 PRs [3] that were integrated into the JDK 8 Updates Project. Sendao intends to contribute more. Most of these PRs fix test bugs or make the test more robust. Some of those fixes were non-trivial. Making him committer would help speed up getting those fixes into the JDK 8 update trees faster without the help of a sponsor. Sendao worked as compiler test engineer for several years and works at Hygon now. They will continue to sustains JDK 8 for the next few years. Votes are due by 23:59 CEST on Monday, September 22nd, 2025. Only current JDK 8 Updates Project Committers [4] are eligible to vote on this nomination. Votes must be cast in the open by replying to this mailing list. For Lazy Consensus voting instructions see [5]. Thanks, Severin [1] https://openjdk.org/census#syan [2] https://openjdk.org/projects/jdk8u/ [3] https://github.com/openjdk/jdk8u-dev/issues?q=is%3Apr%20author%3AsendaoYan%20label%3Aintegrated [4] https://openjdk.org/census#jdk8u [5] https://openjdk.org/projects/#committer-vote From abakhtin at openjdk.org Mon Sep 8 17:30:19 2025 From: abakhtin at openjdk.org (Alexey Bakhtin) Date: Mon, 8 Sep 2025 17:30:19 GMT Subject: [jdk8u-dev] RFR: 8345358: Some DLL Files are missing Windows Properties In-Reply-To: References: Message-ID: <32SxRQn1gqie6ppqb1IrzNyIYRaL04yp5N2dqv6EsLQ=.b8c3b6b1-65de-4854-9b12-ea4b971abcb3@github.com> On Mon, 25 Aug 2025 17:10:32 GMT, Alexey Bakhtin wrote: >> keep open > >> LGTM. @mrserb Could you please confirm before ramp down phase on August 29? > > @mrserb could you please approve the review? Without it, I can not make a fix request. Thank you > @alexeybakhtin this is just waiting for an approval request. Thank you. Just had a missprint in the label ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/611#issuecomment-3267246837 From serb at openjdk.org Mon Sep 8 17:52:18 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Mon, 8 Sep 2025 17:52:18 GMT Subject: [jdk8u-dev] RFR: 8345149: javax/sound/midi/Soundbanks/GetSoundBankSecurityException/GetSoundBankSecurityException.java failes with FileSystemException In-Reply-To: <4c3EWMImHibADVPzaAf7AItE4-rIVPHozAx9AvZLQkI=.860400ce-6565-41fe-9beb-add6835fbaa6@github.com> References: <4c3EWMImHibADVPzaAf7AItE4-rIVPHozAx9AvZLQkI=.860400ce-6565-41fe-9beb-add6835fbaa6@github.com> Message-ID: <75dcin8ljtgUx-g17j522yLZSOdReyhI4ksB7WChymY=.a99bdd9e-5b75-428e-b31a-d0c9ae112988@github.com> On Tue, 17 Dec 2024 21:05:19 GMT, Min Choi wrote: > Added javax/sound/midi/Soundbanks/GetSoundBankSecurityException/GetSoundBankSecurityException.java > to ProblemList.txt to exclude on Windows Machines. Marked as reviewed by serb (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk8u-dev/pull/613#pullrequestreview-3197507895 From serb at openjdk.org Mon Sep 8 23:23:38 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Mon, 8 Sep 2025 23:23:38 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 13:07:32 GMT, Ekaterina Vergizova wrote: >> I'd like to backport it for parity with Oracle 8u. >> 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. >> I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. >> >> GHA testing revealed no regressions, additionally tested manually with J2Ddemo. > > Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into 8297088 > - Backport 01b213899cb8124860441fa26c9652b4a4bff896 >No new issues found in GHA testing, additionally tested manually with J2Ddemo. This is not enough test coverage, CMM tests are not part of GA testing, you need to run at least headless tests from the jdk_desktop group, or directly: java/awt/color* and sun/java2d/cmm*. This is applicable to all backports for lcms including 11u. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3268325815 From jkratochvil at openjdk.org Tue Sep 9 08:09:00 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 08:09:00 GMT Subject: [jdk8u-dev] RFR: 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag [v2] In-Reply-To: References: <9OU59-FHZ2M70rEbLCoTEySTkJO7Js9yIF8f_UuGNlo=.344b0a1f-836b-433b-9d4e-b56bad4b5f23@github.com> Message-ID: On Mon, 8 Sep 2025 14:18:03 GMT, Jan Kratochvil wrote: >> It is needed in jdk8 as a prerequisite for JDK-6986863. >> A testcase is included. >> It is not a clean backport - source files are moved, include files angle vs. quote change, copyright years update. >> This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > > Jan Kratochvil has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: > > 8263622: The java.awt.color.ICC_Profile#setData invert the order of bytes for the "head" tag > > Reviewed-by: azvegint The CI errors are false positives, they are the same as unpatched sources tested in: https://github.com/jankratochvil/jdk8u-dev/pull/1 ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/690#issuecomment-3269426873 From ktakakuri at openjdk.org Tue Sep 9 09:10:28 2025 From: ktakakuri at openjdk.org (Kazuhisa Takakuri) Date: Tue, 9 Sep 2025 09:10:28 GMT Subject: [jdk8u-dev] RFR: 8074883: Tab key should move to focused button in a button group [v2] In-Reply-To: References: Message-ID: > This is a backport of JDK-8074883: Tab key should move to focused button in a button group. > > I would like to backport the patch to OpenJDK8u. > Original patch does not apply cleanly to 8u, because the fix uses a new API published in JDK9. > > * Since RequestFocusController only determines whether or not to set focus, I modified it so that requestFocus/requestFocusInWindow is called by SwingUtilities.invokeLater() and re-set focus and return false. > Without invokeLater(), the focus returns to the first button in case of Cause.ACTIVATION. > > * ToggleButton.getGroupSelection() is defined as a package private method, because it must be called by JCompoennt.focusController. > > * Calling requestFocus()/requestFocusInWindow() will be processed as Cause.UNKNOWN. > ToggleButton.getGroupSelection() returns itself, so no circular call occurs. > > * I moved Component.requestFocusController.acceptRequestFocus because RequestFocusContoroller is not called when Cause.ACTIVATION. > > * Only Swing components replace default RequestFocusController to JComponent.focusController. > The focusController returns true except for Swing, so this change does not affect other components. > > Testing: > build on Windows x86_64 > java/awt, javax/swing and ButtonGroupFocusTest.java on Windows x86_64 Kazuhisa Takakuri has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains four additional commits since the last revision: - Merge branch 'openjdk:master' into 8074883 - Merge branch 'openjdk:master' into 8074883 - Merge branch 'openjdk:master' into 8074883 - Backport eef0714323de ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/212/files - new: https://git.openjdk.org/jdk8u-dev/pull/212/files/c578963c..6e8415e4 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=212&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=212&range=00-01 Stats: 991449 lines in 2264 files changed: 934269 ins; 48303 del; 8877 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/212.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/212/head:pull/212 PR: https://git.openjdk.org/jdk8u-dev/pull/212 From adinn at redhat.com Tue Sep 9 09:11:22 2025 From: adinn at redhat.com (Andrew Dinn) Date: Tue, 9 Sep 2025 10:11:22 +0100 Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan In-Reply-To: References: Message-ID: Vote: yes On 08/09/2025 17:26, Severin Gehwolf wrote: > I hereby nominate Sendao Yan (syan [1]) to JDK 8 Updates Project[2] > committer. Sendao has so far authored 21 PRs [3] that were integrated > into the JDK 8 Updates Project. Sendao intends to contribute more. Most > of these PRs fix test bugs or make the test more robust. Some of those > fixes were non-trivial. > > Making him committer would help speed up getting those fixes into the > JDK 8 update trees faster without the help of a sponsor. > > Sendao worked as compiler test engineer for several years and works at > Hygon now. They will continue to sustains JDK 8 for the next few years. > > Votes are due by 23:59 CEST on Monday, September 22nd, 2025. > > Only current JDK 8 Updates Project Committers [4] are eligible to vote > on this nomination. Votes must be cast in the open by replying to this > mailing list. For Lazy Consensus voting instructions see [5]. > > Thanks, > Severin > > [1] https://openjdk.org/census#syan > [2] https://openjdk.org/projects/jdk8u/ > [3] https://github.com/openjdk/jdk8u-dev/issues?q=is%3Apr%20author%3AsendaoYan%20label%3Aintegrated > [4] https://openjdk.org/census#jdk8u > [5] https://openjdk.org/projects/#committer-vote > -- regards, Andrew Dinn ----------- Red Hat Distinguished Engineer He/Him/His IBM UK Limited Registered in England and Wales with number 741598 Registered office: Building C, IBM Hursley Office, Hursley Park Road, Winchester, Hampshire SO21 2JN From jkratochvil at openjdk.org Tue Sep 9 10:02:54 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 10:02:54 GMT Subject: [jdk8u-dev] RFR: 6986863: ProfileDeferralMgr throwing ConcurrentModificationException [v2] In-Reply-To: References: Message-ID: > It is needed in jdk8 as a prerequisite for JDK-8347377. > A testcase is included. > It is not a clean backport - source files are moved, copyright years update, original patch uses extra function getStandardProfileInputStream(), Anton Voznia hasn't implemented it by himself, he borrowed it from another patch https://bugs.openjdk.org/browse/JDK-8039271 and changed it a bit according to the requirements. He didn't include entire fix because it's redundant. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/690 as a prerequisite. Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: 6986863: ProfileDeferralMgr throwing ConcurrentModificationException Reviewed-by: kizune ------------- Changes: https://git.openjdk.org/jdk8u-dev/pull/691/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=691&range=01 Stats: 389 lines in 7 files changed: 206 ins; 115 del; 68 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/691.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/691/head:pull/691 PR: https://git.openjdk.org/jdk8u-dev/pull/691 From jkratochvil at openjdk.org Tue Sep 9 10:02:56 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 10:02:56 GMT Subject: [jdk8u-dev] RFR: 6986863: ProfileDeferralMgr throwing ConcurrentModificationException In-Reply-To: References: Message-ID: On Sat, 6 Sep 2025 20:24:12 GMT, Jan Kratochvil wrote: > It is needed in jdk8 as a prerequisite for JDK-8347377. > A testcase is included. > It is not a clean backport - source files are moved, copyright years update, original patch uses extra function getStandardProfileInputStream(), Anton Voznia hasn't implemented it by himself, he borrowed it from another patch https://bugs.openjdk.org/browse/JDK-8039271 and changed it a bit according to the requirements. He didn't include entire fix because it's redundant. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/690 as a prerequisite. The CI errors are false positives, they are the same as unpatched sources tested in: https://github.com/jankratochvil/jdk8u-dev/pull/1 ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/691#issuecomment-3269885759 From jkratochvil at openjdk.org Tue Sep 9 10:43:57 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 10:43:57 GMT Subject: [jdk8u-dev] RFR: 8347377: Add validation checks for ICC_Profile header fields [v2] In-Reply-To: References: Message-ID: > ICC_Profile header is validated too late leading later to unexpected exceptions. > A testcase is included. > It is not a clean backport - it has to change return switch expressions to their old format, also source files are moved, copyright years updated. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/691 as a prerequisite. Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: 8347377: Add validation checks for ICC_Profile header fields Reviewed-by: prr, jdv ------------- Changes: https://git.openjdk.org/jdk8u-dev/pull/692/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=692&range=01 Stats: 347 lines in 4 files changed: 336 ins; 2 del; 9 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/692.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/692/head:pull/692 PR: https://git.openjdk.org/jdk8u-dev/pull/692 From jkratochvil at openjdk.org Tue Sep 9 10:43:57 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 10:43:57 GMT Subject: [jdk8u-dev] RFR: 8347377: Add validation checks for ICC_Profile header fields In-Reply-To: References: Message-ID: On Sat, 6 Sep 2025 20:32:28 GMT, Jan Kratochvil wrote: > ICC_Profile header is validated too late leading later to unexpected exceptions. > A testcase is included. > It is not a clean backport - it has to change return switch expressions to their old format, also source files are moved, copyright years updated. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/691 as a prerequisite. The CI errors are false positives, they are the same as unpatched sources tested in: https://github.com/jankratochvil/jdk8u-dev/pull/1 ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/692#issuecomment-3270071088 From jkratochvil at openjdk.org Tue Sep 9 11:22:41 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 11:22:41 GMT Subject: [jdk8u-dev] RFR: 8346465: Add a check in setData() to restrict the update of Built-In ICC_Profiles [v2] In-Reply-To: References: Message-ID: <5KIg7EDn5Ma1bVda1LxIw78PoBlfVqBpa-O8oWtCPrI=.9def494c-0922-4e68-893f-20bf0b27c228@github.com> > Built-in singleton ICC_Profiles could be modified using setData(). Throw an exception for such an attempt. > A testcase is included. > It is not a clean backport - it has to change return switch expressions to their old format and there are also other 2 minor conflicts. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/692 as a prerequisite. Jan Kratochvil has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: 8346465: Add a check in setData() to restrict the update of Built-In ICC_Profiles Reviewed-by: aivanov, jdv, prr, serb ------------- Changes: https://git.openjdk.org/jdk8u-dev/pull/693/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=693&range=01 Stats: 271 lines in 7 files changed: 257 ins; 1 del; 13 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/693.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/693/head:pull/693 PR: https://git.openjdk.org/jdk8u-dev/pull/693 From jkratochvil at openjdk.org Tue Sep 9 11:22:42 2025 From: jkratochvil at openjdk.org (Jan Kratochvil) Date: Tue, 9 Sep 2025 11:22:42 GMT Subject: [jdk8u-dev] RFR: 8346465: Add a check in setData() to restrict the update of Built-In ICC_Profiles In-Reply-To: References: Message-ID: On Sat, 6 Sep 2025 20:38:02 GMT, Jan Kratochvil wrote: > Built-in singleton ICC_Profiles could be modified using setData(). Throw an exception for such an attempt. > A testcase is included. > It is not a clean backport - it has to change return switch expressions to their old format and there are also other 2 minor conflicts. > This backport is authored by Anton Voznia, I am only submitting it. Anton Voznia is an Azul employee so he should be covered by Azul OCA. Anton Voznia has not signed a personal OCA from Oracle. > It requires https://github.com/openjdk/jdk8u-dev/pull/692 as a prerequisite. The CI errors are false positives, they are the same as unpatched sources tested in: https://github.com/jankratochvil/jdk8u-dev/pull/1 ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/693#issuecomment-3270226385 From sgehwolf at openjdk.org Tue Sep 9 12:38:41 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 12:38:41 GMT Subject: [jdk8u-dev] RFR: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test Message-ID: Trivial patch to problem list a test we know fails. This should reduce test noise. ------------- Commit messages: - 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test Changes: https://git.openjdk.org/jdk8u-dev/pull/695/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=695&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8367115 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/695.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/695/head:pull/695 PR: https://git.openjdk.org/jdk8u-dev/pull/695 From sgehwolf at openjdk.org Tue Sep 9 12:38:42 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 12:38:42 GMT Subject: [jdk8u-dev] RFR: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 16:36:38 GMT, Severin Gehwolf wrote: > Trivial patch to problem list a test we know fails. This should reduce test noise. Remaining failures are the intermittent `CAInterop.java#entrustrootcag4` failures. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/695#issuecomment-3270518057 From sgehwolf at openjdk.org Tue Sep 9 13:40:53 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 13:40:53 GMT Subject: [jdk8u-dev] RFR: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test In-Reply-To: References: Message-ID: On Tue, 9 Sep 2025 12:33:36 GMT, Severin Gehwolf wrote: > Remaining failures are the intermittent `CAInterop.java#entrustrootcag4` failures. Filed https://bugs.openjdk.org/browse/JDK-8367256 for this. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/695#issuecomment-3270795083 From sgehwolf at openjdk.org Tue Sep 9 15:30:03 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 15:30:03 GMT Subject: [jdk8u-dev] RFR: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 16:36:38 GMT, Severin Gehwolf wrote: > Trivial patch to problem list a test we know fails. This should reduce test noise. https://github.com/openjdk/jdk8u-dev/pull/696 depends on this and then GHA is green. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/695#issuecomment-3271231182 From sgehwolf at openjdk.org Tue Sep 9 15:32:29 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 15:32:29 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test Message-ID: Another problem list to get GHA green (at last). ------------- Depends on: https://git.openjdk.org/jdk8u-dev/pull/695 Commit messages: - 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test Changes: https://git.openjdk.org/jdk8u-dev/pull/696/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=696&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8367257 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/696.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/696/head:pull/696 PR: https://git.openjdk.org/jdk8u-dev/pull/696 From sgehwolf at openjdk.org Tue Sep 9 15:32:30 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Tue, 9 Sep 2025 15:32:30 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test In-Reply-To: References: Message-ID: On Tue, 9 Sep 2025 13:40:38 GMT, Severin Gehwolf wrote: > Another problem list to get GHA green (at last). GHA is green. Ready for review. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/696#issuecomment-3271229023 From sgehwolf at redhat.com Tue Sep 9 15:33:03 2025 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Tue, 09 Sep 2025 17:33:03 +0200 Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan In-Reply-To: References: Message-ID: <0a09fc4723fe96d258f0e2bd2fcc07a75c1a2f99.camel@redhat.com> Vote: yes On Mon, 2025-09-08 at 18:26 +0200, Severin Gehwolf wrote: > I hereby nominate Sendao Yan (syan [1]) to JDK 8 Updates Project[2] > committer. From yangfei at iscas.ac.cn Wed Sep 10 03:26:10 2025 From: yangfei at iscas.ac.cn (yangfei at iscas.ac.cn) Date: Wed, 10 Sep 2025 11:26:10 +0800 (GMT+08:00) Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan In-Reply-To: References: Message-ID: <5e424151.13d3a.19931a8cdb5.Coremail.yangfei@iscas.ac.cn> Vote: yes > -----Original Messages----- > From: "Severin Gehwolf" > Sent Time: 2025-09-09 00:26:04 (Tuesday) > To: jdk8u-dev at openjdk.org > Cc: > Subject: CFV: New JDK 8 Updates Project Committer: Sendao Yan > > I hereby nominate Sendao Yan (syan [1]) to JDK 8 Updates Project[2] > committer. Sendao has so far authored 21 PRs [3] that were integrated > into the JDK 8 Updates Project. Sendao intends to contribute more. Most > of these PRs fix test bugs or make the test more robust. Some of those > fixes were non-trivial. > > Making him committer would help speed up getting those fixes into the > JDK 8 update trees faster without the help of a sponsor. > > Sendao worked as compiler test engineer for several years and works at > Hygon now. They will continue to sustains JDK 8 for the next few years. > > Votes are due by 23:59 CEST on Monday, September 22nd, 2025. > > Only current JDK 8 Updates Project Committers [4] are eligible to vote > on this nomination. Votes must be cast in the open by replying to this > mailing list. For Lazy Consensus voting instructions see [5]. > > Thanks, > Severin > > [1] https://openjdk.org/census#syan > [2] https://openjdk.org/projects/jdk8u/ > [3] https://github.com/openjdk/jdk8u-dev/issues?q=is%3Apr%20author%3AsendaoYan%20label%3Aintegrated > [4] https://openjdk.org/census#jdk8u > [5] https://openjdk.org/projects/#committer-vote From evergizova at openjdk.org Wed Sep 10 21:54:44 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 10 Sep 2025 21:54:44 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 23:19:01 GMT, Sergey Bylokhov wrote: > > No new issues found in GHA testing, additionally tested manually with J2Ddemo. > > This is not enough test coverage, CMM tests are not part of GA testing, you need to run at least headless tests from the jdk_desktop group, or directly: java/awt/color* and sun/java2d/cmm*. This is applicable to all backports for lcms including 11u. Thanks, java/awt/color/* and sun/java2d/cmm/* tests passed successfully ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3276660614 From evergizova at openjdk.org Wed Sep 10 22:11:10 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 10 Sep 2025 22:11:10 GMT Subject: [jdk8u-dev] RFR: 8303482: Update LCMS to 2.15 In-Reply-To: References: Message-ID: <7pvVZjYuP8wLwXnDqdDmuBAXuIGBeH3BaIDK1mS7GV4=.e99ed043-b301-48a8-92e6-29db1978dcdb@github.com> On Thu, 21 Aug 2025 23:04:18 GMT, Ekaterina Vergizova wrote: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Additional testing: java/awt/color/* and sun/java2d/cmm/* tests passed successfully ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/678#issuecomment-3276697312 From jjiang at openjdk.org Thu Sep 11 03:55:12 2025 From: jjiang at openjdk.org (John Jiang) Date: Thu, 11 Sep 2025 03:55:12 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent [v2] In-Reply-To: References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: On Mon, 8 Sep 2025 14:41:30 GMT, John Jiang wrote: >> Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. >> This backport is not clean, because most files don't exist in jdk8u, >> and the copyright years in the following files are different between the repos. >> - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java >> - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java > > John Jiang has updated the pull request incrementally with one additional commit since the last revision: > > Trigger CI I think my changes didn't cause the failures. --- Pre-submit tests - Linux x64 - Build / test - Linux x64 (jdk/security_infra) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49858665456 Summary: jdk_security_infra FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 TEST STATS: name=jdk_security_infra run=52 pass=50 fail=2 These failures are known issues as the below, - security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 fails with unexpected exception https://bugs.openjdk.org/browse/JDK-8367256 - [8u] Problem list CAInterop.java#actalisauthenticationrootca test https://bugs.openjdk.org/browse/JDK-8367115 - [8u] Problem list CAInterop.java#entrustrootcag4 test https://bugs.openjdk.org/browse/JDK-8367257 --- Pre-submit tests - Linux x86 - Build / test - Linux x86 (jdk/security_infra) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49857754166 Summary: jdk_security_infra FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 TEST STATS: name=jdk_security_infra run=52 pass=50 fail=2 The same as the above. --- Pre-submit tests - macOS x64 - Build / test - macOS x64 (jdk/security_infra) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49859935467 Summary: jdk_security_infra FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 TEST STATS: name=jdk_security_infra run=52 pass=50 fail=2 The same as the above. --- Pre-submit tests - Windows x64 - Build / test - Windows x64 (jdk/security_infra) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49858740728 Summary: jdk_security_infra FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 TEST STATS: name=jdk_security_infra run=52 pass=50 fail=2 The same as the above. --- Pre-submit tests - Windows x86 - Build / test - Windows x86 (jdk/security_infra) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49859916109 Summary: jdk_security_infra FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#actalisauthenticationrootca FAILED: security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 TEST STATS: name=jdk_security_infra run=52 pass=50 fail=2 The same as the above. --- Pre-submit tests - Linux x86 - Test (tier1) - Linux x86 (hotspot/tier1) https://github.com/johnshajiang/jdk8u-dev/actions/runs/17554502230/job/49857754152 Summary: hotspot_tier1 FAILED: gc/logging/TestGCId.java TEST STATS: name=hotspot_tier1 run=802 pass=801 fail=1 `gc/logging/TestGCId.java` timed out. Agent error: java.lang.Exception: Agent 10 timed out with a timeout of 480 seconds; check console log for any additional details ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/689#issuecomment-3277307146 From evergizova at openjdk.org Thu Sep 11 12:12:43 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Thu, 11 Sep 2025 12:12:43 GMT Subject: [jdk8u-dev] RFR: 8321489: Update LCMS to 2.16 In-Reply-To: References: Message-ID: On Fri, 22 Aug 2025 16:49:35 GMT, Ekaterina Vergizova wrote: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Additional testing: java/awt/color/* and sun/java2d/cmm/* tests passed successfully ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/679#issuecomment-3280275789 From evergizova at openjdk.org Thu Sep 11 12:15:59 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Thu, 11 Sep 2025 12:15:59 GMT Subject: [jdk8u-dev] RFR: 8348110: Update LCMS to 2.17 In-Reply-To: References: Message-ID: On Fri, 22 Aug 2025 18:47:47 GMT, Ekaterina Vergizova wrote: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Additional testing: java/awt/color/* and sun/java2d/cmm/* tests passed successfully ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/680#issuecomment-3280277589 From tg at debian.org Sun Sep 14 15:46:25 2025 From: tg at debian.org (Thorsten Glaser) Date: Sun, 14 Sep 2025 17:46:25 +0200 (CEST) Subject: Fwd: Debian #1115228: openjdk-8: FTBFS with glibc 2.42 due to conflict with the newly added uabs function from ISO C2y Message-ID: <78bdc993-a623-5aef-8023-57adbfc3bafc@debian.org> Hi, OpenJDK 8 will not build under C2y (C26?) or with the next glibc any more. Are you going to fix this upstream? The options are probably probing for it and using the standard function when present, or renaming it and always using the own. Thanks in advance, //mirabilos ---------- Forwarded message ---------- From: Aurelien Jarno Message-ID: <175784982235.2811304.14814533927099270669.reportbug at ohm.local> Date: Sun, 14 Sep 2025 13:37:02 Subject: Bug#1115228: openjdk-8: FTBFS with glibc 2.42 due to conflict with the newly added uabs function Source: openjdk-8 Version: 8u462-ga-1 Severity: important Tags: ftbfs forky sid Justification: fails to build from source User: debian-glibc at lists.debian.org Usertags: glibc-2.42 Dear maintainer, During a rebuild of all packages in unstable with glibc 2.42 from experimental, your package failed to build. Below you will find how the build ends. If required, the full build log is available here: https://people.debian.org/~aurel32/glibc-2.42/ The build failure is due to the addition of the uabs function (part of ISO C2Y) to glibc 2.42, now conflicting with the openjdk source code: https://sourceware.org/git/?p=glibc.git;a=blob;f=NEWS;h=f0b0e924a45c7698e6ca6415f2d9a096a24e37df;hb=d2097651cc57834dbfcaa102ddfacae0d86cfb66#l22 About the archive rebuild: The build was made on virtual machines from AWS, using sbuild, a reduced chroot with only build-essential packages and glibc 2.42 from experimental. Regards Aurelien -------------------------------------------------------------------------------- [...] /usr/bin/x86_64-linux-gnu-g++-14 -DLINUX -D_GNU_SOURCE -DAMD64 -DPRODUCT -I. -I/<>/src/hotspot/src/share/vm/prims -I/<>/src/hotspot/src/share/vm -I/<>/src/hotspot/src/share/vm/precompiled -I/<>/src/hotspot/src/cpu/x86/vm -I/<>/src/hotspot/src/os_cpu/linux_x86/vm -I/<>/src/hotspot/src/os/linux/vm -I/<>/src/hotspot/src/os/posix/vm -I../generated -DHOTSPOT_RELEASE_VERSION="\"25.462-b08\"" -DHOTSPOT_BUILD_TARGET="\"product\"" -DHOTSPOT_BUILD_USER="\"buildd\"" -DHOTSPOT_LIB_ARCH=\"amd64\" -DHOTSPOT_VM_DISTRO="\"OpenJDK\"" -DDEB_MULTIARCH="\"x86_64-linux-gnu\"" -fstack-protector -DTARGET_OS_FAMILY_linux -DTARGET_ARCH_x86 -DTARGET_ARCH_MODEL_x86_64 -DTARGET_OS_ARCH_linux_x86 -DTARGET_OS_ARCH_MODEL_linux_x86_64 -DTARGET_COMPILER_gcc -DINCLUDE_JFR=1 -DCOMPILER2 -DCOMPILER1 -fPIC -fno-rtti -fno-exceptions -D_REENTRANT -fcheck-new -fvisibility=hidden -m64 -pipe -fno-strict-aliasing -fno-omit-frame-pointer -O3 -DVM_LITTLE_ENDIAN -D_LP64=1 -Wpointer-arith -Wsign-compare -Wundef -Wunused-function -Wunused-value -Wformat=2 -Wreturn-type -Woverloaded-virtual -fstack-protector -Wdate-time -D_FORTIFY_SOURCE=2 -g -Werror=implicit-function-declaration -ffile-prefix-map=/<>=. -fstack-protector-strong -fstack-clash-protection -Wformat -fcf-protection -Wdate-time -D_FORTIFY_SOURCE=2 -fno-delete-null-pointer-checks -fno-lifetime-dse -std=gnu++98 -c -MMD -MP -MF ../generated/dependencies/precompiled.hpp.gch.d -fpch-deps -x c++-header /<>/src/hotspot/src/share/vm/precompiled/precompiled.hpp -o precompiled.hpp.gch && { echo Done with ; } cc1plus: warning: '-Werror=' argument '-Werror=implicit-function-declaration' is not valid for C++ In file included from /<>/src/hotspot/src/share/vm/utilities/debug.hpp:28, from /<>/src/hotspot/src/share/vm/runtime/globals.hpp:28, from /<>/src/hotspot/src/share/vm/memory/allocation.hpp:28, from /<>/src/hotspot/src/share/vm/memory/iterator.hpp:28, from /<>/src/hotspot/src/share/vm/memory/genOopClosures.hpp:28, from /<>/src/hotspot/src/share/vm/oops/klass.hpp:28, from /<>/src/hotspot/src/share/vm/runtime/handles.hpp:28, from /<>/src/hotspot/src/share/vm/memory/universe.hpp:28, from /<>/src/hotspot/src/share/vm/code/oopRecorder.hpp:28, from /<>/src/hotspot/src/share/vm/asm/codeBuffer.hpp:28, from /<>/src/hotspot/src/share/vm/asm/assembler.hpp:28, from /<>/src/hotspot/src/share/vm/precompiled/precompiled.hpp:29: /<>/src/hotspot/src/share/vm/utilities/globalDefinitions.hpp:1276:28: error: 'unsigned int uabs(int)' was declared 'extern' and later 'static' [-fpermissive] 1276 | static inline unsigned int uabs(int n) { return uabs((unsigned int)n); } | ^~~~ In file included from /usr/include/c++/14/cstdlib:79, from /usr/include/c++/14/stdlib.h:36, from /<>/src/hotspot/src/share/vm/utilities/globalDefinitions_gcc.hpp:39, from /<>/src/hotspot/src/share/vm/utilities/globalDefinitions.hpp:38: /usr/include/stdlib.h:989:21: note: previous declaration of 'unsigned int uabs(int)' 989 | extern unsigned int uabs (int __x) __THROW __attribute__ ((__const__)) __wur; | ^~~~ gmake[7]: *** [/<>/src/hotspot/make/linux/makefiles/vm.make:317: precompiled.hpp.gch] Error 1 gmake[7]: Leaving directory '/<>/build/hotspot/linux_amd64_compiler2/product' gmake[6]: *** [/<>/src/hotspot/make/linux/makefiles/top.make:119: the_vm] Error 2 gmake[6]: Leaving directory '/<>/build/hotspot/linux_amd64_compiler2/product' gmake[5]: *** [/<>/src/hotspot/make/linux/Makefile:284: product] Error 2 gmake[5]: Leaving directory '/<>/build/hotspot' gmake[4]: *** [Makefile:231: generic_build2] Error 2 gmake[4]: Leaving directory '/<>/src/hotspot/make' gmake[3]: *** [Makefile:177: product] Error 2 gmake[3]: Leaving directory '/<>/src/hotspot/make' gmake[2]: *** [HotspotWrapper.gmk:45: /<>/build/hotspot/_hotspot.timestamp] Error 2 gmake[2]: Leaving directory '/<>/src/make' make[1]: *** [/<>/src//make/Main.gmk:110: hotspot-only] Error 2 make[1]: Leaving directory '/<>/build' make: *** [debian/rules:1138: stamps/build] Error 1 dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2 -------------------------------------------------------------------------------- From sgehwolf at redhat.com Mon Sep 15 08:41:03 2025 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Mon, 15 Sep 2025 10:41:03 +0200 Subject: Fwd: Debian #1115228: openjdk-8: FTBFS with glibc 2.42 due to conflict with the newly added uabs function from ISO C2y In-Reply-To: <78bdc993-a623-5aef-8023-57adbfc3bafc@debian.org> References: <78bdc993-a623-5aef-8023-57adbfc3bafc@debian.org> Message-ID: <1f89334e295b2c63f00319a430c6b718a5d4250a.camel@redhat.com> This is tracked in (not in JDK 11 or JDK 8 yet): https://bugs.openjdk.org/browse/JDK-8354941 Thanks, Severin On Sun, 2025-09-14 at 17:46 +0200, Thorsten Glaser wrote: > Hi, > > OpenJDK 8 will not build under C2y (C26?) or with the next glibc > any more. Are you going to fix this upstream? > > The options are probably probing for it and using the standard > function when present, or renaming it and always using the own. > > Thanks in advance, > //mirabilos > > ---------- Forwarded message ---------- > From: Aurelien Jarno > Message-ID: <175784982235.2811304.14814533927099270669.reportbug at ohm.local> > Date: Sun, 14 Sep 2025 13:37:02 > Subject: Bug#1115228: openjdk-8: FTBFS with glibc 2.42 due to conflict with the > ??? newly added uabs function > > Source: openjdk-8 > Version: 8u462-ga-1 > Severity: important > Tags: ftbfs forky sid > Justification: fails to build from source > User: debian-glibc at lists.debian.org > Usertags: glibc-2.42 > > Dear maintainer, > > During a rebuild of all packages in unstable with glibc 2.42 from > experimental, your package failed to build. Below you will find how the > build ends. If required, the full build log is available here: > > https://people.debian.org/~aurel32/glibc-2.42/ > > The build failure is due to the addition of the uabs function (part of > ISO C2Y) to glibc 2.42, now conflicting with the openjdk source code: > > https://sourceware.org/git/?p=glibc.git;a=blob;f=NEWS;h=f0b0e924a45c7698e6ca6415f2d9a096a24e37df;hb=d2097651cc57834dbfcaa102ddfacae0d86cfb66#l22 > > About the archive rebuild: The build was made on virtual machines from > AWS, using sbuild, a reduced chroot with only build-essential packages > and glibc 2.42 from experimental. > > Regards > Aurelien > > -------------------------------------------------------------------------------- > [...] > /usr/bin/x86_64-linux-gnu-g++-14 -DLINUX -D_GNU_SOURCE -DAMD64 -DPRODUCT -I. -I/<>/src/hotspot/src/share/vm/prims -I/<>/src/hotspot/src/share/vm -I/<>/src/hotspot/src/share/vm/precompiled -I/<>/src/hotspot/src/cpu/x86/vm -I/<>/src/hotspot/src/os_cpu/linux_x86/vm -I/<>/src/hotspot/src/os/linux/vm -I/<>/src/hotspot/src/os/posix/vm -I../generated -DHOTSPOT_RELEASE_VERSION="\"25.462-b08\"" -DHOTSPOT_BUILD_TARGET="\"product\"" -DHOTSPOT_BUILD_USER="\"buildd\"" -DHOTSPOT_LIB_ARCH=\"amd64\" -DHOTSPOT_VM_DISTRO="\"OpenJDK\"" -DDEB_MULTIARCH="\"x86_64-linux-gnu\""? -fstack-protector? -DTARGET_OS_FAMILY_linux -DTARGET_ARCH_x86 -DTARGET_ARCH_MODEL_x86_64 -DTARGET_OS_ARCH_linux_x86 -DTARGET_OS_ARCH_MODEL_linux_x86_64 -DTARGET_COMPILER_gcc -DINCLUDE_JFR=1 -DCOMPILER2 -DCOMPILER1 -fPIC -fno-rtti -fno-exceptions -D_REENTRANT -fcheck-new -fvisibility=hidden -m64? -pipe -fno-strict-aliasing? -fno-omit-frame-pointer -O3? -DVM_LITTLE_ENDIAN -D_LP64=1? -Wpointer-arith -Wsign-compare -Wundef -Wunused-function -Wunused-value -Wformat=2 -Wreturn-type -Woverloaded-virtual?? -fstack-protector -Wdate-time -D_FORTIFY_SOURCE=2 -g -Werror=implicit-function-declaration -ffile-prefix-map=/<>=. -fstack-protector-strong -fstack-clash-protection -Wformat -fcf-protection -Wdate-time -D_FORTIFY_SOURCE=2 -fno-delete-null-pointer-checks -fno-lifetime-dse -std=gnu++98 -c -MMD -MP -MF ../generated/dependencies/precompiled.hpp.gch.d -fpch-deps -x c++-header /<>/src/hotspot/src/share/vm/precompiled/precompiled.hpp -o precompiled.hpp.gch && { echo Done with ; } > cc1plus: warning: '-Werror=' argument '-Werror=implicit-function-declaration' is not valid for C++ > In file included from /<>/src/hotspot/src/share/vm/utilities/debug.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/runtime/globals.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/memory/allocation.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/memory/iterator.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/memory/genOopClosures.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/oops/klass.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/runtime/handles.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/memory/universe.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/code/oopRecorder.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/asm/codeBuffer.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/asm/assembler.hpp:28, > ???????????????? from /<>/src/hotspot/src/share/vm/precompiled/precompiled.hpp:29: > /<>/src/hotspot/src/share/vm/utilities/globalDefinitions.hpp:1276:28: error: 'unsigned int uabs(int)' was declared 'extern' and later 'static' [-fpermissive] > ?1276 | static inline unsigned int uabs(int n) { return uabs((unsigned int)n); } > ????? |??????????????????????????? ^~~~ > In file included from /usr/include/c++/14/cstdlib:79, > ???????????????? from /usr/include/c++/14/stdlib.h:36, > ???????????????? from /<>/src/hotspot/src/share/vm/utilities/globalDefinitions_gcc.hpp:39, > ???????????????? from /<>/src/hotspot/src/share/vm/utilities/globalDefinitions.hpp:38: > /usr/include/stdlib.h:989:21: note: previous declaration of 'unsigned int uabs(int)' > ? 989 | extern unsigned int uabs (int __x) __THROW __attribute__ ((__const__)) __wur; > ????? |???????????????????? ^~~~ > gmake[7]: *** [/<>/src/hotspot/make/linux/makefiles/vm.make:317: precompiled.hpp.gch] Error 1 > gmake[7]: Leaving directory '/<>/build/hotspot/linux_amd64_compiler2/product' > gmake[6]: *** [/<>/src/hotspot/make/linux/makefiles/top.make:119: the_vm] Error 2 > gmake[6]: Leaving directory '/<>/build/hotspot/linux_amd64_compiler2/product' > gmake[5]: *** [/<>/src/hotspot/make/linux/Makefile:284: product] Error 2 > gmake[5]: Leaving directory '/<>/build/hotspot' > gmake[4]: *** [Makefile:231: generic_build2] Error 2 > gmake[4]: Leaving directory '/<>/src/hotspot/make' > gmake[3]: *** [Makefile:177: product] Error 2 > gmake[3]: Leaving directory '/<>/src/hotspot/make' > gmake[2]: *** [HotspotWrapper.gmk:45: /<>/build/hotspot/_hotspot.timestamp] Error 2 > gmake[2]: Leaving directory '/<>/src/make' > make[1]: *** [/<>/src//make/Main.gmk:110: hotspot-only] Error 2 > make[1]: Leaving directory '/<>/build' > make: *** [debian/rules:1138: stamps/build] Error 1 > dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2 > -------------------------------------------------------------------------------- From andrew at openjdk.org Mon Sep 15 15:28:58 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Mon, 15 Sep 2025 15:28:58 GMT Subject: [jdk8u-dev] RFR: 8345358: Some DLL Files are missing Windows Properties In-Reply-To: <32SxRQn1gqie6ppqb1IrzNyIYRaL04yp5N2dqv6EsLQ=.b8c3b6b1-65de-4854-9b12-ea4b971abcb3@github.com> References: <32SxRQn1gqie6ppqb1IrzNyIYRaL04yp5N2dqv6EsLQ=.b8c3b6b1-65de-4854-9b12-ea4b971abcb3@github.com> Message-ID: On Mon, 8 Sep 2025 17:27:30 GMT, Alexey Bakhtin wrote: > > @alexeybakhtin this is just waiting for an approval request. > > Thank you. Just had a missprint in the label Ah, no worries. If you use the /approval request command on the PR, you can avoid having to alter the bug manually. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/611#issuecomment-3292753091 From andrew at openjdk.org Mon Sep 15 15:35:02 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Mon, 15 Sep 2025 15:35:02 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent [v2] In-Reply-To: References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: On Mon, 8 Sep 2025 14:41:30 GMT, John Jiang wrote: >> Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. >> This backport is not clean, because most files don't exist in jdk8u, >> and the copyright years in the following files are different between the repos. >> - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java >> - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java > > John Jiang has updated the pull request incrementally with one additional commit since the last revision: > > Trigger CI Yes, these are known issues that we are working to resolve. I don't expect new failures from a comment change. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/689#issuecomment-3292786138 From duke at openjdk.org Mon Sep 15 15:47:02 2025 From: duke at openjdk.org (duke) Date: Mon, 15 Sep 2025 15:47:02 GMT Subject: [jdk8u-dev] RFR: 8364597: Replace THL A29 Limited with Tencent [v2] In-Reply-To: References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: On Mon, 8 Sep 2025 14:41:30 GMT, John Jiang wrote: >> Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. >> This backport is not clean, because most files don't exist in jdk8u, >> and the copyright years in the following files are different between the repos. >> - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java >> - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java > > John Jiang has updated the pull request incrementally with one additional commit since the last revision: > > Trigger CI @johnshajiang Your change (at version 17c4ba3acefeae88f148e561f00b94a9552f1cd7) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/689#issuecomment-3292829705 From jvanek at openjdk.org Mon Sep 15 16:10:54 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Mon, 15 Sep 2025 16:10:54 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on Message-ID: We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. ------------- Commit messages: - 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on Changes: https://git.openjdk.org/jdk8u-dev/pull/697/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8360869 Stats: 11 lines in 1 file changed: 11 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/697.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/697/head:pull/697 PR: https://git.openjdk.org/jdk8u-dev/pull/697 From abakhtin at openjdk.org Mon Sep 15 16:58:18 2025 From: abakhtin at openjdk.org (Alexey Bakhtin) Date: Mon, 15 Sep 2025 16:58:18 GMT Subject: [jdk8u-dev] Integrated: 8345358: Some DLL Files are missing Windows Properties In-Reply-To: References: Message-ID: <3F0kOd2UIsiqvXCQao0K8ZXGFr6-ri62YGmbXcsEebY=.f26e48d0-3622-4202-af59-ab48d5712bfb@github.com> On Tue, 10 Dec 2024 22:45:15 GMT, Alexey Bakhtin wrote: > Add missing properties for the j2gss.dll and sspi_bridge.dll files This pull request has now been integrated. Changeset: 2b36fc1b Author: Alexey Bakhtin URL: https://git.openjdk.org/jdk8u-dev/commit/2b36fc1b924032167b455bdd527cc7f75044fe9d Stats: 99 lines in 2 files changed: 99 ins; 0 del; 0 mod 8345358: Some DLL Files are missing Windows Properties Reviewed-by: andrew, serb ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/611 From jjiang at openjdk.org Mon Sep 15 21:26:19 2025 From: jjiang at openjdk.org (John Jiang) Date: Mon, 15 Sep 2025 21:26:19 GMT Subject: [jdk8u-dev] Integrated: 8364597: Replace THL A29 Limited with Tencent In-Reply-To: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> References: <5qDOXxI-hUoUCnjhBLXSdu9gAifFz5ZpyZQ-HNPnLkI=.a712610f-2685-4795-b2e8-e6b125616808@github.com> Message-ID: On Wed, 3 Sep 2025 15:35:39 GMT, John Jiang wrote: > Please review this backport of `JDK-8364597: Replace THL A29 Limited with Tencent`. > This backport is not clean, because most files don't exist in jdk8u, > and the copyright years in the following files are different between the repos. > - jdk/test/jdk/internal/platform/docker/GetFreeSwapSpaceSize.java > - jdk/test/jdk/internal/platform/docker/TestGetFreeSwapSpaceSize.java This pull request has now been integrated. Changeset: 9a1f0c6f Author: John Jiang Committer: Andrew John Hughes URL: https://git.openjdk.org/jdk8u-dev/commit/9a1f0c6f9f1bc41b651b60f21c9b2b8630851267 Stats: 10 lines in 7 files changed: 0 ins; 3 del; 7 mod 8364597: Replace THL A29 Limited with Tencent Reviewed-by: andrew Backport-of: 4c9eaddaef83c6ba30e27ae3e0d16caeeec206cb ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/689 From andrew at openjdk.org Wed Sep 17 01:23:46 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 17 Sep 2025 01:23:46 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on In-Reply-To: References: Message-ID: <4QNL56V7cVztCPt-m57UI_phdr8Q3DNYRENi8e1yDMk=.2d5b4358-032a-46c1-a38f-58f3abc07e02@github.com> On Mon, 15 Sep 2025 16:00:26 GMT, Ji?? Van?k wrote: > We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. > > I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. I would move the `AC_MSG_RESULT` earlier i.e. ~~~ AC_MSG_RESULT([found ($COMPILER_VERSION_NUMBER]) if test $COMPILER_VERSION_NUMBER_MAJOR -lt 5; then AC_MSG_ERROR([GCC < 5 is known to lead to incorrect compilation on aarch64. See JDK-8360869.]) fi ~~~ And yes, configure needs to be regenerated at present. I've removed the approval request. This should only be done after successful review and with a comment as to why it is suitable for inclusion. Easier way is with the `/approval request` command. ------------- Changes requested by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/697#pullrequestreview-3232159871 From andrew at openjdk.org Wed Sep 17 01:27:47 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 17 Sep 2025 01:27:47 GMT Subject: [jdk8u-dev] RFR: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test In-Reply-To: References: Message-ID: On Mon, 8 Sep 2025 16:36:38 GMT, Severin Gehwolf wrote: > Trivial patch to problem list a test we know fails. This should reduce test noise. Looks fine. Please apply for approval. ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/695#pullrequestreview-3232164532 From andrew at openjdk.org Wed Sep 17 01:30:46 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 17 Sep 2025 01:30:46 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 13:07:32 GMT, Ekaterina Vergizova wrote: >> I'd like to backport it for parity with Oracle 8u. >> 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. >> I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. >> >> GHA testing revealed no regressions, additionally tested manually with J2Ddemo. > > Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into 8297088 > - Backport 01b213899cb8124860441fa26c9652b4a4bff896 Sorry for the delay in getting back to this. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3300865790 From schernyshev at openjdk.org Wed Sep 17 06:27:37 2025 From: schernyshev at openjdk.org (Sergey Chernyshev) Date: Wed, 17 Sep 2025 06:27:37 GMT Subject: [jdk8u-dev] RFR: 8224267: JOptionPane message string with 5000+ newlines produces StackOverflowError [v3] In-Reply-To: References: Message-ID: > Hi all, > > These are parity backports with Oracle's 8u461, OpenJDK 8u462 is affected. The commit [46251bc6](https://github.com/openjdk/jdk/commit/46251bc6e248a19e8d78173ff8d0502c68ee1acb) didn't apply cleanly as JDK-8049870 hasn't been backported to jdk8u-dev. JDK-8290162 is basically a clean backport, the paths changed to 8u path scheme. The functional part of 5074006 applied cleanly in BasicOptionPaneUI.java. The test in TestJOptionHTMLTag.java was rewritten to get rid of dependency on PassFailJFrame (not in 8). Replaced the text block (JEP 355 style) with traditional string literals. The paths have been changed to the 8u path scheme. The test passes with the fix and fails otherwise. > > Thanks! Sergey Chernyshev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains four commits: - Merge remote-tracking branch 'dev/master' into backport-sercher-46251bc6-master # Conflicts: # jdk/src/share/classes/javax/swing/plaf/basic/BasicOptionPaneUI.java - Backport 91072ee3934616ab2edc4850a59c0a25fd0de3b4 - Backport 6e18883d8ffd9a7b7d495da05e9859dc1d1a2677 - Backport 46251bc6e248a19e8d78173ff8d0502c68ee1acb ------------- Changes: https://git.openjdk.org/jdk8u-dev/pull/659/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=659&range=02 Stats: 233 lines in 3 files changed: 202 ins; 17 del; 14 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/659.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/659/head:pull/659 PR: https://git.openjdk.org/jdk8u-dev/pull/659 From jvanek at openjdk.org Wed Sep 17 07:06:13 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Wed, 17 Sep 2025 07:06:13 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v2] In-Reply-To: References: Message-ID: > We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. > > I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Moved compiler version result to earlier stage ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/697/files - new: https://git.openjdk.org/jdk8u-dev/pull/697/files/1a23a459..dc7efed9 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=00-01 Stats: 3 lines in 1 file changed: 1 ins; 1 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/697.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/697/head:pull/697 PR: https://git.openjdk.org/jdk8u-dev/pull/697 From jvanek at openjdk.org Wed Sep 17 07:08:56 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Wed, 17 Sep 2025 07:08:56 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v2] In-Reply-To: <4QNL56V7cVztCPt-m57UI_phdr8Q3DNYRENi8e1yDMk=.2d5b4358-032a-46c1-a38f-58f3abc07e02@github.com> References: <4QNL56V7cVztCPt-m57UI_phdr8Q3DNYRENi8e1yDMk=.2d5b4358-032a-46c1-a38f-58f3abc07e02@github.com> Message-ID: On Wed, 17 Sep 2025 01:21:23 GMT, Andrew John Hughes wrote: > I would move the `AC_MSG_RESULT` earlier i.e. > > ``` > AC_MSG_RESULT([found ($COMPILER_VERSION_NUMBER]) > if test $COMPILER_VERSION_NUMBER_MAJOR -lt 5; then > AC_MSG_ERROR([GCC < 5 is known to lead to incorrect compilation on aarch64. See JDK-8360869.]) > fi > ``` > Sure, sounds good. Done. > And yes, configure needs to be regenerated at present. Any recommended version of autoconf to use? When I use 2.72, the diff is 90% of the file. > > I've removed the approval request. This should only be done after successful review and with a comment as to why it is suitable for inclusion. Easier way is with the `/approval request` command. Sure thing. Whether to do it or so, is if course on you and @theRealAph . I will do my best on how to do that. Thanx! ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/697#issuecomment-3301600008 From aph-open at littlepinkcloud.com Wed Sep 17 08:43:48 2025 From: aph-open at littlepinkcloud.com (Andrew Haley) Date: Wed, 17 Sep 2025 09:43:48 +0100 Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v2] In-Reply-To: References: <4QNL56V7cVztCPt-m57UI_phdr8Q3DNYRENi8e1yDMk=.2d5b4358-032a-46c1-a38f-58f3abc07e02@github.com> Message-ID: <2f950e9b-f3df-4803-910e-645e680de29c@littlepinkcloud.com> On 17/09/2025 08:08, Ji?? Van?k wrote: > AC_MSG_ERROR([GCC < 5 is known to lead to incorrect compilation on aarch64. See JDK-8360869.]) "Known" is a bit too strong. "GCC < 5 may incorrectly compile HotSpot" is better. -- Andrew Haley (he/him) Java Platform Lead Engineer https://keybase.io/andrewhaley EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671 From jvanek at openjdk.org Wed Sep 17 09:20:09 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Wed, 17 Sep 2025 09:20:09 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v3] In-Reply-To: References: Message-ID: > We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. > > I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: Improved error message ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/697/files - new: https://git.openjdk.org/jdk8u-dev/pull/697/files/dc7efed9..f4c31107 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=01-02 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/697.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/697/head:pull/697 PR: https://git.openjdk.org/jdk8u-dev/pull/697 From jvanek at openjdk.org Wed Sep 17 09:30:19 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Wed, 17 Sep 2025 09:30:19 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on In-Reply-To: <2f950e9b-f3df-4803-910e-645e680de29c@littlepinkcloud.com> References: <2f950e9b-f3df-4803-910e-645e680de29c@littlepinkcloud.com> Message-ID: On Wed, 17 Sep 2025 08:45:17 GMT, Andrew Haley wrote: > "Known" is a bit too strong. "GCC < 5 may incorrectly compile HotSpot" sure. TY! ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/697#issuecomment-3302121799 From duke at openjdk.org Wed Sep 17 14:17:55 2025 From: duke at openjdk.org (duke) Date: Wed, 17 Sep 2025 14:17:55 GMT Subject: [jdk8u-dev] RFR: 8297088: Update LCMS to 2.14 [v2] In-Reply-To: References: Message-ID: On Mon, 1 Sep 2025 13:07:32 GMT, Ekaterina Vergizova wrote: >> I'd like to backport it for parity with Oracle 8u. >> 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. >> I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. >> >> GHA testing revealed no regressions, additionally tested manually with J2Ddemo. > > Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: > > - Merge branch 'master' into 8297088 > - Backport 01b213899cb8124860441fa26c9652b4a4bff896 @kvergizova Your change (at version 6f31652bed309f3772df581e24129dd79d557b55) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/677#issuecomment-3303221075 From andrew at openjdk.org Wed Sep 17 16:00:08 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 17 Sep 2025 16:00:08 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v3] In-Reply-To: References: <4QNL56V7cVztCPt-m57UI_phdr8Q3DNYRENi8e1yDMk=.2d5b4358-032a-46c1-a38f-58f3abc07e02@github.com> Message-ID: <9rg1xmYconClGDYL_C7KjUd4_1UrdVxcLUG3L0BBnds=.132725ba-e5f6-4a45-9eb4-9c0fd043eb0b@github.com> On Wed, 17 Sep 2025 07:06:10 GMT, Ji?? Van?k wrote: > > I would move the `AC_MSG_RESULT` earlier i.e. > > ``` > > AC_MSG_RESULT([found ($COMPILER_VERSION_NUMBER]) > > if test $COMPILER_VERSION_NUMBER_MAJOR -lt 5; then > > AC_MSG_ERROR([GCC < 5 is known to lead to incorrect compilation on aarch64. See JDK-8360869.]) > > fi > > ``` > > Sure, sounds good. Done. Thanks. This looks good with aph's amendments as well. It should mean the checking line is always completed with the version number and the error is on its own line. > > > And yes, configure needs to be regenerated at present. > > Any recommended version of autoconf to use? When I use 2.72, the diff is 90% of the file. I'll look at what I used. You may need to dust off an older version (2.72 sounds very new, not sure I've seen that one yet) It can produce differing results even with the same version from different distro packages, so I would like to drop it as we have on later JDKs. If you would rather wait on that backport, that's an alternative option. I would like to finally get that into the January update. > > > I've removed the approval request. This should only be done after successful review and with a comment as to why it is suitable for inclusion. Easier way is with the `/approval request` command. > > Sure thing. Whether to do it or so, is if course on you and @theRealAph . I will do my best on how to do that. Thanx! This is just the general process across all update releases. If there are PRs in the approval queue that still need a review, it clogs it up and can mean that ones that are ready to go are missed. In theory, there are a lot more reviewers than maintainers :) ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/697#issuecomment-3303642343 From sgehwolf at openjdk.org Wed Sep 17 16:15:21 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Wed, 17 Sep 2025 16:15:21 GMT Subject: [jdk8u-dev] Integrated: 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test In-Reply-To: References: Message-ID: <4rQ59kOM437m849CH92kEg7y6nI_8lMGa8zUJ07ZQkI=.2f1b5f2d-5173-4cc5-ab00-6c742e3316e1@github.com> On Mon, 8 Sep 2025 16:36:38 GMT, Severin Gehwolf wrote: > Trivial patch to problem list a test we know fails. This should reduce test noise. This pull request has now been integrated. Changeset: 9948bd6b Author: Severin Gehwolf URL: https://git.openjdk.org/jdk8u-dev/commit/9948bd6b9a7247535ca754a35b7aa755b8a49f82 Stats: 1 line in 1 file changed: 1 ins; 0 del; 0 mod 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test Reviewed-by: andrew ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/695 From sgehwolf at openjdk.org Wed Sep 17 16:23:20 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Wed, 17 Sep 2025 16:23:20 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test [v2] In-Reply-To: References: Message-ID: > Another problem list to get GHA green (at last). Severin Gehwolf has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/696/files - new: https://git.openjdk.org/jdk8u-dev/pull/696/files/41e4028e..41e4028e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=696&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=696&range=00-01 Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/696.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/696/head:pull/696 PR: https://git.openjdk.org/jdk8u-dev/pull/696 From sgehwolf at openjdk.org Wed Sep 17 16:37:54 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Wed, 17 Sep 2025 16:37:54 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test [v3] In-Reply-To: References: Message-ID: > Another problem list to get GHA green (at last). Severin Gehwolf has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: - Merge branch 'master' into jdk-8367257-problemlist-entrustrootcag4 - 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test - 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/696/files - new: https://git.openjdk.org/jdk8u-dev/pull/696/files/41e4028e..3856b3e2 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=696&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=696&range=01-02 Stats: 109 lines in 9 files changed: 99 ins; 3 del; 7 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/696.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/696/head:pull/696 PR: https://git.openjdk.org/jdk8u-dev/pull/696 From andrew at openjdk.org Wed Sep 17 20:46:22 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Wed, 17 Sep 2025 20:46:22 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test [v3] In-Reply-To: References: Message-ID: On Wed, 17 Sep 2025 16:37:54 GMT, Severin Gehwolf wrote: >> Another problem list to get GHA green (at last). > > Severin Gehwolf has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: > > - Merge branch 'master' into jdk-8367257-problemlist-entrustrootcag4 > - 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test > - 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test Looks fine. Nice you're keeping them in alphabetical order :) ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/696#pullrequestreview-3236215895 From evergizova at openjdk.org Wed Sep 17 20:51:24 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 17 Sep 2025 20:51:24 GMT Subject: [jdk8u-dev] Integrated: 8297088: Update LCMS to 2.14 In-Reply-To: References: Message-ID: On Thu, 21 Aug 2025 20:35:46 GMT, Ekaterina Vergizova wrote: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > I also fixed the file paths in UPDATING.txt instructions so that they match the paths in OpenJDK8. > > GHA testing revealed no regressions, additionally tested manually with J2Ddemo. This pull request has now been integrated. Changeset: 9a4bc2d2 Author: Ekaterina Vergizova Committer: Andrew John Hughes URL: https://git.openjdk.org/jdk8u-dev/commit/9a4bc2d205c9b8b5f743f16a2ea8b85d6cb6924b Stats: 2141 lines in 31 files changed: 1549 ins; 141 del; 451 mod 8297088: Update LCMS to 2.14 Reviewed-by: abakhtin, andrew Backport-of: 01b213899cb8124860441fa26c9652b4a4bff896 ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/677 From evergizova at openjdk.org Wed Sep 17 21:00:16 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 17 Sep 2025 21:00:16 GMT Subject: [jdk8u-dev] RFR: 8303482: Update LCMS to 2.15 [v2] In-Reply-To: References: Message-ID: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/678/files - new: https://git.openjdk.org/jdk8u-dev/pull/678/files/5517fec4..5517fec4 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=678&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=678&range=00-01 Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/678.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/678/head:pull/678 PR: https://git.openjdk.org/jdk8u-dev/pull/678 From evergizova at openjdk.org Wed Sep 17 21:44:22 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 17 Sep 2025 21:44:22 GMT Subject: [jdk8u-dev] RFR: 8303482: Update LCMS to 2.15 [v3] In-Reply-To: References: Message-ID: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision: Backport e28b210d5020fc7cc3a52c58f0a8ce14e16dd5fb ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/678/files - new: https://git.openjdk.org/jdk8u-dev/pull/678/files/5517fec4..ee135890 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=678&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=678&range=01-02 Stats: 2357 lines in 118 files changed: 843 ins; 1105 del; 409 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/678.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/678/head:pull/678 PR: https://git.openjdk.org/jdk8u-dev/pull/678 From evergizova at openjdk.org Wed Sep 17 21:50:03 2025 From: evergizova at openjdk.org (Ekaterina Vergizova) Date: Wed, 17 Sep 2025 21:50:03 GMT Subject: [jdk8u-dev] RFR: 8321489: Update LCMS to 2.16 [v2] In-Reply-To: References: Message-ID: > I'd like to backport it for parity with Oracle 8u. > 11u patch applies almost cleanly, except for a license update in THIRD_PARTY_README instead of lcms.md. > > GHA testing passes without regressions, test failures look unrelated. Additionally tested manually with J2Ddemo. Ekaterina Vergizova has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/679/files - new: https://git.openjdk.org/jdk8u-dev/pull/679/files/8bacf0b5..8bacf0b5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=679&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=679&range=00-01 Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/679.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/679/head:pull/679 PR: https://git.openjdk.org/jdk8u-dev/pull/679 From jjiang at openjdk.org Thu Sep 18 07:13:44 2025 From: jjiang at openjdk.org (John Jiang) Date: Thu, 18 Sep 2025 07:13:44 GMT Subject: [jdk8u-dev] RFR: 8240226: DeflateIn_InflateOut.java test incorrectly assumes size of compressed file Message-ID: `java/util/zip/DeflateIn_InflateOut.java` failed with a system zlib. The backport is clean. ------------- Commit messages: - Backport 69f4ac1e0d61804b1bf306ee728a4de4397aeb47 Changes: https://git.openjdk.org/jdk8u-dev/pull/698/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=698&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8240226 Stats: 5 lines in 1 file changed: 2 ins; 2 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/698.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/698/head:pull/698 PR: https://git.openjdk.org/jdk8u-dev/pull/698 From jvanek at openjdk.org Thu Sep 18 07:56:39 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Thu, 18 Sep 2025 07:56:39 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v4] In-Reply-To: References: Message-ID: > We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. > > I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: generad generated-configure Skipping about 50 hunks like: @@ -50158,7 +50167,7 @@ ac_compiler_gnu=$ac_cv_cxx_compiler_gnu /* end confdefs.h. */ int -main (void) +main () { return 0; ; ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/697/files - new: https://git.openjdk.org/jdk8u-dev/pull/697/files/f4c31107..e06588df Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=03 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=02-03 Stats: 13 lines in 1 file changed: 13 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/697.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/697/head:pull/697 PR: https://git.openjdk.org/jdk8u-dev/pull/697 From jvanek at openjdk.org Thu Sep 18 08:00:59 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Thu, 18 Sep 2025 08:00:59 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v5] In-Reply-To: References: Message-ID: <196Y7TKF9GNgwVRcKposFa5qZ_kfOvdgDAEgckqyRPg=.dba882cd-f17d-44e1-9db0-fd2c8cbfb5a4@github.com> > We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. > > I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. Ji?? Van?k has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision: regenerated generated-configure by self built autoconf 2.69 Skipping about 50 hunks like: @@ -50158,7 +50167,7 @@ ac_compiler_gnu=$ac_cv_cxx_compiler_gnu /* end confdefs.h. */ int -main (void) +main () { return 0; ; ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/697/files - new: https://git.openjdk.org/jdk8u-dev/pull/697/files/e06588df..78ee959b Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=04 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=697&range=03-04 Stats: 0 lines in 0 files changed: 0 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/697.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/697/head:pull/697 PR: https://git.openjdk.org/jdk8u-dev/pull/697 From jvanek at openjdk.org Thu Sep 18 08:01:03 2025 From: jvanek at openjdk.org (=?UTF-8?B?SmnFmcOt?= =?UTF-8?B?IA==?= =?UTF-8?B?VmFuxJtr?=) Date: Thu, 18 Sep 2025 08:01:03 GMT Subject: [jdk8u-dev] RFR: 8360869: jcstress is able to crash jdk8 on aarch64 with jfr on [v3] In-Reply-To: References: Message-ID: On Wed, 17 Sep 2025 09:20:09 GMT, Ji?? Van?k wrote: >> We are seeing rare crashes in jcstress with jdk8 on aarch64 with jfron. Those crashes are real, however we were not able to reproduce with builds on gcc 5 and up. The real culprint was not found. but gcc version on aarch64 should be checked. Ideal version seems 7, but I'm not that bold. >> >> I think this change must be accompanied by regenerating `generated-configure` but am not sure with process on it. > > Ji?? Van?k has updated the pull request incrementally with one additional commit since the last revision: > > Improved error message Thanx! Regenerated with self built 2.69. Output was sane this time, only I manually skipped about 50 hunks like this: @@ -50158,7 +50167,7 @@ ac_compiler_gnu=$ac_cv_cxx_compiler_gnu /* end confdefs.h. */ int -main (void) +main () { return 0; ; I guess thats done by too new ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/697#issuecomment-3305957685 From jjiang at openjdk.org Thu Sep 18 09:37:14 2025 From: jjiang at openjdk.org (John Jiang) Date: Thu, 18 Sep 2025 09:37:14 GMT Subject: [jdk8u-dev] RFR: 8240226: DeflateIn_InflateOut.java test incorrectly assumes size of compressed file In-Reply-To: References: Message-ID: On Thu, 18 Sep 2025 07:07:08 GMT, John Jiang wrote: > `java/util/zip/DeflateIn_InflateOut.java` failed with a system zlib. > The backport is clean. This backport is test-only and should not caused those failures. They look known issues, e.g. JDK-8303159 and JDK-8367256. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/698#issuecomment-3306481642 From sgehwolf at openjdk.org Thu Sep 18 09:40:50 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Thu, 18 Sep 2025 09:40:50 GMT Subject: [jdk8u-dev] RFR: 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test [v3] In-Reply-To: References: Message-ID: On Wed, 17 Sep 2025 16:37:54 GMT, Severin Gehwolf wrote: >> Another problem list to get GHA green (at last). > > Severin Gehwolf has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: > > - Merge branch 'master' into jdk-8367257-problemlist-entrustrootcag4 > - 8367257: [8u] Problem list CAInterop.java#entrustrootcag4 test > - 8367115: [8u] Problem list CAInterop.java#actalisauthenticationrootca test Of course we see the intermittent digicert failures now. `ssslrootevrsaca` seems to be intermittent as well. Good times. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/696#issuecomment-3306496710 From fferrari at openjdk.org Fri Sep 19 10:28:56 2025 From: fferrari at openjdk.org (Francisco Ferrari Bihurriet) Date: Fri, 19 Sep 2025 10:28:56 GMT Subject: [jdk8u-dev] RFR: 8339280: jarsigner -verify performs cross-checking between CEN and LOC Message-ID: Hi, this is a backport of openjdk/jdk11u-dev#3098. The backport isn't clean, because of the following: * File paths that require adjustment for 8u * In `Main.java`, trivial context differences and code that needs adjustment from 11u to 8u: * `java.nio.file.Path.of()` ? `java.nio.file.Paths.get()` * `InputStream::transferTo(OutputStream.nullOutputStream())` ? [original code before a review suggestion](https://github.com/openjdk/jdk/pull/23532/commits/ef546f4b778e8a1f43dbedc56a79bb53d169a1e7#diff-7083af3b8473a092987afa0bbb4d1694664649534bac716f6b9cd3c3b9833219L1185-L1186) * In the test (`VerifyJarEntryName.java`), code that needs adjustment from 11u to 8u: * `@library /test/lib` ? `@library /lib/testlibrary` * JUnit 5 ? JUnit 4 * `org.junit.jupiter.api.BeforeAll` ? `org.junit.BeforeClass` * `org.junit.jupiter.api.BeforeEach` ? `org.junit.Before` * `org.junit.jupiter.api.Test` ? `org.junit.Test` * `java.nio.file.Path.of()` ? `java.nio.file.Paths.get()` * `@BeforeClass`, `@Before` and `@Test` methods must be public * Can't use `var` * `Arrays.equals(a, start, end, b, 0, b.length)` ? `Arrays.equals(Arrays.copyOfRange(a, start, end), b)` (`Arrays::equals` with offsets not present in 8u) * NOTE: for the Japanese manpages translation approach, please refer to openjdk/jdk11u-dev#3098 * I updated `jdk/src/linux/doc/man/ja/jarsigner.1` and `jdk/src/solaris/doc/sun/man/man1/ja/jarsigner.1` (identical), and left `dk/src/bsd/doc/man/ja/jarsigner.1` untouched (doesn't have any content besides the headers) #### Related issues ("relates to" Jira issue links) [JDK-8353299](https://bugs.openjdk.org/browse/JDK-8353299 "VerifyJarEntryName.java test fails") (openjdk/jdk at acd4da49a01760599ec4c325ff6c56f53ba5cc9c) and [JDK-8367782](https://bugs.openjdk.org/browse/JDK-8367782 "VerifyJarEntryName.java: Fix modifyJarEntryName to operate on bytes and re-introduce verifySignatureEntryName") (openjdk/jdk at TBD) were also included as part of this backport. They are test-only changes that improve the reliability and coverage of `VerifyJarEntryName.java`. Since `test/hotspot/jtreg/runtime/appcds/SignedJar.java` is not present in 8u, [JDK-8353330](https://bugs.openjdk.org/browse/JDK-8353330 "Test runtime/cds/appcds/SignedJar.java fails in CDSHeapVerifier") was not included. #### Testing * Besides the `tier1` run from the [GitHub actions](https://github.com/franferrax/jdk8u-dev/actions/runs/17843526748) (similar results as the [current `master` run](https://github.com/kvergizova/jdk8u-dev/actions/runs/17378479373/job/49331554816)), I ran a regression using the following categories and individual tests: * `jdk/test/com/sun/jarsigner` * `jdk/test/java/security/SignedJar` * `jdk/test/java/util/jar` * `jdk/test/sun/security/pkcs/pkcs7` * `jdk/test/sun/security/tools/jarsigner` * Includes `VerifyJarEntryName.java`, created for this issue * `jdk/test/sun/security/tools/keytool` No regressions were found against the current `master` branch (9a4bc2d205c9b8b5f743f16a2ea8b85d6cb6924b). ------------- Commit messages: - Backport 8353299 & 8367782 - Backport bbd5b174c50346152a624317b6bd76ec48f7e551 Changes: https://git.openjdk.org/jdk8u-dev/pull/699/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=699&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8339280 Stats: 401 lines in 10 files changed: 401 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/699.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/699/head:pull/699 PR: https://git.openjdk.org/jdk8u-dev/pull/699 From aph at openjdk.org Fri Sep 19 13:42:22 2025 From: aph at openjdk.org (Andrew Haley) Date: Fri, 19 Sep 2025 13:42:22 GMT Subject: [jdk8u-dev] RFR: 8219006: AArch64: Register corruption in slow subtype check Message-ID: The backport is clean except for changed pathnames. ------------- Commit messages: - Backport 4ac927853957fe691692137a6cabbd1b141b95d4 Changes: https://git.openjdk.org/jdk8u-dev/pull/700/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=700&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8219006 Stats: 15 lines in 1 file changed: 5 ins; 1 del; 9 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/700.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/700/head:pull/700 PR: https://git.openjdk.org/jdk8u-dev/pull/700 From phh at openjdk.org Fri Sep 19 14:40:22 2025 From: phh at openjdk.org (Paul Hohensee) Date: Fri, 19 Sep 2025 14:40:22 GMT Subject: [jdk8u-dev] RFR: 8219006: AArch64: Register corruption in slow subtype check In-Reply-To: References: Message-ID: On Fri, 19 Sep 2025 13:33:30 GMT, Andrew Haley wrote: > The backport is clean except for changed pathnames. Marked as reviewed by phh (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk8u-dev/pull/700#pullrequestreview-3245218398 From shade at openjdk.org Fri Sep 19 15:17:59 2025 From: shade at openjdk.org (Aleksey Shipilev) Date: Fri, 19 Sep 2025 15:17:59 GMT Subject: [jdk8u-dev] RFR: 8219006: AArch64: Register corruption in slow subtype check In-Reply-To: References: Message-ID: <9OAL4aaBQIhlCM0RtV01zOgIGkDVobNwttmxGLofzq4=.f7bce6e4-62bd-439c-b691-cbe97acd58de@github.com> On Fri, 19 Sep 2025 13:33:30 GMT, Andrew Haley wrote: > The backport is clean except for changed pathnames. The backport looks good. I see the original analysis figured this is a problem for debug builds. Have you seen in failing in release builds? ------------- Marked as reviewed by shade (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/700#pullrequestreview-3245407861 From aph at openjdk.org Fri Sep 19 15:22:29 2025 From: aph at openjdk.org (Andrew Haley) Date: Fri, 19 Sep 2025 15:22:29 GMT Subject: [jdk8u-dev] RFR: 8219006: AArch64: Register corruption in slow subtype check In-Reply-To: <9OAL4aaBQIhlCM0RtV01zOgIGkDVobNwttmxGLofzq4=.f7bce6e4-62bd-439c-b691-cbe97acd58de@github.com> References: <9OAL4aaBQIhlCM0RtV01zOgIGkDVobNwttmxGLofzq4=.f7bce6e4-62bd-439c-b691-cbe97acd58de@github.com> Message-ID: On Fri, 19 Sep 2025 15:15:51 GMT, Aleksey Shipilev wrote: > I see the original analysis figured this is a problem for debug builds. Have you seen in failing in release builds? No, and I guess that's why we didn't backport it at the time, but we're testing non-product builds. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/700#issuecomment-3312618169 From shade at openjdk.org Fri Sep 19 15:29:53 2025 From: shade at openjdk.org (Aleksey Shipilev) Date: Fri, 19 Sep 2025 15:29:53 GMT Subject: [jdk8u-dev] RFR: 8219006: AArch64: Register corruption in slow subtype check In-Reply-To: References: <9OAL4aaBQIhlCM0RtV01zOgIGkDVobNwttmxGLofzq4=.f7bce6e4-62bd-439c-b691-cbe97acd58de@github.com> Message-ID: On Fri, 19 Sep 2025 15:19:50 GMT, Andrew Haley wrote: > > I see the original analysis figured this is a problem for debug builds. Have you seen in failing in release builds? > > No, and I guess that's why we didn't backport it at the time, but we're testing non-product builds. OK! ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/700#issuecomment-3312643578 From fferrari at openjdk.org Fri Sep 19 18:51:57 2025 From: fferrari at openjdk.org (Francisco Ferrari Bihurriet) Date: Fri, 19 Sep 2025 18:51:57 GMT Subject: [jdk8u-dev] RFR: 8339280: jarsigner -verify performs cross-checking between CEN and LOC [v2] In-Reply-To: References: Message-ID: > Hi, this is a backport of openjdk/jdk11u-dev#3098. The backport isn't clean, because of the following: > > * File paths that require adjustment for 8u > * In `Main.java`, trivial context differences and code that needs adjustment from 11u to 8u: > * `java.nio.file.Path.of()` ? `java.nio.file.Paths.get()` > * `InputStream::transferTo(OutputStream.nullOutputStream())` ? [original code before a review suggestion](https://github.com/openjdk/jdk/pull/23532/commits/ef546f4b778e8a1f43dbedc56a79bb53d169a1e7#diff-7083af3b8473a092987afa0bbb4d1694664649534bac716f6b9cd3c3b9833219L1185-L1186) > * In the test (`VerifyJarEntryName.java`), code that needs adjustment from 11u to 8u: > * `@library /test/lib` ? `@library /lib/testlibrary` > * JUnit 5 ? JUnit 4 > * `org.junit.jupiter.api.BeforeAll` ? `org.junit.BeforeClass` > * `org.junit.jupiter.api.BeforeEach` ? `org.junit.Before` > * `org.junit.jupiter.api.Test` ? `org.junit.Test` > * `java.nio.file.Path.of()` ? `java.nio.file.Paths.get()` > * `@BeforeClass`, `@Before` and `@Test` methods must be public > * Can't use `var` > * `Arrays.equals(a, start, end, b, 0, b.length)` ? `Arrays.equals(Arrays.copyOfRange(a, start, end), b)` (`Arrays::equals` with offsets not present in 8u) > * ~~NOTE: for the Japanese manpages translation approach, please refer to openjdk/jdk11u-dev#3098~~ > * ~~I updated `jdk/src/linux/doc/man/ja/jarsigner.1` and `jdk/src/solaris/doc/sun/man/man1/ja/jarsigner.1` (identical), and left `dk/src/bsd/doc/man/ja/jarsigner.1` untouched (doesn't have any content besides the headers)~~ > * $\mbox{\color{red}UPDATE}$ (28348b29f892b685d9098c4560128a8bfb206afd): all the internationalized messages have been removed, as they aren't typically included in backports (thanks @jerboaa for letting me know). > > #### Related issues ("relates to" Jira issue links) > > [JDK-8353299](https://bugs.openjdk.org/browse/JDK-8353299 "VerifyJarEntryName.java test fails") (openjdk/jdk at acd4da49a01760599ec4c325ff6c56f53ba5cc9c) and [JDK-8367782](https://bugs.openjdk.org/browse/JDK-8367782 "VerifyJarEntryName.java: Fix modifyJarEntryName to operate on bytes and re-introduce verifySignatureEntryName") (openjdk/jdk at 1b9a11682d5f73885213822423bfce8dfc17febd) were also included as part of this backport. They are test-only changes that improve the reliability and coverage of `VerifyJarEntryName.java`. > > Since `test/hotspot/jtreg/runtime/appcds/SignedJar.java` is not present in 8u, [JDK-8353330](https://bugs.open... Francisco Ferrari Bihurriet has updated the pull request incrementally with one additional commit since the last revision: Remove internationalized messages ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/699/files - new: https://git.openjdk.org/jdk8u-dev/pull/699/files/d1c0f3ef..28348b29 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=699&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=699&range=00-01 Stats: 66 lines in 4 files changed: 0 ins; 66 del; 0 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/699.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/699/head:pull/699 PR: https://git.openjdk.org/jdk8u-dev/pull/699 From aph at openjdk.org Mon Sep 22 12:51:01 2025 From: aph at openjdk.org (Andrew Haley) Date: Mon, 22 Sep 2025 12:51:01 GMT Subject: [jdk8u-dev] Integrated: 8219006: AArch64: Register corruption in slow subtype check In-Reply-To: References: Message-ID: On Fri, 19 Sep 2025 13:33:30 GMT, Andrew Haley wrote: > The backport is clean except for changed pathnames. This pull request has now been integrated. Changeset: 26a41cde Author: Andrew Haley URL: https://git.openjdk.org/jdk8u-dev/commit/26a41cde17a326807c070367a62b9213ce5a8de1 Stats: 15 lines in 1 file changed: 5 ins; 1 del; 9 mod 8219006: AArch64: Register corruption in slow subtype check Reviewed-by: phh, shade Backport-of: 4ac927853957fe691692137a6cabbd1b141b95d4 ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/700 From avieiro at openjdk.org Tue Sep 23 08:32:54 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Tue, 23 Sep 2025 08:32:54 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel Message-ID: Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from 17 (11 is still pending approval) that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. The issue is reproducible in OpenJDK 8 with the Windows Look and Feel on Windows 11. - Passes tier1 tests - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) The backport is separated in three commits: - Backport of JDK-8348760 - Modify the new test `jdk/test/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` to not depend on `@library /java/awt/regtesthelpers` / `@PassFailJFrame`, since `PassFailJFrame` is not available in JDK8. - Backport of JDK-8365375 NOTE: Related issues [JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. ------------- Commit messages: - Backport 3758e584e70c230e8d0294b4e9e7723ae2ed0a13 - Update test not to use PassFailJFrame not available in JDK8 - Backport 2ed2edb244d0a6bf1a156f2a4a346986388c4236 Changes: https://git.openjdk.org/jdk8u/pull/77/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u&pr=77&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8348760 Stats: 670 lines in 8 files changed: 564 ins; 82 del; 24 mod Patch: https://git.openjdk.org/jdk8u/pull/77.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/77/head:pull/77 PR: https://git.openjdk.org/jdk8u/pull/77 From avieiro at openjdk.org Tue Sep 23 08:32:55 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Tue, 23 Sep 2025 08:32:55 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: On Tue, 23 Sep 2025 08:22:41 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from 17 (11 is still pending approval) that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The issue is reproducible in OpenJDK 8 with the Windows Look and Feel on Windows 11. > > - Passes tier1 tests > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > The backport is separated in three commits: > > - Backport of JDK-8348760 > - Modify the new test `jdk/test/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` to not depend on `@library /java/awt/regtesthelpers` / `@PassFailJFrame`, since `PassFailJFrame` is not available in JDK8. > - Backport of JDK-8365375 > > > > NOTE: Related issues [JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. The updated `jdk/test/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` test, showing the `JRadioButtonMenuItem`s in Windows 11 / Windows Look and Feel. backports-jdk-8348760-jdk8 ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/77#issuecomment-3322921839 From sgehwolf at redhat.com Tue Sep 23 08:36:17 2025 From: sgehwolf at redhat.com (Severin Gehwolf) Date: Tue, 23 Sep 2025 10:36:17 +0200 Subject: Result: New JDK 8 Updates Project Committer: Sendao Yan Message-ID: Voting for Sendao Yan, OpenJDK ID 'syan', is now closed [0]. Yes: 4 Veto: 0 Abstain: 0 According to the Bylaws definition of Lazy Consensus [1], this is sufficient to approve the nomination. Thanks, Severin [0] https://mail.openjdk.org/pipermail/jdk8u-dev/2025-September/020465.html [1] https://openjdk.java.net/bylaws#lazy-consensus From rmarchenko at openjdk.org Tue Sep 23 12:18:36 2025 From: rmarchenko at openjdk.org (Roman Marchenko) Date: Tue, 23 Sep 2025 12:18:36 GMT Subject: [jdk8u-dev] RFR: 8350813: Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError Message-ID: This is backport of "[JDK-8350813](https://bugs.openjdk.org/browse/JDK-8350813) Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError" The problem exists in jdk8 also. Almost clean backport, resolved conflict in copyright, and put `BulkSoundBank.java` to `jdk/test/javax/sound/midi/BulkSoundBank/`. Test fails before the fix, and passes with the fix. ------------- Commit messages: - Backport fcc2a24291d499f7149debad1250903ddc369d91 Changes: https://git.openjdk.org/jdk8u-dev/pull/702/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=702&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8350813 Stats: 61 lines in 2 files changed: 60 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/702.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/702/head:pull/702 PR: https://git.openjdk.org/jdk8u-dev/pull/702 From andrew at openjdk.org Tue Sep 23 15:29:39 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Tue, 23 Sep 2025 15:29:39 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: <_WunWwyTd7TkQ6_7ybJ7WVsH6wVilDNkme-odBUeOXM=.f36e0099-e79e-48c9-9ee8-9bec04201842@github.com> On Tue, 23 Sep 2025 08:22:41 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from 17 (11 is still pending approval) that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The issue is reproducible in OpenJDK 8 with the Windows Look and Feel on Windows 11. > > - Passes tier1 tests > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > The backport is separated in three commits: > > - Backport of JDK-8348760 > - Modify the new test `jdk/test/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` to not depend on `@library /java/awt/regtesthelpers` / `@PassFailJFrame`, since `PassFailJFrame` is not available in JDK8. > - Backport of JDK-8365375 > > > > NOTE: Related issues [JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. This seems a bit premature when the 11u one is still in review. I just left comments about further changes needed there. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/77#issuecomment-3324512498 From avieiro at openjdk.org Wed Sep 24 08:00:49 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Wed, 24 Sep 2025 08:00:49 GMT Subject: [jdk8u] Withdrawn: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: On Tue, 23 Sep 2025 08:22:41 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from 17 (11 is still pending approval) that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The issue is reproducible in OpenJDK 8 with the Windows Look and Feel on Windows 11. > > - Passes tier1 tests > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > The backport is separated in three commits: > > - Backport of JDK-8348760 > - Modify the new test `jdk/test/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` to not depend on `@library /java/awt/regtesthelpers` / `@PassFailJFrame`, since `PassFailJFrame` is not available in JDK8. > - Backport of JDK-8365375 > > > > NOTE: Related issues [JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. This pull request has been closed without being integrated. ------------- PR: https://git.openjdk.org/jdk8u/pull/77 From rmarchenko at openjdk.org Wed Sep 24 08:11:12 2025 From: rmarchenko at openjdk.org (Roman Marchenko) Date: Wed, 24 Sep 2025 08:11:12 GMT Subject: [jdk8u-dev] RFR: 8350813: Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError In-Reply-To: References: Message-ID: On Tue, 23 Sep 2025 12:13:43 GMT, Roman Marchenko wrote: > This is backport of "[JDK-8350813](https://bugs.openjdk.org/browse/JDK-8350813) Rendering of bulky sound bank from MIDI sequence can cause OutOfMemoryError" > > The problem exists in jdk8 also. > > Almost clean backport, resolved conflict in copyright, and put `BulkSoundBank.java` to `jdk/test/javax/sound/midi/BulkSoundBank/`. > > Test fails before the fix, and passes with the fix. The failed CertPathValidator tests don't look related to the change. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/702#issuecomment-3327121884 From serb at openjdk.org Thu Sep 25 01:15:59 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Thu, 25 Sep 2025 01:15:59 GMT Subject: [jdk8u-dev] RFR: 8305072: Win32ShellFolder2.compareTo is inconsistent [v3] In-Reply-To: References: Message-ID: > Hi all, > > This pull request contains a backport of commit [2fcb8168](https://github.com/openjdk/jdk/commit/2fcb816858406f33cefef3164b2c85f9f996c7da) from the [openjdk/jdk](https://git.openjdk.org/jdk) repository. > > The commit being backported was authored by Alexey Ivanov on 9 Apr 2024 and was reviewed by Phil Race and Sergey Bylokhov. > > The patch is mostly clean, I had to delete `@modules` and `--add-opens` in the test see https://github.com/openjdk/jdk8u-dev/pull/522/commits/2dcba6b8048de90e29c5cd8eaca3be407db2c24e > > The fix verified by the test, no new issues were found by the `:jdk_desktop` tests > > Thanks! Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision: - Merge branch 'openjdk:master' into backport-mrserb-2fcb8168-master - Merge branch 'openjdk:master' into backport-mrserb-2fcb8168-master - Delete @modules - Merge branch 'openjdk:master' into backport-mrserb-2fcb8168-master - Merge branch 'openjdk:master' into backport-mrserb-2fcb8168-master - Backport 2fcb816858406f33cefef3164b2c85f9f996c7da ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/522/files - new: https://git.openjdk.org/jdk8u-dev/pull/522/files/f76a2bc3..0decb232 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=522&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=522&range=01-02 Stats: 13353 lines in 398 files changed: 8609 ins; 2884 del; 1860 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/522.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/522/head:pull/522 PR: https://git.openjdk.org/jdk8u-dev/pull/522 From syan at openjdk.org Thu Sep 25 02:57:04 2025 From: syan at openjdk.org (SendaoYan) Date: Thu, 25 Sep 2025 02:57:04 GMT Subject: [jdk8u-dev] RFR: 8186149: quarantine gc/survivorAlignment/TestPromotionFromSurvivorToTenuredAfterMinorGC.java [v2] In-Reply-To: References: Message-ID: > Hi all, > > I found that gc/survivorAlignment/TestPromotionFromSurvivorToTenuredAfterMinorGC.java intermittent fails which has been fixed by jdk13 in [JDK-8218049](https://bugs.openjdk.org/browse/JDK-8218049). I think this fixed PR do not suitable to backport from jdk13 to jdk8u. So I want to backport [JDK-8186149](https://bugs.openjdk.org/browse/JDK-8186149) to quarantine this fragile test. > > Backport not clean because there are many different context between jdk8u and jdk13 for hotspot/test/ProblemList.txt > > Only touch the Problemlist, no risk. SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision: - Merge branch 'openjdk:master' into jbs8186149 - Backport 19ead83208381512fa2c89673b8d992b6106cc1c ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/683/files - new: https://git.openjdk.org/jdk8u-dev/pull/683/files/2d8289b0..ee59d45d Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=683&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=683&range=00-01 Stats: 4025 lines in 70 files changed: 2258 ins; 1214 del; 553 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/683.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/683/head:pull/683 PR: https://git.openjdk.org/jdk8u-dev/pull/683 From serb at openjdk.org Thu Sep 25 05:15:39 2025 From: serb at openjdk.org (Sergey Bylokhov) Date: Thu, 25 Sep 2025 05:15:39 GMT Subject: [jdk8u-dev] RFR: 8160974: [TESTBUG] Mark more headful tests with @key headful. [v6] In-Reply-To: References: Message-ID: On Wed, 12 Feb 2025 08:41:05 GMT, Kazuhisa Takakuri wrote: >> Kazuhisa Takakuri has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits: >> >> - Merge branch 'master' into 8160974 >> - Backport e52b963d0cdc49e2048757eae2e4b10bb279f296 >> - Backport e52b963d0cdc49e2048757eae2e4b10bb279f296 >> - Merge branch 'master' into 8160974 >> - Merge branch 'master' into 8160974 >> - Backport e52b963d0cdc49e2048757eae2e4b10bb279f296 > > This pull request is pending approval of the Fix Request. > I comment to not close. Since https://github.com/openjdk/jdk8u-dev/pull/306 has been integrated, we can revisit this now. @ktakakuri, do you plan to reopen it? ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/383#issuecomment-3332174293 From avieiro at openjdk.org Thu Sep 25 07:20:47 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Thu, 25 Sep 2025 07:20:47 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel Message-ID: Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. - Passes tier1 tests on all platforms. - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. ------------- Commit messages: - Update test not to use PassFailJFrame - Backport 31fc05c4721aa1aaf12e5f326a52c1db48006abb Changes: https://git.openjdk.org/jdk8u/pull/78/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u&pr=78&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8348760 Stats: 698 lines in 8 files changed: 592 ins; 82 del; 24 mod Patch: https://git.openjdk.org/jdk8u/pull/78.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/78/head:pull/78 PR: https://git.openjdk.org/jdk8u/pull/78 From avieiro at openjdk.org Thu Sep 25 07:20:47 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Thu, 25 Sep 2025 07:20:47 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: <7xYVggvYeMVOT0gSs_E5DPQhFfs2hqkuj-_1xlrgf0Y=.cc7114b2-7e07-4eec-b71c-9b404826e06c@github.com> On Thu, 25 Sep 2025 07:11:15 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. > > - Passes tier1 tests on all platforms. > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. The result of the refactored test in Windows 11. backports-2025-09-24 ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/78#issuecomment-3332506480 From syan at openjdk.org Thu Sep 25 14:36:57 2025 From: syan at openjdk.org (SendaoYan) Date: Thu, 25 Sep 2025 14:36:57 GMT Subject: [jdk8u-dev] RFR: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer [v5] In-Reply-To: References: Message-ID: > Hi all, > > When running hotspot/test/gc/g1/TestG1TraceEagerReclaimHumongousObjects.java on a CPU with more than 200 physical threads, the jvm will crashes. The reason is that the testcase turn on the gc log, which prints the statistics of each gc thread. If the machine has more cores, more gc threads will be turned on (143 gc threads on a machine with 224 physical threads). In the G1GCParPhasePrinter::print_time_values function (hotspot/src/share/vm/gc_implementation/g1/g1GCPhaseTimes.cpp), the relevant statistics of all gc threads are concatenated into one line, and the string concatenation content is saved in the array defined by g1GCPhaseTimes.cpp::LineBuffer::_buffer. Therefore, on machines with a large number of physical threads, it is easy for the GC log output line length to exceed the predefined buffer size. When the buffer size is exceeded, an error occurs when calling the os::vsnprintf function. > In JDK9, JDK-8150068 refactors the relevant GC log output, so buffer overflow will no longer occur. However, JDK-8150068 is a new feature, and JDK-8150068 cannot be directly backported to jdk8u. In addition, the amount of JDK-8150068 code is large, and the risk of backporting to jdk8u is also very high. Therefore, this PR changes the buffer length to 1024*3 to ensure that there will be no problems with GC log output in some scenarios, and leave a certain margin. > > In addition, this PR adds a guarantee statement to ensure that an error is reported before calling os::vsnprintf when the buffer overflows, which is conducive to the rapid location of the problem > > Change has been verified locally, risk is low. > > Additional testing: > > - [x] jtreg tests include tier1/2/3 etc.. on linux-x64 with release build SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision: - Merge branch 'openjdk:master' into jbs8362208 - Should be _cur >= BUFFER_LEN - fix bugs - Remove guarantee and add debug only warning, then return early when "previous LineBuffer overflow, request ignored" - 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/668/files - new: https://git.openjdk.org/jdk8u-dev/pull/668/files/811ce78a..5b616299 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=668&range=04 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=668&range=03-04 Stats: 5964 lines in 192 files changed: 3395 ins; 1547 del; 1022 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/668.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/668/head:pull/668 PR: https://git.openjdk.org/jdk8u-dev/pull/668 From syan at openjdk.org Thu Sep 25 14:42:22 2025 From: syan at openjdk.org (SendaoYan) Date: Thu, 25 Sep 2025 14:42:22 GMT Subject: [jdk8u-dev] RFR: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer [v5] In-Reply-To: References: Message-ID: On Thu, 25 Sep 2025 14:36:57 GMT, SendaoYan wrote: >> Hi all, >> >> When running hotspot/test/gc/g1/TestG1TraceEagerReclaimHumongousObjects.java on a CPU with more than 200 physical threads, the jvm will crashes. The reason is that the testcase turn on the gc log, which prints the statistics of each gc thread. If the machine has more cores, more gc threads will be turned on (143 gc threads on a machine with 224 physical threads). In the G1GCParPhasePrinter::print_time_values function (hotspot/src/share/vm/gc_implementation/g1/g1GCPhaseTimes.cpp), the relevant statistics of all gc threads are concatenated into one line, and the string concatenation content is saved in the array defined by g1GCPhaseTimes.cpp::LineBuffer::_buffer. Therefore, on machines with a large number of physical threads, it is easy for the GC log output line length to exceed the predefined buffer size. When the buffer size is exceeded, an error occurs when calling the os::vsnprintf function. >> In JDK9, JDK-8150068 refactors the relevant GC log output, so buffer overflow will no longer occur. However, JDK-8150068 is a new feature, and JDK-8150068 cannot be directly backported to jdk8u. In addition, the amount of JDK-8150068 code is large, and the risk of backporting to jdk8u is also very high. Therefore, this PR changes the buffer length to 1024*3 to ensure that there will be no problems with GC log output in some scenarios, and leave a certain margin. >> >> In addition, this PR adds a guarantee statement to ensure that an error is reported before calling os::vsnprintf when the buffer overflows, which is conducive to the rapid location of the problem >> >> Change has been verified locally, risk is low. >> >> Additional testing: >> >> - [x] jtreg tests include tier1/2/3 etc.. on linux-x64 with release build > > SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision: > > - Merge branch 'openjdk:master' into jbs8362208 > - Should be _cur >= BUFFER_LEN > - fix bugs > - Remove guarantee and add debug only warning, then return early when "previous LineBuffer overflow, request ignored" > - 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer Looking for the 2rd reviewer for this bug fix. The buffer is used for gc log concatenation ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/668#issuecomment-3334504090 From stuefe at openjdk.org Thu Sep 25 14:48:07 2025 From: stuefe at openjdk.org (Thomas Stuefe) Date: Thu, 25 Sep 2025 14:48:07 GMT Subject: [jdk8u-dev] RFR: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer [v5] In-Reply-To: References: Message-ID: <-Kfq0TetKjixWsKeCYWzhhY3Z8Jw4sWgfRHAyzTpds8=.b2c72b8e-886d-49e9-a726-d17add121d1d@github.com> On Thu, 25 Sep 2025 14:36:57 GMT, SendaoYan wrote: >> Hi all, >> >> When running hotspot/test/gc/g1/TestG1TraceEagerReclaimHumongousObjects.java on a CPU with more than 200 physical threads, the jvm will crashes. The reason is that the testcase turn on the gc log, which prints the statistics of each gc thread. If the machine has more cores, more gc threads will be turned on (143 gc threads on a machine with 224 physical threads). In the G1GCParPhasePrinter::print_time_values function (hotspot/src/share/vm/gc_implementation/g1/g1GCPhaseTimes.cpp), the relevant statistics of all gc threads are concatenated into one line, and the string concatenation content is saved in the array defined by g1GCPhaseTimes.cpp::LineBuffer::_buffer. Therefore, on machines with a large number of physical threads, it is easy for the GC log output line length to exceed the predefined buffer size. When the buffer size is exceeded, an error occurs when calling the os::vsnprintf function. >> In JDK9, JDK-8150068 refactors the relevant GC log output, so buffer overflow will no longer occur. However, JDK-8150068 is a new feature, and JDK-8150068 cannot be directly backported to jdk8u. In addition, the amount of JDK-8150068 code is large, and the risk of backporting to jdk8u is also very high. Therefore, this PR changes the buffer length to 1024*3 to ensure that there will be no problems with GC log output in some scenarios, and leave a certain margin. >> >> In addition, this PR adds a guarantee statement to ensure that an error is reported before calling os::vsnprintf when the buffer overflows, which is conducive to the rapid location of the problem >> >> Change has been verified locally, risk is low. >> >> Additional testing: >> >> - [x] jtreg tests include tier1/2/3 etc.. on linux-x64 with release build > > SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision: > > - Merge branch 'openjdk:master' into jbs8362208 > - Should be _cur >= BUFFER_LEN > - fix bugs > - Remove guarantee and add debug only warning, then return early when "previous LineBuffer overflow, request ignored" > - 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer Ok. ------------- Marked as reviewed by stuefe (Reviewer). PR Review: https://git.openjdk.org/jdk8u-dev/pull/668#pullrequestreview-3268155178 From sgehwolf at openjdk.org Thu Sep 25 18:42:36 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Thu, 25 Sep 2025 18:42:36 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: <1IViJTR2rdZzkwtUHLVloUWbdLjmqPhXajZuRhwEKCc=.334578e3-a4ab-4935-9599-5ba1540df071@github.com> On Thu, 25 Sep 2025 07:11:15 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. > > - Passes tier1 tests on all platforms. > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. There were small differences to the JDK 11 backport: diff --git a/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java b/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java index c0286a74e1..c29297a630 100644 --- a/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java +++ b/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java @@ -1,5 +1,5 @@ /* - * Copyright (c) 2002, 2010, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2002, 2025, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it diff --git a/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java b/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java index 78b11147a7..7fda45b254 100644 --- a/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java +++ b/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java @@ -785,6 +785,7 @@ public class WindowsIconFactory implements Serializable } assert menuItem == null || c == menuItem; Icon icon = getIcon(); + if (type == JCheckBoxMenuItem.class || type == JRadioButtonMenuItem.class) { AbstractButton b = (AbstractButton) c; Please include them so that the patches are more alike. Seems fine otherwise. ------------- Changes requested by sgehwolf (Reviewer). PR Review: https://git.openjdk.org/jdk8u/pull/78#pullrequestreview-3269029414 From avieiro at openjdk.org Thu Sep 25 19:08:29 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Thu, 25 Sep 2025 19:08:29 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: References: Message-ID: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. > > - Passes tier1 tests on all platforms. > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: Copyright and newline typo ------------- Changes: - all: https://git.openjdk.org/jdk8u/pull/78/files - new: https://git.openjdk.org/jdk8u/pull/78/files/a114cf3c..8086c91e Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u&pr=78&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk8u&pr=78&range=00-01 Stats: 2 lines in 2 files changed: 1 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk8u/pull/78.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/78/head:pull/78 PR: https://git.openjdk.org/jdk8u/pull/78 From avieiro at openjdk.org Thu Sep 25 19:08:30 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Thu, 25 Sep 2025 19:08:30 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: <1IViJTR2rdZzkwtUHLVloUWbdLjmqPhXajZuRhwEKCc=.334578e3-a4ab-4935-9599-5ba1540df071@github.com> References: <1IViJTR2rdZzkwtUHLVloUWbdLjmqPhXajZuRhwEKCc=.334578e3-a4ab-4935-9599-5ba1540df071@github.com> Message-ID: On Thu, 25 Sep 2025 18:39:24 GMT, Severin Gehwolf wrote: >> Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: >> >> Copyright and newline typo > > There were small differences to the JDK 11 backport: > > > diff --git a/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java b/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java > index c0286a74e1..c29297a630 100644 > --- a/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java > +++ b/jdk/src/share/classes/com/sun/java/swing/SwingUtilities3.java > @@ -1,5 +1,5 @@ > /* > - * Copyright (c) 2002, 2010, Oracle and/or its affiliates. All rights reserved. > + * Copyright (c) 2002, 2025, Oracle and/or its affiliates. All rights reserved. > * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. > * > * This code is free software; you can redistribute it and/or modify it > diff --git a/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java b/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java > index 78b11147a7..7fda45b254 100644 > --- a/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java > +++ b/jdk/src/share/classes/com/sun/java/swing/plaf/windows/WindowsIconFactory.java > @@ -785,6 +785,7 @@ public class WindowsIconFactory implements Serializable > } > assert menuItem == null || c == menuItem; > Icon icon = getIcon(); > + > if (type == JCheckBoxMenuItem.class > || type == JRadioButtonMenuItem.class) { > AbstractButton b = (AbstractButton) c; > > > Please include them so that the patches are more alike. Seems fine otherwise. Fixed. Thanks @jerboaa ! ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/78#issuecomment-3335573202 From sgehwolf at openjdk.org Thu Sep 25 19:36:15 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Thu, 25 Sep 2025 19:36:15 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: References: Message-ID: <41ZbUgIH2hQzeRHPX2BoC2tm5KYJEzMaYLebT-nLHkU=.1a521b95-e36f-4a01-8d6e-2ecee0125e7f@github.com> On Thu, 25 Sep 2025 19:08:29 GMT, Antonio Vieiro wrote: >> Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. >> >> The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. >> >> - Passes tier1 tests on all platforms. >> - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) >> >> NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. > > Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: > > Copyright and newline typo Looks good. ------------- Marked as reviewed by sgehwolf (Reviewer). PR Review: https://git.openjdk.org/jdk8u/pull/78#pullrequestreview-3269191689 From andrew at openjdk.org Thu Sep 25 20:56:47 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Thu, 25 Sep 2025 20:56:47 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: References: Message-ID: On Thu, 25 Sep 2025 19:08:29 GMT, Antonio Vieiro wrote: >> Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. >> >> The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. >> >> - Passes tier1 tests on all platforms. >> - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) >> >> NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. > > Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: > > Copyright and newline typo Looks good to me too. ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u/pull/78#pullrequestreview-3269439495 From duke at openjdk.org Thu Sep 25 21:24:21 2025 From: duke at openjdk.org (duke) Date: Thu, 25 Sep 2025 21:24:21 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: References: Message-ID: On Thu, 25 Sep 2025 19:08:29 GMT, Antonio Vieiro wrote: >> Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. >> >> The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. >> >> - Passes tier1 tests on all platforms. >> - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) >> >> NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. > > Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: > > Copyright and newline typo @vieiro Your change (at version 8086c91ec1114e636cfbc06c2da685fcd88ba8d2) is now ready to be sponsored by a Committer. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/78#issuecomment-3335981804 From andrew at openjdk.org Fri Sep 26 00:14:23 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 00:14:23 GMT Subject: [jdk8u] RFR: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel [v2] In-Reply-To: References: Message-ID: On Thu, 25 Sep 2025 19:08:29 GMT, Antonio Vieiro wrote: >> Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. >> >> The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. >> >> - Passes tier1 tests on all platforms. >> - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) >> >> NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. > > Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision: > > Copyright and newline typo > Related issues should be backported too, ideally for 2025/10. I've done 8365389 for [11u](https://github.com/openjdk/jdk11u/pull/105) and [8u](https://github.com/openjdk/jdk8u/pull/79). Testing would be welcome. If we can get those in tomorrow, they will make it before freeze. The other changes hasn't been pushed to trunk yet, but it is only further cleanup and a test. 8365389 is the fix. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/78#issuecomment-3336306031 From avieiro at openjdk.org Fri Sep 26 00:14:23 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Fri, 26 Sep 2025 00:14:23 GMT Subject: [jdk8u] Integrated: 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel In-Reply-To: References: Message-ID: <0zjHFpxCWTiZo3mqxwDbkDVaghvOXYWWyMKava0qoCE=.9ad431a4-203c-4947-97e0-79b74ec1d288@github.com> On Thu, 25 Sep 2025 07:11:15 GMT, Antonio Vieiro wrote: > Backport of [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) (P1 - CPU25_10-critical-approved) and [JDK-8365375](https://bugs.openjdk.org/browse/JDK-8365375) from jdk11u, that properly renders JRadioButtonMenuItems with ImageIcons in the Windows Look and Feel on Windows 11. > > The new test `test/jdk/javax/swing/JMenuItem/TestRadioAndCheckMenuItemWithIcon.java` has been slightly refactored in a second commit, since `PassFailJFrame` is not available in `/java/awt/regtesthelpers` in JDK8. > > - Passes tier1 tests on all platforms. > - Passes the new (manual) test on Windows 11/x86_64 (24H2/26100.6584) > > NOTE: Related issues[JDK-8365389](https://bugs.openjdk.org/browse/JDK-8365389) and [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) (all P3) will also need to be backported. This pull request has now been integrated. Changeset: 333a5510 Author: Antonio Vieiro Committer: Andrew John Hughes URL: https://git.openjdk.org/jdk8u/commit/333a55103ed3e480793539f7329bff6e23771047 Stats: 700 lines in 8 files changed: 593 ins; 82 del; 25 mod 8348760: RadioButton is not shown if JRadioButtonMenuItem is rendered with ImageIcon in WindowsLookAndFeel 8365375: Method SU3.setAcceleratorSelectionForeground assigns to acceleratorForeground Reviewed-by: sgehwolf, andrew Backport-of: 31fc05c4721aa1aaf12e5f326a52c1db48006abb ------------- PR: https://git.openjdk.org/jdk8u/pull/78 From andrew at openjdk.org Fri Sep 26 00:15:02 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 00:15:02 GMT Subject: [jdk8u] RFR: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI Message-ID: I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) Re-testing by those with access to an appropriate Windows 11 environment would be helpful. ------------- Depends on: https://git.openjdk.org/jdk8u/pull/78 Commit messages: - Backport 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d Changes: https://git.openjdk.org/jdk8u/pull/79/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u&pr=79&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8365389 Stats: 79 lines in 6 files changed: 14 ins; 51 del; 14 mod Patch: https://git.openjdk.org/jdk8u/pull/79.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/79/head:pull/79 PR: https://git.openjdk.org/jdk8u/pull/79 From andrew at openjdk.org Fri Sep 26 00:21:57 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 00:21:57 GMT Subject: [jdk8u] RFR: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI [v2] In-Reply-To: References: Message-ID: > I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. > Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. > > Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) > > Re-testing by those with access to an appropriate Windows 11 environment would be helpful. Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: Backport 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d ------------- Changes: https://git.openjdk.org/jdk8u/pull/79/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u&pr=79&range=01 Stats: 79 lines in 6 files changed: 14 ins; 51 del; 14 mod Patch: https://git.openjdk.org/jdk8u/pull/79.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/79/head:pull/79 PR: https://git.openjdk.org/jdk8u/pull/79 From syan at openjdk.org Fri Sep 26 02:01:11 2025 From: syan at openjdk.org (SendaoYan) Date: Fri, 26 Sep 2025 02:01:11 GMT Subject: [jdk8u-dev] RFR: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer [v5] In-Reply-To: <-cnWapK8H_C31U4_zgXA04r1oxKKVxRy7klxMKtZR7c=.8d4d3e89-4b5e-4d4c-b45c-c80e80fd97a3@github.com> References: <2z91S7Q94O7xylUdIfLJqM05kjDo_RTMA5hzXJEfHBs=.ebffdee5-5c22-4041-a5ea-e9ed6e86fb6e@github.com> <-cnWapK8H_C31U4_zgXA04r1oxKKVxRy7klxMKtZR7c=.8d4d3e89-4b5e-4d4c-b45c-c80e80fd97a3@github.com> Message-ID: On Fri, 18 Jul 2025 16:54:05 GMT, Paul Hohensee wrote: >> SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision: >> >> - Merge branch 'openjdk:master' into jbs8362208 >> - Should be _cur >= BUFFER_LEN >> - fix bugs >> - Remove guarantee and add debug only warning, then return early when "previous LineBuffer overflow, request ignored" >> - 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer > > Thanks, looks good. Thanks for the suggestions and reviews @phohensee @tstuefe GHA report two jtreg tests security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#entrustrootcag4 security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java#sslrootevrsaca fails. I think it's unrelated to this PR. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/668#issuecomment-3336511587 From avieiro at openjdk.org Fri Sep 26 07:59:58 2025 From: avieiro at openjdk.org (Antonio Vieiro) Date: Fri, 26 Sep 2025 07:59:58 GMT Subject: [jdk8u] RFR: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI [v2] In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 00:21:57 GMT, Andrew John Hughes wrote: >> I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. >> Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. >> >> Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) >> >> Re-testing by those with access to an appropriate Windows 11 environment would be helpful. > > Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: > > Backport 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d Manual test passes on Windows 11: imagen ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/79#issuecomment-3337228478 From sgehwolf at openjdk.org Fri Sep 26 08:47:40 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Fri, 26 Sep 2025 08:47:40 GMT Subject: [jdk8u] RFR: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI [v2] In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 00:21:57 GMT, Andrew John Hughes wrote: >> I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. >> Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. >> >> Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) >> >> Re-testing by those with access to an appropriate Windows 11 environment would be helpful. > > Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: > > Backport 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d > Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) Indeed. Looks good to me. ------------- Marked as reviewed by sgehwolf (Reviewer). PR Review: https://git.openjdk.org/jdk8u/pull/79#pullrequestreview-3270925056 From syan at openjdk.org Fri Sep 26 09:36:10 2025 From: syan at openjdk.org (SendaoYan) Date: Fri, 26 Sep 2025 09:36:10 GMT Subject: [jdk8u-dev] RFR: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer [v4] In-Reply-To: References: <2z91S7Q94O7xylUdIfLJqM05kjDo_RTMA5hzXJEfHBs=.ebffdee5-5c22-4041-a5ea-e9ed6e86fb6e@github.com> Message-ID: On Thu, 28 Aug 2025 08:38:54 GMT, Severin Gehwolf wrote: >> SendaoYan has updated the pull request incrementally with one additional commit since the last revision: >> >> Should be _cur >= BUFFER_LEN > > @tstuefe Could you perhaps help get this reviewed? Thanks! Thanks for the approved @jerboaa ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/668#issuecomment-3337656635 From syan at openjdk.org Fri Sep 26 09:36:12 2025 From: syan at openjdk.org (SendaoYan) Date: Fri, 26 Sep 2025 09:36:12 GMT Subject: [jdk8u-dev] Integrated: 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer In-Reply-To: References: Message-ID: <0KIm41jAD9nENnkVr2QYkF_H1-_gcMct32eFtR66-k4=.1aece475-59b1-4a4b-9ffe-305218a81906@github.com> On Tue, 15 Jul 2025 06:47:42 GMT, SendaoYan wrote: > Hi all, > > When running hotspot/test/gc/g1/TestG1TraceEagerReclaimHumongousObjects.java on a CPU with more than 200 physical threads, the jvm will crashes. The reason is that the testcase turn on the gc log, which prints the statistics of each gc thread. If the machine has more cores, more gc threads will be turned on (143 gc threads on a machine with 224 physical threads). In the G1GCParPhasePrinter::print_time_values function (hotspot/src/share/vm/gc_implementation/g1/g1GCPhaseTimes.cpp), the relevant statistics of all gc threads are concatenated into one line, and the string concatenation content is saved in the array defined by g1GCPhaseTimes.cpp::LineBuffer::_buffer. Therefore, on machines with a large number of physical threads, it is easy for the GC log output line length to exceed the predefined buffer size. When the buffer size is exceeded, an error occurs when calling the os::vsnprintf function. > In JDK9, JDK-8150068 refactors the relevant GC log output, so buffer overflow will no longer occur. However, JDK-8150068 is a new feature, and JDK-8150068 cannot be directly backported to jdk8u. In addition, the amount of JDK-8150068 code is large, and the risk of backporting to jdk8u is also very high. Therefore, this PR changes the buffer length to 1024*3 to ensure that there will be no problems with GC log output in some scenarios, and leave a certain margin. > > In addition, this PR adds a guarantee statement to ensure that an error is reported before calling os::vsnprintf when the buffer overflows, which is conducive to the rapid location of the problem > > Change has been verified locally, risk is low. > > Additional testing: > > - [x] jtreg tests include tier1/2/3 etc.. on linux-x64 with release build This pull request has now been integrated. Changeset: 7bbd26b3 Author: SendaoYan URL: https://git.openjdk.org/jdk8u-dev/commit/7bbd26b3263a774731f233a92dc081119eb4f8d5 Stats: 6 lines in 1 file changed: 4 ins; 0 del; 2 mod 8362208: [8u] Buffer overflow in g1GCPhaseTimes.cpp::LineBuffer::_buffer Reviewed-by: phh, stuefe ------------- PR: https://git.openjdk.org/jdk8u-dev/pull/668 From sgehwolf at openjdk.org Fri Sep 26 12:52:31 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Fri, 26 Sep 2025 12:52:31 GMT Subject: [jdk8u] RFR: 8368308: ISO 4217 Amendment 180 Update Message-ID: Not a clean backport to the [JDK 11u version](https://github.com/openjdk/jdk11u/pull/106), but resolves were trivial. Changes to `jdk/test/java/util/Currency/ValidateISO4217.java` where manually redone. Testing: `java/util/Currency/` tests pass. ------------- Commit messages: - Backport 85f5bf3f415cc3d44d1618ec574e73f846bb91c4 Changes: https://git.openjdk.org/jdk8u/pull/80/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u&pr=80&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8368308 Stats: 12 lines in 3 files changed: 0 ins; 0 del; 12 mod Patch: https://git.openjdk.org/jdk8u/pull/80.diff Fetch: git fetch https://git.openjdk.org/jdk8u.git pull/80/head:pull/80 PR: https://git.openjdk.org/jdk8u/pull/80 From andrew at openjdk.org Fri Sep 26 12:52:46 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 12:52:46 GMT Subject: [jdk8u] RFR: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI [v2] In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 00:21:57 GMT, Andrew John Hughes wrote: >> I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. >> Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. >> >> Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) >> >> Re-testing by those with access to an appropriate Windows 11 environment would be helpful. > > Andrew John Hughes has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains one commit: > > Backport 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d Thanks both of you for the quick response and especially the testing. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/79#issuecomment-3338584299 From andrew at openjdk.org Fri Sep 26 13:33:40 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 13:33:40 GMT Subject: [jdk8u] RFR: 8368308: ISO 4217 Amendment 180 Update In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 12:46:21 GMT, Severin Gehwolf wrote: > resolves were trivial The only one I see is the manual application in `ValidateISO4217.java` of the new currency code because of the different indentation. This is due to 8u not having the refactoring in [JDK-8310923](https://bugs.openjdk.org/browse/JDK-8310923) which would in turn require a new JTreg and the headaches that involves (it took a long time to go to 11u , IIRC). Were there any others? ------------- Marked as reviewed by andrew (Reviewer). PR Review: https://git.openjdk.org/jdk8u/pull/80#pullrequestreview-3272304575 From sgehwolf at openjdk.org Fri Sep 26 13:43:53 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Fri, 26 Sep 2025 13:43:53 GMT Subject: [jdk8u] RFR: 8368308: ISO 4217 Amendment 180 Update In-Reply-To: References: Message-ID: <2Vfvx7J3z71qkxtRMJO5Tp9wKQJa2lVDqzBo30OF5pQ=.3432d0ac-5b64-481b-81df-71a85c6a2c8c@github.com> On Fri, 26 Sep 2025 13:30:30 GMT, Andrew John Hughes wrote: > > resolves were trivial > > The only one I see is the manual application in `ValidateISO4217.java` of the new currency code because of the different indentation. This is due to 8u not having the refactoring in [JDK-8310923](https://bugs.openjdk.org/browse/JDK-8310923) which would in turn require a new JTreg and the headaches that involves (it took a long time to go to 11u , IIRC). Were there any others? No. But paths to files are different in 8u as well. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/80#issuecomment-3338774914 From sgehwolf at openjdk.org Fri Sep 26 13:43:54 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Fri, 26 Sep 2025 13:43:54 GMT Subject: [jdk8u] RFR: 8368308: ISO 4217 Amendment 180 Update In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 12:46:21 GMT, Severin Gehwolf wrote: > Not a clean backport to the [JDK 11u version](https://github.com/openjdk/jdk11u/pull/106), but resolves were trivial. Changes to `jdk/test/java/util/Currency/ValidateISO4217.java` where manually redone. > > Testing: `java/util/Currency/` tests pass. Thanks for the quick review! ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/80#issuecomment-3338782952 From andrew at openjdk.org Fri Sep 26 13:52:01 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 13:52:01 GMT Subject: [jdk8u] RFR: 8368308: ISO 4217 Amendment 180 Update In-Reply-To: <2Vfvx7J3z71qkxtRMJO5Tp9wKQJa2lVDqzBo30OF5pQ=.3432d0ac-5b64-481b-81df-71a85c6a2c8c@github.com> References: <2Vfvx7J3z71qkxtRMJO5Tp9wKQJa2lVDqzBo30OF5pQ=.3432d0ac-5b64-481b-81df-71a85c6a2c8c@github.com> Message-ID: On Fri, 26 Sep 2025 13:38:50 GMT, Severin Gehwolf wrote: > > > resolves were trivial > > > > > > The only one I see is the manual application in `ValidateISO4217.java` of the new currency code because of the different indentation. This is due to 8u not having the refactoring in [JDK-8310923](https://bugs.openjdk.org/browse/JDK-8310923) which would in turn require a new JTreg and the headaches that involves (it took a long time to go to 11u , IIRC). Were there any others? > > No. But paths to files are different in 8u as well. Oh yeah, I tend to still count that as clean. The only ones to really worry about are new files. We've had the odd test sneak into the wrong tree (particularly with the big JFR change). If you've patched the same context in an existing file, then this should imply the paths have been shuffled correctly. ------------- PR Comment: https://git.openjdk.org/jdk8u/pull/80#issuecomment-3338823355 From sgehwolf at openjdk.org Fri Sep 26 15:50:12 2025 From: sgehwolf at openjdk.org (Severin Gehwolf) Date: Fri, 26 Sep 2025 15:50:12 GMT Subject: [jdk8u] Integrated: 8368308: ISO 4217 Amendment 180 Update In-Reply-To: References: Message-ID: <7PwmZeKGQ9VH4EGIuoOlPxbNLn9Ww7rBbP03Jsacnhg=.ccba6424-54ba-4740-a0b0-e1a6702cd950@github.com> On Fri, 26 Sep 2025 12:46:21 GMT, Severin Gehwolf wrote: > Not a clean backport to the [JDK 11u version](https://github.com/openjdk/jdk11u/pull/106), but resolves were trivial. Changes to `jdk/test/java/util/Currency/ValidateISO4217.java` where manually redone. > > Testing: `java/util/Currency/` tests pass. This pull request has now been integrated. Changeset: 2e6cbc97 Author: Severin Gehwolf URL: https://git.openjdk.org/jdk8u/commit/2e6cbc976f558ccbf004f2803d82be70ad20dfde Stats: 12 lines in 3 files changed: 0 ins; 0 del; 12 mod 8368308: ISO 4217 Amendment 180 Update Reviewed-by: andrew Backport-of: 85f5bf3f415cc3d44d1618ec574e73f846bb91c4 ------------- PR: https://git.openjdk.org/jdk8u/pull/80 From andrew at openjdk.org Fri Sep 26 15:56:10 2025 From: andrew at openjdk.org (Andrew John Hughes) Date: Fri, 26 Sep 2025 15:56:10 GMT Subject: [jdk8u] Integrated: 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI In-Reply-To: References: Message-ID: On Fri, 26 Sep 2025 00:08:32 GMT, Andrew John Hughes wrote: > I'd like to bring this late to 8u to assure the regression introduced by [JDK-8348760](https://bugs.openjdk.org/browse/JDK-8348760) doesn't get into the October release. > Unfortunately the follow-up change [JDK-8365625](https://bugs.openjdk.org/browse/JDK-8365625) is not yet pushed, but this is further cleanup and a testcase; it is this change that contains the fix for the issue. > > Backport to 8u is clean from the 11u patch, after [shuffling](https://github.com/gnu-andrew/jdk/tree/jdk9u-updated/common/bin) > > Re-testing by those with access to an appropriate Windows 11 environment would be helpful. This pull request has now been integrated. Changeset: 5cffbcb0 Author: Andrew John Hughes URL: https://git.openjdk.org/jdk8u/commit/5cffbcb0344f2cf16682a09519894ba705182241 Stats: 79 lines in 6 files changed: 14 ins; 51 del; 14 mod 8365389: Remove static color fields from SwingUtilities3 and WindowsMenuItemUI Reviewed-by: sgehwolf Backport-of: 3468c6e5ef7e7592cf9484736ce333fbe0eaf34d ------------- PR: https://git.openjdk.org/jdk8u/pull/79 From syan at openjdk.org Sat Sep 27 02:51:26 2025 From: syan at openjdk.org (SendaoYan) Date: Sat, 27 Sep 2025 02:51:26 GMT Subject: [jdk8u-dev] RFR: 8353657: [8u] Test tools/launcher/VersionCheck.java fails with debug build [v3] In-Reply-To: <-1iYYpTJ_TFZx6RhRJOw-_nVQhhivtOY0zFWg8E_AgM=.7ade1453-9e17-488d-84a0-449415ee21de@github.com> References: <-1iYYpTJ_TFZx6RhRJOw-_nVQhhivtOY0zFWg8E_AgM=.7ade1453-9e17-488d-84a0-449415ee21de@github.com> Message-ID: > Hi all, > > Test tools/launcher/VersionCheck.java fails with fastdebug build which report 'jstack.debuginfo: cannot execute binary file'. This failure fixed by [JDK-8237192](https://bugs.openjdk.org/browse/JDK-8237192) in jdk15, only a subset of [JDK-8237192](https://bugs.openjdk.org/browse/JDK-8237192) to fix this test failure. So it's not suitable to backport [JDK-8237192](https://bugs.openjdk.org/browse/JDK-8237192) to jdk8u-dev, then I create this PR to fix this testbug. > > This PR skip all the `*.debuginfo` files to check the '-version' or '-J-version'. Change has been verified localy, test-fix only, no risk. SendaoYan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision: - Merge branch 'openjdk:master' into jbs8353657 - Use NIX_DBGINFO_EXT instead of DEBUGINFO_FILE_EXT - 8353657: [8u] Test tools/launcher/VersionCheck.java fails with debug build ------------- Changes: - all: https://git.openjdk.org/jdk8u-dev/pull/685/files - new: https://git.openjdk.org/jdk8u-dev/pull/685/files/847ec546..6f33ed69 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=685&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=685&range=01-02 Stats: 3282 lines in 66 files changed: 2122 ins; 651 del; 509 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/685.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/685/head:pull/685 PR: https://git.openjdk.org/jdk8u-dev/pull/685 From syan at openjdk.org Tue Sep 30 01:52:14 2025 From: syan at openjdk.org (SendaoYan) Date: Tue, 30 Sep 2025 01:52:14 GMT Subject: [jdk8u-dev] RFR: 8153147: Mark java/net/BindException/Test.java as intermittently failing Message-ID: Hi all, I want to backport JDK-8153147 to mark java/net/BindException/Test.java as intermittently failing. The `@key intermittent` will enable this test disable or auto rerun when fails for some setups. ------------- Commit messages: - Backport c240356dba403e2864b1c21f9133cbc19bb6cbcb Changes: https://git.openjdk.org/jdk8u-dev/pull/703/files Webrev: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=703&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8153147 Stats: 2 lines in 1 file changed: 1 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jdk8u-dev/pull/703.diff Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/703/head:pull/703 PR: https://git.openjdk.org/jdk8u-dev/pull/703 From syan at openjdk.org Tue Sep 30 07:39:45 2025 From: syan at openjdk.org (SendaoYan) Date: Tue, 30 Sep 2025 07:39:45 GMT Subject: [jdk8u-dev] RFR: 8153147: Mark java/net/BindException/Test.java as intermittently failing In-Reply-To: References: Message-ID: On Tue, 30 Sep 2025 01:45:33 GMT, SendaoYan wrote: > Hi all, > > I want to backport JDK-8153147 to mark java/net/BindException/Test.java as intermittently failing. The `@key intermittent` will enable this test disable or auto rerun when fails for some setups. GHA report several test failures - Several security/infra/java/security/cert/CertPathValidator/certification/CAInterop.java tests fails, which recorded by https://bugs.openjdk.org/browse/JDK-8367256. It's unrelated to this backport PR. - Several com/sun/jdi fails on macos-x64, tests fails because of `gethostbyname: unknown host`, seems like test environmental issue - gc/6581734/Test6581734.java timed out, this failure has been recorded by https://bugs.openjdk.org/browse/JDK-8332853, it's unrelated to this backport PR. ------------- PR Comment: https://git.openjdk.org/jdk8u-dev/pull/703#issuecomment-3350412980