On Fri, 19 Dec 2025 11:13:01 GMT, Patricio Chilano Mateo
<[email protected]> wrote:
>> David Holmes has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - Delay suspender whilst target is in-critical
>> - Revert "8373839: Disable JVM TI suspension during JNI critical regions"
>>
>> This reverts commit 7723275e4495cc1f514c531afe752210209617cc.
>
> src/hotspot/share/runtime/suspendResumeManager.cpp line 64:
>
>> 62: }
>> 63: void do_thread(Thread* thr) {
>> 64: assert(thr != Thread::current(), "must be");
>
> I think the target can execute this operation.
We already checked for current thread in `SuspendResumeManager::suspend` and
took a different path before calling this.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/28884#discussion_r2634990833
