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