On Wed, 28 Oct 2026 16:39:08 GMT, Daniel D. Daugherty <[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.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/33609#discussion_r4403679115

Reply via email to