ViewVC Help
View File | Revision Log | Show Annotations | Download File | Root Listing
root/jsr166/jsr166/src/test/tck/ReentrantReadWriteLockTest.java
(Generate patch)

Comparing jsr166/src/test/tck/ReentrantReadWriteLockTest.java (file contents):
Revision 1.36 by jsr166, Sat Nov 21 02:33:20 2009 UTC vs.
Revision 1.58 by jsr166, Sat May 7 14:43:13 2011 UTC

# Line 1 | Line 1
1   /*
2   * Written by Doug Lea with assistance from members of JCP JSR-166
3   * Expert Group and released to the public domain, as explained at
4 < * http://creativecommons.org/licenses/publicdomain
4 > * http://creativecommons.org/publicdomain/zero/1.0/
5   * Other contributors include Andrew Wright, Jeffrey Hayes,
6   * Pat Fisher, Mike Judd.
7   */
8  
9   import junit.framework.*;
10 + import java.util.concurrent.atomic.AtomicBoolean;
11   import java.util.concurrent.locks.*;
12   import java.util.concurrent.*;
13   import static java.util.concurrent.TimeUnit.MILLISECONDS;
# Line 15 | Line 16 | import java.util.*;
16  
17   public class ReentrantReadWriteLockTest extends JSR166TestCase {
18      public static void main(String[] args) {
19 <        junit.textui.TestRunner.run (suite());
19 >        junit.textui.TestRunner.run(suite());
20      }
21      public static Test suite() {
22          return new TestSuite(ReentrantReadWriteLockTest.class);
# Line 32 | Line 33 | public class ReentrantReadWriteLockTest
33          }
34      }
35  
35
36      /**
37       * A runnable calling lockInterruptibly that expects to be
38       * interrupted
# Line 50 | Line 50 | public class ReentrantReadWriteLockTest
50       */
51      static class PublicReentrantReadWriteLock extends ReentrantReadWriteLock {
52          PublicReentrantReadWriteLock() { super(); }
53 +        PublicReentrantReadWriteLock(boolean fair) { super(fair); }
54 +        public Thread getOwner() {
55 +            return super.getOwner();
56 +        }
57          public Collection<Thread> getQueuedThreads() {
58              return super.getQueuedThreads();
59          }
# Line 59 | Line 63 | public class ReentrantReadWriteLockTest
63      }
64  
65      /**
66 +     * Releases write lock, checking that it had a hold count of 1.
67 +     */
68 +    void releaseWriteLock(PublicReentrantReadWriteLock lock) {
69 +        ReentrantReadWriteLock.WriteLock writeLock = lock.writeLock();
70 +        assertWriteLockedBy(lock, Thread.currentThread());
71 +        assertEquals(1, lock.getWriteHoldCount());
72 +        writeLock.unlock();
73 +        assertNotWriteLocked(lock);
74 +    }
75 +
76 +    /**
77 +     * Spin-waits until lock.hasQueuedThread(t) becomes true.
78 +     */
79 +    void waitForQueuedThread(PublicReentrantReadWriteLock lock, Thread t) {
80 +        long startTime = System.nanoTime();
81 +        while (!lock.hasQueuedThread(t)) {
82 +            if (millisElapsedSince(startTime) > LONG_DELAY_MS)
83 +                throw new AssertionError("timed out");
84 +            Thread.yield();
85 +        }
86 +        assertTrue(t.isAlive());
87 +        assertTrue(lock.getOwner() != t);
88 +    }
89 +
90 +    /**
91 +     * Checks that lock is not write-locked.
92 +     */
93 +    void assertNotWriteLocked(PublicReentrantReadWriteLock lock) {
94 +        assertFalse(lock.isWriteLocked());
95 +        assertFalse(lock.isWriteLockedByCurrentThread());
96 +        assertFalse(lock.writeLock().isHeldByCurrentThread());
97 +        assertNull(lock.getOwner());
98 +        assertEquals(0, lock.getWriteHoldCount());
99 +    }
100 +
101 +    /**
102 +     * Checks that lock is write-locked by the given thread.
103 +     */
104 +    void assertWriteLockedBy(PublicReentrantReadWriteLock lock, Thread t) {
105 +        assertTrue(lock.isWriteLocked());
106 +        assertSame(t, lock.getOwner());
107 +        assertEquals(t == Thread.currentThread(),
108 +                     lock.isWriteLockedByCurrentThread());
109 +        assertEquals(t == Thread.currentThread(),
110 +                     lock.writeLock().isHeldByCurrentThread());
111 +        assertEquals(t == Thread.currentThread(),
112 +                     lock.getWriteHoldCount() > 0);
113 +        assertEquals(0, lock.getReadLockCount());
114 +    }
115 +
116 +    /**
117 +     * Checks that condition c has no waiters.
118 +     */
119 +    void assertHasNoWaiters(PublicReentrantReadWriteLock lock, Condition c) {
120 +        assertHasWaiters(lock, c, new Thread[] {});
121 +    }
122 +
123 +    /**
124 +     * Checks that condition c has exactly the given waiter threads.
125 +     */
126 +    void assertHasWaiters(PublicReentrantReadWriteLock lock, Condition c,
127 +                          Thread... threads) {
128 +        lock.writeLock().lock();
129 +        assertEquals(threads.length > 0, lock.hasWaiters(c));
130 +        assertEquals(threads.length, lock.getWaitQueueLength(c));
131 +        assertEquals(threads.length == 0, lock.getWaitingThreads(c).isEmpty());
132 +        assertEquals(threads.length, lock.getWaitingThreads(c).size());
133 +        assertEquals(new HashSet<Thread>(lock.getWaitingThreads(c)),
134 +                     new HashSet<Thread>(Arrays.asList(threads)));
135 +        lock.writeLock().unlock();
136 +    }
137 +
138 +    /**
139       * Constructor sets given fairness, and is in unlocked state
140       */
141      public void testConstructor() {
142 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
143 <        assertFalse(rl.isFair());
144 <        assertFalse(rl.isWriteLocked());
145 <        assertEquals(0, rl.getReadLockCount());
146 <        ReentrantReadWriteLock r2 = new ReentrantReadWriteLock(true);
147 <        assertTrue(r2.isFair());
148 <        assertFalse(r2.isWriteLocked());
149 <        assertEquals(0, r2.getReadLockCount());
150 <        ReentrantReadWriteLock r3 = new ReentrantReadWriteLock(false);
151 <        assertFalse(r3.isFair());
152 <        assertFalse(r3.isWriteLocked());
153 <        assertEquals(0, r3.getReadLockCount());
142 >        PublicReentrantReadWriteLock lock;
143 >
144 >        lock = new PublicReentrantReadWriteLock();
145 >        assertFalse(lock.isFair());
146 >        assertNotWriteLocked(lock);
147 >        assertEquals(0, lock.getReadLockCount());
148 >
149 >        lock = new PublicReentrantReadWriteLock(true);
150 >        assertTrue(lock.isFair());
151 >        assertNotWriteLocked(lock);
152 >        assertEquals(0, lock.getReadLockCount());
153 >
154 >        lock = new PublicReentrantReadWriteLock(false);
155 >        assertFalse(lock.isFair());
156 >        assertNotWriteLocked(lock);
157 >        assertEquals(0, lock.getReadLockCount());
158      }
159  
160      /**
161       * write-locking and read-locking an unlocked lock succeed
162       */
163      public void testLock() {
164 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
165 <        rl.writeLock().lock();
166 <        assertTrue(rl.isWriteLocked());
167 <        assertTrue(rl.isWriteLockedByCurrentThread());
168 <        assertTrue(rl.writeLock().isHeldByCurrentThread());
169 <        assertEquals(0, rl.getReadLockCount());
170 <        rl.writeLock().unlock();
171 <        assertFalse(rl.isWriteLocked());
172 <        assertFalse(rl.isWriteLockedByCurrentThread());
173 <        assertFalse(rl.writeLock().isHeldByCurrentThread());
174 <        assertEquals(0, rl.getReadLockCount());
175 <        rl.readLock().lock();
176 <        assertFalse(rl.isWriteLocked());
96 <        assertFalse(rl.isWriteLockedByCurrentThread());
97 <        assertEquals(1, rl.getReadLockCount());
98 <        rl.readLock().unlock();
99 <        assertFalse(rl.isWriteLocked());
100 <        assertFalse(rl.isWriteLockedByCurrentThread());
101 <        assertEquals(0, rl.getReadLockCount());
164 >        PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
165 >        assertNotWriteLocked(lock);
166 >        lock.writeLock().lock();
167 >        assertWriteLockedBy(lock, Thread.currentThread());
168 >        lock.writeLock().unlock();
169 >        assertNotWriteLocked(lock);
170 >        assertEquals(0, lock.getReadLockCount());
171 >        lock.readLock().lock();
172 >        assertNotWriteLocked(lock);
173 >        assertEquals(1, lock.getReadLockCount());
174 >        lock.readLock().unlock();
175 >        assertNotWriteLocked(lock);
176 >        assertEquals(0, lock.getReadLockCount());
177      }
178  
104
179      /**
180       * locking an unlocked fair lock succeeds
181       */
182      public void testFairLock() {
183 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock(true);
184 <        rl.writeLock().lock();
185 <        assertTrue(rl.isWriteLocked());
186 <        assertTrue(rl.isWriteLockedByCurrentThread());
187 <        assertTrue(rl.writeLock().isHeldByCurrentThread());
188 <        assertEquals(0, rl.getReadLockCount());
189 <        rl.writeLock().unlock();
190 <        assertFalse(rl.isWriteLocked());
191 <        assertFalse(rl.isWriteLockedByCurrentThread());
192 <        assertFalse(rl.writeLock().isHeldByCurrentThread());
193 <        assertEquals(0, rl.getReadLockCount());
194 <        rl.readLock().lock();
195 <        assertFalse(rl.isWriteLocked());
122 <        assertFalse(rl.isWriteLockedByCurrentThread());
123 <        assertEquals(1, rl.getReadLockCount());
124 <        rl.readLock().unlock();
125 <        assertFalse(rl.isWriteLocked());
126 <        assertFalse(rl.isWriteLockedByCurrentThread());
127 <        assertEquals(0, rl.getReadLockCount());
183 >        PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock(true);
184 >        assertNotWriteLocked(lock);
185 >        lock.writeLock().lock();
186 >        assertWriteLockedBy(lock, Thread.currentThread());
187 >        lock.writeLock().unlock();
188 >        assertNotWriteLocked(lock);
189 >        assertEquals(0, lock.getReadLockCount());
190 >        lock.readLock().lock();
191 >        assertNotWriteLocked(lock);
192 >        assertEquals(1, lock.getReadLockCount());
193 >        lock.readLock().unlock();
194 >        assertNotWriteLocked(lock);
195 >        assertEquals(0, lock.getReadLockCount());
196      }
197  
198      /**
# Line 172 | Line 240 | public class ReentrantReadWriteLockTest
240          }
241      }
242  
175
243      /**
244       * write-unlocking an unlocked lock throws IllegalMonitorStateException
245       */
246 <    public void testUnlock_IllegalMonitorStateException() {
247 <        ReentrantReadWriteLock rl = new ReentrantReadWriteLock();
246 >    public void testWriteUnlock_IllegalMonitorStateException() {
247 >        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
248          try {
249 <            rl.writeLock().unlock();
249 >            lock.writeLock().unlock();
250              shouldThrow();
251          } catch (IllegalMonitorStateException success) {}
252      }
253  
254 +    /**
255 +     * read-unlocking an unlocked lock throws IllegalMonitorStateException
256 +     */
257 +    public void testReadUnlock_IllegalMonitorStateException() {
258 +        ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
259 +        try {
260 +            lock.readLock().unlock();
261 +            shouldThrow();
262 +        } catch (IllegalMonitorStateException success) {}
263 +    }
264  
265      /**
266       * write-lockInterruptibly is interruptible
267       */
268      public void testWriteLockInterruptibly_Interrupted() throws Exception {
269 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
270 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
269 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
270 >        lock.writeLock().lock();
271 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
272              public void realRun() throws InterruptedException {
273                  lock.writeLock().lockInterruptibly();
196                lock.writeLock().unlock();
197                lock.writeLock().lockInterruptibly();
198                lock.writeLock().unlock();
274              }});
275  
276 <        lock.writeLock().lock();
202 <        t.start();
203 <        Thread.sleep(SHORT_DELAY_MS);
276 >        waitForQueuedThread(lock, t);
277          t.interrupt();
278 <        Thread.sleep(SHORT_DELAY_MS);
279 <        lock.writeLock().unlock();
207 <        t.join();
278 >        awaitTermination(t);
279 >        releaseWriteLock(lock);
280      }
281  
282      /**
283       * timed write-tryLock is interruptible
284       */
285      public void testWriteTryLock_Interrupted() throws InterruptedException {
286 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
286 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
287          lock.writeLock().lock();
288 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
288 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
289              public void realRun() throws InterruptedException {
290 <                lock.writeLock().tryLock(1000,MILLISECONDS);
290 >                lock.writeLock().tryLock(LONG_DELAY_MS, MILLISECONDS);
291              }});
292  
293 <        t.start();
293 >        waitForQueuedThread(lock, t);
294          t.interrupt();
295 <        lock.writeLock().unlock();
296 <        t.join();
295 >        awaitTermination(t);
296 >        releaseWriteLock(lock);
297      }
298  
299      /**
300       * read-lockInterruptibly is interruptible
301       */
302      public void testReadLockInterruptibly_Interrupted() throws InterruptedException {
303 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
303 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
304          lock.writeLock().lock();
305 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
305 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
306              public void realRun() throws InterruptedException {
307                  lock.readLock().lockInterruptibly();
308              }});
309  
310 <        t.start();
239 <        Thread.sleep(SHORT_DELAY_MS);
310 >        waitForQueuedThread(lock, t);
311          t.interrupt();
312 <        Thread.sleep(SHORT_DELAY_MS);
313 <        lock.writeLock().unlock();
243 <        t.join();
312 >        awaitTermination(t);
313 >        releaseWriteLock(lock);
314      }
315  
316      /**
317       * timed read-tryLock is interruptible
318       */
319      public void testReadTryLock_Interrupted() throws InterruptedException {
320 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
320 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
321          lock.writeLock().lock();
322 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
322 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
323              public void realRun() throws InterruptedException {
324 <                lock.readLock().tryLock(1000,MILLISECONDS);
324 >                lock.readLock().tryLock(LONG_DELAY_MS, MILLISECONDS);
325              }});
326  
327 <        t.start();
327 >        waitForQueuedThread(lock, t);
328          t.interrupt();
329 <        t.join();
329 >        awaitTermination(t);
330 >        releaseWriteLock(lock);
331      }
332  
262
333      /**
334       * write-tryLock fails if locked
335       */
336      public void testWriteTryLockWhenLocked() throws InterruptedException {
337 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
337 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
338          lock.writeLock().lock();
339 <        Thread t = new Thread(new Runnable() {
340 <                public void run() {
341 <                    threadAssertFalse(lock.writeLock().tryLock());
342 <                }
273 <            });
339 >        Thread t = newStartedThread(new CheckedRunnable() {
340 >            public void realRun() {
341 >                assertFalse(lock.writeLock().tryLock());
342 >            }});
343  
344 <        t.start();
345 <        t.join();
277 <        lock.writeLock().unlock();
344 >        awaitTermination(t);
345 >        releaseWriteLock(lock);
346      }
347  
348      /**
349       * read-tryLock fails if locked
350       */
351      public void testReadTryLockWhenLocked() throws InterruptedException {
352 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
352 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
353          lock.writeLock().lock();
354 <        Thread t = new Thread(new Runnable() {
355 <                public void run() {
356 <                    threadAssertFalse(lock.readLock().tryLock());
357 <                }
290 <            });
354 >        Thread t = newStartedThread(new CheckedRunnable() {
355 >            public void realRun() {
356 >                assertFalse(lock.readLock().tryLock());
357 >            }});
358  
359 <        t.start();
360 <        t.join();
294 <        lock.writeLock().unlock();
359 >        awaitTermination(t);
360 >        releaseWriteLock(lock);
361      }
362  
363      /**
# Line 300 | Line 366 | public class ReentrantReadWriteLockTest
366      public void testMultipleReadLocks() throws InterruptedException {
367          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
368          lock.readLock().lock();
369 <        Thread t = new Thread(new Runnable() {
370 <                public void run() {
371 <                    threadAssertTrue(lock.readLock().tryLock());
372 <                    lock.readLock().unlock();
373 <                }
308 <            });
369 >        Thread t = newStartedThread(new CheckedRunnable() {
370 >            public void realRun() {
371 >                assertTrue(lock.readLock().tryLock());
372 >                lock.readLock().unlock();
373 >            }});
374  
375 <        t.start();
311 <        t.join();
375 >        awaitTermination(t);
376          lock.readLock().unlock();
377      }
378  
379      /**
380 <     * A writelock succeeds after reading threads unlock
380 >     * A writelock succeeds only after a reading thread unlocks
381 >     */
382 >    public void testWriteAfterReadLock() throws InterruptedException {
383 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
384 >        lock.readLock().lock();
385 >
386 >        Thread t = newStartedThread(new CheckedRunnable() {
387 >            public void realRun() {
388 >                assertEquals(1, lock.getReadLockCount());
389 >                lock.writeLock().lock();
390 >                assertEquals(0, lock.getReadLockCount());
391 >                lock.writeLock().unlock();
392 >            }});
393 >        waitForQueuedThread(lock, t);
394 >        assertNotWriteLocked(lock);
395 >        assertEquals(1, lock.getReadLockCount());
396 >        lock.readLock().unlock();
397 >        assertEquals(0, lock.getReadLockCount());
398 >        awaitTermination(t);
399 >        assertNotWriteLocked(lock);
400 >    }
401 >
402 >    /**
403 >     * A writelock succeeds only after reading threads unlock
404       */
405      public void testWriteAfterMultipleReadLocks() throws InterruptedException {
406 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
406 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
407          lock.readLock().lock();
408 <        Thread t1 = new Thread(new Runnable() {
409 <                public void run() {
410 <                    lock.readLock().lock();
411 <                    lock.readLock().unlock();
412 <                }
413 <            });
414 <        Thread t2 = new Thread(new Runnable() {
415 <                public void run() {
329 <                    lock.writeLock().lock();
330 <                    lock.writeLock().unlock();
331 <                }
332 <            });
408 >        lock.readLock().lock();
409 >        Thread t1 = newStartedThread(new CheckedRunnable() {
410 >            public void realRun() {
411 >                lock.readLock().lock();
412 >                assertEquals(3, lock.getReadLockCount());
413 >                lock.readLock().unlock();
414 >            }});
415 >        awaitTermination(t1);
416  
417 <        t1.start();
418 <        t2.start();
419 <        Thread.sleep(SHORT_DELAY_MS);
417 >        Thread t2 = newStartedThread(new CheckedRunnable() {
418 >            public void realRun() {
419 >                assertEquals(2, lock.getReadLockCount());
420 >                lock.writeLock().lock();
421 >                assertEquals(0, lock.getReadLockCount());
422 >                lock.writeLock().unlock();
423 >            }});
424 >        waitForQueuedThread(lock, t2);
425 >        assertNotWriteLocked(lock);
426 >        assertEquals(2, lock.getReadLockCount());
427          lock.readLock().unlock();
428 <        t1.join(MEDIUM_DELAY_MS);
429 <        t2.join(MEDIUM_DELAY_MS);
430 <        assertTrue(!t1.isAlive());
431 <        assertTrue(!t2.isAlive());
428 >        lock.readLock().unlock();
429 >        assertEquals(0, lock.getReadLockCount());
430 >        awaitTermination(t2);
431 >        assertNotWriteLocked(lock);
432      }
433  
434      /**
435 <     * Readlocks succeed after a writing thread unlocks
435 >     * A thread that tries to acquire a fair read lock (non-reentrantly)
436 >     * will block if there is a waiting writer thread.
437 >     */
438 >    public void testReaderWriterReaderFairFifo() throws InterruptedException {
439 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock(true);
440 >        final AtomicBoolean t1GotLock = new AtomicBoolean(false);
441 >
442 >        lock.readLock().lock();
443 >        Thread t1 = newStartedThread(new CheckedRunnable() {
444 >            public void realRun() {
445 >                assertEquals(1, lock.getReadLockCount());
446 >                lock.writeLock().lock();
447 >                assertEquals(0, lock.getReadLockCount());
448 >                t1GotLock.set(true);
449 >                lock.writeLock().unlock();
450 >            }});
451 >        waitForQueuedThread(lock, t1);
452 >
453 >        Thread t2 = newStartedThread(new CheckedRunnable() {
454 >            public void realRun() {
455 >                assertEquals(1, lock.getReadLockCount());
456 >                lock.readLock().lock();
457 >                assertEquals(1, lock.getReadLockCount());
458 >                assertTrue(t1GotLock.get());
459 >                lock.readLock().unlock();
460 >            }});
461 >        waitForQueuedThread(lock, t2);
462 >        assertTrue(t1.isAlive());
463 >        assertNotWriteLocked(lock);
464 >        assertEquals(1, lock.getReadLockCount());
465 >        lock.readLock().unlock();
466 >        awaitTermination(t1);
467 >        awaitTermination(t2);
468 >        assertNotWriteLocked(lock);
469 >    }
470 >
471 >    /**
472 >     * Readlocks succeed only after a writing thread unlocks
473       */
474      public void testReadAfterWriteLock() throws InterruptedException {
475 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
475 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
476          lock.writeLock().lock();
477 <        Thread t1 = new Thread(new Runnable() {
478 <                public void run() {
479 <                    lock.readLock().lock();
480 <                    lock.readLock().unlock();
481 <                }
482 <            });
483 <        Thread t2 = new Thread(new Runnable() {
484 <                public void run() {
485 <                    lock.readLock().lock();
486 <                    lock.readLock().unlock();
360 <                }
361 <            });
477 >        Thread t1 = newStartedThread(new CheckedRunnable() {
478 >            public void realRun() {
479 >                lock.readLock().lock();
480 >                lock.readLock().unlock();
481 >            }});
482 >        Thread t2 = newStartedThread(new CheckedRunnable() {
483 >            public void realRun() {
484 >                lock.readLock().lock();
485 >                lock.readLock().unlock();
486 >            }});
487  
488 <        t1.start();
489 <        t2.start();
490 <        Thread.sleep(SHORT_DELAY_MS);
491 <        lock.writeLock().unlock();
492 <        t1.join(MEDIUM_DELAY_MS);
368 <        t2.join(MEDIUM_DELAY_MS);
369 <        assertTrue(!t1.isAlive());
370 <        assertTrue(!t2.isAlive());
488 >        waitForQueuedThread(lock, t1);
489 >        waitForQueuedThread(lock, t2);
490 >        releaseWriteLock(lock);
491 >        awaitTermination(t1);
492 >        awaitTermination(t2);
493      }
494  
495      /**
# Line 386 | Line 508 | public class ReentrantReadWriteLockTest
508       * other threads are waiting for readlock
509       */
510      public void testReadHoldingWriteLock2() throws InterruptedException {
511 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
511 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
512          lock.writeLock().lock();
391        Thread t1 = new Thread(new Runnable() {
392                public void run() {
393                    lock.readLock().lock();
394                    lock.readLock().unlock();
395                }
396            });
397        Thread t2 = new Thread(new Runnable() {
398                public void run() {
399                    lock.readLock().lock();
400                    lock.readLock().unlock();
401                }
402            });
403
404        t1.start();
405        t2.start();
513          lock.readLock().lock();
514          lock.readLock().unlock();
515 <        Thread.sleep(SHORT_DELAY_MS);
515 >
516 >        Thread t1 = newStartedThread(new CheckedRunnable() {
517 >            public void realRun() {
518 >                lock.readLock().lock();
519 >                lock.readLock().unlock();
520 >            }});
521 >        Thread t2 = newStartedThread(new CheckedRunnable() {
522 >            public void realRun() {
523 >                lock.readLock().lock();
524 >                lock.readLock().unlock();
525 >            }});
526 >
527 >        waitForQueuedThread(lock, t1);
528 >        waitForQueuedThread(lock, t2);
529 >        assertWriteLockedBy(lock, Thread.currentThread());
530          lock.readLock().lock();
531          lock.readLock().unlock();
532 <        lock.writeLock().unlock();
533 <        t1.join(MEDIUM_DELAY_MS);
534 <        t2.join(MEDIUM_DELAY_MS);
414 <        assertTrue(!t1.isAlive());
415 <        assertTrue(!t2.isAlive());
532 >        releaseWriteLock(lock);
533 >        awaitTermination(t1);
534 >        awaitTermination(t2);
535      }
536  
537      /**
538 <     *  Read lock succeeds if write locked by current thread even if
538 >     * Read lock succeeds if write locked by current thread even if
539       * other threads are waiting for writelock
540       */
541      public void testReadHoldingWriteLock3() throws InterruptedException {
542 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
542 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
543          lock.writeLock().lock();
425        Thread t1 = new Thread(new Runnable() {
426                public void run() {
427                    lock.writeLock().lock();
428                    lock.writeLock().unlock();
429                }
430            });
431        Thread t2 = new Thread(new Runnable() {
432                public void run() {
433                    lock.writeLock().lock();
434                    lock.writeLock().unlock();
435                }
436            });
437
438        t1.start();
439        t2.start();
544          lock.readLock().lock();
545          lock.readLock().unlock();
546 <        Thread.sleep(SHORT_DELAY_MS);
546 >
547 >        Thread t1 = newStartedThread(new CheckedRunnable() {
548 >            public void realRun() {
549 >                lock.writeLock().lock();
550 >                lock.writeLock().unlock();
551 >            }});
552 >        Thread t2 = newStartedThread(new CheckedRunnable() {
553 >            public void realRun() {
554 >                lock.writeLock().lock();
555 >                lock.writeLock().unlock();
556 >            }});
557 >
558 >        waitForQueuedThread(lock, t1);
559 >        waitForQueuedThread(lock, t2);
560 >        assertWriteLockedBy(lock, Thread.currentThread());
561          lock.readLock().lock();
562          lock.readLock().unlock();
563 <        lock.writeLock().unlock();
564 <        t1.join(MEDIUM_DELAY_MS);
565 <        t2.join(MEDIUM_DELAY_MS);
448 <        assertTrue(!t1.isAlive());
449 <        assertTrue(!t2.isAlive());
563 >        releaseWriteLock(lock);
564 >        awaitTermination(t1);
565 >        awaitTermination(t2);
566      }
567  
452
568      /**
569 <     *  Write lock succeeds if write locked by current thread even if
569 >     * Write lock succeeds if write locked by current thread even if
570       * other threads are waiting for writelock
571       */
572      public void testWriteHoldingWriteLock4() throws InterruptedException {
573 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
459 <        lock.writeLock().lock();
460 <        Thread t1 = new Thread(new Runnable() {
461 <                public void run() {
462 <                    lock.writeLock().lock();
463 <                    lock.writeLock().unlock();
464 <                }
465 <            });
466 <        Thread t2 = new Thread(new Runnable() {
467 <                public void run() {
468 <                    lock.writeLock().lock();
469 <                    lock.writeLock().unlock();
470 <                }
471 <            });
472 <
473 <        t1.start();
474 <        t2.start();
573 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
574          lock.writeLock().lock();
476        lock.writeLock().unlock();
477        Thread.sleep(SHORT_DELAY_MS);
575          lock.writeLock().lock();
576          lock.writeLock().unlock();
480        lock.writeLock().unlock();
481        t1.join(MEDIUM_DELAY_MS);
482        t2.join(MEDIUM_DELAY_MS);
483        assertTrue(!t1.isAlive());
484        assertTrue(!t2.isAlive());
485    }
577  
578 +        Thread t1 = newStartedThread(new CheckedRunnable() {
579 +            public void realRun() {
580 +                lock.writeLock().lock();
581 +                lock.writeLock().unlock();
582 +            }});
583 +        Thread t2 = newStartedThread(new CheckedRunnable() {
584 +            public void realRun() {
585 +                lock.writeLock().lock();
586 +                lock.writeLock().unlock();
587 +            }});
588 +
589 +        waitForQueuedThread(lock, t1);
590 +        waitForQueuedThread(lock, t2);
591 +        assertWriteLockedBy(lock, Thread.currentThread());
592 +        assertEquals(1, lock.getWriteHoldCount());
593 +        lock.writeLock().lock();
594 +        assertWriteLockedBy(lock, Thread.currentThread());
595 +        assertEquals(2, lock.getWriteHoldCount());
596 +        lock.writeLock().unlock();
597 +        releaseWriteLock(lock);
598 +        awaitTermination(t1);
599 +        awaitTermination(t2);
600 +    }
601  
602      /**
603       * Fair Read trylock succeeds if write locked by current thread
# Line 501 | Line 615 | public class ReentrantReadWriteLockTest
615       * other threads are waiting for readlock
616       */
617      public void testReadHoldingWriteLockFair2() throws InterruptedException {
618 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
618 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock(true);
619          lock.writeLock().lock();
506        Thread t1 = new Thread(new Runnable() {
507                public void run() {
508                    lock.readLock().lock();
509                    lock.readLock().unlock();
510                }
511            });
512        Thread t2 = new Thread(new Runnable() {
513                public void run() {
514                    lock.readLock().lock();
515                    lock.readLock().unlock();
516                }
517            });
518
519        t1.start();
520        t2.start();
620          lock.readLock().lock();
621          lock.readLock().unlock();
622 <        Thread.sleep(SHORT_DELAY_MS);
622 >
623 >        Thread t1 = newStartedThread(new CheckedRunnable() {
624 >            public void realRun() {
625 >                lock.readLock().lock();
626 >                lock.readLock().unlock();
627 >            }});
628 >        Thread t2 = newStartedThread(new CheckedRunnable() {
629 >            public void realRun() {
630 >                lock.readLock().lock();
631 >                lock.readLock().unlock();
632 >            }});
633 >
634 >        waitForQueuedThread(lock, t1);
635 >        waitForQueuedThread(lock, t2);
636 >        assertWriteLockedBy(lock, Thread.currentThread());
637          lock.readLock().lock();
638          lock.readLock().unlock();
639 <        lock.writeLock().unlock();
640 <        t1.join(MEDIUM_DELAY_MS);
641 <        t2.join(MEDIUM_DELAY_MS);
529 <        assertTrue(!t1.isAlive());
530 <        assertTrue(!t2.isAlive());
639 >        releaseWriteLock(lock);
640 >        awaitTermination(t1);
641 >        awaitTermination(t2);
642      }
643  
533
644      /**
645       * Fair Read lock succeeds if write locked by current thread even if
646       * other threads are waiting for writelock
647       */
648      public void testReadHoldingWriteLockFair3() throws InterruptedException {
649 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
649 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock(true);
650          lock.writeLock().lock();
541        Thread t1 = new Thread(new Runnable() {
542                public void run() {
543                    lock.writeLock().lock();
544                    lock.writeLock().unlock();
545                }
546            });
547        Thread t2 = new Thread(new Runnable() {
548                public void run() {
549                    lock.writeLock().lock();
550                    lock.writeLock().unlock();
551                }
552            });
553
554        t1.start();
555        t2.start();
651          lock.readLock().lock();
652          lock.readLock().unlock();
653 <        Thread.sleep(SHORT_DELAY_MS);
653 >
654 >        Thread t1 = newStartedThread(new CheckedRunnable() {
655 >            public void realRun() {
656 >                lock.writeLock().lock();
657 >                lock.writeLock().unlock();
658 >            }});
659 >        Thread t2 = newStartedThread(new CheckedRunnable() {
660 >            public void realRun() {
661 >                lock.writeLock().lock();
662 >                lock.writeLock().unlock();
663 >            }});
664 >
665 >        waitForQueuedThread(lock, t1);
666 >        waitForQueuedThread(lock, t2);
667 >        assertWriteLockedBy(lock, Thread.currentThread());
668          lock.readLock().lock();
669          lock.readLock().unlock();
670 <        lock.writeLock().unlock();
671 <        t1.join(MEDIUM_DELAY_MS);
672 <        t2.join(MEDIUM_DELAY_MS);
564 <        assertTrue(!t1.isAlive());
565 <        assertTrue(!t2.isAlive());
670 >        releaseWriteLock(lock);
671 >        awaitTermination(t1);
672 >        awaitTermination(t2);
673      }
674  
568
675      /**
676       * Fair Write lock succeeds if write locked by current thread even if
677       * other threads are waiting for writelock
678       */
679      public void testWriteHoldingWriteLockFair4() throws InterruptedException {
680 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
680 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock(true);
681          lock.writeLock().lock();
682 <        Thread t1 = new Thread(new Runnable() {
683 <                public void run() {
684 <                    lock.writeLock().lock();
685 <                    lock.writeLock().unlock();
686 <                }
687 <            });
688 <        Thread t2 = new Thread(new Runnable() {
689 <                public void run() {
690 <                    lock.writeLock().lock();
691 <                    lock.writeLock().unlock();
586 <                }
587 <            });
682 >        Thread t1 = newStartedThread(new CheckedRunnable() {
683 >            public void realRun() {
684 >                lock.writeLock().lock();
685 >                lock.writeLock().unlock();
686 >            }});
687 >        Thread t2 = newStartedThread(new CheckedRunnable() {
688 >            public void realRun() {
689 >                lock.writeLock().lock();
690 >                lock.writeLock().unlock();
691 >            }});
692  
693 <        t1.start();
694 <        t2.start();
695 <        Thread.sleep(SHORT_DELAY_MS);
696 <        assertTrue(lock.isWriteLockedByCurrentThread());
593 <        assertTrue(lock.getWriteHoldCount() == 1);
693 >        waitForQueuedThread(lock, t1);
694 >        waitForQueuedThread(lock, t2);
695 >        assertWriteLockedBy(lock, Thread.currentThread());
696 >        assertEquals(1, lock.getWriteHoldCount());
697          lock.writeLock().lock();
698 <        assertTrue(lock.getWriteHoldCount() == 2);
698 >        assertEquals(2, lock.getWriteHoldCount());
699          lock.writeLock().unlock();
700          lock.writeLock().lock();
701          lock.writeLock().unlock();
702 <        lock.writeLock().unlock();
703 <        t1.join(MEDIUM_DELAY_MS);
704 <        t2.join(MEDIUM_DELAY_MS);
602 <        assertTrue(!t1.isAlive());
603 <        assertTrue(!t2.isAlive());
702 >        releaseWriteLock(lock);
703 >        awaitTermination(t1);
704 >        awaitTermination(t2);
705      }
706  
606
707      /**
708       * Read tryLock succeeds if readlocked but not writelocked
709       */
710      public void testTryLockWhenReadLocked() throws InterruptedException {
711          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
712          lock.readLock().lock();
713 <        Thread t = new Thread(new Runnable() {
714 <                public void run() {
715 <                    threadAssertTrue(lock.readLock().tryLock());
716 <                    lock.readLock().unlock();
717 <                }
618 <            });
713 >        Thread t = newStartedThread(new CheckedRunnable() {
714 >            public void realRun() {
715 >                assertTrue(lock.readLock().tryLock());
716 >                lock.readLock().unlock();
717 >            }});
718  
719 <        t.start();
621 <        t.join();
719 >        awaitTermination(t);
720          lock.readLock().unlock();
721      }
722  
625
626
723      /**
724       * write tryLock fails when readlocked
725       */
726      public void testWriteTryLockWhenReadLocked() throws InterruptedException {
727          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
728          lock.readLock().lock();
729 <        Thread t = new Thread(new Runnable() {
730 <                public void run() {
731 <                    threadAssertFalse(lock.writeLock().tryLock());
732 <                }
637 <            });
729 >        Thread t = newStartedThread(new CheckedRunnable() {
730 >            public void realRun() {
731 >                assertFalse(lock.writeLock().tryLock());
732 >            }});
733  
734 <        t.start();
640 <        t.join();
734 >        awaitTermination(t);
735          lock.readLock().unlock();
736      }
737  
644
738      /**
739       * Fair Read tryLock succeeds if readlocked but not writelocked
740       */
741      public void testTryLockWhenReadLockedFair() throws InterruptedException {
742          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
743          lock.readLock().lock();
744 <        Thread t = new Thread(new Runnable() {
745 <                public void run() {
746 <                    threadAssertTrue(lock.readLock().tryLock());
747 <                    lock.readLock().unlock();
748 <                }
656 <            });
744 >        Thread t = newStartedThread(new CheckedRunnable() {
745 >            public void realRun() {
746 >                assertTrue(lock.readLock().tryLock());
747 >                lock.readLock().unlock();
748 >            }});
749  
750 <        t.start();
659 <        t.join();
750 >        awaitTermination(t);
751          lock.readLock().unlock();
752      }
753  
663
664
754      /**
755       * Fair write tryLock fails when readlocked
756       */
757      public void testWriteTryLockWhenReadLockedFair() throws InterruptedException {
758          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock(true);
759          lock.readLock().lock();
760 <        Thread t = new Thread(new Runnable() {
761 <                public void run() {
762 <                    threadAssertFalse(lock.writeLock().tryLock());
763 <                }
675 <            });
760 >        Thread t = newStartedThread(new CheckedRunnable() {
761 >            public void realRun() {
762 >                assertFalse(lock.writeLock().tryLock());
763 >            }});
764  
765 <        t.start();
678 <        t.join();
765 >        awaitTermination(t);
766          lock.readLock().unlock();
767      }
768  
682
683
769      /**
770       * write timed tryLock times out if locked
771       */
772      public void testWriteTryLock_Timeout() throws InterruptedException {
773          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
774          lock.writeLock().lock();
775 <        Thread t = new Thread(new CheckedRunnable() {
775 >        Thread t = newStartedThread(new CheckedRunnable() {
776              public void realRun() throws InterruptedException {
777 <                threadAssertFalse(lock.writeLock().tryLock(1, MILLISECONDS));
777 >                assertFalse(lock.writeLock().tryLock(1, MILLISECONDS));
778              }});
779  
780 <        t.start();
696 <        t.join();
780 >        awaitTermination(t);
781          assertTrue(lock.writeLock().isHeldByCurrentThread());
782          lock.writeLock().unlock();
783      }
# Line 704 | Line 788 | public class ReentrantReadWriteLockTest
788      public void testReadTryLock_Timeout() throws InterruptedException {
789          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
790          lock.writeLock().lock();
791 <        Thread t = new Thread(new CheckedRunnable() {
791 >        Thread t = newStartedThread(new CheckedRunnable() {
792              public void realRun() throws InterruptedException {
793 <                threadAssertFalse(lock.readLock().tryLock(1, MILLISECONDS));
793 >                assertFalse(lock.readLock().tryLock(1, MILLISECONDS));
794              }});
795  
796 <        t.start();
713 <        t.join();
796 >        awaitTermination(t);
797          assertTrue(lock.writeLock().isHeldByCurrentThread());
798          lock.writeLock().unlock();
799      }
800  
718
801      /**
802       * write lockInterruptibly succeeds if lock free else is interruptible
803       */
804      public void testWriteLockInterruptibly() throws InterruptedException {
805 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
805 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
806          lock.writeLock().lockInterruptibly();
807 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
807 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
808              public void realRun() throws InterruptedException {
809                  lock.writeLock().lockInterruptibly();
810              }});
811  
812 <        t.start();
731 <        Thread.sleep(SHORT_DELAY_MS);
812 >        waitForQueuedThread(lock, t);
813          t.interrupt();
814 <        Thread.sleep(SHORT_DELAY_MS);
815 <        t.join();
735 <        lock.writeLock().unlock();
814 >        awaitTermination(t);
815 >        releaseWriteLock(lock);
816      }
817  
818      /**
819 <     *  read lockInterruptibly succeeds if lock free else is interruptible
819 >     * read lockInterruptibly succeeds if lock free else is interruptible
820       */
821      public void testReadLockInterruptibly() throws InterruptedException {
822 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
822 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
823          lock.writeLock().lockInterruptibly();
824 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
824 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
825              public void realRun() throws InterruptedException {
826                  lock.readLock().lockInterruptibly();
827              }});
828  
829 <        t.start();
750 <        Thread.sleep(SHORT_DELAY_MS);
829 >        waitForQueuedThread(lock, t);
830          t.interrupt();
831 <        t.join();
832 <        lock.writeLock().unlock();
831 >        awaitTermination(t);
832 >        releaseWriteLock(lock);
833      }
834  
835      /**
# Line 763 | Line 842 | public class ReentrantReadWriteLockTest
842              c.await();
843              shouldThrow();
844          } catch (IllegalMonitorStateException success) {}
845 +        try {
846 +            c.await(LONG_DELAY_MS, MILLISECONDS);
847 +            shouldThrow();
848 +        } catch (IllegalMonitorStateException success) {}
849 +        try {
850 +            c.awaitNanos(100);
851 +            shouldThrow();
852 +        } catch (IllegalMonitorStateException success) {}
853 +        try {
854 +            c.awaitUninterruptibly();
855 +            shouldThrow();
856 +        } catch (IllegalMonitorStateException success) {}
857      }
858  
859      /**
# Line 778 | Line 869 | public class ReentrantReadWriteLockTest
869      }
870  
871      /**
872 +     * Calling signalAll without holding lock throws IllegalMonitorStateException
873 +     */
874 +    public void testSignalAll_IllegalMonitor() {
875 +        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
876 +        final Condition c = lock.writeLock().newCondition();
877 +        try {
878 +            c.signalAll();
879 +            shouldThrow();
880 +        } catch (IllegalMonitorStateException success) {}
881 +    }
882 +
883 +    /**
884       * awaitNanos without a signal times out
885       */
886      public void testAwaitNanos_Timeout() throws InterruptedException {
887          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
888          final Condition c = lock.writeLock().newCondition();
786
889          lock.writeLock().lock();
890 <        long t = c.awaitNanos(100);
891 <        assertTrue(t <= 0);
890 >        long startTime = System.nanoTime();
891 >        long timeoutMillis = 10;
892 >        long timeoutNanos = MILLISECONDS.toNanos(timeoutMillis);
893 >        long nanosRemaining = c.awaitNanos(timeoutNanos);
894 >        assertTrue(nanosRemaining <= 0);
895 >        assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
896          lock.writeLock().unlock();
897      }
898  
793
899      /**
900 <     *  timed await without a signal times out
900 >     * timed await without a signal times out
901       */
902      public void testAwait_Timeout() throws InterruptedException {
903          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
904          final Condition c = lock.writeLock().newCondition();
905          lock.writeLock().lock();
906 <        assertFalse(c.await(SHORT_DELAY_MS, MILLISECONDS));
906 >        long startTime = System.nanoTime();
907 >        long timeoutMillis = 10;
908 >        assertFalse(c.await(timeoutMillis, MILLISECONDS));
909 >        assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
910          lock.writeLock().unlock();
911      }
912  
# Line 820 | Line 928 | public class ReentrantReadWriteLockTest
928      public void testAwait() throws InterruptedException {
929          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
930          final Condition c = lock.writeLock().newCondition();
931 <        Thread t = new Thread(new CheckedRunnable() {
931 >        final CountDownLatch locked = new CountDownLatch(1);
932 >        Thread t = newStartedThread(new CheckedRunnable() {
933              public void realRun() throws InterruptedException {
934                  lock.writeLock().lock();
935 +                locked.countDown();
936                  c.await();
937                  lock.writeLock().unlock();
938              }});
939  
940 <        t.start();
831 <        Thread.sleep(SHORT_DELAY_MS);
940 >        locked.await();
941          lock.writeLock().lock();
942          c.signal();
943 +        assertTrue(t.isAlive());
944          lock.writeLock().unlock();
945 <        t.join(SHORT_DELAY_MS);
836 <        assertFalse(t.isAlive());
837 <    }
838 <
839 <    /** A helper class for uninterruptible wait tests */
840 <    class UninterruptableThread extends Thread {
841 <        private Lock lock;
842 <        private Condition c;
843 <
844 <        public volatile boolean canAwake = false;
845 <        public volatile boolean interrupted = false;
846 <        public volatile boolean lockStarted = false;
847 <
848 <        public UninterruptableThread(Lock lock, Condition c) {
849 <            this.lock = lock;
850 <            this.c = c;
851 <        }
852 <
853 <        public synchronized void run() {
854 <            lock.lock();
855 <            lockStarted = true;
856 <
857 <            while (!canAwake) {
858 <                c.awaitUninterruptibly();
859 <            }
860 <
861 <            interrupted = isInterrupted();
862 <            lock.unlock();
863 <        }
945 >        awaitTermination(t);
946      }
947  
948      /**
# Line 869 | Line 951 | public class ReentrantReadWriteLockTest
951      public void testAwaitUninterruptibly() throws InterruptedException {
952          final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
953          final Condition c = lock.writeLock().newCondition();
954 <        UninterruptableThread thread = new UninterruptableThread(lock.writeLock(), c);
955 <
956 <        thread.start();
957 <
958 <        while (!thread.lockStarted) {
959 <            Thread.sleep(100);
960 <        }
954 >        final CountDownLatch locked = new CountDownLatch(1);
955 >        final AtomicBoolean canAwake = new AtomicBoolean(false);
956 >        Thread t = newStartedThread(new CheckedRunnable() {
957 >            public void realRun() {
958 >                lock.writeLock().lock();
959 >                locked.countDown();
960 >                c.awaitUninterruptibly();
961 >                assertTrue(Thread.interrupted());
962 >                lock.writeLock().unlock();
963 >            }});
964  
965 +        locked.await();
966          lock.writeLock().lock();
967 <        try {
968 <            thread.interrupt();
969 <            thread.canAwake = true;
970 <            c.signal();
971 <        } finally {
972 <            lock.writeLock().unlock();
973 <        }
974 <
889 <        thread.join();
890 <        assertTrue(thread.interrupted);
891 <        assertFalse(thread.isAlive());
967 >        lock.writeLock().unlock();
968 >        t.interrupt();
969 >        t.join(10);
970 >        assertTrue(t.isAlive());
971 >        lock.writeLock().lock();
972 >        c.signal();
973 >        lock.writeLock().unlock();
974 >        awaitTermination(t);
975      }
976  
977      /**
978       * await is interruptible
979       */
980      public void testAwait_Interrupt() throws InterruptedException {
981 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
981 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
982          final Condition c = lock.writeLock().newCondition();
983 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
983 >        final CountDownLatch locked = new CountDownLatch(1);
984 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
985              public void realRun() throws InterruptedException {
986                  lock.writeLock().lock();
987 <                c.await();
988 <                lock.writeLock().unlock();
987 >                assertWriteLockedBy(lock, Thread.currentThread());
988 >                assertHasNoWaiters(lock, c);
989 >                locked.countDown();
990 >                try {
991 >                    c.await();
992 >                } finally {
993 >                    assertWriteLockedBy(lock, Thread.currentThread());
994 >                    assertHasNoWaiters(lock, c);
995 >                    lock.writeLock().unlock();
996 >                    assertFalse(Thread.interrupted());
997 >                }
998              }});
999  
1000 <        t.start();
1001 <        Thread.sleep(SHORT_DELAY_MS);
1000 >        locked.await();
1001 >        assertHasWaiters(lock, c, t);
1002          t.interrupt();
1003 <        t.join(SHORT_DELAY_MS);
1004 <        assertFalse(t.isAlive());
1003 >        awaitTermination(t);
1004 >        assertNotWriteLocked(lock);
1005      }
1006  
1007      /**
1008       * awaitNanos is interruptible
1009       */
1010      public void testAwaitNanos_Interrupt() throws InterruptedException {
1011 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1011 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1012          final Condition c = lock.writeLock().newCondition();
1013 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
1013 >        final CountDownLatch locked = new CountDownLatch(1);
1014 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
1015              public void realRun() throws InterruptedException {
1016                  lock.writeLock().lock();
1017 <                c.awaitNanos(SHORT_DELAY_MS * 2 * 1000000);
1018 <                lock.writeLock().unlock();
1017 >                assertWriteLockedBy(lock, Thread.currentThread());
1018 >                assertHasNoWaiters(lock, c);
1019 >                locked.countDown();
1020 >                try {
1021 >                    c.awaitNanos(MILLISECONDS.toNanos(LONG_DELAY_MS));
1022 >                } finally {
1023 >                    assertWriteLockedBy(lock, Thread.currentThread());
1024 >                    assertHasNoWaiters(lock, c);
1025 >                    lock.writeLock().unlock();
1026 >                    assertFalse(Thread.interrupted());
1027 >                }
1028              }});
1029  
1030 <        t.start();
1031 <        Thread.sleep(SHORT_DELAY_MS);
1030 >        locked.await();
1031 >        assertHasWaiters(lock, c, t);
1032          t.interrupt();
1033 <        t.join(SHORT_DELAY_MS);
1034 <        assertFalse(t.isAlive());
1033 >        awaitTermination(t);
1034 >        assertNotWriteLocked(lock);
1035      }
1036  
1037      /**
1038       * awaitUntil is interruptible
1039       */
1040      public void testAwaitUntil_Interrupt() throws InterruptedException {
1041 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1041 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1042          final Condition c = lock.writeLock().newCondition();
1043 <        Thread t = new Thread(new CheckedInterruptedRunnable() {
1043 >        final CountDownLatch locked = new CountDownLatch(1);
1044 >        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
1045              public void realRun() throws InterruptedException {
1046                  lock.writeLock().lock();
1047 +                assertWriteLockedBy(lock, Thread.currentThread());
1048 +                assertHasNoWaiters(lock, c);
1049 +                locked.countDown();
1050                  java.util.Date d = new java.util.Date();
1051 <                c.awaitUntil(new java.util.Date(d.getTime() + 10000));
1052 <                lock.writeLock().unlock();
1051 >                try {
1052 >                    c.awaitUntil(new java.util.Date(d.getTime() + 10000));
1053 >                } finally {
1054 >                    assertWriteLockedBy(lock, Thread.currentThread());
1055 >                    assertHasNoWaiters(lock, c);
1056 >                    lock.writeLock().unlock();
1057 >                    assertFalse(Thread.interrupted());
1058 >                }
1059              }});
1060  
1061 <        t.start();
1062 <        Thread.sleep(SHORT_DELAY_MS);
1061 >        locked.await();
1062 >        assertHasWaiters(lock, c, t);
1063          t.interrupt();
1064 <        t.join(SHORT_DELAY_MS);
1065 <        assertFalse(t.isAlive());
1064 >        awaitTermination(t);
1065 >        assertNotWriteLocked(lock);
1066      }
1067  
1068      /**
1069       * signalAll wakes up all threads
1070       */
1071      public void testSignalAll() throws InterruptedException {
1072 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1072 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1073          final Condition c = lock.writeLock().newCondition();
1074 <        Thread t1 = new Thread(new CheckedRunnable() {
1074 >        final CountDownLatch locked = new CountDownLatch(2);
1075 >        final Lock writeLock = lock.writeLock();
1076 >        Thread t1 = newStartedThread(new CheckedRunnable() {
1077              public void realRun() throws InterruptedException {
1078 <                lock.writeLock().lock();
1078 >                writeLock.lock();
1079 >                locked.countDown();
1080                  c.await();
1081 <                lock.writeLock().unlock();
1081 >                writeLock.unlock();
1082              }});
1083  
1084 <        Thread t2 = new Thread(new CheckedRunnable() {
1084 >        Thread t2 = newStartedThread(new CheckedRunnable() {
1085              public void realRun() throws InterruptedException {
1086 <                lock.writeLock().lock();
1086 >                writeLock.lock();
1087 >                locked.countDown();
1088                  c.await();
1089 <                lock.writeLock().unlock();
1089 >                writeLock.unlock();
1090              }});
1091  
1092 <        t1.start();
1093 <        t2.start();
1094 <        Thread.sleep(SHORT_DELAY_MS);
978 <        lock.writeLock().lock();
1092 >        locked.await();
1093 >        writeLock.lock();
1094 >        assertHasWaiters(lock, c, t1, t2);
1095          c.signalAll();
1096 <        lock.writeLock().unlock();
1097 <        t1.join(SHORT_DELAY_MS);
1098 <        t2.join(SHORT_DELAY_MS);
1099 <        assertFalse(t1.isAlive());
1100 <        assertFalse(t2.isAlive());
1096 >        assertHasNoWaiters(lock, c);
1097 >        writeLock.unlock();
1098 >        awaitTermination(t1);
1099 >        awaitTermination(t2);
1100 >    }
1101 >
1102 >    /**
1103 >     * signal wakes up waiting threads in FIFO order.
1104 >     */
1105 >    public void testSignalWakesFifo() throws InterruptedException {
1106 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1107 >        final Condition c = lock.writeLock().newCondition();
1108 >        final CountDownLatch locked1 = new CountDownLatch(1);
1109 >        final CountDownLatch locked2 = new CountDownLatch(1);
1110 >        final Lock writeLock = lock.writeLock();
1111 >        Thread t1 = newStartedThread(new CheckedRunnable() {
1112 >            public void realRun() throws InterruptedException {
1113 >                writeLock.lock();
1114 >                locked1.countDown();
1115 >                c.await();
1116 >                writeLock.unlock();
1117 >            }});
1118 >
1119 >        locked1.await();
1120 >
1121 >        Thread t2 = newStartedThread(new CheckedRunnable() {
1122 >            public void realRun() throws InterruptedException {
1123 >                writeLock.lock();
1124 >                locked2.countDown();
1125 >                c.await();
1126 >                writeLock.unlock();
1127 >            }});
1128 >
1129 >        locked2.await();
1130 >
1131 >        writeLock.lock();
1132 >        assertHasWaiters(lock, c, t1, t2);
1133 >        assertFalse(lock.hasQueuedThreads());
1134 >        c.signal();
1135 >        assertHasWaiters(lock, c, t2);
1136 >        assertTrue(lock.hasQueuedThread(t1));
1137 >        assertFalse(lock.hasQueuedThread(t2));
1138 >        c.signal();
1139 >        assertHasNoWaiters(lock, c);
1140 >        assertTrue(lock.hasQueuedThread(t1));
1141 >        assertTrue(lock.hasQueuedThread(t2));
1142 >        writeLock.unlock();
1143 >        awaitTermination(t1);
1144 >        awaitTermination(t2);
1145      }
1146  
1147      /**
# Line 1008 | Line 1168 | public class ReentrantReadWriteLockTest
1168       * hasQueuedThreads reports whether there are waiting threads
1169       */
1170      public void testhasQueuedThreads() throws InterruptedException {
1171 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1171 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1172          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1173          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1174          assertFalse(lock.hasQueuedThreads());
1175          lock.writeLock().lock();
1176 +        assertFalse(lock.hasQueuedThreads());
1177 +        long startTime = System.nanoTime();
1178          t1.start();
1179 <        Thread.sleep(SHORT_DELAY_MS);
1018 <        assertTrue(lock.hasQueuedThreads());
1179 >        waitForQueuedThread(lock, t1);
1180          t2.start();
1181 <        Thread.sleep(SHORT_DELAY_MS);
1181 >        waitForQueuedThread(lock, t2);
1182          assertTrue(lock.hasQueuedThreads());
1183          t1.interrupt();
1184 <        Thread.sleep(SHORT_DELAY_MS);
1184 >        awaitTermination(t1);
1185          assertTrue(lock.hasQueuedThreads());
1186          lock.writeLock().unlock();
1187 <        Thread.sleep(SHORT_DELAY_MS);
1187 >        awaitTermination(t2);
1188          assertFalse(lock.hasQueuedThreads());
1028        t1.join();
1029        t2.join();
1189      }
1190  
1191      /**
# Line 1044 | Line 1203 | public class ReentrantReadWriteLockTest
1203       * hasQueuedThread reports whether a thread is queued.
1204       */
1205      public void testHasQueuedThread() throws InterruptedException {
1206 <        final ReentrantReadWriteLock sync = new ReentrantReadWriteLock();
1207 <        Thread t1 = new Thread(new InterruptedLockRunnable(sync));
1208 <        Thread t2 = new Thread(new InterruptibleLockRunnable(sync));
1209 <        assertFalse(sync.hasQueuedThread(t1));
1210 <        assertFalse(sync.hasQueuedThread(t2));
1211 <        sync.writeLock().lock();
1206 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1207 >        Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1208 >        Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1209 >        assertFalse(lock.hasQueuedThread(t1));
1210 >        assertFalse(lock.hasQueuedThread(t2));
1211 >        lock.writeLock().lock();
1212 >        long startTime = System.nanoTime();
1213          t1.start();
1214 <        Thread.sleep(SHORT_DELAY_MS);
1215 <        assertTrue(sync.hasQueuedThread(t1));
1214 >        waitForQueuedThread(lock, t1);
1215 >        assertTrue(lock.hasQueuedThread(t1));
1216 >        assertFalse(lock.hasQueuedThread(t2));
1217          t2.start();
1218 <        Thread.sleep(SHORT_DELAY_MS);
1219 <        assertTrue(sync.hasQueuedThread(t1));
1220 <        assertTrue(sync.hasQueuedThread(t2));
1218 >        waitForQueuedThread(lock, t2);
1219 >        assertTrue(lock.hasQueuedThread(t1));
1220 >        assertTrue(lock.hasQueuedThread(t2));
1221          t1.interrupt();
1222 <        Thread.sleep(SHORT_DELAY_MS);
1223 <        assertFalse(sync.hasQueuedThread(t1));
1224 <        assertTrue(sync.hasQueuedThread(t2));
1225 <        sync.writeLock().unlock();
1226 <        Thread.sleep(SHORT_DELAY_MS);
1227 <        assertFalse(sync.hasQueuedThread(t1));
1228 <        Thread.sleep(SHORT_DELAY_MS);
1068 <        assertFalse(sync.hasQueuedThread(t2));
1069 <        t1.join();
1070 <        t2.join();
1222 >        awaitTermination(t1);
1223 >        assertFalse(lock.hasQueuedThread(t1));
1224 >        assertTrue(lock.hasQueuedThread(t2));
1225 >        lock.writeLock().unlock();
1226 >        awaitTermination(t2);
1227 >        assertFalse(lock.hasQueuedThread(t1));
1228 >        assertFalse(lock.hasQueuedThread(t2));
1229      }
1230  
1073
1231      /**
1232       * getQueueLength reports number of waiting threads
1233       */
1234      public void testGetQueueLength() throws InterruptedException {
1235 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1235 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1236          Thread t1 = new Thread(new InterruptedLockRunnable(lock));
1237          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1238          assertEquals(0, lock.getQueueLength());
1239          lock.writeLock().lock();
1240 +        long startTime = System.nanoTime();
1241          t1.start();
1242 <        Thread.sleep(SHORT_DELAY_MS);
1242 >        waitForQueuedThread(lock, t1);
1243          assertEquals(1, lock.getQueueLength());
1244          t2.start();
1245 <        Thread.sleep(SHORT_DELAY_MS);
1245 >        waitForQueuedThread(lock, t2);
1246          assertEquals(2, lock.getQueueLength());
1247          t1.interrupt();
1248 <        Thread.sleep(SHORT_DELAY_MS);
1248 >        awaitTermination(t1);
1249          assertEquals(1, lock.getQueueLength());
1250          lock.writeLock().unlock();
1251 <        Thread.sleep(SHORT_DELAY_MS);
1251 >        awaitTermination(t2);
1252          assertEquals(0, lock.getQueueLength());
1095        t1.join();
1096        t2.join();
1253      }
1254  
1255      /**
# Line 1105 | Line 1261 | public class ReentrantReadWriteLockTest
1261          Thread t2 = new Thread(new InterruptibleLockRunnable(lock));
1262          assertTrue(lock.getQueuedThreads().isEmpty());
1263          lock.writeLock().lock();
1264 +        long startTime = System.nanoTime();
1265          assertTrue(lock.getQueuedThreads().isEmpty());
1266          t1.start();
1267 <        Thread.sleep(SHORT_DELAY_MS);
1267 >        waitForQueuedThread(lock, t1);
1268 >        assertEquals(1, lock.getQueuedThreads().size());
1269          assertTrue(lock.getQueuedThreads().contains(t1));
1270          t2.start();
1271 <        Thread.sleep(SHORT_DELAY_MS);
1271 >        waitForQueuedThread(lock, t2);
1272 >        assertEquals(2, lock.getQueuedThreads().size());
1273          assertTrue(lock.getQueuedThreads().contains(t1));
1274          assertTrue(lock.getQueuedThreads().contains(t2));
1275          t1.interrupt();
1276 <        Thread.sleep(SHORT_DELAY_MS);
1276 >        awaitTermination(t1);
1277          assertFalse(lock.getQueuedThreads().contains(t1));
1278          assertTrue(lock.getQueuedThreads().contains(t2));
1279 +        assertEquals(1, lock.getQueuedThreads().size());
1280          lock.writeLock().unlock();
1281 <        Thread.sleep(SHORT_DELAY_MS);
1281 >        awaitTermination(t2);
1282          assertTrue(lock.getQueuedThreads().isEmpty());
1123        t1.join();
1124        t2.join();
1283      }
1284  
1285      /**
# Line 1146 | Line 1304 | public class ReentrantReadWriteLockTest
1304          } catch (NullPointerException success) {}
1305      }
1306  
1149
1307      /**
1308       * getWaitingThreads throws NPE if null
1309       */
# Line 1183 | Line 1340 | public class ReentrantReadWriteLockTest
1340          } catch (IllegalMonitorStateException success) {}
1341      }
1342  
1186
1343      /**
1344       * getWaitQueueLength throws IAE if not owned
1345       */
# Line 1209 | Line 1365 | public class ReentrantReadWriteLockTest
1365          } catch (IllegalMonitorStateException success) {}
1366      }
1367  
1212
1368      /**
1369       * getWaitingThreads throws IAE if not owned
1370       */
# Line 1235 | Line 1390 | public class ReentrantReadWriteLockTest
1390          } catch (IllegalMonitorStateException success) {}
1391      }
1392  
1238
1393      /**
1394       * hasWaiters returns true when a thread is waiting, else false
1395       */
1396      public void testHasWaiters() throws InterruptedException {
1397 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1397 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1398          final Condition c = lock.writeLock().newCondition();
1399 <        Thread t = new Thread(new CheckedRunnable() {
1399 >        final CountDownLatch locked = new CountDownLatch(1);
1400 >        Thread t = newStartedThread(new CheckedRunnable() {
1401              public void realRun() throws InterruptedException {
1402                  lock.writeLock().lock();
1403 <                threadAssertFalse(lock.hasWaiters(c));
1404 <                threadAssertEquals(0, lock.getWaitQueueLength(c));
1403 >                assertFalse(lock.hasWaiters(c));
1404 >                locked.countDown();
1405 >                assertEquals(0, lock.getWaitQueueLength(c));
1406                  c.await();
1407                  lock.writeLock().unlock();
1408              }});
1409  
1410 <        t.start();
1255 <        Thread.sleep(SHORT_DELAY_MS);
1410 >        locked.await();
1411          lock.writeLock().lock();
1412          assertTrue(lock.hasWaiters(c));
1413          assertEquals(1, lock.getWaitQueueLength(c));
1414          c.signal();
1415 +        assertHasNoWaiters(lock, c);
1416          lock.writeLock().unlock();
1417 <        Thread.sleep(SHORT_DELAY_MS);
1418 <        lock.writeLock().lock();
1263 <        assertFalse(lock.hasWaiters(c));
1264 <        assertEquals(0, lock.getWaitQueueLength(c));
1265 <        lock.writeLock().unlock();
1266 <        t.join(SHORT_DELAY_MS);
1267 <        assertFalse(t.isAlive());
1417 >        awaitTermination(t);
1418 >        assertHasNoWaiters(lock, c);
1419      }
1420  
1421      /**
1422       * getWaitQueueLength returns number of waiting threads
1423       */
1424      public void testGetWaitQueueLength() throws InterruptedException {
1425 <        final ReentrantReadWriteLock lock = new ReentrantReadWriteLock();
1425 >        final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1426          final Condition c = lock.writeLock().newCondition();
1427 <        Thread t = new Thread(new CheckedRunnable() {
1427 >        final CountDownLatch locked = new CountDownLatch(1);
1428 >        Thread t = newStartedThread(new CheckedRunnable() {
1429              public void realRun() throws InterruptedException {
1430                  lock.writeLock().lock();
1431 <                threadAssertFalse(lock.hasWaiters(c));
1432 <                threadAssertEquals(0, lock.getWaitQueueLength(c));
1431 >                assertEquals(0, lock.getWaitQueueLength(c));
1432 >                locked.countDown();
1433                  c.await();
1434                  lock.writeLock().unlock();
1435              }});
1436  
1437 <        t.start();
1286 <        Thread.sleep(SHORT_DELAY_MS);
1437 >        locked.await();
1438          lock.writeLock().lock();
1439 <        assertTrue(lock.hasWaiters(c));
1439 >        assertHasWaiters(lock, c, t);
1440          assertEquals(1, lock.getWaitQueueLength(c));
1441          c.signal();
1442 <        lock.writeLock().unlock();
1292 <        Thread.sleep(SHORT_DELAY_MS);
1293 <        lock.writeLock().lock();
1294 <        assertFalse(lock.hasWaiters(c));
1442 >        assertHasNoWaiters(lock, c);
1443          assertEquals(0, lock.getWaitQueueLength(c));
1444          lock.writeLock().unlock();
1445 <        t.join(SHORT_DELAY_MS);
1298 <        assertFalse(t.isAlive());
1445 >        awaitTermination(t);
1446      }
1447  
1301
1448      /**
1449       * getWaitingThreads returns only and all waiting threads
1450       */
1451      public void testGetWaitingThreads() throws InterruptedException {
1452          final PublicReentrantReadWriteLock lock = new PublicReentrantReadWriteLock();
1453          final Condition c = lock.writeLock().newCondition();
1454 +        final CountDownLatch locked1 = new CountDownLatch(1);
1455 +        final CountDownLatch locked2 = new CountDownLatch(1);
1456          Thread t1 = new Thread(new CheckedRunnable() {
1457              public void realRun() throws InterruptedException {
1458                  lock.writeLock().lock();
1459 <                threadAssertTrue(lock.getWaitingThreads(c).isEmpty());
1459 >                assertTrue(lock.getWaitingThreads(c).isEmpty());
1460 >                locked1.countDown();
1461                  c.await();
1462                  lock.writeLock().unlock();
1463              }});
# Line 1316 | Line 1465 | public class ReentrantReadWriteLockTest
1465          Thread t2 = new Thread(new CheckedRunnable() {
1466              public void realRun() throws InterruptedException {
1467                  lock.writeLock().lock();
1468 <                threadAssertFalse(lock.getWaitingThreads(c).isEmpty());
1468 >                assertFalse(lock.getWaitingThreads(c).isEmpty());
1469 >                locked2.countDown();
1470                  c.await();
1471                  lock.writeLock().unlock();
1472              }});
# Line 1324 | Line 1474 | public class ReentrantReadWriteLockTest
1474          lock.writeLock().lock();
1475          assertTrue(lock.getWaitingThreads(c).isEmpty());
1476          lock.writeLock().unlock();
1477 +
1478          t1.start();
1479 <        Thread.sleep(SHORT_DELAY_MS);
1479 >        locked1.await();
1480          t2.start();
1481 <        Thread.sleep(SHORT_DELAY_MS);
1481 >        locked2.await();
1482 >
1483          lock.writeLock().lock();
1484          assertTrue(lock.hasWaiters(c));
1485          assertTrue(lock.getWaitingThreads(c).contains(t1));
1486          assertTrue(lock.getWaitingThreads(c).contains(t2));
1487 +        assertEquals(2, lock.getWaitingThreads(c).size());
1488          c.signalAll();
1489 +        assertHasNoWaiters(lock, c);
1490          lock.writeLock().unlock();
1491 <        Thread.sleep(SHORT_DELAY_MS);
1492 <        lock.writeLock().lock();
1493 <        assertFalse(lock.hasWaiters(c));
1494 <        assertTrue(lock.getWaitingThreads(c).isEmpty());
1495 <        lock.writeLock().unlock();
1342 <        t1.join(SHORT_DELAY_MS);
1343 <        t2.join(SHORT_DELAY_MS);
1344 <        assertFalse(t1.isAlive());
1345 <        assertFalse(t2.isAlive());
1491 >
1492 >        awaitTermination(t1);
1493 >        awaitTermination(t2);
1494 >
1495 >        assertHasNoWaiters(lock, c);
1496      }
1497  
1498      /**

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines