diff options
author | Steve Wise <swise@opengridcomputing.com> | 2013-08-06 17:34:36 +0200 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2013-08-13 20:55:46 +0200 |
commit | 27ca34f54a70cb85895aa7147a6c35f1cd07fa55 (patch) | |
tree | 460c3cad1c580ce0070c8312c0311f982d6fffda /drivers/infiniband/hw/cxgb4 | |
parent | RDMA/cxgb4: Fix QP flush logic (diff) | |
download | linux-27ca34f54a70cb85895aa7147a6c35f1cd07fa55.tar.xz linux-27ca34f54a70cb85895aa7147a6c35f1cd07fa55.zip |
RDMA/cxgb4: Fix accounting for unsignaled SQ WRs to deal with wrap
When determining how many WRs are completed with a signaled CQE,
correctly deal with queue wraps.
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Vipul Pandya <vipul@chelsio.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/hw/cxgb4')
-rw-r--r-- | drivers/infiniband/hw/cxgb4/cq.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/cxgb4/cq.c b/drivers/infiniband/hw/cxgb4/cq.c index 6657390de956..88de3aa9c5b0 100644 --- a/drivers/infiniband/hw/cxgb4/cq.c +++ b/drivers/infiniband/hw/cxgb4/cq.c @@ -611,9 +611,12 @@ proc_cqe: * to the first unsignaled one, and idx points to the * signaled one. So adjust in_use based on this delta. * if this is not completing any unsigned wrs, then the - * delta will be 0. + * delta will be 0. Handle wrapping also! */ - wq->sq.in_use -= idx - wq->sq.cidx; + if (idx < wq->sq.cidx) + wq->sq.in_use -= wq->sq.size + idx - wq->sq.cidx; + else + wq->sq.in_use -= idx - wq->sq.cidx; BUG_ON(wq->sq.in_use < 0 && wq->sq.in_use < wq->sq.size); wq->sq.cidx = (uint16_t)idx; |