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

Comparing jsr166/src/test/tck/StampedLockTest.java (file contents):
Revision 1.26 by jsr166, Sun Jul 17 03:06:50 2016 UTC vs.
Revision 1.40 by jsr166, Tue Sep 26 01:38:55 2017 UTC

# Line 11 | Line 11 | import static java.util.concurrent.TimeU
11   import java.util.ArrayList;
12   import java.util.List;
13   import java.util.concurrent.CountDownLatch;
14 + import java.util.concurrent.Future;
15   import java.util.concurrent.TimeUnit;
16   import java.util.concurrent.locks.Lock;
17   import java.util.concurrent.locks.StampedLock;
18   import java.util.function.BiConsumer;
18 import java.util.function.Consumer;
19   import java.util.function.Function;
20  
21   import junit.framework.Test;
# Line 30 | Line 30 | public class StampedLockTest extends JSR
30      }
31  
32      /**
33     * A runnable calling writeLockInterruptibly
34     */
35    class InterruptibleLockRunnable extends CheckedRunnable {
36        final StampedLock lock;
37        InterruptibleLockRunnable(StampedLock l) { lock = l; }
38        public void realRun() throws InterruptedException {
39            lock.writeLockInterruptibly();
40        }
41    }
42
43    /**
44     * A runnable calling writeLockInterruptibly that expects to be
45     * interrupted
46     */
47    class InterruptedLockRunnable extends CheckedInterruptedRunnable {
48        final StampedLock lock;
49        InterruptedLockRunnable(StampedLock l) { lock = l; }
50        public void realRun() throws InterruptedException {
51            lock.writeLockInterruptibly();
52        }
53    }
54
55    /**
33       * Releases write lock, checking isWriteLocked before and after
34       */
35 <    void releaseWriteLock(StampedLock lock, long s) {
35 >    void releaseWriteLock(StampedLock lock, long stamp) {
36          assertTrue(lock.isWriteLocked());
37 <        lock.unlockWrite(s);
37 >        assertValid(lock, stamp);
38 >        lock.unlockWrite(stamp);
39          assertFalse(lock.isWriteLocked());
40 +        assertFalse(lock.validate(stamp));
41      }
42  
43      /**
44 <     * Constructed StampedLock is in unlocked state
44 >     * Releases read lock, checking isReadLocked before and after
45       */
46 <    public void testConstructor() {
47 <        StampedLock lock;
48 <        lock = new StampedLock();
49 <        assertFalse(lock.isWriteLocked());
46 >    void releaseReadLock(StampedLock lock, long stamp) {
47 >        assertTrue(lock.isReadLocked());
48 >        assertValid(lock, stamp);
49 >        lock.unlockRead(stamp);
50          assertFalse(lock.isReadLocked());
51 <        assertEquals(lock.getReadLockCount(), 0);
51 >        assertTrue(lock.validate(stamp));
52      }
53  
54 <    /**
55 <     * write-locking and read-locking an unlocked lock succeed
56 <     */
78 <    public void testLock() {
79 <        StampedLock lock = new StampedLock();
80 <        assertFalse(lock.isWriteLocked());
81 <        assertFalse(lock.isReadLocked());
82 <        assertEquals(lock.getReadLockCount(), 0);
83 <        long s = lock.writeLock();
84 <        assertTrue(lock.isWriteLocked());
85 <        assertFalse(lock.isReadLocked());
86 <        assertEquals(lock.getReadLockCount(), 0);
87 <        lock.unlockWrite(s);
88 <        assertFalse(lock.isWriteLocked());
89 <        assertFalse(lock.isReadLocked());
90 <        assertEquals(lock.getReadLockCount(), 0);
91 <        long rs = lock.readLock();
92 <        assertFalse(lock.isWriteLocked());
93 <        assertTrue(lock.isReadLocked());
94 <        assertEquals(lock.getReadLockCount(), 1);
95 <        lock.unlockRead(rs);
96 <        assertFalse(lock.isWriteLocked());
97 <        assertFalse(lock.isReadLocked());
98 <        assertEquals(lock.getReadLockCount(), 0);
54 >    long assertNonZero(long v) {
55 >        assertTrue(v != 0L);
56 >        return v;
57      }
58  
59 <    /**
60 <     * unlock releases either a read or write lock
61 <     */
62 <    public void testUnlock() {
105 <        StampedLock lock = new StampedLock();
106 <        assertFalse(lock.isWriteLocked());
107 <        assertFalse(lock.isReadLocked());
108 <        assertEquals(lock.getReadLockCount(), 0);
109 <        long s = lock.writeLock();
110 <        assertTrue(lock.isWriteLocked());
111 <        assertFalse(lock.isReadLocked());
112 <        assertEquals(lock.getReadLockCount(), 0);
113 <        lock.unlock(s);
114 <        assertFalse(lock.isWriteLocked());
115 <        assertFalse(lock.isReadLocked());
116 <        assertEquals(lock.getReadLockCount(), 0);
117 <        long rs = lock.readLock();
118 <        assertFalse(lock.isWriteLocked());
119 <        assertTrue(lock.isReadLocked());
120 <        assertEquals(lock.getReadLockCount(), 1);
121 <        lock.unlock(rs);
122 <        assertFalse(lock.isWriteLocked());
123 <        assertFalse(lock.isReadLocked());
124 <        assertEquals(lock.getReadLockCount(), 0);
59 >    long assertValid(StampedLock lock, long stamp) {
60 >        assertTrue(stamp != 0L);
61 >        assertTrue(lock.validate(stamp));
62 >        return stamp;
63      }
64  
65 <    /**
128 <     * tryUnlockRead/Write succeeds if locked in associated mode else
129 <     * returns false
130 <     */
131 <    public void testTryUnlock() {
132 <        StampedLock lock = new StampedLock();
133 <        assertFalse(lock.isWriteLocked());
134 <        assertFalse(lock.isReadLocked());
135 <        assertEquals(lock.getReadLockCount(), 0);
136 <        long s = lock.writeLock();
137 <        assertTrue(lock.isWriteLocked());
65 >    void assertUnlocked(StampedLock lock) {
66          assertFalse(lock.isReadLocked());
139        assertEquals(lock.getReadLockCount(), 0);
140        assertFalse(lock.tryUnlockRead());
141        assertTrue(lock.tryUnlockWrite());
142        assertFalse(lock.tryUnlockWrite());
143        assertFalse(lock.tryUnlockRead());
67          assertFalse(lock.isWriteLocked());
68 <        assertFalse(lock.isReadLocked());
69 <        assertEquals(lock.getReadLockCount(), 0);
147 <        long rs = lock.readLock();
148 <        assertFalse(lock.isWriteLocked());
149 <        assertTrue(lock.isReadLocked());
150 <        assertEquals(lock.getReadLockCount(), 1);
151 <        assertFalse(lock.tryUnlockWrite());
152 <        assertTrue(lock.tryUnlockRead());
153 <        assertFalse(lock.tryUnlockRead());
154 <        assertFalse(lock.tryUnlockWrite());
155 <        assertFalse(lock.isWriteLocked());
156 <        assertFalse(lock.isReadLocked());
157 <        assertEquals(lock.getReadLockCount(), 0);
68 >        assertEquals(0, lock.getReadLockCount());
69 >        assertValid(lock, lock.tryOptimisticRead());
70      }
71  
72 <    /**
73 <     * write-unlocking an unlocked lock throws IllegalMonitorStateException
74 <     */
75 <    public void testWriteUnlock_IMSE() {
76 <        StampedLock lock = new StampedLock();
77 <        try {
78 <            lock.unlockWrite(0L);
79 <            shouldThrow();
80 <        } catch (IllegalMonitorStateException success) {}
72 >    List<Action> lockLockers(Lock lock) {
73 >        List<Action> lockers = new ArrayList<>();
74 >        lockers.add(() -> lock.lock());
75 >        lockers.add(() -> lock.lockInterruptibly());
76 >        lockers.add(() -> lock.tryLock());
77 >        lockers.add(() -> lock.tryLock(Long.MIN_VALUE, DAYS));
78 >        lockers.add(() -> lock.tryLock(0L, DAYS));
79 >        lockers.add(() -> lock.tryLock(Long.MAX_VALUE, DAYS));
80 >        return lockers;
81 >    }
82 >
83 >    List<Function<StampedLock, Long>> readLockers() {
84 >        List<Function<StampedLock, Long>> readLockers = new ArrayList<>();
85 >        readLockers.add(sl -> sl.readLock());
86 >        readLockers.add(sl -> sl.tryReadLock());
87 >        readLockers.add(sl -> readLockInterruptiblyUninterrupted(sl));
88 >        readLockers.add(sl -> tryReadLockUninterrupted(sl, Long.MIN_VALUE, DAYS));
89 >        readLockers.add(sl -> tryReadLockUninterrupted(sl, 0L, DAYS));
90 >        readLockers.add(sl -> sl.tryConvertToReadLock(sl.tryOptimisticRead()));
91 >        return readLockers;
92 >    }
93 >
94 >    List<BiConsumer<StampedLock, Long>> readUnlockers() {
95 >        List<BiConsumer<StampedLock, Long>> readUnlockers = new ArrayList<>();
96 >        readUnlockers.add((sl, stamp) -> sl.unlockRead(stamp));
97 >        readUnlockers.add((sl, stamp) -> assertTrue(sl.tryUnlockRead()));
98 >        readUnlockers.add((sl, stamp) -> sl.asReadLock().unlock());
99 >        readUnlockers.add((sl, stamp) -> sl.unlock(stamp));
100 >        readUnlockers.add((sl, stamp) -> assertValid(sl, sl.tryConvertToOptimisticRead(stamp)));
101 >        return readUnlockers;
102 >    }
103 >
104 >    List<Function<StampedLock, Long>> writeLockers() {
105 >        List<Function<StampedLock, Long>> writeLockers = new ArrayList<>();
106 >        writeLockers.add(sl -> sl.writeLock());
107 >        writeLockers.add(sl -> sl.tryWriteLock());
108 >        writeLockers.add(sl -> writeLockInterruptiblyUninterrupted(sl));
109 >        writeLockers.add(sl -> tryWriteLockUninterrupted(sl, Long.MIN_VALUE, DAYS));
110 >        writeLockers.add(sl -> tryWriteLockUninterrupted(sl, 0L, DAYS));
111 >        writeLockers.add(sl -> sl.tryConvertToWriteLock(sl.tryOptimisticRead()));
112 >        return writeLockers;
113 >    }
114 >
115 >    List<BiConsumer<StampedLock, Long>> writeUnlockers() {
116 >        List<BiConsumer<StampedLock, Long>> writeUnlockers = new ArrayList<>();
117 >        writeUnlockers.add((sl, stamp) -> sl.unlockWrite(stamp));
118 >        writeUnlockers.add((sl, stamp) -> assertTrue(sl.tryUnlockWrite()));
119 >        writeUnlockers.add((sl, stamp) -> sl.asWriteLock().unlock());
120 >        writeUnlockers.add((sl, stamp) -> sl.unlock(stamp));
121 >        writeUnlockers.add((sl, stamp) -> assertValid(sl, sl.tryConvertToOptimisticRead(stamp)));
122 >        return writeUnlockers;
123      }
124  
125      /**
126 <     * write-unlocking an unlocked lock throws IllegalMonitorStateException
126 >     * Constructed StampedLock is in unlocked state
127       */
128 <    public void testWriteUnlock_IMSE2() {
129 <        StampedLock lock = new StampedLock();
176 <        long s = lock.writeLock();
177 <        lock.unlockWrite(s);
178 <        try {
179 <            lock.unlockWrite(s);
180 <            shouldThrow();
181 <        } catch (IllegalMonitorStateException success) {}
128 >    public void testConstructor() {
129 >        assertUnlocked(new StampedLock());
130      }
131  
132      /**
133 <     * write-unlocking after readlock throws IllegalMonitorStateException
133 >     * write-locking, then unlocking, an unlocked lock succeed
134       */
135 <    public void testWriteUnlock_IMSE3() {
135 >    public void testWriteLock_lockUnlock() {
136          StampedLock lock = new StampedLock();
137 <        long s = lock.readLock();
138 <        try {
139 <            lock.unlockWrite(s);
140 <            shouldThrow();
141 <        } catch (IllegalMonitorStateException success) {}
137 >
138 >        for (Function<StampedLock, Long> writeLocker : writeLockers())
139 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
140 >            assertFalse(lock.isWriteLocked());
141 >            assertFalse(lock.isReadLocked());
142 >            assertEquals(0, lock.getReadLockCount());
143 >
144 >            long s = writeLocker.apply(lock);
145 >            assertValid(lock, s);
146 >            assertTrue(lock.isWriteLocked());
147 >            assertFalse(lock.isReadLocked());
148 >            assertEquals(0, lock.getReadLockCount());
149 >            writeUnlocker.accept(lock, s);
150 >            assertUnlocked(lock);
151 >        }
152      }
153  
154      /**
155 <     * read-unlocking an unlocked lock throws IllegalMonitorStateException
155 >     * read-locking, then unlocking, an unlocked lock succeed
156       */
157 <    public void testReadUnlock_IMSE() {
157 >    public void testReadLock_lockUnlock() {
158          StampedLock lock = new StampedLock();
159 <        long s = lock.readLock();
160 <        lock.unlockRead(s);
161 <        try {
162 <            lock.unlockRead(s);
163 <            shouldThrow();
164 <        } catch (IllegalMonitorStateException success) {}
159 >
160 >        for (Function<StampedLock, Long> readLocker : readLockers())
161 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
162 >            long s = 42;
163 >            for (int i = 0; i < 2; i++) {
164 >                s = assertValid(lock, readLocker.apply(lock));
165 >                assertFalse(lock.isWriteLocked());
166 >                assertTrue(lock.isReadLocked());
167 >                assertEquals(i + 1, lock.getReadLockCount());
168 >            }
169 >            for (int i = 0; i < 2; i++) {
170 >                assertFalse(lock.isWriteLocked());
171 >                assertTrue(lock.isReadLocked());
172 >                assertEquals(2 - i, lock.getReadLockCount());
173 >                readUnlocker.accept(lock, s);
174 >            }
175 >            assertUnlocked(lock);
176 >        }
177      }
178  
179      /**
180 <     * read-unlocking an unlocked lock throws IllegalMonitorStateException
180 >     * tryUnlockWrite fails if not write locked
181       */
182 <    public void testReadUnlock_IMSE2() {
182 >    public void testTryUnlockWrite_failure() {
183          StampedLock lock = new StampedLock();
184 <        try {
185 <            lock.unlockRead(0L);
186 <            shouldThrow();
187 <        } catch (IllegalMonitorStateException success) {}
184 >        assertFalse(lock.tryUnlockWrite());
185 >
186 >        for (Function<StampedLock, Long> readLocker : readLockers())
187 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
188 >            long s = assertValid(lock, readLocker.apply(lock));
189 >            assertFalse(lock.tryUnlockWrite());
190 >            assertTrue(lock.isReadLocked());
191 >            readUnlocker.accept(lock, s);
192 >            assertUnlocked(lock);
193 >        }
194      }
195  
196      /**
197 <     * read-unlocking after writeLock throws IllegalMonitorStateException
197 >     * tryUnlockRead fails if not read locked
198       */
199 <    public void testReadUnlock_IMSE3() {
199 >    public void testTryUnlockRead_failure() {
200          StampedLock lock = new StampedLock();
201 <        long s = lock.writeLock();
202 <        try {
203 <            lock.unlockRead(s);
204 <            shouldThrow();
205 <        } catch (IllegalMonitorStateException success) {}
201 >        assertFalse(lock.tryUnlockRead());
202 >
203 >        for (Function<StampedLock, Long> writeLocker : writeLockers())
204 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
205 >            long s = writeLocker.apply(lock);
206 >            assertFalse(lock.tryUnlockRead());
207 >            assertTrue(lock.isWriteLocked());
208 >            writeUnlocker.accept(lock, s);
209 >            assertUnlocked(lock);
210 >        }
211      }
212  
213      /**
214 <     * validate(0) fails
214 >     * validate(0L) fails
215       */
216      public void testValidate0() {
217          StampedLock lock = new StampedLock();
# Line 238 | Line 219 | public class StampedLockTest extends JSR
219      }
220  
221      /**
222 <     * A stamp obtained from a successful lock operation validates
222 >     * A stamp obtained from a successful lock operation validates while the lock is held
223       */
224      public void testValidate() throws InterruptedException {
225          StampedLock lock = new StampedLock();
226 <        long s = lock.writeLock();
227 <        assertTrue(lock.validate(s));
228 <        lock.unlockWrite(s);
229 <        s = lock.readLock();
230 <        assertTrue(lock.validate(s));
231 <        lock.unlockRead(s);
232 <        assertTrue((s = lock.tryWriteLock()) != 0L);
233 <        assertTrue(lock.validate(s));
234 <        lock.unlockWrite(s);
235 <        assertTrue((s = lock.tryReadLock()) != 0L);
236 <        assertTrue(lock.validate(s));
237 <        lock.unlockRead(s);
238 <        assertTrue((s = lock.tryWriteLock(100L, MILLISECONDS)) != 0L);
239 <        assertTrue(lock.validate(s));
259 <        lock.unlockWrite(s);
260 <        assertTrue((s = lock.tryReadLock(100L, MILLISECONDS)) != 0L);
261 <        assertTrue(lock.validate(s));
262 <        lock.unlockRead(s);
263 <        assertTrue((s = lock.tryOptimisticRead()) != 0L);
226 >
227 >        for (Function<StampedLock, Long> readLocker : readLockers())
228 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
229 >            long s = assertNonZero(readLocker.apply(lock));
230 >            assertTrue(lock.validate(s));
231 >            readUnlocker.accept(lock, s);
232 >        }
233 >
234 >        for (Function<StampedLock, Long> writeLocker : writeLockers())
235 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
236 >            long s = assertNonZero(writeLocker.apply(lock));
237 >            assertTrue(lock.validate(s));
238 >            writeUnlocker.accept(lock, s);
239 >        }
240      }
241  
242      /**
# Line 268 | Line 244 | public class StampedLockTest extends JSR
244       */
245      public void testValidate2() throws InterruptedException {
246          StampedLock lock = new StampedLock();
247 <        long s;
272 <        assertTrue((s = lock.writeLock()) != 0L);
247 >        long s = assertNonZero(lock.writeLock());
248          assertTrue(lock.validate(s));
249          assertFalse(lock.validate(lock.tryWriteLock()));
250 <        assertFalse(lock.validate(lock.tryWriteLock(10L, MILLISECONDS)));
250 >        assertFalse(lock.validate(lock.tryWriteLock(randomExpiredTimeout(),
251 >                                                    randomTimeUnit())));
252          assertFalse(lock.validate(lock.tryReadLock()));
253 <        assertFalse(lock.validate(lock.tryReadLock(10L, MILLISECONDS)));
253 >        assertFalse(lock.validate(lock.tryWriteLock(randomExpiredTimeout(),
254 >                                                    randomTimeUnit())));
255          assertFalse(lock.validate(lock.tryOptimisticRead()));
256          lock.unlockWrite(s);
257      }
258  
259 +    void assertThrowInterruptedExceptionWhenPreInterrupted(Action[] actions) {
260 +        for (Action action : actions) {
261 +            Thread.currentThread().interrupt();
262 +            try {
263 +                action.run();
264 +                shouldThrow();
265 +            }
266 +            catch (InterruptedException success) {}
267 +            catch (Throwable fail) { threadUnexpectedException(fail); }
268 +            assertFalse(Thread.interrupted());
269 +        }
270 +    }
271 +
272      /**
273 <     * writeLockInterruptibly is interruptible
273 >     * interruptible operations throw InterruptedException when pre-interrupted
274       */
275 <    public void testWriteLockInterruptibly_Interruptible()
286 <            throws InterruptedException {
287 <        final CountDownLatch running = new CountDownLatch(1);
275 >    public void testInterruptibleOperationsThrowInterruptedExceptionWhenPreInterrupted() {
276          final StampedLock lock = new StampedLock();
289        long s = lock.writeLock();
290        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
291            public void realRun() throws InterruptedException {
292                running.countDown();
293                lock.writeLockInterruptibly();
294            }});
277  
278 <        running.await();
279 <        waitForThreadToEnterWaitState(t, 100);
280 <        t.interrupt();
281 <        awaitTermination(t);
282 <        releaseWriteLock(lock, s);
278 >        Action[] interruptibleLockActions = {
279 >            () -> lock.writeLockInterruptibly(),
280 >            () -> lock.tryWriteLock(Long.MIN_VALUE, DAYS),
281 >            () -> lock.tryWriteLock(Long.MAX_VALUE, DAYS),
282 >            () -> lock.readLockInterruptibly(),
283 >            () -> lock.tryReadLock(Long.MIN_VALUE, DAYS),
284 >            () -> lock.tryReadLock(Long.MAX_VALUE, DAYS),
285 >            () -> lock.asWriteLock().lockInterruptibly(),
286 >            () -> lock.asWriteLock().tryLock(0L, DAYS),
287 >            () -> lock.asWriteLock().tryLock(Long.MAX_VALUE, DAYS),
288 >            () -> lock.asReadLock().lockInterruptibly(),
289 >            () -> lock.asReadLock().tryLock(0L, DAYS),
290 >            () -> lock.asReadLock().tryLock(Long.MAX_VALUE, DAYS),
291 >        };
292 >        shuffle(interruptibleLockActions);
293 >
294 >        assertThrowInterruptedExceptionWhenPreInterrupted(interruptibleLockActions);
295 >        {
296 >            long s = lock.writeLock();
297 >            assertThrowInterruptedExceptionWhenPreInterrupted(interruptibleLockActions);
298 >            lock.unlockWrite(s);
299 >        }
300 >        {
301 >            long s = lock.readLock();
302 >            assertThrowInterruptedExceptionWhenPreInterrupted(interruptibleLockActions);
303 >            lock.unlockRead(s);
304 >        }
305 >    }
306 >
307 >    void assertThrowInterruptedExceptionWhenInterrupted(Action[] actions) {
308 >        int n = actions.length;
309 >        Future<?>[] futures = new Future<?>[n];
310 >        CountDownLatch threadsStarted = new CountDownLatch(n);
311 >        CountDownLatch done = new CountDownLatch(n);
312 >
313 >        for (int i = 0; i < n; i++) {
314 >            Action action = actions[i];
315 >            futures[i] = cachedThreadPool.submit(new CheckedRunnable() {
316 >                public void realRun() throws Throwable {
317 >                    threadsStarted.countDown();
318 >                    try {
319 >                        action.run();
320 >                        shouldThrow();
321 >                    }
322 >                    catch (InterruptedException success) {}
323 >                    catch (Throwable fail) { threadUnexpectedException(fail); }
324 >                    assertFalse(Thread.interrupted());
325 >                    done.countDown();
326 >                }});
327 >        }
328 >
329 >        await(threadsStarted);
330 >        assertEquals(n, done.getCount());
331 >        for (Future<?> future : futures) // Interrupt all the tasks
332 >            future.cancel(true);
333 >        await(done);
334      }
335  
336      /**
337 <     * timed tryWriteLock is interruptible
337 >     * interruptible operations throw InterruptedException when write locked and interrupted
338       */
339 <    public void testWriteTryLock_Interruptible() throws InterruptedException {
307 <        final CountDownLatch running = new CountDownLatch(1);
339 >    public void testInterruptibleOperationsThrowInterruptedExceptionWriteLockedInterrupted() {
340          final StampedLock lock = new StampedLock();
341          long s = lock.writeLock();
310        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
311            public void realRun() throws InterruptedException {
312                running.countDown();
313                lock.tryWriteLock(2 * LONG_DELAY_MS, MILLISECONDS);
314            }});
342  
343 <        running.await();
344 <        waitForThreadToEnterWaitState(t, 100);
345 <        t.interrupt();
346 <        awaitTermination(t);
347 <        releaseWriteLock(lock, s);
343 >        Action[] interruptibleLockBlockingActions = {
344 >            () -> lock.writeLockInterruptibly(),
345 >            () -> lock.tryWriteLock(Long.MAX_VALUE, DAYS),
346 >            () -> lock.readLockInterruptibly(),
347 >            () -> lock.tryReadLock(Long.MAX_VALUE, DAYS),
348 >            () -> lock.asWriteLock().lockInterruptibly(),
349 >            () -> lock.asWriteLock().tryLock(Long.MAX_VALUE, DAYS),
350 >            () -> lock.asReadLock().lockInterruptibly(),
351 >            () -> lock.asReadLock().tryLock(Long.MAX_VALUE, DAYS),
352 >        };
353 >        shuffle(interruptibleLockBlockingActions);
354 >
355 >        assertThrowInterruptedExceptionWhenInterrupted(interruptibleLockBlockingActions);
356      }
357  
358      /**
359 <     * readLockInterruptibly is interruptible
359 >     * interruptible operations throw InterruptedException when read locked and interrupted
360       */
361 <    public void testReadLockInterruptibly_Interruptible()
327 <            throws InterruptedException {
328 <        final CountDownLatch running = new CountDownLatch(1);
361 >    public void testInterruptibleOperationsThrowInterruptedExceptionReadLockedInterrupted() {
362          final StampedLock lock = new StampedLock();
363 <        long s = lock.writeLock();
331 <        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
332 <            public void realRun() throws InterruptedException {
333 <                running.countDown();
334 <                lock.readLockInterruptibly();
335 <            }});
363 >        long s = lock.readLock();
364  
365 <        running.await();
366 <        waitForThreadToEnterWaitState(t, 100);
367 <        t.interrupt();
368 <        awaitTermination(t);
369 <        releaseWriteLock(lock, s);
365 >        Action[] interruptibleLockBlockingActions = {
366 >            () -> lock.writeLockInterruptibly(),
367 >            () -> lock.tryWriteLock(Long.MAX_VALUE, DAYS),
368 >            () -> lock.asWriteLock().lockInterruptibly(),
369 >            () -> lock.asWriteLock().tryLock(Long.MAX_VALUE, DAYS),
370 >        };
371 >        shuffle(interruptibleLockBlockingActions);
372 >
373 >        assertThrowInterruptedExceptionWhenInterrupted(interruptibleLockBlockingActions);
374      }
375  
376      /**
377 <     * timed tryReadLock is interruptible
377 >     * Non-interruptible operations ignore and preserve interrupt status
378       */
379 <    public void testReadTryLock_Interruptible() throws InterruptedException {
348 <        final CountDownLatch running = new CountDownLatch(1);
379 >    public void testNonInterruptibleOperationsIgnoreInterrupts() {
380          final StampedLock lock = new StampedLock();
381 <        long s = lock.writeLock();
351 <        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
352 <            public void realRun() throws InterruptedException {
353 <                running.countDown();
354 <                lock.tryReadLock(2 * LONG_DELAY_MS, MILLISECONDS);
355 <            }});
381 >        Thread.currentThread().interrupt();
382  
383 <        running.await();
384 <        waitForThreadToEnterWaitState(t, 100);
385 <        t.interrupt();
386 <        awaitTermination(t);
387 <        releaseWriteLock(lock, s);
383 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
384 >            long s = assertValid(lock, lock.readLock());
385 >            readUnlocker.accept(lock, s);
386 >            s = assertValid(lock, lock.tryReadLock());
387 >            readUnlocker.accept(lock, s);
388 >        }
389 >
390 >        lock.asReadLock().lock();
391 >        lock.asReadLock().unlock();
392 >
393 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
394 >            long s = assertValid(lock, lock.writeLock());
395 >            writeUnlocker.accept(lock, s);
396 >            s = assertValid(lock, lock.tryWriteLock());
397 >            writeUnlocker.accept(lock, s);
398 >        }
399 >
400 >        lock.asWriteLock().lock();
401 >        lock.asWriteLock().unlock();
402 >
403 >        assertTrue(Thread.interrupted());
404      }
405  
406      /**
407       * tryWriteLock on an unlocked lock succeeds
408       */
409 <    public void testWriteTryLock() {
409 >    public void testTryWriteLock() {
410          final StampedLock lock = new StampedLock();
411          long s = lock.tryWriteLock();
412          assertTrue(s != 0L);
413          assertTrue(lock.isWriteLocked());
414 <        long s2 = lock.tryWriteLock();
373 <        assertEquals(s2, 0L);
414 >        assertEquals(0L, lock.tryWriteLock());
415          releaseWriteLock(lock, s);
416      }
417  
418      /**
419       * tryWriteLock fails if locked
420       */
421 <    public void testWriteTryLockWhenLocked() {
421 >    public void testTryWriteLockWhenLocked() {
422          final StampedLock lock = new StampedLock();
423          long s = lock.writeLock();
424          Thread t = newStartedThread(new CheckedRunnable() {
425              public void realRun() {
426 <                long ws = lock.tryWriteLock();
386 <                assertTrue(ws == 0L);
426 >                assertEquals(0L, lock.tryWriteLock());
427              }});
428  
429 +        assertEquals(0L, lock.tryWriteLock());
430          awaitTermination(t);
431          releaseWriteLock(lock, s);
432      }
# Line 393 | Line 434 | public class StampedLockTest extends JSR
434      /**
435       * tryReadLock fails if write-locked
436       */
437 <    public void testReadTryLockWhenLocked() {
437 >    public void testTryReadLockWhenLocked() {
438          final StampedLock lock = new StampedLock();
439          long s = lock.writeLock();
440          Thread t = newStartedThread(new CheckedRunnable() {
441              public void realRun() {
442 <                long rs = lock.tryReadLock();
402 <                assertEquals(rs, 0L);
442 >                assertEquals(0L, lock.tryReadLock());
443              }});
444  
445 +        assertEquals(0L, lock.tryReadLock());
446          awaitTermination(t);
447          releaseWriteLock(lock, s);
448      }
# Line 415 | Line 456 | public class StampedLockTest extends JSR
456          Thread t = newStartedThread(new CheckedRunnable() {
457              public void realRun() throws InterruptedException {
458                  long s2 = lock.tryReadLock();
459 <                assertTrue(s2 != 0L);
459 >                assertValid(lock, s2);
460                  lock.unlockRead(s2);
461                  long s3 = lock.tryReadLock(LONG_DELAY_MS, MILLISECONDS);
462 <                assertTrue(s3 != 0L);
462 >                assertValid(lock, s3);
463                  lock.unlockRead(s3);
464                  long s4 = lock.readLock();
465 +                assertValid(lock, s4);
466                  lock.unlockRead(s4);
467 +                lock.asReadLock().lock();
468 +                lock.asReadLock().unlock();
469 +                lock.asReadLock().lockInterruptibly();
470 +                lock.asReadLock().unlock();
471 +                lock.asReadLock().tryLock(Long.MIN_VALUE, DAYS);
472 +                lock.asReadLock().unlock();
473              }});
474  
475          awaitTermination(t);
# Line 429 | Line 477 | public class StampedLockTest extends JSR
477      }
478  
479      /**
480 <     * A writelock succeeds only after a reading thread unlocks
480 >     * writeLock() succeeds only after a reading thread unlocks
481       */
482      public void testWriteAfterReadLock() throws InterruptedException {
483 <        final CountDownLatch running = new CountDownLatch(1);
483 >        final CountDownLatch aboutToLock = new CountDownLatch(1);
484          final StampedLock lock = new StampedLock();
485          long rs = lock.readLock();
486          Thread t = newStartedThread(new CheckedRunnable() {
487              public void realRun() {
488 <                running.countDown();
488 >                aboutToLock.countDown();
489                  long s = lock.writeLock();
490 +                assertTrue(lock.isWriteLocked());
491 +                assertFalse(lock.isReadLocked());
492                  lock.unlockWrite(s);
493              }});
494  
495 <        running.await();
496 <        waitForThreadToEnterWaitState(t, 100);
495 >        await(aboutToLock);
496 >        assertThreadBlocks(t, Thread.State.WAITING);
497          assertFalse(lock.isWriteLocked());
498 +        assertTrue(lock.isReadLocked());
499          lock.unlockRead(rs);
500          awaitTermination(t);
501 <        assertFalse(lock.isWriteLocked());
501 >        assertUnlocked(lock);
502      }
503  
504      /**
505 <     * A writelock succeeds only after reading threads unlock
505 >     * writeLock() succeeds only after reading threads unlock
506       */
507      public void testWriteAfterMultipleReadLocks() {
508          final StampedLock lock = new StampedLock();
# Line 470 | Line 521 | public class StampedLockTest extends JSR
521                  lock.unlockWrite(ws);
522              }});
523  
524 +        assertTrue(lock.isReadLocked());
525          assertFalse(lock.isWriteLocked());
526          lock.unlockRead(s);
527          awaitTermination(t2);
528 <        assertFalse(lock.isWriteLocked());
528 >        assertUnlocked(lock);
529      }
530  
531      /**
532 <     * Readlocks succeed only after a writing thread unlocks
532 >     * readLock() succeed only after a writing thread unlocks
533       */
534      public void testReadAfterWriteLock() {
535          final StampedLock lock = new StampedLock();
536 +        final CountDownLatch threadsStarted = new CountDownLatch(2);
537          final long s = lock.writeLock();
538 <        Thread t1 = newStartedThread(new CheckedRunnable() {
486 <            public void realRun() {
487 <                long rs = lock.readLock();
488 <                lock.unlockRead(rs);
489 <            }});
490 <        Thread t2 = newStartedThread(new CheckedRunnable() {
538 >        final Runnable acquireReleaseReadLock = new CheckedRunnable() {
539              public void realRun() {
540 +                threadsStarted.countDown();
541                  long rs = lock.readLock();
542 +                assertTrue(lock.isReadLocked());
543 +                assertFalse(lock.isWriteLocked());
544                  lock.unlockRead(rs);
545 <            }});
546 <
545 >            }};
546 >        Thread t1 = newStartedThread(acquireReleaseReadLock);
547 >        Thread t2 = newStartedThread(acquireReleaseReadLock);
548 >
549 >        await(threadsStarted);
550 >        assertThreadBlocks(t1, Thread.State.WAITING);
551 >        assertThreadBlocks(t2, Thread.State.WAITING);
552 >        assertTrue(lock.isWriteLocked());
553 >        assertFalse(lock.isReadLocked());
554          releaseWriteLock(lock, s);
555          awaitTermination(t1);
556          awaitTermination(t2);
557 +        assertUnlocked(lock);
558      }
559  
560      /**
561 <     * tryReadLock succeeds if readlocked but not writelocked
561 >     * tryReadLock succeeds if read locked but not write locked
562       */
563      public void testTryLockWhenReadLocked() {
564          final StampedLock lock = new StampedLock();
# Line 507 | Line 566 | public class StampedLockTest extends JSR
566          Thread t = newStartedThread(new CheckedRunnable() {
567              public void realRun() {
568                  long rs = lock.tryReadLock();
569 <                threadAssertTrue(rs != 0L);
569 >                assertValid(lock, rs);
570                  lock.unlockRead(rs);
571              }});
572  
# Line 516 | Line 575 | public class StampedLockTest extends JSR
575      }
576  
577      /**
578 <     * tryWriteLock fails when readlocked
578 >     * tryWriteLock fails when read locked
579       */
580 <    public void testWriteTryLockWhenReadLocked() {
580 >    public void testTryWriteLockWhenReadLocked() {
581          final StampedLock lock = new StampedLock();
582          long s = lock.readLock();
583          Thread t = newStartedThread(new CheckedRunnable() {
584              public void realRun() {
585 <                long ws = lock.tryWriteLock();
527 <                threadAssertEquals(ws, 0L);
585 >                threadAssertEquals(0L, lock.tryWriteLock());
586              }});
587  
588          awaitTermination(t);
# Line 532 | Line 590 | public class StampedLockTest extends JSR
590      }
591  
592      /**
593 <     * timed tryWriteLock times out if locked
593 >     * timed lock operations time out if lock not available
594       */
595 <    public void testWriteTryLock_Timeout() {
595 >    public void testTimedLock_Timeout() throws Exception {
596 >        ArrayList<Future<?>> futures = new ArrayList<>();
597 >
598 >        // Write locked
599          final StampedLock lock = new StampedLock();
600 <        long s = lock.writeLock();
601 <        Thread t = newStartedThread(new CheckedRunnable() {
600 >        long stamp = lock.writeLock();
601 >        assertEquals(0L, lock.tryReadLock(0L, DAYS));
602 >        assertEquals(0L, lock.tryReadLock(Long.MIN_VALUE, DAYS));
603 >        assertFalse(lock.asReadLock().tryLock(0L, DAYS));
604 >        assertFalse(lock.asReadLock().tryLock(Long.MIN_VALUE, DAYS));
605 >        assertEquals(0L, lock.tryWriteLock(0L, DAYS));
606 >        assertEquals(0L, lock.tryWriteLock(Long.MIN_VALUE, DAYS));
607 >        assertFalse(lock.asWriteLock().tryLock(0L, DAYS));
608 >        assertFalse(lock.asWriteLock().tryLock(Long.MIN_VALUE, DAYS));
609 >
610 >        futures.add(cachedThreadPool.submit(new CheckedRunnable() {
611              public void realRun() throws InterruptedException {
612                  long startTime = System.nanoTime();
613 <                long timeoutMillis = 10;
614 <                long ws = lock.tryWriteLock(timeoutMillis, MILLISECONDS);
615 <                assertEquals(ws, 0L);
546 <                assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
547 <            }});
613 >                assertEquals(0L, lock.tryWriteLock(timeoutMillis(), MILLISECONDS));
614 >                assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
615 >            }}));
616  
617 <        awaitTermination(t);
618 <        releaseWriteLock(lock, s);
619 <    }
617 >        futures.add(cachedThreadPool.submit(new CheckedRunnable() {
618 >            public void realRun() throws InterruptedException {
619 >                long startTime = System.nanoTime();
620 >                assertEquals(0L, lock.tryReadLock(timeoutMillis(), MILLISECONDS));
621 >                assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
622 >            }}));
623 >
624 >        // Read locked
625 >        final StampedLock lock2 = new StampedLock();
626 >        long stamp2 = lock2.readLock();
627 >        assertEquals(0L, lock2.tryWriteLock(0L, DAYS));
628 >        assertEquals(0L, lock2.tryWriteLock(Long.MIN_VALUE, DAYS));
629 >        assertFalse(lock2.asWriteLock().tryLock(0L, DAYS));
630 >        assertFalse(lock2.asWriteLock().tryLock(Long.MIN_VALUE, DAYS));
631  
632 <    /**
554 <     * timed tryReadLock times out if write-locked
555 <     */
556 <    public void testReadTryLock_Timeout() {
557 <        final StampedLock lock = new StampedLock();
558 <        long s = lock.writeLock();
559 <        Thread t = newStartedThread(new CheckedRunnable() {
632 >        futures.add(cachedThreadPool.submit(new CheckedRunnable() {
633              public void realRun() throws InterruptedException {
634                  long startTime = System.nanoTime();
635 <                long timeoutMillis = 10;
636 <                long rs = lock.tryReadLock(timeoutMillis, MILLISECONDS);
637 <                assertEquals(rs, 0L);
565 <                assertTrue(millisElapsedSince(startTime) >= timeoutMillis);
566 <            }});
635 >                assertEquals(0L, lock2.tryWriteLock(timeoutMillis(), MILLISECONDS));
636 >                assertTrue(millisElapsedSince(startTime) >= timeoutMillis());
637 >            }}));
638  
639 <        awaitTermination(t);
640 <        assertTrue(lock.isWriteLocked());
641 <        lock.unlockWrite(s);
639 >        for (Future<?> future : futures)
640 >            assertNull(future.get());
641 >
642 >        releaseWriteLock(lock, stamp);
643 >        releaseReadLock(lock2, stamp2);
644      }
645  
646      /**
647 <     * writeLockInterruptibly succeeds if unlocked, else is interruptible
647 >     * writeLockInterruptibly succeeds if unlocked
648       */
649      public void testWriteLockInterruptibly() throws InterruptedException {
577        final CountDownLatch running = new CountDownLatch(1);
650          final StampedLock lock = new StampedLock();
651          long s = lock.writeLockInterruptibly();
580        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
581            public void realRun() throws InterruptedException {
582                running.countDown();
583                lock.writeLockInterruptibly();
584            }});
585
586        running.await();
587        waitForThreadToEnterWaitState(t, 100);
588        t.interrupt();
652          assertTrue(lock.isWriteLocked());
590        awaitTermination(t);
653          releaseWriteLock(lock, s);
654      }
655  
656      /**
657 <     * readLockInterruptibly succeeds if lock free else is interruptible
657 >     * readLockInterruptibly succeeds if lock free
658       */
659      public void testReadLockInterruptibly() throws InterruptedException {
598        final CountDownLatch running = new CountDownLatch(1);
660          final StampedLock lock = new StampedLock();
661 <        long s;
662 <        s = lock.readLockInterruptibly();
661 >
662 >        long s = assertValid(lock, lock.readLockInterruptibly());
663 >        assertTrue(lock.isReadLocked());
664          lock.unlockRead(s);
603        s = lock.writeLockInterruptibly();
604        Thread t = newStartedThread(new CheckedInterruptedRunnable() {
605            public void realRun() throws InterruptedException {
606                running.countDown();
607                lock.readLockInterruptibly();
608            }});
665  
666 <        running.await();
667 <        waitForThreadToEnterWaitState(t, 100);
668 <        t.interrupt();
613 <        awaitTermination(t);
614 <        releaseWriteLock(lock, s);
666 >        lock.asReadLock().lockInterruptibly();
667 >        assertTrue(lock.isReadLocked());
668 >        lock.asReadLock().unlock();
669      }
670  
671      /**
# Line 643 | Line 697 | public class StampedLockTest extends JSR
697      }
698  
699      /**
700 <     * tryOptimisticRead succeeds and validates if unlocked, fails if locked
700 >     * tryOptimisticRead succeeds and validates if unlocked, fails if
701 >     * exclusively locked
702       */
703      public void testValidateOptimistic() throws InterruptedException {
704          StampedLock lock = new StampedLock();
705 <        long s, p;
706 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
707 <        assertTrue(lock.validate(p));
708 <        assertTrue((s = lock.writeLock()) != 0L);
709 <        assertFalse((p = lock.tryOptimisticRead()) != 0L);
710 <        assertTrue(lock.validate(s));
711 <        lock.unlockWrite(s);
712 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
713 <        assertTrue(lock.validate(p));
714 <        assertTrue((s = lock.readLock()) != 0L);
715 <        assertTrue(lock.validate(s));
716 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
717 <        assertTrue(lock.validate(p));
718 <        lock.unlockRead(s);
719 <        assertTrue((s = lock.tryWriteLock()) != 0L);
720 <        assertTrue(lock.validate(s));
721 <        assertFalse((p = lock.tryOptimisticRead()) != 0L);
667 <        lock.unlockWrite(s);
668 <        assertTrue((s = lock.tryReadLock()) != 0L);
669 <        assertTrue(lock.validate(s));
670 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
671 <        lock.unlockRead(s);
672 <        assertTrue(lock.validate(p));
673 <        assertTrue((s = lock.tryWriteLock(100L, MILLISECONDS)) != 0L);
674 <        assertFalse((p = lock.tryOptimisticRead()) != 0L);
675 <        assertTrue(lock.validate(s));
676 <        lock.unlockWrite(s);
677 <        assertTrue((s = lock.tryReadLock(100L, MILLISECONDS)) != 0L);
678 <        assertTrue(lock.validate(s));
679 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
680 <        lock.unlockRead(s);
681 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
705 >
706 >        assertValid(lock, lock.tryOptimisticRead());
707 >
708 >        for (Function<StampedLock, Long> writeLocker : writeLockers()) {
709 >            long s = assertValid(lock, writeLocker.apply(lock));
710 >            assertEquals(0L, lock.tryOptimisticRead());
711 >            releaseWriteLock(lock, s);
712 >        }
713 >
714 >        for (Function<StampedLock, Long> readLocker : readLockers()) {
715 >            long s = assertValid(lock, readLocker.apply(lock));
716 >            long p = assertValid(lock, lock.tryOptimisticRead());
717 >            releaseReadLock(lock, s);
718 >            assertTrue(lock.validate(p));
719 >        }
720 >
721 >        assertValid(lock, lock.tryOptimisticRead());
722      }
723  
724      /**
725       * tryOptimisticRead stamp does not validate if a write lock intervenes
726       */
727      public void testValidateOptimisticWriteLocked() {
728 <        StampedLock lock = new StampedLock();
729 <        long s, p;
730 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
691 <        assertTrue((s = lock.writeLock()) != 0L);
728 >        final StampedLock lock = new StampedLock();
729 >        final long p = assertValid(lock, lock.tryOptimisticRead());
730 >        final long s = assertValid(lock, lock.writeLock());
731          assertFalse(lock.validate(p));
732 <        assertFalse((p = lock.tryOptimisticRead()) != 0L);
732 >        assertEquals(0L, lock.tryOptimisticRead());
733          assertTrue(lock.validate(s));
734          lock.unlockWrite(s);
735      }
# Line 701 | Line 740 | public class StampedLockTest extends JSR
740       */
741      public void testValidateOptimisticWriteLocked2()
742              throws InterruptedException {
743 <        final CountDownLatch running = new CountDownLatch(1);
743 >        final CountDownLatch locked = new CountDownLatch(1);
744          final StampedLock lock = new StampedLock();
745 <        long s, p;
746 <        assertTrue((p = lock.tryOptimisticRead()) != 0L);
745 >        final long p = assertValid(lock, lock.tryOptimisticRead());
746 >
747          Thread t = newStartedThread(new CheckedInterruptedRunnable() {
748              public void realRun() throws InterruptedException {
749                  lock.writeLockInterruptibly();
750 <                running.countDown();
750 >                locked.countDown();
751                  lock.writeLockInterruptibly();
752              }});
753  
754 <        running.await();
754 >        await(locked);
755          assertFalse(lock.validate(p));
756 <        assertFalse((p = lock.tryOptimisticRead()) != 0L);
756 >        assertEquals(0L, lock.tryOptimisticRead());
757 >        assertThreadBlocks(t, Thread.State.WAITING);
758          t.interrupt();
759          awaitTermination(t);
760 +        assertTrue(lock.isWriteLocked());
761      }
762  
763      /**
764 <     * tryConvertToOptimisticRead succeeds and validates if successfully locked,
764 >     * tryConvertToOptimisticRead succeeds and validates if successfully locked
765       */
766      public void testTryConvertToOptimisticRead() throws InterruptedException {
767          StampedLock lock = new StampedLock();
768 <        long s, p;
768 >        long s, p, q;
769          assertEquals(0L, lock.tryConvertToOptimisticRead(0L));
770  
771 <        assertTrue((s = lock.tryOptimisticRead()) != 0L);
731 <        assertEquals(s, lock.tryConvertToOptimisticRead(s));
732 <        assertTrue(lock.validate(s));
733 <
734 <        assertTrue((p = lock.readLock()) != 0L);
735 <        assertTrue((s = lock.tryOptimisticRead()) != 0L);
771 >        s = assertValid(lock, lock.tryOptimisticRead());
772          assertEquals(s, lock.tryConvertToOptimisticRead(s));
773          assertTrue(lock.validate(s));
738        lock.unlockRead(p);
739
740        assertTrue((s = lock.writeLock()) != 0L);
741        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
742        assertTrue(lock.validate(p));
743
744        assertTrue((s = lock.readLock()) != 0L);
745        assertTrue(lock.validate(s));
746        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
747        assertTrue(lock.validate(p));
748
749        assertTrue((s = lock.tryWriteLock()) != 0L);
750        assertTrue(lock.validate(s));
751        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
752        assertTrue(lock.validate(p));
774  
775 <        assertTrue((s = lock.tryReadLock()) != 0L);
776 <        assertTrue(lock.validate(s));
777 <        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
778 <        assertTrue(lock.validate(p));
779 <
780 <        assertTrue((s = lock.tryWriteLock(100L, MILLISECONDS)) != 0L);
781 <        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
761 <        assertTrue(lock.validate(p));
775 >        for (Function<StampedLock, Long> writeLocker : writeLockers()) {
776 >            s = assertValid(lock, writeLocker.apply(lock));
777 >            p = assertValid(lock, lock.tryConvertToOptimisticRead(s));
778 >            assertFalse(lock.validate(s));
779 >            assertTrue(lock.validate(p));
780 >            assertUnlocked(lock);
781 >        }
782  
783 <        assertTrue((s = lock.tryReadLock(100L, MILLISECONDS)) != 0L);
784 <        assertTrue(lock.validate(s));
785 <        assertTrue((p = lock.tryConvertToOptimisticRead(s)) != 0L);
786 <        assertTrue(lock.validate(p));
783 >        for (Function<StampedLock, Long> readLocker : readLockers()) {
784 >            s = assertValid(lock, readLocker.apply(lock));
785 >            q = assertValid(lock, lock.tryOptimisticRead());
786 >            assertEquals(q, lock.tryConvertToOptimisticRead(q));
787 >            assertTrue(lock.validate(q));
788 >            assertTrue(lock.isReadLocked());
789 >            p = assertValid(lock, lock.tryConvertToOptimisticRead(s));
790 >            assertTrue(lock.validate(p));
791 >            assertTrue(lock.validate(s));
792 >            assertUnlocked(lock);
793 >            assertEquals(q, lock.tryConvertToOptimisticRead(q));
794 >            assertTrue(lock.validate(q));
795 >        }
796      }
797  
798      /**
799 <     * tryConvertToReadLock succeeds and validates if successfully locked
771 <     * or lock free;
799 >     * tryConvertToReadLock succeeds for valid stamps
800       */
801      public void testTryConvertToReadLock() throws InterruptedException {
802          StampedLock lock = new StampedLock();
803          long s, p;
804  
805 <        assertFalse((p = lock.tryConvertToReadLock(0L)) != 0L);
805 >        assertEquals(0L, lock.tryConvertToReadLock(0L));
806  
807 <        assertTrue((s = lock.tryOptimisticRead()) != 0L);
808 <        assertTrue((p = lock.tryConvertToReadLock(s)) != 0L);
807 >        s = assertValid(lock, lock.tryOptimisticRead());
808 >        p = assertValid(lock, lock.tryConvertToReadLock(s));
809          assertTrue(lock.isReadLocked());
810          assertEquals(1, lock.getReadLockCount());
811 +        assertTrue(lock.validate(s));
812          lock.unlockRead(p);
813  
814 <        assertTrue((s = lock.tryOptimisticRead()) != 0L);
814 >        s = assertValid(lock, lock.tryOptimisticRead());
815          lock.readLock();
816 <        assertTrue((p = lock.tryConvertToReadLock(s)) != 0L);
816 >        p = assertValid(lock, lock.tryConvertToReadLock(s));
817          assertTrue(lock.isReadLocked());
818          assertEquals(2, lock.getReadLockCount());
819          lock.unlockRead(p);
820          lock.unlockRead(p);
821 +        assertUnlocked(lock);
822  
823 <        assertTrue((s = lock.writeLock()) != 0L);
824 <        assertTrue((p = lock.tryConvertToReadLock(s)) != 0L);
825 <        assertTrue(lock.validate(p));
826 <        assertTrue(lock.isReadLocked());
827 <        assertEquals(1, lock.getReadLockCount());
828 <        lock.unlockRead(p);
829 <
830 <        assertTrue((s = lock.readLock()) != 0L);
831 <        assertTrue(lock.validate(s));
832 <        assertEquals(s, lock.tryConvertToReadLock(s));
833 <        assertTrue(lock.validate(s));
834 <        assertTrue(lock.isReadLocked());
835 <        assertEquals(1, lock.getReadLockCount());
836 <        lock.unlockRead(s);
837 <
838 <        assertTrue((s = lock.tryWriteLock()) != 0L);
839 <        assertTrue(lock.validate(s));
840 <        assertTrue((p = lock.tryConvertToReadLock(s)) != 0L);
841 <        assertTrue(lock.validate(p));
812 <        assertEquals(1, lock.getReadLockCount());
813 <        lock.unlockRead(p);
814 <
815 <        assertTrue((s = lock.tryReadLock()) != 0L);
816 <        assertTrue(lock.validate(s));
817 <        assertEquals(s, lock.tryConvertToReadLock(s));
818 <        assertTrue(lock.validate(s));
819 <        assertTrue(lock.isReadLocked());
820 <        assertEquals(1, lock.getReadLockCount());
821 <        lock.unlockRead(s);
822 <
823 <        assertTrue((s = lock.tryWriteLock(100L, MILLISECONDS)) != 0L);
824 <        assertTrue((p = lock.tryConvertToReadLock(s)) != 0L);
825 <        assertTrue(lock.validate(p));
826 <        assertTrue(lock.isReadLocked());
827 <        assertEquals(1, lock.getReadLockCount());
828 <        lock.unlockRead(p);
829 <
830 <        assertTrue((s = lock.tryReadLock(100L, MILLISECONDS)) != 0L);
831 <        assertTrue(lock.validate(s));
832 <        assertEquals(s, lock.tryConvertToReadLock(s));
833 <        assertTrue(lock.validate(s));
834 <        assertTrue(lock.isReadLocked());
835 <        assertEquals(1, lock.getReadLockCount());
836 <        lock.unlockRead(s);
823 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
824 >            for (Function<StampedLock, Long> writeLocker : writeLockers()) {
825 >                s = assertValid(lock, writeLocker.apply(lock));
826 >                p = assertValid(lock, lock.tryConvertToReadLock(s));
827 >                assertFalse(lock.validate(s));
828 >                assertTrue(lock.isReadLocked());
829 >                assertEquals(1, lock.getReadLockCount());
830 >                readUnlocker.accept(lock, p);
831 >            }
832 >
833 >            for (Function<StampedLock, Long> readLocker : readLockers()) {
834 >                s = assertValid(lock, readLocker.apply(lock));
835 >                assertEquals(s, lock.tryConvertToReadLock(s));
836 >                assertTrue(lock.validate(s));
837 >                assertTrue(lock.isReadLocked());
838 >                assertEquals(1, lock.getReadLockCount());
839 >                readUnlocker.accept(lock, s);
840 >            }
841 >        }
842      }
843  
844      /**
845 <     * tryConvertToWriteLock succeeds and validates if successfully locked
841 <     * or lock free;
845 >     * tryConvertToWriteLock succeeds if lock available; fails if multiply read locked
846       */
847      public void testTryConvertToWriteLock() throws InterruptedException {
848          StampedLock lock = new StampedLock();
849          long s, p;
850  
851 <        assertFalse((p = lock.tryConvertToWriteLock(0L)) != 0L);
851 >        assertEquals(0L, lock.tryConvertToWriteLock(0L));
852  
853          assertTrue((s = lock.tryOptimisticRead()) != 0L);
854          assertTrue((p = lock.tryConvertToWriteLock(s)) != 0L);
855          assertTrue(lock.isWriteLocked());
856          lock.unlockWrite(p);
857  
858 <        assertTrue((s = lock.writeLock()) != 0L);
859 <        assertEquals(s, lock.tryConvertToWriteLock(s));
860 <        assertTrue(lock.validate(s));
861 <        assertTrue(lock.isWriteLocked());
862 <        lock.unlockWrite(s);
863 <
864 <        assertTrue((s = lock.readLock()) != 0L);
865 <        assertTrue(lock.validate(s));
866 <        assertTrue((p = lock.tryConvertToWriteLock(s)) != 0L);
867 <        assertTrue(lock.validate(p));
868 <        assertTrue(lock.isWriteLocked());
869 <        lock.unlockWrite(p);
870 <
871 <        assertTrue((s = lock.tryWriteLock()) != 0L);
872 <        assertTrue(lock.validate(s));
873 <        assertEquals(s, lock.tryConvertToWriteLock(s));
874 <        assertTrue(lock.validate(s));
875 <        assertTrue(lock.isWriteLocked());
872 <        lock.unlockWrite(s);
873 <
874 <        assertTrue((s = lock.tryReadLock()) != 0L);
875 <        assertTrue(lock.validate(s));
876 <        assertTrue((p = lock.tryConvertToWriteLock(s)) != 0L);
877 <        assertTrue(lock.validate(p));
878 <        assertTrue(lock.isWriteLocked());
879 <        lock.unlockWrite(p);
880 <
881 <        assertTrue((s = lock.tryWriteLock(100L, MILLISECONDS)) != 0L);
882 <        assertTrue((p = lock.tryConvertToWriteLock(s)) != 0L);
883 <        assertTrue(lock.validate(p));
884 <        assertTrue(lock.isWriteLocked());
885 <        lock.unlockWrite(p);
858 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
859 >            for (Function<StampedLock, Long> writeLocker : writeLockers()) {
860 >                s = assertValid(lock, writeLocker.apply(lock));
861 >                assertEquals(s, lock.tryConvertToWriteLock(s));
862 >                assertTrue(lock.validate(s));
863 >                assertTrue(lock.isWriteLocked());
864 >                writeUnlocker.accept(lock, s);
865 >            }
866 >
867 >            for (Function<StampedLock, Long> readLocker : readLockers()) {
868 >                s = assertValid(lock, readLocker.apply(lock));
869 >                p = assertValid(lock, lock.tryConvertToWriteLock(s));
870 >                assertFalse(lock.validate(s));
871 >                assertTrue(lock.validate(p));
872 >                assertTrue(lock.isWriteLocked());
873 >                writeUnlocker.accept(lock, p);
874 >            }
875 >        }
876  
877 <        assertTrue((s = lock.tryReadLock(100L, MILLISECONDS)) != 0L);
878 <        assertTrue(lock.validate(s));
879 <        assertTrue((p = lock.tryConvertToWriteLock(s)) != 0L);
880 <        assertTrue(lock.validate(p));
881 <        assertTrue(lock.isWriteLocked());
882 <        lock.unlockWrite(p);
877 >        // failure if multiply read locked
878 >        for (Function<StampedLock, Long> readLocker : readLockers()) {
879 >            s = assertValid(lock, readLocker.apply(lock));
880 >            p = assertValid(lock, readLocker.apply(lock));
881 >            assertEquals(0L, lock.tryConvertToWriteLock(s));
882 >            assertTrue(lock.validate(s));
883 >            assertTrue(lock.validate(p));
884 >            assertEquals(2, lock.getReadLockCount());
885 >            lock.unlock(p);
886 >            lock.unlock(s);
887 >            assertUnlocked(lock);
888 >        }
889      }
890  
891      /**
892       * asWriteLock can be locked and unlocked
893       */
894 <    public void testAsWriteLock() {
894 >    public void testAsWriteLock() throws Throwable {
895          StampedLock sl = new StampedLock();
896          Lock lock = sl.asWriteLock();
897 <        lock.lock();
898 <        assertFalse(lock.tryLock());
899 <        lock.unlock();
900 <        assertTrue(lock.tryLock());
897 >        for (Action locker : lockLockers(lock)) {
898 >            locker.run();
899 >            assertTrue(sl.isWriteLocked());
900 >            assertFalse(sl.isReadLocked());
901 >            assertFalse(lock.tryLock());
902 >            lock.unlock();
903 >            assertUnlocked(sl);
904 >        }
905      }
906  
907      /**
908       * asReadLock can be locked and unlocked
909       */
910 <    public void testAsReadLock() {
910 >    public void testAsReadLock() throws Throwable {
911          StampedLock sl = new StampedLock();
912          Lock lock = sl.asReadLock();
913 <        lock.lock();
914 <        lock.unlock();
915 <        assertTrue(lock.tryLock());
913 >        for (Action locker : lockLockers(lock)) {
914 >            locker.run();
915 >            assertTrue(sl.isReadLocked());
916 >            assertFalse(sl.isWriteLocked());
917 >            assertEquals(1, sl.getReadLockCount());
918 >            locker.run();
919 >            assertTrue(sl.isReadLocked());
920 >            assertEquals(2, sl.getReadLockCount());
921 >            lock.unlock();
922 >            lock.unlock();
923 >            assertUnlocked(sl);
924 >        }
925      }
926  
927      /**
928       * asReadWriteLock.writeLock can be locked and unlocked
929       */
930 <    public void testAsReadWriteLockWriteLock() {
930 >    public void testAsReadWriteLockWriteLock() throws Throwable {
931          StampedLock sl = new StampedLock();
932          Lock lock = sl.asReadWriteLock().writeLock();
933 <        lock.lock();
934 <        assertFalse(lock.tryLock());
935 <        lock.unlock();
936 <        assertTrue(lock.tryLock());
933 >        for (Action locker : lockLockers(lock)) {
934 >            locker.run();
935 >            assertTrue(sl.isWriteLocked());
936 >            assertFalse(sl.isReadLocked());
937 >            assertFalse(lock.tryLock());
938 >            lock.unlock();
939 >            assertUnlocked(sl);
940 >        }
941      }
942  
943      /**
944       * asReadWriteLock.readLock can be locked and unlocked
945       */
946 <    public void testAsReadWriteLockReadLock() {
946 >    public void testAsReadWriteLockReadLock() throws Throwable {
947          StampedLock sl = new StampedLock();
948          Lock lock = sl.asReadWriteLock().readLock();
949 <        lock.lock();
950 <        lock.unlock();
951 <        assertTrue(lock.tryLock());
949 >        for (Action locker : lockLockers(lock)) {
950 >            locker.run();
951 >            assertTrue(sl.isReadLocked());
952 >            assertFalse(sl.isWriteLocked());
953 >            assertEquals(1, sl.getReadLockCount());
954 >            locker.run();
955 >            assertTrue(sl.isReadLocked());
956 >            assertEquals(2, sl.getReadLockCount());
957 >            lock.unlock();
958 >            lock.unlock();
959 >            assertUnlocked(sl);
960 >        }
961      }
962  
963      /**
# Line 958 | Line 980 | public class StampedLockTest extends JSR
980          Runnable[] actions = {
981              () -> {
982                  StampedLock sl = new StampedLock();
983 <                long stamp = sl.tryOptimisticRead();
962 <                assertTrue(stamp != 0);
983 >                long stamp = assertValid(sl, sl.tryOptimisticRead());
984                  sl.unlockRead(stamp);
985              },
986              () -> {
# Line 976 | Line 997 | public class StampedLockTest extends JSR
997              },
998              () -> {
999                  StampedLock sl = new StampedLock();
979                long stamp = sl.tryOptimisticRead();
1000                  sl.readLock();
1001 +                long stamp = assertValid(sl, sl.tryOptimisticRead());
1002                  sl.unlockRead(stamp);
1003              },
1004              () -> {
1005                  StampedLock sl = new StampedLock();
985                long stamp = sl.tryOptimisticRead();
1006                  sl.readLock();
1007 +                long stamp = assertValid(sl, sl.tryOptimisticRead());
1008                  sl.unlock(stamp);
1009              },
1010  
1011              () -> {
1012                  StampedLock sl = new StampedLock();
1013                  long stamp = sl.tryConvertToOptimisticRead(sl.writeLock());
1014 <                assertTrue(stamp != 0);
1014 >                assertValid(sl, stamp);
1015                  sl.writeLock();
1016                  sl.unlockWrite(stamp);
1017              },
# Line 1016 | Line 1037 | public class StampedLockTest extends JSR
1037              () -> {
1038                  StampedLock sl = new StampedLock();
1039                  long stamp = sl.tryConvertToOptimisticRead(sl.readLock());
1040 <                assertTrue(stamp != 0);
1040 >                assertValid(sl, stamp);
1041                  sl.writeLock();
1042                  sl.unlockWrite(stamp);
1043              },
# Line 1036 | Line 1057 | public class StampedLockTest extends JSR
1057                  StampedLock sl = new StampedLock();
1058                  sl.readLock();
1059                  long stamp = sl.tryConvertToOptimisticRead(sl.readLock());
1060 <                assertTrue(stamp != 0);
1060 >                assertValid(sl, stamp);
1061                  sl.readLock();
1062                  sl.unlockRead(stamp);
1063              },
# Line 1079 | Line 1100 | public class StampedLockTest extends JSR
1100      }
1101  
1102      /**
1103 <     * Invalid write stamps result in IllegalMonitorStateException
1103 >     * Invalid stamps result in IllegalMonitorStateException
1104       */
1105 <    public void testInvalidWriteStampsThrowIllegalMonitorStateException() {
1106 <        List<Function<StampedLock, Long>> writeLockers = new ArrayList<>();
1086 <        writeLockers.add((sl) -> sl.writeLock());
1087 <        writeLockers.add((sl) -> writeLockInterruptiblyUninterrupted(sl));
1088 <        writeLockers.add((sl) -> tryWriteLockUninterrupted(sl, Long.MIN_VALUE, DAYS));
1089 <        writeLockers.add((sl) -> tryWriteLockUninterrupted(sl, 0, DAYS));
1105 >    public void testInvalidStampsThrowIllegalMonitorStateException() {
1106 >        final StampedLock sl = new StampedLock();
1107  
1108 <        List<BiConsumer<StampedLock, Long>> writeUnlockers = new ArrayList<>();
1109 <        writeUnlockers.add((sl, stamp) -> sl.unlockWrite(stamp));
1110 <        writeUnlockers.add((sl, stamp) -> assertTrue(sl.tryUnlockWrite()));
1111 <        writeUnlockers.add((sl, stamp) -> sl.asWriteLock().unlock());
1112 <        writeUnlockers.add((sl, stamp) -> sl.unlock(stamp));
1108 >        assertThrows(IllegalMonitorStateException.class,
1109 >                     () -> sl.unlockWrite(0L),
1110 >                     () -> sl.unlockRead(0L),
1111 >                     () -> sl.unlock(0L));
1112 >
1113 >        final long optimisticStamp = sl.tryOptimisticRead();
1114 >        final long readStamp = sl.readLock();
1115 >        sl.unlockRead(readStamp);
1116 >        final long writeStamp = sl.writeLock();
1117 >        sl.unlockWrite(writeStamp);
1118 >        assertTrue(optimisticStamp != 0L && readStamp != 0L && writeStamp != 0L);
1119 >        final long[] noLongerValidStamps = { optimisticStamp, readStamp, writeStamp };
1120 >        final Runnable assertNoLongerValidStampsThrow = () -> {
1121 >            for (long noLongerValidStamp : noLongerValidStamps)
1122 >                assertThrows(IllegalMonitorStateException.class,
1123 >                             () -> sl.unlockWrite(noLongerValidStamp),
1124 >                             () -> sl.unlockRead(noLongerValidStamp),
1125 >                             () -> sl.unlock(noLongerValidStamp));
1126 >        };
1127 >        assertNoLongerValidStampsThrow.run();
1128  
1129 <        List<Consumer<StampedLock>> mutaters = new ArrayList<>();
1130 <        mutaters.add((sl) -> {});
1131 <        mutaters.add((sl) -> sl.readLock());
1132 <        for (Function<StampedLock, Long> writeLocker : writeLockers)
1133 <            mutaters.add((sl) -> writeLocker.apply(sl));
1102 <
1103 <        for (Function<StampedLock, Long> writeLocker : writeLockers)
1104 <        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers)
1105 <        for (Consumer<StampedLock> mutater : mutaters) {
1106 <            final StampedLock sl = new StampedLock();
1107 <            final long stamp = writeLocker.apply(sl);
1108 <            assertTrue(stamp != 0L);
1129 >        for (Function<StampedLock, Long> readLocker : readLockers())
1130 >        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers()) {
1131 >            final long stamp = readLocker.apply(sl);
1132 >            assertValid(sl, stamp);
1133 >            assertNoLongerValidStampsThrow.run();
1134              assertThrows(IllegalMonitorStateException.class,
1135 <                         () -> sl.unlockRead(stamp));
1136 <            writeUnlocker.accept(sl, stamp);
1137 <            mutater.accept(sl);
1135 >                         () -> sl.unlockWrite(stamp),
1136 >                         () -> sl.unlockRead(sl.tryOptimisticRead()),
1137 >                         () -> sl.unlockRead(0L));
1138 >            readUnlocker.accept(sl, stamp);
1139 >            assertUnlocked(sl);
1140 >            assertNoLongerValidStampsThrow.run();
1141 >        }
1142 >
1143 >        for (Function<StampedLock, Long> writeLocker : writeLockers())
1144 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
1145 >            final long stamp = writeLocker.apply(sl);
1146 >            assertValid(sl, stamp);
1147 >            assertNoLongerValidStampsThrow.run();
1148              assertThrows(IllegalMonitorStateException.class,
1114                         () -> sl.unlock(stamp),
1149                           () -> sl.unlockRead(stamp),
1150 <                         () -> sl.unlockWrite(stamp));
1150 >                         () -> sl.unlockWrite(0L));
1151 >            writeUnlocker.accept(sl, stamp);
1152 >            assertUnlocked(sl);
1153 >            assertNoLongerValidStampsThrow.run();
1154          }
1155      }
1156  
1157      /**
1158 <     * Invalid read stamps result in IllegalMonitorStateException
1158 >     * Read locks can be very deeply nested
1159       */
1160 <    public void testInvalidReadStampsThrowIllegalMonitorStateException() {
1161 <        List<Function<StampedLock, Long>> readLockers = new ArrayList<>();
1162 <        readLockers.add((sl) -> sl.readLock());
1163 <        readLockers.add((sl) -> readLockInterruptiblyUninterrupted(sl));
1164 <        readLockers.add((sl) -> tryReadLockUninterrupted(sl, Long.MIN_VALUE, DAYS));
1165 <        readLockers.add((sl) -> tryReadLockUninterrupted(sl, 0, DAYS));
1160 >    public void testDeeplyNestedReadLocks() {
1161 >        final StampedLock lock = new StampedLock();
1162 >        final int depth = 300;
1163 >        final long[] stamps = new long[depth];
1164 >        final List<Function<StampedLock, Long>> readLockers = readLockers();
1165 >        final List<BiConsumer<StampedLock, Long>> readUnlockers = readUnlockers();
1166 >        for (int i = 0; i < depth; i++) {
1167 >            Function<StampedLock, Long> readLocker
1168 >                = readLockers.get(i % readLockers.size());
1169 >            long stamp = readLocker.apply(lock);
1170 >            assertEquals(i + 1, lock.getReadLockCount());
1171 >            assertTrue(lock.isReadLocked());
1172 >            stamps[i] = stamp;
1173 >        }
1174 >        for (int i = 0; i < depth; i++) {
1175 >            BiConsumer<StampedLock, Long> readUnlocker
1176 >                = readUnlockers.get(i % readUnlockers.size());
1177 >            assertEquals(depth - i, lock.getReadLockCount());
1178 >            assertTrue(lock.isReadLocked());
1179 >            readUnlocker.accept(lock, stamps[depth - 1 - i]);
1180 >        }
1181 >        assertUnlocked(lock);
1182 >    }
1183  
1184 <        List<BiConsumer<StampedLock, Long>> readUnlockers = new ArrayList<>();
1185 <        readUnlockers.add((sl, stamp) -> sl.unlockRead(stamp));
1186 <        readUnlockers.add((sl, stamp) -> assertTrue(sl.tryUnlockRead()));
1187 <        readUnlockers.add((sl, stamp) -> sl.asReadLock().unlock());
1188 <        readUnlockers.add((sl, stamp) -> sl.unlock(stamp));
1184 >    /**
1185 >     * Stamped locks are not reentrant.
1186 >     */
1187 >    public void testNonReentrant() throws InterruptedException {
1188 >        final StampedLock lock = new StampedLock();
1189 >        long stamp;
1190  
1191 <        List<Function<StampedLock, Long>> writeLockers = new ArrayList<>();
1192 <        writeLockers.add((sl) -> sl.writeLock());
1193 <        writeLockers.add((sl) -> writeLockInterruptiblyUninterrupted(sl));
1194 <        writeLockers.add((sl) -> tryWriteLockUninterrupted(sl, Long.MIN_VALUE, DAYS));
1195 <        writeLockers.add((sl) -> tryWriteLockUninterrupted(sl, 0, DAYS));
1191 >        stamp = lock.writeLock();
1192 >        assertValid(lock, stamp);
1193 >        assertEquals(0L, lock.tryWriteLock(0L, DAYS));
1194 >        assertEquals(0L, lock.tryReadLock(0L, DAYS));
1195 >        assertValid(lock, stamp);
1196 >        lock.unlockWrite(stamp);
1197 >
1198 >        stamp = lock.tryWriteLock(1L, DAYS);
1199 >        assertEquals(0L, lock.tryWriteLock(0L, DAYS));
1200 >        assertValid(lock, stamp);
1201 >        lock.unlockWrite(stamp);
1202 >
1203 >        stamp = lock.readLock();
1204 >        assertEquals(0L, lock.tryWriteLock(0L, DAYS));
1205 >        assertValid(lock, stamp);
1206 >        lock.unlockRead(stamp);
1207 >    }
1208  
1209 <        List<BiConsumer<StampedLock, Long>> writeUnlockers = new ArrayList<>();
1210 <        writeUnlockers.add((sl, stamp) -> sl.unlockWrite(stamp));
1211 <        writeUnlockers.add((sl, stamp) -> assertTrue(sl.tryUnlockWrite()));
1212 <        writeUnlockers.add((sl, stamp) -> sl.asWriteLock().unlock());
1213 <        writeUnlockers.add((sl, stamp) -> sl.unlock(stamp));
1209 >    /**
1210 >     * """StampedLocks have no notion of ownership. Locks acquired in
1211 >     * one thread can be released or converted in another."""
1212 >     */
1213 >    public void testNoOwnership() throws Throwable {
1214 >        ArrayList<Future<?>> futures = new ArrayList<>();
1215 >        for (Function<StampedLock, Long> writeLocker : writeLockers())
1216 >        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers()) {
1217 >            StampedLock lock = new StampedLock();
1218 >            long stamp = writeLocker.apply(lock);
1219 >            futures.add(cachedThreadPool.submit(new CheckedRunnable() {
1220 >                public void realRun() {
1221 >                    writeUnlocker.accept(lock, stamp);
1222 >                    assertUnlocked(lock);
1223 >                    assertFalse(lock.validate(stamp));
1224 >                }}));
1225 >        }
1226 >        for (Future<?> future : futures)
1227 >            assertNull(future.get());
1228 >    }
1229  
1230 +    /** Tries out sample usage code from StampedLock javadoc. */
1231 +    public void testSampleUsage() throws Throwable {
1232 +        class Point {
1233 +            private double x, y;
1234 +            private final StampedLock sl = new StampedLock();
1235 +
1236 +            void move(double deltaX, double deltaY) { // an exclusively locked method
1237 +                long stamp = sl.writeLock();
1238 +                try {
1239 +                    x += deltaX;
1240 +                    y += deltaY;
1241 +                } finally {
1242 +                    sl.unlockWrite(stamp);
1243 +                }
1244 +            }
1245  
1246 <        for (Function<StampedLock, Long> readLocker : readLockers)
1247 <        for (BiConsumer<StampedLock, Long> readUnlocker : readUnlockers)
1248 <        for (Function<StampedLock, Long> writeLocker : writeLockers)
1249 <        for (BiConsumer<StampedLock, Long> writeUnlocker : writeUnlockers) {
1250 <            final StampedLock sl = new StampedLock();
1251 <            final long stamp = readLocker.apply(sl);
1252 <            assertTrue(stamp != 0L);
1253 <            assertThrows(IllegalMonitorStateException.class,
1254 <                         () -> sl.unlockWrite(stamp));
1255 <            readUnlocker.accept(sl, stamp);
1256 <            assertThrows(IllegalMonitorStateException.class,
1257 <                         () -> sl.unlock(stamp),
1258 <                         () -> sl.unlockRead(stamp),
1259 <                         () -> sl.unlockWrite(stamp));
1260 <            final long writeStamp = writeLocker.apply(sl);
1261 <            assertTrue(writeStamp != 0L);
1165 <            assertTrue(writeStamp != stamp);
1166 <            assertThrows(IllegalMonitorStateException.class,
1167 <                         () -> sl.unlock(stamp),
1168 <                         () -> sl.unlockRead(stamp),
1169 <                         () -> sl.unlockWrite(stamp));
1170 <            writeUnlocker.accept(sl, writeStamp);
1171 <            assertThrows(IllegalMonitorStateException.class,
1172 <                         () -> sl.unlock(stamp),
1173 <                         () -> sl.unlockRead(stamp),
1174 <                         () -> sl.unlockWrite(stamp));
1246 >            double distanceFromOrigin() { // A read-only method
1247 >                double currentX, currentY;
1248 >                long stamp = sl.tryOptimisticRead();
1249 >                do {
1250 >                    if (stamp == 0L)
1251 >                        stamp = sl.readLock();
1252 >                    try {
1253 >                        // possibly racy reads
1254 >                        currentX = x;
1255 >                        currentY = y;
1256 >                    } finally {
1257 >                        stamp = sl.tryConvertToOptimisticRead(stamp);
1258 >                    }
1259 >                } while (stamp == 0);
1260 >                return Math.hypot(currentX, currentY);
1261 >            }
1262          }
1263 +
1264 +        Point p = new Point();
1265 +        p.move(3.0, 4.0);
1266 +        assertEquals(5.0, p.distanceFromOrigin());
1267      }
1268  
1269   }

Diff Legend

Removed lines
+ Added lines
< Changed lines
> Changed lines