(13) RFR (S): 8226595: jvmti/scenarios/contention/TC04/tc04t001/TestDescription.java still fails due to wrong number of MonitorContendedEntered events

[serguei.spitsyn at oracle.com]

Thanks, Alex!
Serguei

On 6/21/19 17:51, Alex Menkov wrote:
> LGTM
>
> --alex
>
> On 06/21/2019 16:48, serguei.spitsyn at oracle.com wrote:
>> Please, review a test bug fix for:
>> https://bugs.openjdk.java.net/browse/JDK-8226595
>>
>> Webrev:
>> http://cr.openjdk.java.net/~sspitsyn/webrevs/2019/8226595-mon-events-correction.1/ 
>>
>>
>>
>> Summary:
>>    The recent fix of https://bugs.openjdk.java.net/browse/JDK-8223736 
>> introduced another issue.
>>    Now, there is a whole in synchronization in updating the static 
>> field 'lastEnterEventsCount'
>>    in the method run().
>>    Additionally, this update does a cleanup of the commented out 
>> debugging code.
>>
>>    Thanks a lot to Alex for filing this new bug and it's analysis!
>>
>>
>> Testing:
>>    A mach5 submission is in progress.
>>
>>
>> Thanks,
>> Serguei