RFR: 8324582: Replace -Djava.util.concurrent.ForkJoinPool.common.parallelism to -Djdk.virtualThreadScheduler.parallelism in jvmti vthread tests

Chris Plummer cjplummer at openjdk.org
Wed Jan 24 19:30:27 UTC 2024


On Wed, 24 Jan 2024 02:42:29 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> Some jvmti tests use
> -Djava.util.concurrent.ForkJoinPool.common.parallelism
> to control the pool of virtual threads. However, it is controlled by
> jdk.virtualThreadScheduler.parallelism property.
> 
> The non-continuations implementation doesn't use any of these properties and it was just deleted.
> 
> I verified the fix using jcmd Thread.dump and ran all jvmti tests in the default configuration.

test/hotspot/jtreg/serviceability/jvmti/thread/GetAllThreads/allthr01/liballthr01.cpp line 49:

> 47: static const char thread1_name[] = "thread1";
> 48: static const char sys_thread_name[] = "SysThread";
> 49: // Tes uses -Djdk.virtualThreadScheduler.parallelism=1

Maybe you can also fix the typo: Tes -> Test.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/17547#discussion_r1465440169


More information about the serviceability-dev mailing list