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

[Alex Menkov]

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