(13) RFR (XS): 8226917: jvmti/scenarios/contention/TC04/tc04t001/TestDescription.java fails on jvmti->InterruptThread (JVMTI_ERROR_THREAD_NOT_ALIVE)

[serguei.spitsyn at oracle.com]

Thank you a lot, Gary!
Serguei

On 6/28/19 11:58 AM, gary.adams at oracle.com wrote:
> Looks good to me.
>
> On 6/28/19 2:09 PM, serguei.spitsyn at oracle.com wrote:
>> Please, review a test bug fix for:
>> https://bugs.openjdk.java.net/browse/JDK-8226917
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8226917-mon-events-correction2.1/ 
>>
>>
>>
>> Summary:
>>   One more fragment in the native agent was overlooked and is not 
>> needed anymore
>>   after the fix of JDK-8223736 which implemented more reliable sync 
>> approach.
>>   The error code JVMTI_ERROR_THREAD_NOT_ALIVE started to be seen because
>>   the sleep timeout changed to be shorter. Now, the thread holding 
>> the monitor
>>   is able to notice the enterEventsCount was increased by the 
>> MonitorContendedEnter
>>   event callback and finish before the callback attempts to interrupt 
>> it with the
>>   JVMTI InterruptThread.
>>
>>
>> Testing:
>>   A mach5 submission is in progress.
>>
>> Thanks,
>> Serguei
>