On Thu, 11 Dec 2025 00:54:38 GMT, Leonid Mesnik <[email protected]> wrote:
>> Serguei Spitsyn has updated the pull request incrementally with one >> additional commit since the last revision: >> >> minor correction for sleep > > test/hotspot/jtreg/serviceability/jvmti/vthread/ThreadListStackTracesTest/ThreadListStackTracesTest.java > line 100: > >> 98: TestTask task = new ReentrantLockTestTask(); >> 99: Thread vt = Thread.ofVirtual().name(name).start(task); >> 100: TestTask.sleep(50); // allow potentially needed class loading >> to complete > > The just sleep might be unreliable. > > Would it makes sense to wait until `lock.hasQueuedThreads()` becomes true? > > If 'hasQueuedThreads' doesn't work then makes to pre-load classes used by > lock before running test. Nice suggestion, thanks! It is working. Update now. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/28734#discussion_r2608900674
