diff options
author | Roland Dreier <rolandd@cisco.com> | 2007-10-18 00:54:44 -0400 |
---|---|---|
committer | Roland Dreier <rolandd@cisco.com> | 2007-10-18 00:54:44 -0400 |
commit | fd312561adcc90e924f35d3032d5493aeb4c3017 (patch) | |
tree | 5a69996b93124a8beeb589007324a8a33a7c2fa8 /drivers | |
parent | 8da9ee9c1ec9036d8fad82f6ddbc4d15143cc6ca (diff) |
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>
Diffstat (limited to 'drivers')
-rw-r--r-- | drivers/infiniband/ulp/ipoib/ipoib_cm.c | 2 |
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) | |||
434 | goto repost; | 434 | goto repost; |
435 | } | 435 | } |
436 | 436 | ||
437 | if (!likely(wr_id & IPOIB_CM_RX_UPDATE_MASK)) { | 437 | if (unlikely(!(wr_id & IPOIB_CM_RX_UPDATE_MASK))) { |
438 | p = wc->qp->qp_context; | 438 | p = wc->qp->qp_context; |
439 | if (p && time_after_eq(jiffies, p->jiffies + IPOIB_CM_RX_UPDATE_TIME)) { | 439 | if (p && time_after_eq(jiffies, p->jiffies + IPOIB_CM_RX_UPDATE_TIME)) { |
440 | spin_lock_irqsave(&priv->lock, flags); | 440 | spin_lock_irqsave(&priv->lock, flags); |