RFR: 8370905: Update vm.defmeth tests to use virtual threads

Patricio Chilano Mateo pchilanomate at openjdk.org
Mon Dec 1 16:47:25 UTC 2025


On Wed, 29 Oct 2025 18:54:09 GMT, Leonid Mesnik <lmesnik at openjdk.org> wrote:

> The vm.defmeth stress tests should use
> `TestThreadFactory.newThread()`
> to start virtual threads if test thread factory is used.

Looks good to me.

test/hotspot/jtreg/vmTestbase/vm/runtime/defmeth/StressTest.java line 98:

> 96:              this.rand = new Random(seed);
> 97:              this.thread = TestThreadFactory.newThread(this, id);
> 98:              thread.setName(id);

Isn't this already set by `TestThreadFactory.newThread(this, id)`?

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

Marked as reviewed by pchilanomate (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/28048#pullrequestreview-3525996141
PR Review Comment: https://git.openjdk.org/jdk/pull/28048#discussion_r2577844273


More information about the hotspot-runtime-dev mailing list