diff options
author | Jay Vosburgh <fubar@us.ibm.com> | 2005-05-26 15:56:59 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2005-05-26 15:56:59 -0400 |
commit | 2f872f0401d4b470990864fbf99c19130f25ad4d (patch) | |
tree | 715de6e359e359321df14fe5d21c9d50076be761 | |
parent | 0dca51d362b8e4af6b0dbc9e54d1e5165341918a (diff) |
[BONDING]: bonding using arp_ip_target may stay down with active path
Correcting the list traversal makes the problem go away.
Signed-off-by: Jay Vosburgh <fubar@us.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/bonding/bond_main.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 770e28f98fd5..269a5e407349 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c | |||
@@ -3037,7 +3037,7 @@ static void bond_activebackup_arp_mon(struct net_device *bond_dev) | |||
3037 | bond_set_slave_inactive_flags(bond->current_arp_slave); | 3037 | bond_set_slave_inactive_flags(bond->current_arp_slave); |
3038 | 3038 | ||
3039 | /* search for next candidate */ | 3039 | /* search for next candidate */ |
3040 | bond_for_each_slave_from(bond, slave, i, bond->current_arp_slave) { | 3040 | bond_for_each_slave_from(bond, slave, i, bond->current_arp_slave->next) { |
3041 | if (IS_UP(slave->dev)) { | 3041 | if (IS_UP(slave->dev)) { |
3042 | slave->link = BOND_LINK_BACK; | 3042 | slave->link = BOND_LINK_BACK; |
3043 | bond_set_slave_active_flags(slave); | 3043 | bond_set_slave_active_flags(slave); |