RFR: JDK-8201603: MonitorContendedEnter failure in nsk/jvmti/scenarios/contention/TC02/tc02t001

[Gary Adams]

The main thread in tc02t001 is using a sleep(100) to cause a
monitor contention for the debuggee thread. On some slower
platform builds the main thread continues before the debuggee
thread observes the contention.

Similar to the fix for cm02t001, this is a simple change to increase
the pauses to allow the debuggee thread to run.

Webrev: http://cr.openjdk.java.net/~gadams/8201603/webrev/
Issue: https://bugs.openjdk.java.net/browse/JDK-8201603

[Chris Plummer]

Hi Gary,

Looks good. Please add the comment "Wait for contended "sychronized (M)".

thanks,

Chris

Post by Gary Adams
The main thread in tc02t001 is using a sleep(100) to cause a
monitor contention for the debuggee thread. On some slower
platform builds the main thread continues before the debuggee
thread observes the contention.
Similar to the fix for cm02t001, this is a simple change to increase
the pauses to allow the debuggee thread to run.
  Webrev: http://cr.openjdk.java.net/~gadams/8201603/webrev/
  Issue: https://bugs.openjdk.java.net/browse/JDK-8201603

[Gary Adams]

Patch attached.

I think one reviewer is sufficient since this mimics
the cm02t001 bug fix.

Post by Chris Plummer
Hi Gary,
Looks good. Please add the comment "Wait for contended "sychronized (M)".
thanks,
Chris

/ The main thread in tc02t001 is using a sleep(100) to cause a

/>/ monitor contention for the debuggee thread. On some slower
/>/ platform builds the main thread continues before the debuggee
/>/ thread observes the contention.
/>/
/>/ Similar to the fix for cm02t001, this is a simple change to increase
/>/ the pauses to allow the debuggee thread to run.
/>/
/>/ Webrev:http://cr.openjdk.java.net/~gadams/8201603/webrev/ <http://cr.openjdk.java.net/%7Egadams/8201603/webrev/>
/>/ Issue:https://bugs.openjdk.java.net/browse/JDK-8201603
/