RFR(S): 6952105 TEST_BUG: testcase failure, not very often, com/sun/jdi/SuspendThreadTest.java

[David Holmes]

Hi Staffan,

If you get a spurious wakeup from wait():

  151             try {
  152                 synchronized (bkptSignal) {
  153                     bkptSignal.wait(5000);
  154                 }
  155             } catch (InterruptedException ee) {
  156             }
  157             if (prevBkptCount == bkptCount) {
  158                 failure("failure: test hung");

you could report failure. But that is far less likely than the current 
problem using sleep.

David

On 18/02/2014 8:19 PM, Staffan Larsen wrote:
> Still looking for Reviewer for this change.
>
> Thanks,
> /Staffan
>
> On 11 feb 2014, at 15:12, Staffan Larsen <staffan.larsen at oracle.com> wrote:
>
>> Updated the test to use proper synchronization and notification between threads. Should be more stable and much faster.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-6952105
>> webrev: http://cr.openjdk.java.net/~sla/6952105/webrev.00/
>>
>> Thanks,
>> /Staffan
>