bonding: Remove unnecessary else
authorJoe Perches <joe@perches.com>
Tue, 18 Feb 2014 17:42:46 +0000 (09:42 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 19 Feb 2014 20:37:51 +0000 (15:37 -0500)
It's unnecessary and less readable after a clause ending in a goto.

Signed-off-by: Joe Perches <joe@perches.com>
Reviewed-by: Ding Tianhong <dingtianhong@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/bonding/bond_alb.c

index 97a43a2..aaeeacf 100644 (file)
@@ -1455,12 +1455,12 @@ int bond_alb_xmit(struct sk_buff *skb, struct net_device *bond_dev)
 
                bond_dev_queue_xmit(bond, skb, tx_slave->dev);
                goto out;
-       } else {
-               if (tx_slave) {
-                       _lock_tx_hashtbl(bond);
-                       __tlb_clear_slave(bond, tx_slave, 0);
-                       _unlock_tx_hashtbl(bond);
-               }
+       }
+
+       if (tx_slave) {
+               _lock_tx_hashtbl(bond);
+               __tlb_clear_slave(bond, tx_slave, 0);
+               _unlock_tx_hashtbl(bond);
        }
 
        /* no suitable interface, frame not sent */