On Fri, 5 Jun 2026 13:59:06 GMT, Patricio Chilano Mateo 
<[email protected]> wrote:

> Please review this small test fix. We need at least two workers in the test 
> since one of the virtual threads will be pinned in the class initializer. The 
> timeout can be reproduced when running the test with 
> `-XX:ActiveProcessorCount=1`.
> 
> Thanks,
> Patricio
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Marked as reviewed by alanb (Reviewer).

test/jdk/java/lang/Thread/virtual/KlassInit.java line 105:

> 103:     static void setup() {
> 104:         // need >=2 carriers for testing pinning
> 105:         VThreadRunner.ensureParallelism(2);

All of the `@run` tags use `/othervm` so no need to reset, good.

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

PR Review: https://git.openjdk.org/jdk/pull/31403#pullrequestreview-4437071917
PR Review Comment: https://git.openjdk.org/jdk/pull/31403#discussion_r3363314349

Reply via email to