On Wed, 31 Mar 2021 04:56:32 GMT, David Holmes <[email protected]> wrote:
>> Robbin Ehn has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains two commits:
>>
>> - Merge branch 'master' into SuspendInHandshake
>> - 8257831: Suspend with handshake (review baseline)
>
> src/hotspot/share/prims/jvmtiRawMonitor.cpp line 428:
>
>> 426: jt->set_thread_state_fence(_thread_in_native_trans);
>> 427: SafepointMechanism::process_if_requested(jt);
>> 428: if (jt->is_interrupted(true)) {
>
> A thread must be _thread_in_vm to safely query is_interrupted() as it
> accesses the threadObj.
Any unsafe (not native/blocked) is fine, therefore I never completed the
transition.
I set the state to _thread_in_vm before.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3191