RFR 8067241 DeadlockTest.java failed with negative timeout value
Daniel Fuchs
daniel.fuchs at oracle.com
Mon Dec 15 17:38:39 UTC 2014
On 12/12/14 09:56, shanliang wrote:
> Updated.
>
> Here is the new version:
> http://cr.openjdk.java.net/~sjiang/JDK-8067241/01/
>
> Thanks,
> Shanliang
Hi Shanliang,
Your changes looks good to me. WRT using a Phaser, it would
require a careful analysis to assert that modifying the
locking strategy - especially things like
90 synchronized(o) {
91 synchronized(this) {
92 gotLock = true;
93
94 this.notify();
95 }
still preserve the nature of the test.
I wonder if it's worth the trouble.
best regards,
-- daniel
More information about the serviceability-dev
mailing list