RFR: 8311218: fatal error: stuck in JvmtiVTMSTransitionDisabler::VTMS_transition_disable [v3]
Serguei Spitsyn
sspitsyn at openjdk.org
Thu Dec 14 16:59:40 UTC 2023
On Thu, 14 Dec 2023 12:11:42 GMT, Serguei Spitsyn <sspitsyn at openjdk.org> wrote:
>> src/java.base/share/classes/java/lang/VirtualThread.java line 1164:
>>
>>> 1162:
>>> 1163: @IntrinsicCandidate
>>> 1164: private native void notifyJvmtiCriticalLock(boolean enter);
>>
>> The name is confusing to me, the CriticalLock looks like it is the section is critical and might be taken by a single thread only. Or it's just unclear what is critical here.
>> However, the purpose is to disable suspend
>> Wouldn't be 'notifyJvmtiSuspendLock notifyJvmtiDisableSuspend' better name here?
>> or comment what critical means here.
>
> Okay, thanks. I like your name suggestion but let's check with Alan first.
Implemented this renaming suggestion. Let's wait if Alan ia okay with it.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/17011#discussion_r1426990736
More information about the core-libs-dev
mailing list