RFR: 8364343: Virtual Thread transition management needs to be independent of JVM TI [v9]
Patricio Chilano Mateo
pchilanomate at openjdk.org
Mon Dec 1 23:00:17 UTC 2025
On Mon, 1 Dec 2025 19:14:37 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:
> Excellent work. Thank you for answering my questions. There's one click suggested change.
>
Thanks for the review Coleen!
> src/hotspot/share/opto/library_call.cpp line 3052:
>
>> 3050: //
>> 3051: // java_lang_Thread::set_is_in_VTMS_transition(vthread, true);
>> 3052: // carrier->set_is_in_VTMS_transition(true);
>
> Suggestion:
>
> // java_lang_Thread::set_is_in_vthread_transition(vthread, true);
> // carrier->set_is_in_vthread_transition(true);
Fixed. Also fixed the comment above: the other caller is `startFinalTransition`.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28361#issuecomment-3599315673
PR Review Comment: https://git.openjdk.org/jdk/pull/28361#discussion_r2578999900
More information about the serviceability-dev
mailing list