RFR: 8257831: Suspend with handshakes [v2]

Robbin Ehn rehn at openjdk.java.net
Wed Apr 7 07:28:48 UTC 2021


On Wed, 31 Mar 2021 05:21:03 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/runtime/handshake.cpp line 463:
> 
>> 461:   ThreadInVMForHandshake tivm(_handshakee);
>> 462:   {
>> 463:     ttyLocker::break_tty_lock_for_safepoint(os::current_thread_id());
> 
> Why is this needed when it is inside ThreadInVMForHandshake constructor ??

If we process the async suspension handshake we can go to safepoint.
And before safepoint we must drop the tty lock.

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

PR: https://git.openjdk.java.net/jdk/pull/3191


More information about the serviceability-dev mailing list