(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 you a lot, Dan!
Serguei


On 6/22/19 06:01, Daniel D. Daugherty wrote:
> 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
>