RFR (M) CR 8023234: StampedLock serializes readers on writer unlock

Martin Buchholz martinrb at google.com
Mon Aug 19 23:06:55 UTC 2013


Thanks.  Looks good.

Style suggestions for the test:

+            while(!isDone && !isInterrupted()) {
SPC after keywords.

I think it's more readable to use CountDownLatch instead of CyclicBarrier
for writerHasLock, because of asymmetric API.
Doers call countDown(), waiters call await().



On Mon, Aug 19, 2013 at 7:50 AM, Aleksey Shipilev <
aleksey.shipilev at oracle.com> wrote:

> Hi,
>
> This is the review for the issue originally found by Dr. Heinz Kabutz
> and Victor Grazi. The issue is consequently diagnosed and fixed by Doug
> Lea in JSR 166 CVS.
>
> The webrev which brings in the fix and the regression tests in JDK 8:
>    http://cr.openjdk.java.net/~shade/8023234/webrev.00/
>
> Testing:
>   - builds on Linux x86_64/release
>   - targeted regression test failure/pass before/after the fix
>   - java/util/concurrent/locks/StampedLock basic tests
>
> Thanks,
> -Aleksey.
>



More information about the core-libs-dev mailing list