diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-06-18 09:27:38 +0200 |
---|---|---|
committer | Roland Dreier <roland@purestorage.com> | 2013-06-20 13:51:52 +0200 |
commit | 21bfd4706268c85c9d17ca2fd48de3b19c9d40db (patch) | |
tree | 5627c83e66849ee3107de6ccf42502aa2b669a20 /drivers | |
parent | ndisc: Convert use of typedef ctl_table to struct ctl_table (diff) | |
download | linux-21bfd4706268c85c9d17ca2fd48de3b19c9d40db.tar.xz linux-21bfd4706268c85c9d17ca2fd48de3b19c9d40db.zip |
RDMA/cxgb3: Timeout condition is never true
This is a static checker fix. "count" is unsigned so it's never -1.
Since "count" is 16 bits and the addition operation is implicitly
casted to int then there is no wrapping here.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Steve Wise <swise@opengridcomputing.com>
Signed-off-by: Roland Dreier <roland@purestorage.com>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/hw/cxgb3/iwch_qp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/cxgb3/iwch_qp.c b/drivers/infiniband/hw/cxgb3/iwch_qp.c index e5649e8b215d..b57c0befd962 100644 --- a/drivers/infiniband/hw/cxgb3/iwch_qp.c +++ b/drivers/infiniband/hw/cxgb3/iwch_qp.c @@ -883,7 +883,8 @@ u16 iwch_rqes_posted(struct iwch_qp *qhp) { union t3_wr *wqe = qhp->wq.queue; u16 count = 0; - while ((count+1) != 0 && fw_riwrh_opcode((struct fw_riwrh *)wqe) == T3_WR_RCV) { + + while (count < USHRT_MAX && fw_riwrh_opcode((struct fw_riwrh *)wqe) == T3_WR_RCV) { count++; wqe++; } |