Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 09:14:06 GMT, Nizar Benalla wrote: >> Nizar Benalla has updated the pull request incrementally with one additional >> commit since the last revision: >> >> use existing DOCS_JDK_IMAGE_DIR to be consistent. >> >> no ifneq check before assigning the env variable because

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 08:39:49 GMT, Nizar Benalla wrote: >> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following P

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Nizar Benalla
On Fri, 18 Oct 2024 08:39:49 GMT, Nizar Benalla wrote: >> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following P

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 12:46:36 GMT, Nizar Benalla wrote: > I'd like to avoid integrating this right before the weekend, and I still want > to add copyright headers to the other TEST.ROOT files if I can find out when > they were added. I roughly know how to do it. I would say fixing copyrights in

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v12]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 12:51:46 GMT, Nizar Benalla wrote: >> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following P

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Magnus Ihse Bursie
On Fri, 18 Oct 2024 12:21:33 GMT, Erik Joelsson wrote: >> make/RunTests.gmk line 872: >> >>> 870: $1_JTREG_BASIC_OPTIONS += -e:TEST_IMAGE_DIR=$(TEST_IMAGE_DIR) >>> 871: >>> 872: $1_JTREG_BASIC_OPTIONS += >>> -e:DOCS_JDK_IMAGE_DIR=$$(DOCS_JDK_IMAGE_DIR) >> >> I didn't include a null check

Integrated: 8337536: AArch64: Enable BTI branch protection for runtime part

2024-10-18 Thread Fei Gao
On Wed, 7 Aug 2024 10:40:09 GMT, Fei Gao wrote: > This patch enables BTI branch protection for runtime part on Linux/aarch64 > platform. > > Motivation > > 1. Since Fedora 33, glibc+kernel are PAC/BTI enabled by default. User-level > packages can gain additional hardening by compiling with th

Re: RFR: 8337536: AArch64: Enable BTI branch protection for runtime part [v3]

2024-10-18 Thread Fei Gao
On Thu, 15 Aug 2024 15:32:28 GMT, Fei Gao wrote: >> This patch enables BTI branch protection for runtime part on Linux/aarch64 >> platform. >> >> Motivation >> >> 1. Since Fedora 33, glibc+kernel are PAC/BTI enabled by default. User-level >> packages can gain additional hardening by compiling

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
> This is the implementation of JEP 486: Permanently Disable the Security > Manager. See [JEP 486](https://openjdk.org/jeps/486) for more details. The > [CSR](https://bugs.openjdk.org/browse/JDK-8338412) describes in detail the > main changes in the JEP and also includes an apidiff of the specif

Re: RFR: 8342578: GHA: RISC-V: Bootstrap using Debian snapshot is still failing

2024-10-18 Thread Aleksey Shipilev
On Fri, 18 Oct 2024 01:33:18 GMT, Fei Yang wrote: > In JDK-8339548, we switched to use Debian snapshot > (https://snapshot.debian.org/archive/debian/20240228T034848Z/) for bootstrap. > The reason is that we don't have a stable Debian release for RISC-V yet and > Debian "sid" (https://httpredir

Re: RFR: 8342578: GHA: RISC-V: Bootstrap using Debian snapshot is still failing

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 01:33:18 GMT, Fei Yang wrote: > In JDK-8339548, we switched to use Debian snapshot > (https://snapshot.debian.org/archive/debian/20240228T034848Z/) for bootstrap. > The reason is that we don't have a stable Debian release for RISC-V yet and > Debian "sid" (https://httpredir

Re: RFR: 8339570: Add Tidy build support for JDK tests [v13]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 08:47:31 GMT, Nizar Benalla wrote: >> Can I get a review for this patch that adds the necessary changes for local >> support of the `tidy` library. >> >> The dependency can be retrieved by running `make/devkit/createTidyBundle.sh` >> on Linux and MacOs systems. >> >> This

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Kevin Walls
On Fri, 18 Oct 2024 19:03:30 GMT, Sean Mullan wrote: >> This is the implementation of JEP 486: Permanently Disable the Security >> Manager. See [JEP 486](https://openjdk.org/jeps/486) for more details. The >> [CSR](https://bugs.openjdk.org/browse/JDK-8338412) describes in detail the >> main ch

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Thu, 17 Oct 2024 05:54:24 GMT, Alan Bateman wrote: >> Ok, I'll also add an API note to `getClassContext()` to use `StackWalker` >> instead. > > Okay, it already has `@see StackWalker`. My guess is that anything extending > SM to call getClassContext is very old code. If that old code is comp

Re: RFR: 8339570: Add Tidy build support for JDK tests [v13]

2024-10-18 Thread Nizar Benalla
On Fri, 18 Oct 2024 12:26:13 GMT, Erik Joelsson wrote: >> Nizar Benalla 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 15 additional >> commits

Re: RFR: 8339570: Add Tidy build support for JDK tests [v14]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 12:44:36 GMT, Nizar Benalla wrote: >> Can I get a review for this patch that adds the necessary changes for local >> support of the `tidy` library. >> >> The dependency can be retrieved by running `make/devkit/createTidyBundle.sh` >> on Linux and MacOs systems. >> >> This

Re: RFR: 8339480: Build static-jdk image with a statically linked launcher [v7]

2024-10-18 Thread Johan Vos
On Tue, 15 Oct 2024 20:22:52 GMT, Magnus Ihse Bursie wrote: >> As a prerequisite for Hermetic Java, we need a statically linked `java` >> launcher. It should behave like the normal, dynamically linked `java` >> launcher, except that all JDK native libraries should be statically, not >> dynamic

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v12]

2024-10-18 Thread Nizar Benalla
> Please review this change that adds a new test root `docs` dedicated to > testing the documentation, which has been a work in progress for a while. > Tests for links, encoding, HTML, accessibility will be later added in > following PRs. > > We also define a new make target `test-docs` meant

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Nizar Benalla
On Fri, 18 Oct 2024 08:39:49 GMT, Nizar Benalla wrote: >> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following P

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Erik Joelsson
On Fri, 18 Oct 2024 12:29:30 GMT, Magnus Ihse Bursie wrote: >> It's fine to leave out the conditional, but the double `$$` isn't needed and >> I'm surprised if it actually works. That must be some lucky double >> evaluation somewhere in that case. >> >> Suggestion: >> >> $1_JTREG_BASIC_OPT

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Wed, 16 Oct 2024 20:42:11 GMT, Sean Mullan wrote: >> The description for the SecurityException thrown by these methods were >> adjusted to "if access to the screen is denied by desktop environment". If >> you bring back the paragraphs that were removed then you might have to >> adjust the w

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Tue, 15 Oct 2024 17:01:59 GMT, Sean Mullan wrote: >>> While making `LogManager.checkAccess` be a no-op might be more convenient, >>> it could unconditionally permit operations that formerly required a >>> permission check: clearly a bad idea. Always throwing a `SecurityException` >>> is the

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Tue, 15 Oct 2024 14:50:54 GMT, Daniel Fuchs wrote: >> Sean Mullan has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains 97 commits: >> >> - Merge remote-tracking branch 'jdk-sandbox/jep486' into JDK-8338411 >> - Change apiNote to d

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Thu, 17 Oct 2024 17:59:20 GMT, Sean Mullan wrote: >> All JGSS permission classes follow the same style: >> >> In `javax.security.auth.kerberos.DelegationPermission`: >> >> * This class is used to restrict the usage of the Kerberos >> * delegation model, ie: forwardable and proxiable ticket

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread David M . Lloyd
On Fri, 18 Oct 2024 19:03:30 GMT, Sean Mullan wrote: >> This is the implementation of JEP 486: Permanently Disable the Security >> Manager. See [JEP 486](https://openjdk.org/jeps/486) for more details. The >> [CSR](https://bugs.openjdk.org/browse/JDK-8338412) describes in detail the >> main ch

Re: RFR: 8338411: Implement JEP 486: Permanently Disable the Security Manager [v2]

2024-10-18 Thread Sean Mullan
On Tue, 15 Oct 2024 22:14:00 GMT, Sean Mullan wrote: >> src/java.management/share/classes/javax/management/remote/JMXConnectorFactory.java >> line 225: >> >>> 223: */ >>> 224: public static JMXConnector connect(JMXServiceURL serviceURL) >>> 225: throws IOException { >> >>

Re: RFR: 8339570: Add Tidy build support for JDK tests [v14]

2024-10-18 Thread Nizar Benalla
> Can I get a review for this patch that adds the necessary changes for local > support of the `tidy` library. > > The dependency can be retrieved by running `make/devkit/createTidyBundle.sh` > on Linux and MacOs systems. > > This dependency is primarily going to be used to test the generated

Re: RFR: 8341692: Implement JEP 490: ZGC: Remove the Non-Generational Mode [v4]

2024-10-18 Thread Axel Boldt-Christmas
> This is the implementation task for `JEP 490: ZGC: Remove the > Non-Generational Mode`. See the JEP for details. > [JDK-8335850](https://bugs.openjdk.org/browse/JDK-8335850) Axel Boldt-Christmas has updated the pull request with a new target base due to a merge or a rebase. The pull request n

Re: RFR: 8341692: Implement JEP 490: ZGC: Remove the Non-Generational Mode [v4]

2024-10-18 Thread Stefan Karlsson
On Fri, 18 Oct 2024 06:46:22 GMT, Axel Boldt-Christmas wrote: >> This is the implementation task for `JEP 490: ZGC: Remove the >> Non-Generational Mode`. See the JEP for details. >> [JDK-8335850](https://bugs.openjdk.org/browse/JDK-8335850) > > Axel Boldt-Christmas has updated the pull request

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Nizar Benalla
> Please review this change that adds a new test root `docs` dedicated to > testing the documentation, which has been a work in progress for a while. > Tests for links, encoding, HTML, accessibility will be later added in > following PRs. > > We also define a new make target `test-docs` meant

Re: RFR: 8339570: Add Tidy build support for JDK tests [v12]

2024-10-18 Thread Nizar Benalla
> Can I get a review for this patch that adds the necessary changes for local > support of the `tidy` library. > > The dependency can be retrieved by running `make/devkit/createTidyBundle.sh` > on Linux and MacOs systems. > > This dependency is primarily going to be used to test the generated

Re: RFR: 8339570: Add Tidy build support for JDK tests [v11]

2024-10-18 Thread Nizar Benalla
On Wed, 16 Oct 2024 18:36:14 GMT, Mikael Vidstedt wrote: >> Nizar Benalla has updated the pull request incrementally with one additional >> commit since the last revision: >> >> revert changes to RunTestsPrebuiltSpec.gmk > > make/devkit/createTidyBundle.sh line 79: > >> 77: >> 78: cd "$SCRI

Re: RFR: 8339570: Add Tidy build support for JDK tests [v11]

2024-10-18 Thread Nizar Benalla
On Thu, 17 Oct 2024 15:10:29 GMT, Magnus Ihse Bursie wrote: >> make/autoconf/lib-tests.m4 line 330: >> >>> 328: AC_MSG_RESULT([yes]) >>> 329: AC_MSG_CHECKING([for tidy version]) >>> 330: TIDY_VERSION=`$ECHO $tidy_output | $SED -e 's/.*version //g'` >> >> Looks like `TIDY_VERSI

Re: RFR: 8339570: Add Tidy build support for JDK tests [v13]

2024-10-18 Thread Nizar Benalla
> Can I get a review for this patch that adds the necessary changes for local > support of the `tidy` library. > > The dependency can be retrieved by running `make/devkit/createTidyBundle.sh` > on Linux and MacOs systems. > > This dependency is primarily going to be used to test the generated

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v11]

2024-10-18 Thread Nizar Benalla
On Fri, 18 Oct 2024 08:39:49 GMT, Nizar Benalla wrote: >> Please review this change that adds a new test root `docs` dedicated to >> testing the documentation, which has been a work in progress for a while. >> Tests for links, encoding, HTML, accessibility will be later added in >> following P

Re: RFR: 8340818: Add a new jtreg test root to test the generated documentation [v10]

2024-10-18 Thread Nizar Benalla
On Wed, 16 Oct 2024 17:37:45 GMT, Erik Joelsson wrote: >> Nizar Benalla has updated the pull request incrementally with one additional >> commit since the last revision: >> >> ADD TEST_DEPS when calling tests using test-$1 >> >> drop docs/req.flg > > make/RunTests.gmk line 877: > >> 875