RFR: 8367725: Incorrect reading of oop in SuspendResumeManager::suspend while thread is blocked [v10]

[Leonid Mesnik]

On Thu, 18 Sep 2025 04:42:47 GMT, David Holmes <dholmes at openjdk.org> 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