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

[Daniel D. Daugherty]

On 6/21/19 7:48 PM, 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/

test/hotspot/jtreg/vmTestbase/nsk/jvmti/scenarios/contention/TC04/tc04t001.java
     No comments.

Thumbs up.

Dan


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190622/cbc98fc4/attachment.html>