diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-09-06 04:50:46 -0400 |
|---|---|---|
| committer | Roland Dreier <roland@purestorage.com> | 2013-11-08 17:43:05 -0500 |
| commit | 6ebacdfc07ca61ba258693c9b4c88f1ffbe8ccd7 (patch) | |
| tree | 41dd5f8e054c318bdd689f828c532db653230315 | |
| parent | 959f58544b7f20c92d5eb43d1232c96c15c01bfb (diff) | |
RDMA/ocrdma: Silence an integer underflow warning
We recently added a cap on "max_wqe_allocated" in 43a6b4025c
('RDMA/ocrdma: Create IRD queue fix').
My static checker complains that the cap has a problem because it
casts large values to negative. "attrs->cap.max_send_wr" is a u32.
It comes from the user, but it's capped in ocrdma_check_qp_params() so
it can't wrap here.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
| -rw-r--r-- | drivers/infiniband/hw/ocrdma/ocrdma_hw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/ocrdma/ocrdma_hw.c b/drivers/infiniband/hw/ocrdma/ocrdma_hw.c index 50219ab2279d..56bf32fcb62c 100644 --- a/drivers/infiniband/hw/ocrdma/ocrdma_hw.c +++ b/drivers/infiniband/hw/ocrdma/ocrdma_hw.c | |||
| @@ -1783,7 +1783,7 @@ static int ocrdma_set_create_qp_sq_cmd(struct ocrdma_create_qp_req *cmd, | |||
| 1783 | u32 max_sges = attrs->cap.max_send_sge; | 1783 | u32 max_sges = attrs->cap.max_send_sge; |
| 1784 | 1784 | ||
| 1785 | /* QP1 may exceed 127 */ | 1785 | /* QP1 may exceed 127 */ |
| 1786 | max_wqe_allocated = min_t(int, attrs->cap.max_send_wr + 1, | 1786 | max_wqe_allocated = min_t(u32, attrs->cap.max_send_wr + 1, |
| 1787 | dev->attr.max_wqe); | 1787 | dev->attr.max_wqe); |
| 1788 | 1788 | ||
| 1789 | status = ocrdma_build_q_conf(&max_wqe_allocated, | 1789 | status = ocrdma_build_q_conf(&max_wqe_allocated, |