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