Skip to content

Commit

Permalink
IPoIB: Rewrite "if (!likely(...))" as "if (unlikely(!(...)))"
Browse files Browse the repository at this point in the history
    
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>
  • Loading branch information
Roland Dreier committed Oct 18, 2007
1 parent 8da9ee9 commit fd31256
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion drivers/infiniband/ulp/ipoib/ipoib_cm.c
Original file line number Diff line number Diff line change
Expand Up @@ -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);
Expand Down

0 comments on commit fd31256

Please sign in to comment.