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