On Thu, 7 Nov 2024 00:38:57 GMT, Patricio Chilano Mateo <pchilanom...@openjdk.org> wrote:
>>> the call to java_lang_Thread::set_is_in_VTMS_transition()is not needed in >>> JvmtiUnmountBeginMark >>> >> Why is not needed? I guess you meant to say we should use >> `JvmtiVTMSTransitionDisabler::set_is_in_VTMS_transition()` which does both? > >> the function is_vthread_safe_to_preempt() does not need the vthread parameter >> > Removed. Thank you for the update! It looks okay to me. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/21565#discussion_r1833168776