即使等待条件已经改变,AbstractQueuedSynchronizer.acquireShared也会无限期地等待

Question

我写了一个使用AbstractQueuedSynchronizer的简单类.我写了一个代表"门"的类,如果打开则可以传递,如果关闭则可以阻塞.这是代码:

public class GateBlocking {

  final class Sync extends AbstractQueuedSynchronizer {
    public Sync() {
      setState(0);
    }

    @Override
    protected int tryAcquireShared(int ignored) {
      return getState() == 1 ? 1 : -1;
    }

    public void reset(int newState) {
      setState(newState);
    }
  };

  private Sync sync = new Sync();

  public void open() {
    sync.reset(1);
  }

  public void close() {
    sync.reset(0);
  }

public void pass() throws InterruptedException {
    sync.acquireShared(1);
  }

};

不幸的是,如果一个线程阻塞传递方法,因为门被关闭而另一些线​​程同时打开门,被阻塞的线程不会被中断 - 它无限地阻塞.这是一个测试,显示它:

public class GateBlockingTest {

    @Test
    public void parallelPassClosedAndOpenGate() throws Exception{
        final GateBlocking g = new GateBlocking();
        Thread t = new Thread(new Runnable() {
            @Override
            public void run() {
                try {
                    Thread.sleep(2000);
                    g.open();
                } catch (InterruptedException e) {
                }
            }
        });


        t.start();
        g.pass();
    }
}

请帮助,我应该更改什么来使门传递线程成功获取锁.

Answer 1

它看起来setState()只更改状态，但不会通知阻塞线程有关更改的信息。

因此，您应该使用获取/释放方法：

@Override
protected boolean tryReleaseShared(int ignored) {
    setState(1);
    return true;
}
...
public void open() {
   sync.releaseShared(1);
}

因此，整体工作流程AbstractQueuedSynchronizer如下：

• 客户端调用public获取/释放方法

• 这些方法安排所有同步功能并将实际锁定策略委托给protected try*()方法

• 您可以使用//在protected try*()方法中定义锁定策略getState()setState()compareAndSetState()