(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]

Thanks a lot, Alex!
Serguei

On 6/28/19 3:56 PM, Alex Menkov wrote:
> +1
>
> --alex
>
> On 06/28/2019 11:58, 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
>>