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