diff options
author | Veaceslav Falico <vfalico@gmail.com> | 2014-05-15 15:39:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-05-16 16:34:33 -0400 |
commit | 8557cd74ca8af9a71ae19d445e33d92bd50a6dc5 (patch) | |
tree | c6ad54191e3ed0afc12f62cd5f546189058ec096 /drivers/net/bonding/bond_3ad.c | |
parent | 891ab54d6636b7aa0da48b5a5dd738af8b75cafe (diff) |
bonding: replace SLAVE_IS_OK() with bond_slave_can_tx()
They're verifying the same thing (except of IFF_UP, which is implied for
netif_running(), which is also a prerequisite).
CC: Jay Vosburgh <j.vosburgh@gmail.com>
CC: Andy Gospodarek <andy@greyhouse.net>
Signed-off-by: Veaceslav Falico <vfalico@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding/bond_3ad.c')
-rw-r--r-- | drivers/net/bonding/bond_3ad.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/bonding/bond_3ad.c b/drivers/net/bonding/bond_3ad.c index 7997a1e7cfd0..0dfeaf5da3f2 100644 --- a/drivers/net/bonding/bond_3ad.c +++ b/drivers/net/bonding/bond_3ad.c | |||
@@ -2449,13 +2449,13 @@ int bond_3ad_xmit_xor(struct sk_buff *skb, struct net_device *dev) | |||
2449 | continue; | 2449 | continue; |
2450 | 2450 | ||
2451 | if (slave_agg_no >= 0) { | 2451 | if (slave_agg_no >= 0) { |
2452 | if (!first_ok_slave && SLAVE_IS_OK(slave)) | 2452 | if (!first_ok_slave && bond_slave_can_tx(slave)) |
2453 | first_ok_slave = slave; | 2453 | first_ok_slave = slave; |
2454 | slave_agg_no--; | 2454 | slave_agg_no--; |
2455 | continue; | 2455 | continue; |
2456 | } | 2456 | } |
2457 | 2457 | ||
2458 | if (SLAVE_IS_OK(slave)) { | 2458 | if (bond_slave_can_tx(slave)) { |
2459 | bond_dev_queue_xmit(bond, skb, slave->dev); | 2459 | bond_dev_queue_xmit(bond, skb, slave->dev); |
2460 | goto out; | 2460 | goto out; |
2461 | } | 2461 | } |