RFR(xxs): 8224795: some runtime/SelectionResolution tests are timing out

[Robbin Ehn]

Thanks David!

/Robbin

On 2019-05-27 23:39, David Holmes wrote:
> Looks good Robbin!
> 
> Thank,
> David
> 
> On 27/05/2019 11:32 pm, Robbin Ehn wrote:
>> Hi all,
>>
>> There is a fault in changeset 8221734: Deoptimize with handshakes.
>> Which causes us clear the method's code unconditionally.
>> It must be checked that it's pointing to this nmethod before clearing.
>> I made an error while changing some code after reviews of 8221734 here:
>> http://cr.openjdk.java.net/~rehn/8221734/v3/inc_review/webrev/src/hotspot/share/code/nmethod.cpp.udiff.html 
>>
>>
>> This patch restores correct behavior:
>> diff -r d871ce8ab96b src/hotspot/share/code/nmethod.cpp
>> --- a/src/hotspot/share/code/nmethod.cpp    Sat May 25 20:55:33 2019 +0900
>> +++ b/src/hotspot/share/code/nmethod.cpp    Mon May 27 12:29:26 2019 +0200
>> @@ -1261,7 +1261,7 @@
>>
>>   void nmethod::unlink_from_method() {
>>     if (method() != NULL) {
>> -    method()->unlink_code();
>> +    method()->unlink_code(this);
>>     }
>>   }
>>
>> Issue:
>> https://bugs.openjdk.java.net/browse/JDK-8224795
>>
>> I do not see any new issues in t1-7 and SelectionResolution (executed in t4 
>> and t6) now passes. Also locally verified SelectionResolution tests with Xcomp.
>>
>> Thanks, Robbin