RFR: 8298853: JvmtiVTMSTransitionDisabler should support disabling one virtual thread transitions [v4]
Serguei Spitsyn
sspitsyn at openjdk.org
Thu Dec 22 23:13:54 UTC 2022
On Thu, 22 Dec 2022 22:06:19 GMT, Patricio Chilano Mateo <pchilanomate at openjdk.org> wrote:
> Hi Serguei,
>
> Some comments on the changes below.
>
> Thanks, Patricio
Thank you for reviewing it, Patricio!
> src/hotspot/share/prims/jvmtiThreadState.cpp line 310:
>
>> 308: ml.wait(10); // wait while there is an active suspender or resumer
>> 309: }
>> 310: Atomic::inc(&_VTMS_transition_disable_for_one_count);
>
> I don't understand the purpose of this counter. We only seem to write to it and the only place where it is checked is to do a notify, but I don't see a corresponding check-and-wait on that counter somewhere else, as we do with _VTMS_transition_disable_for_all_count.
Good catch.
> Hi Serguei,
>
> Some comments on the changes below.
>
> Thanks, Patricio
Thank you for reviewing it, Patricio!
-------------
PR: https://git.openjdk.org/jdk/pull/11690
More information about the hotspot-dev
mailing list