Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar [v2]

2022-08-12 Thread Christian Stein
On Sat, 13 Aug 2022 03:41:04 GMT, Jaikiran Pai wrote: >> Christian Stein has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unused import statement > > test/jaxp/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java line 217: >

Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar [v2]

2022-08-12 Thread Jaikiran Pai
On Fri, 12 Aug 2022 20:41:24 GMT, Christian Stein wrote: >> Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by >> accept version information in JAR file names. >> >> Find details in https://bugs.openjdk.org/browse/JDK-8292182 > > Christian Stein has updated the pull

Withdrawn: JDK-8288475: Initializing RandomGeneratorFactory.FactoryMapHolder fails if a SecurityManager is installed

2022-08-12 Thread duke
On Thu, 16 Jun 2022 07:08:20 GMT, Johannes Kuhn wrote: > * This adds additional permissions to the jdk.random module > (`RuntimePermission "accessClassInPackage.jdk.internal.util.random"`) > * The annotations of the provider classes are now parsed early. > This avoids putting the parts that

Re: RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

2022-08-12 Thread Jie Fu
On Thu, 11 Aug 2022 20:02:06 GMT, Daniel D. Daugherty wrote: > A trivial fix so that Continuation/Fuzz.java honors the timeoutFactor JTREG > setting > when waiting for a compilation to finish. > > This fix is being tested in my jdk-20+10 stress testing run. > > The usual Mach5 timeoutFactor

Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar [v2]

2022-08-12 Thread Christian Stein
On Fri, 12 Aug 2022 20:18:50 GMT, Naoto Sato wrote: >> Christian Stein has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove unused import statement > > test/jaxp/javax/xml/jaxp/libs/jaxp/library/JAXPPolicyManager.java line 27: > >>

Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar

2022-08-12 Thread Naoto Sato
On Fri, 12 Aug 2022 16:26:30 GMT, Christian Stein wrote: > Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by > accept version information in JAR file names. > > Find details in https://bugs.openjdk.org/browse/JDK-8292182 LGTM

RFR: 8292196: Reduce runtime of java.util.regex microbenchmarks

2022-08-12 Thread Eric Caspole
These changes reduce the run time to 45 minutes from about 1h40m with stability good enough for weekly promo build testing. - Commit messages: - 8292196: Reduce runtime of java.util.regex microbenchmarks Changes: https://git.openjdk.org/jdk/pull/9861/files Webrev:

Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar

2022-08-12 Thread Iris Clark
On Fri, 12 Aug 2022 16:26:30 GMT, Christian Stein wrote: > Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by > accept version information in JAR file names. > > Find details in https://bugs.openjdk.org/browse/JDK-8292182 Marked as reviewed by iris (Reviewer).

Re: RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar

2022-08-12 Thread Lance Andersen
On Fri, 12 Aug 2022 16:26:30 GMT, Christian Stein wrote: > Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by > accept version information in JAR file names. > > Find details in https://bugs.openjdk.org/browse/JDK-8292182 Marked as reviewed by lancea (Reviewer).

RFR: 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for jtreg version 7 jar

2022-08-12 Thread Christian Stein
Enhance `JAXPPolicyManager` to setup required permissions for `jtreg` 7 by accept version information in JAR file names. Find details in https://bugs.openjdk.org/browse/JDK-8292182 - Commit messages: - 8292182: [TESTLIB] Enhance JAXPPolicyManager to setup required permissions for

Re: RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

2022-08-12 Thread Daniel D . Daugherty
On Fri, 12 Aug 2022 04:05:47 GMT, Jie Fu wrote: >> test/jdk/jdk/internal/vm/Continuation/Fuzz.java line 81: >> >>> 79: static final boolean VERBOSE = false; >>> 80: >>> 81: static float timeoutFactor = >>> Float.parseFloat(System.getProperty("test.timeout.factor", "1.0")); >> >> How

Re: RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

2022-08-12 Thread Daniel D . Daugherty
On Fri, 12 Aug 2022 03:43:46 GMT, Leonid Mesnik wrote: >> A trivial fix so that Continuation/Fuzz.java honors the timeoutFactor JTREG >> setting >> when waiting for a compilation to finish. >> >> This fix is being tested in my jdk-20+10 stress testing run. >> >> The usual Mach5 timeoutFactor

Re: RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

2022-08-12 Thread Daniel D . Daugherty
On Thu, 11 Aug 2022 20:02:06 GMT, Daniel D. Daugherty wrote: > A trivial fix so that Continuation/Fuzz.java honors the timeoutFactor JTREG > setting > when waiting for a compilation to finish. > > This fix is being tested in my jdk-20+10 stress testing run. > > The usual Mach5 timeoutFactor

Re: RFR: 8291911: java/io/File/GetXSpace.java fails with "53687091200 != 161051996160"

2022-08-12 Thread Brian Burkhalter
On Fri, 12 Aug 2022 14:52:23 GMT, Brian Burkhalter wrote: > On Windows, suppress failure if the total space indicated by `df` is less > than `FileStore::getTotalSpace` and the free space indicated by `df` equals > `FileStore::getUnallocatedSpace`. test/jdk/java/io/File/GetXSpace.java line

Re: RFR: 8291911: java/io/File/GetXSpace.java fails with "53687091200 != 161051996160"

2022-08-12 Thread Brian Burkhalter
On Fri, 12 Aug 2022 14:52:23 GMT, Brian Burkhalter wrote: > On Windows, suppress failure if the total space indicated by `df` is less > than `FileStore::getTotalSpace` and the free space indicated by `df` equals > `FileStore::getUnallocatedSpace`. Passed on 20 arbitrary nodes and on one where

RFR: 8291911: java/io/File/GetXSpace.java fails with "53687091200 != 161051996160"

2022-08-12 Thread Brian Burkhalter
On Windows, suppress failure if the total space indicated by `df` is less than `FileStore::getTotalSpace` and the free space indicated by `df` equals `FileStore::getUnallocatedSpace`. - Commit messages: - 8291911: java/io/File/GetXSpace.java fails with "53687091200 !=

Re: [BUG]Collections.unmodifiableMap(map).entrySet().iterator().forEachRemaining ismissing null check

2022-08-12 Thread Rob Spoor
I found one similar issue, https://bugs.openjdk.org/browse/JDK-8015008, but that was for an empty primitive iterator. I've created a bug report with id 9073784. On 11/08/2022 01:24, some-java-user-99206970363698485...@vodafonemail.de wrote: Hi, could you please report this at

Re: RFR: 8292051: jdk/internal/misc/TerminatingThreadLocal/TestTerminatingThreadLocal.java failed "AssertionError: Expected terminated values: [666] but got: []"

2022-08-12 Thread Alan Bateman
On Thu, 11 Aug 2022 08:26:12 GMT, Peter Levart wrote: > Now that we have control over termination of carrier thread which can be > stopped at will without waiting for 30s or more, we can again test all cases > even with virtual threads... Yes, we can. I'll convert the test to run as a TestNG

Re: RFR: 8292051: jdk/internal/misc/TerminatingThreadLocal/TestTerminatingThreadLocal.java failed "AssertionError: Expected terminated values: [666] but got: []"

2022-08-12 Thread Peter Levart
On Wed, 10 Aug 2022 09:18:01 GMT, Alan Bateman wrote: > Change the test to use a custom scheduler so there is better control on when > the carrier threads terminates. Also convert test to TestNG so that all test > cases can be run in the context of a virtual thread. Now that we have control

RFR: 8292051: jdk/internal/misc/TerminatingThreadLocal/TestTerminatingThreadLocal.java failed "AssertionError: Expected terminated values: [666] but got: []"

2022-08-12 Thread Alan Bateman
Change the test to use a custom scheduler so there is better control on when the carrier threads terminates. Also convert test to TestNG so that all test cases can be run in the context of a virtual thread. - Commit messages: - Convert to TestNG and run all tests on a virtual

Re: RFR: 8290211: jdk/internal/vm/Continuation/Fuzz.java failed with "AssertionError: Failed to compile int Fuzz.com_int(int, int) in 5000ms"

2022-08-12 Thread Alan Bateman
On Thu, 11 Aug 2022 20:02:06 GMT, Daniel D. Daugherty wrote: > A trivial fix so that Continuation/Fuzz.java honors the timeoutFactor JTREG > setting > when waiting for a compilation to finish. > > This fix is being tested in my jdk-20+10 stress testing run. > > The usual Mach5 timeoutFactor