RFR: 8375457: Test serviceability/jvmti/vthread/SelfSuspendDisablerTest/SelfSuspendDisablerTest.java#default timed out [v2]
Serguei Spitsyn
sspitsyn at openjdk.org
Sat Feb 21 08:14:48 UTC 2026
On Fri, 20 Feb 2026 22:19:13 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
>> Thank you for checking. Current virtual thread can be in the `except_list`. Then it has already been resumed at the line 1077.
>
> But if it's in the list then `self_tobj()` should be `nullptr`, i.e non `nullptr` implies it must be suspended. We could just add it as an assert inside the conditional.
> Also when iterating the `elist` above, we just called `JvmtiVTSuspender::register_all_vthreads_suspend()` so `JvmtiVTSuspender::is_vthread_suspended` should be true for all vthreads.
You are right, thanks. Replaced this check with assert now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/29802#discussion_r2835967688
More information about the serviceability-dev
mailing list