On Thu, 12 Mar 2026 06:36:47 GMT, David Holmes <[email protected]> wrote:
>> test/hotspot/jtreg/serviceability/jvmti/vthread/SuspendResume3/SuspendResume3.java
>> line 163:
>>
>>> 161: while (true) {
>>> 162: ThreadInfo threadInfo =
>>> ManagementFactory.getThreadMXBean().getThreadInfo(thread.threadId());
>>> 163: assertTrue(threadInfo != null, "getThreadInfo() failed");
>>
>> Nit: If `threadInfo` is expected to be `non-null` then the lines 162-163 can
>> be placed before the loop.
>
> The `ThreadInfo` is a snapshot of the thread's current state so we have to
> re-sample each time through the loop.
Agreed, thanks.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30199#discussion_r2922808830
