RFR: 8257831: Suspend with handshakes [v2]

Robbin Ehn rehn at openjdk.java.net
Wed Apr 7 07:23:32 UTC 2021


On Wed, 31 Mar 2021 04:56:32 GMT, David Holmes <dholmes at openjdk.org> 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


More information about the serviceability-dev mailing list