[NET]: Fix reversed error test in netif_tx_trylock
authorHerbert Xu <herbert@gondor.apana.org.au>
Fri, 21 Jul 2006 21:55:38 +0000 (14:55 -0700)
committerDavid S. Miller <davem@davemloft.net>
Fri, 21 Jul 2006 21:55:38 +0000 (14:55 -0700)
A non-zero return value indicates success from spin_trylock,
not error.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netdevice.h

index 76cc099..75f02d8 100644 (file)
@@ -924,10 +924,10 @@ static inline void netif_tx_lock_bh(struct net_device *dev)
 
 static inline int netif_tx_trylock(struct net_device *dev)
 {
-       int err = spin_trylock(&dev->_xmit_lock);
-       if (!err)
+       int ok = spin_trylock(&dev->_xmit_lock);
+       if (likely(ok))
                dev->xmit_lock_owner = smp_processor_id();
-       return err;
+       return ok;
 }
 
 static inline void netif_tx_unlock(struct net_device *dev)