On Wed, 21 Dec 2022 09:31:51 GMT, David Holmes <dhol...@openjdk.org> wrote:

>> Serguei Spitsyn has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   address wqsecond round review comments
>
> src/hotspot/share/classfile/javaClasses.cpp line 1739:
> 
>> 1737: void java_lang_Thread::inc_VTMS_transition_disable_count(oop 
>> java_thread) {
>> 1738:   int val = VTMS_transition_disable_count(java_thread);
>> 1739:   assert(JvmtiVTMSTransition_lock->is_locked(), "Must be locked");
> 
> You want to use `owned_by_self()` for these assertions - thanks.

You are right. Fixed now.

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

PR: https://git.openjdk.org/jdk/pull/11690

Reply via email to