diff options
| author | Steve Wise <swise@opengridcomputing.com> | 2014-03-19 16:14:42 +0400 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2014-03-20 21:01:30 +0400 |
| commit | 8a9c399eeee8c2d99e22b975f6023001a1fde88f (patch) | |
| tree | 6ba1c08811ed6877d33a1762f4f26c47c0ceb4ac /drivers/infiniband/hw/cxgb4/cq.c | |
| parent | ebf00060c33b9d0946384fa6f440df7ea35a569e (diff) | |
| download | linux-8a9c399eeee8c2d99e22b975f6023001a1fde88f.tar.xz | |
RDMA/cxgb4: Fix incorrect BUG_ON conditions
Based on original work from Jay Hernandez <jay@chelsio.com>
Signed-off-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers/infiniband/hw/cxgb4/cq.c')
| -rw-r--r-- | drivers/infiniband/hw/cxgb4/cq.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/hw/cxgb4/cq.c b/drivers/infiniband/hw/cxgb4/cq.c index 906119f7a43f..d6a7db2c42f6 100644 --- a/drivers/infiniband/hw/cxgb4/cq.c +++ b/drivers/infiniband/hw/cxgb4/cq.c @@ -603,7 +603,7 @@ proc_cqe: */ if (SQ_TYPE(hw_cqe)) { int idx = CQE_WRID_SQ_IDX(hw_cqe); - BUG_ON(idx > wq->sq.size); + BUG_ON(idx >= wq->sq.size); /* * Account for any unsignaled completions completed by @@ -617,7 +617,7 @@ proc_cqe: 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); + BUG_ON(wq->sq.in_use <= 0 && wq->sq.in_use >= wq->sq.size); wq->sq.cidx = (uint16_t)idx; PDBG("%s completing sq idx %u\n", __func__, wq->sq.cidx); |