bonding: enable netpoll without checking link status
authorAndy Gospodarek <andy@greyhouse.net>
Mon, 14 Mar 2011 12:05:21 +0000 (12:05 +0000)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Mar 2011 19:57:53 +0000 (12:57 -0700)
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>
drivers/net/bonding/bond_main.c

index 27c413aa15da06c24b711c8d493682c8d056c80e..1a6e9eb7af431b8d7d833da233f7a44106775874 100644 (file)
@@ -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);