RFR: 8238761: Asynchronous handshakes [v2]

Robbin Ehn rehn at openjdk.java.net
Mon Sep 21 08:41:56 UTC 2020


On Fri, 18 Sep 2020 20:01:02 GMT, Daniel D. Daugherty <dcubed at openjdk.org> wrote:

>> Robbin Ehn has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Fixed double checks
>>   Added NSV
>>   ProcessResult to enum
>>   Fixed logging
>>   Moved _active_handshaker to private
>
> src/hotspot/share/prims/jvmtiEventController.cpp line 340:
> 
>> 338:   } else {
>> 339:     Handshake::execute(&hs, target);
>> 340:   }
> 
> This guarantee() that the handshake has executed doesn't have an
> equivalent in the rewritten code. Should there be some way of verifying
> this condition from this location?

Fixed

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

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


More information about the hotspot-runtime-dev mailing list