summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRoland Dreier <rolandd@cisco.com>2007-10-17 21:54:44 -0700
committerRoland Dreier <rolandd@cisco.com>2007-10-17 21:54:44 -0700
commitfd312561adcc90e924f35d3032d5493aeb4c3017 (patch)
tree5a69996b93124a8beeb589007324a8a33a7c2fa8
parent8da9ee9c1ec9036d8fad82f6ddbc4d15143cc6ca (diff)
downloadtalos-obmc-linux-fd312561adcc90e924f35d3032d5493aeb4c3017.tar.gz
talos-obmc-linux-fd312561adcc90e924f35d3032d5493aeb4c3017.zip
IPoIB: Rewrite "if (!likely(...))" as "if (unlikely(!(...)))"
It's too hard to figure out what "!likely(...)" really means, and who knows how compilers interpret the hint. Signed-off-by: Roland Dreier <rolandd@cisco.com>
-rw-r--r--drivers/infiniband/ulp/ipoib/ipoib_cm.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/infiniband/ulp/ipoib/ipoib_cm.c b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
index 0a0dcb8fdfd1..ddf0c54493c8 100644
--- a/drivers/infiniband/ulp/ipoib/ipoib_cm.c
+++ b/drivers/infiniband/ulp/ipoib/ipoib_cm.c
@@ -434,7 +434,7 @@ void ipoib_cm_handle_rx_wc(struct net_device *dev, struct ib_wc *wc)
goto repost;
}
- if (!likely(wr_id & IPOIB_CM_RX_UPDATE_MASK)) {
+ if (unlikely(!(wr_id & IPOIB_CM_RX_UPDATE_MASK))) {
p = wc->qp->qp_context;
if (p && time_after_eq(jiffies, p->jiffies + IPOIB_CM_RX_UPDATE_TIME)) {
spin_lock_irqsave(&priv->lock, flags);
OpenPOWER on IntegriCloud