--- jsr166/src/test/tck/AbstractQueuedLongSynchronizerTest.java 2011/06/03 05:07:14 1.25 +++ jsr166/src/test/tck/AbstractQueuedLongSynchronizerTest.java 2013/05/02 18:01:09 1.27 @@ -117,7 +117,7 @@ public class AbstractQueuedLongSynchroni } /** A constant to clarify calls to checking methods below. */ - final static Thread[] NO_THREADS = new Thread[0]; + static final Thread[] NO_THREADS = new Thread[0]; /** * Spin-waits until sync.isQueued(t) becomes true. @@ -1187,17 +1187,24 @@ public class AbstractQueuedLongSynchroni */ public void testTryAcquireSharedNanos_Timeout() { final BooleanLatch l = new BooleanLatch(); + final BooleanLatch observedQueued = new BooleanLatch(); + final long timeoutMillis = timeoutMillis(); Thread t = newStartedThread(new CheckedRunnable() { public void realRun() throws InterruptedException { assertFalse(l.isSignalled()); - long startTime = System.nanoTime(); - long nanos = MILLISECONDS.toNanos(timeoutMillis()); - assertFalse(l.tryAcquireSharedNanos(0, nanos)); - assertTrue(millisElapsedSince(startTime) >= timeoutMillis()); + for (long millis = timeoutMillis(); + !observedQueued.isSignalled(); + millis *= 2) { + long nanos = MILLISECONDS.toNanos(millis); + long startTime = System.nanoTime(); + assertFalse(l.tryAcquireSharedNanos(0, nanos)); + assertTrue(millisElapsedSince(startTime) >= millis); + } assertFalse(l.isSignalled()); }}); waitForQueuedThread(l, t); + observedQueued.releaseShared(0); assertFalse(l.isSignalled()); awaitTermination(t); assertFalse(l.isSignalled());