Re: (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.spit...@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