On Wed, 28 Oct 2026 17:07:34 GMT, Alan Bateman <[email protected]> wrote:
>> Trivial fixes to ProblemList various tests: >> [JDK-8396054](https://bugs.openjdk.org/browse/JDK-8396054) ProblemList >> tools/launcher/MiscTests.java with virtual threads >> [JDK-8396057](https://bugs.openjdk.org/browse/JDK-8396057) ProblemList >> jdk/jfr/event/oldobject/TestEmergencyDumpAtOOM.java with enable-preview >> [JDK-8396059](https://bugs.openjdk.org/browse/JDK-8396059) ProblemList >> java/net/URLPermission/OpenURL.java with virtual threads on windows-x64 >> >> --------- >> - [x] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > test/jdk/ProblemList-Virtual.txt line 37: > >> 35: java/lang/ScopedValue/StressStackOverflow.java#TieredStopAtLevel1 >> 8309646 generic-all >> 36: >> 37: tools/launcher/MiscTests.java 8394341 generic-all > > Can you add`@requires test.thread.factory != "Virtual"` to the test instead? > The issue is that it's using ProcessTools so should be skipped in this > config, like we do with other tests. I saw that mentioned in the PR for [JDK-8394341](https://bugs.openjdk.org/browse/JDK-8394341), but it seems like folks are not in agreement in the right way to handle this failure. As far as I can see, the PR has stalled and I simply want to reduce the noise. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/33609#discussion_r4403803045
