summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2013-03-18 20:25:26 +0000
committerRoland Dreier <roland@purestorage.com>2013-03-22 18:07:04 -0700
commit3c32869f7afe40ff7372e5bb7cd3d8b4520711bb (patch)
tree4af3793eec037017e8dfd73ad82f8f877ba8f7f4
parenta937536b868b8369b98967929045f1df54234323 (diff)
downloadop-kernel-dev-3c32869f7afe40ff7372e5bb7cd3d8b4520711bb.zip
op-kernel-dev-3c32869f7afe40ff7372e5bb7cd3d8b4520711bb.tar.gz
IB/ipath: Silence a static checker warning
I have a static checker which complains that 0x255 is too high for the "dev->opstats[opcode]" array. It turns out that the hardware has already validated the opcode at this point so it can't actually overflow. However, silencing the warning is good and this matches how the opcode is treated in qib_ib_rcv() as well. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: Mike Marciniszyn <mike.marciniszyn@intel.com> Signed-off-by: Roland Dreier <roland@purestorage.com>
-rw-r--r--drivers/infiniband/hw/ipath/ipath_verbs.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/hw/ipath/ipath_verbs.c b/drivers/infiniband/hw/ipath/ipath_verbs.c
index 439c35d..ea93870 100644
--- a/drivers/infiniband/hw/ipath/ipath_verbs.c
+++ b/drivers/infiniband/hw/ipath/ipath_verbs.c
@@ -620,7 +620,7 @@ void ipath_ib_rcv(struct ipath_ibdev *dev, void *rhdr, void *data,
goto bail;
}
- opcode = be32_to_cpu(ohdr->bth[0]) >> 24;
+ opcode = (be32_to_cpu(ohdr->bth[0]) >> 24) & 0x7f;
dev->opstats[opcode].n_bytes += tlen;
dev->opstats[opcode].n_packets++;
OpenPOWER on IntegriCloud