On Thu, 18 Sep 2025 04:42:47 GMT, David Holmes <[email protected]> wrote:

>> Leonid Mesnik has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   renamed method
>
> src/hotspot/share/runtime/suspendResumeManager.cpp line 88:
> 
>> 86:   if (register_vthread_SR) {
>> 87:     assert(_target->is_vthread_mounted(), "sanity check");
>> 88:     assert(_target == JavaThread::current(), "should be current thread");
> 
> So this assert should be outside the INCLUDE_JVMTI section - right?

ough,
yes, should be outside INCLUDE_JVMTI 
However, I just can't understand how those methods could be called if JVMTI is 
not included. Do we still have any mechanisms to suspend thread without jvmti 
nowdays?

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27317#discussion_r2357528204

Reply via email to