diff options
author | Steve Wise <swise@opengridcomputing.com> | 2009-04-29 22:15:59 (GMT) |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2009-04-29 22:15:59 (GMT) |
commit | ec6995ddaa3db2c5f028184ed9b74c0c448c4947 (patch) | |
tree | 1d4635d10ed0a9f1e4d6f1893321d9dadc0b4cf6 /drivers | |
parent | 56a50adda49b2020156616c4eb15353e0f9ad7de (diff) | |
download | linux-ec6995ddaa3db2c5f028184ed9b74c0c448c4947.tar.xz |
RDMA/cxgb3: Don't complete flushed send work requests twice
When the SQ is flushed, mark the flushed entries as not signaled so
the poll logic doesn't re-insert the CQ entry thinking its an out of
order completion.
The bug can cause the NFS/RDMA server to crash due to processing the
same completed work request twice.
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Roland Dreier <rolandd@cisco.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/hw/cxgb3/cxio_hal.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/infiniband/hw/cxgb3/cxio_hal.c b/drivers/infiniband/hw/cxgb3/cxio_hal.c index 8d71086..62f9cf2 100644 --- a/drivers/infiniband/hw/cxgb3/cxio_hal.c +++ b/drivers/infiniband/hw/cxgb3/cxio_hal.c @@ -410,6 +410,7 @@ int cxio_flush_sq(struct t3_wq *wq, struct t3_cq *cq, int count) ptr = wq->sq_rptr + count; sqp = wq->sq + Q_PTR2IDX(ptr, wq->sq_size_log2); while (ptr != wq->sq_wptr) { + sqp->signaled = 0; insert_sq_cqe(wq, cq, sqp); ptr++; sqp = wq->sq + Q_PTR2IDX(ptr, wq->sq_size_log2); |