RFR for JDK-6963118 Intermittent test failure: test/java/nio/channels/Selector/Wakeup.java fail intermittently (win)

[Alan Bateman]

On 19/12/2013 19:16, srikalyan wrote:
> Thanks and the new webrev is available here 
> http://cr.openjdk.java.net/~srikchan/Regression/6963118-Wakeup/
>
Thank's for the update. 10 seconds for the timeout is probably okay. In 
other areas then we've used 30 or 60s to be ultra safe. In this case 
then the test has always used 1s and I'm not aware of any reports of it 
failing. I see you've changed the sleep(50) to sleep(1). I don't know if 
the granularity is available everywhere so the overall loop time might 
be longer.

In the other thread on this topic then David Holmes suggests changing 
the yield so a short sleep. That would avoid potential spin and maybe a 
bit better for the system when running tests concurrently. I realize the 
test has always used yield but changing while we are here would be good.

-Alan.