diff options
| author | Eli Cohen <eli@dev.mellanox.co.il> | 2013-10-31 13:26:35 (GMT) |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2013-11-15 18:24:17 (GMT) |
| commit | 93b80ac297b1cfdf3bea771a5ce6ea4ff5d25d1d (patch) | |
| tree | c8cb81c349cb9740b358b9670ecbe5fb8904aa35 | |
| parent | 959f58544b7f20c92d5eb43d1232c96c15c01bfb (diff) | |
| download | linux-93b80ac297b1cfdf3bea771a5ce6ea4ff5d25d1d.tar.xz | |
IB/mlx4: Fix endless loop in resize CQ
When calling get_sw_cqe() we need pass the consumer_index and not the
masked value. Failure to do so will cause incorrect result of
get_sw_cqe() possibly leading to endless loop.
This problem was reported and analyzed by Michael Rice from HP.
Signed-off-by: Eli Cohen <eli@mellanox.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
| -rw-r--r-- | drivers/infiniband/hw/mlx4/cq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/mlx4/cq.c b/drivers/infiniband/hw/mlx4/cq.c index d5e60f4..3fe1a68a 100644 --- a/drivers/infiniband/hw/mlx4/cq.c +++ b/drivers/infiniband/hw/mlx4/cq.c @@ -324,7 +324,7 @@ static int mlx4_ib_get_outstanding_cqes(struct mlx4_ib_cq *cq) u32 i; i = cq->mcq.cons_index; - while (get_sw_cqe(cq, i & cq->ibcq.cqe)) + while (get_sw_cqe(cq, i)) ++i; return i - cq->mcq.cons_index; |