RFR: 8346727: JvmtiVTMSTransitionDisabler deadlock [v3]

Serguei Spitsyn sspitsyn at openjdk.org
Sat Jan 11 06:01:36 UTC 2025


On Fri, 10 Jan 2025 18:10:34 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review: tweak some assert messages
>
> src/hotspot/share/runtime/javaThread.hpp line 724:
> 
>> 722:   void set_VTMS_transition_mark(bool val)        { Atomic::store(&_VTMS_transition_mark, val); }
>> 723: 
>> 724:   // Temporarily skip posting JVMTI events for safety reasons when executions is in ia critical section:
> 
> Suggestion:
> 
>   // Temporarily skip posting JVMTI events for safety reasons when execution is in a critical section:

Thank you. Fixed now.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/22997#discussion_r1911903539


More information about the hotspot-dev mailing list