diff options
| author | Andy Gospodarek <andy@greyhouse.net> | 2011-03-14 13:05:21 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-03-16 20:57:53 +0100 |
| commit | ceda86a108671294052cbf51660097b6534672f5 (patch) | |
| tree | fd82cf21c099ba455e429bac7b8b0bbb9bd0d7ea /drivers/net/bonding | |
| parent | xfrm: Refcount destination entry on xfrm_lookup (diff) | |
| download | linux-ceda86a108671294052cbf51660097b6534672f5.tar.xz linux-ceda86a108671294052cbf51660097b6534672f5.zip | |
bonding: enable netpoll without checking link status
Only slaves that are up should transmit netpoll frames, so there is no
need to check to see if a slave is up before enabling netpoll on it.
This resolves a reported failure on active-backup bonds where a slave
interface is down when netpoll was enabled.
Signed-off-by: Andy Gospodarek <andy@greyhouse.net>
Tested-by: WANG Cong <amwang@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/bonding')
| -rw-r--r-- | drivers/net/bonding/bond_main.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 27c413aa15da..1a6e9eb7af43 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -1347,8 +1347,6 @@ static int bond_netpoll_setup(struct net_device *dev, struct netpoll_info *ni) read_lock(&bond->lock); bond_for_each_slave(bond, slave, i) { - if (!IS_UP(slave->dev)) - continue; err = slave_enable_netpoll(slave); if (err) { __bond_netpoll_cleanup(bond); |