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

时间:2012-12-07 14:17:30

标签: java java.util.concurrent

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

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();
    }
}

请帮助,我应该更改什么才能使门传递线程成功获得锁定。

1 个答案:

答案 0 :(得分:2)

看起来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()方法中定义锁定政策