Re: [PATCH] bonding using arp_ip_target may stay down with active path
From: Eric Paris <hidden>
Date: 2005-05-24 18:31:02
Possibly related (same subject, not in this thread)
- 2005-05-23 · Re: [PATCH] bonding using arp_ip_target may stay down with active path · "David S. Miller" <davem@davemloft.net>
- 2005-05-16 · Re: [PATCH] bonding using arp_ip_target may stay down with active path · Jay Vosburgh <hidden>
- 2005-05-16 · [PATCH] bonding using arp_ip_target may stay down with active path · Eric Paris <hidden>
It ate my tabs again. This mailer is going out the window. -Eric On Tue, 2005-05-24 at 14:26 -0400, Eric Paris wrote:
quoted hunk ↗ jump to hunk
quoted
On Mon, 2005-05-23 at 14:21 -0700, Jay Vosburgh wrote:David S. Miller [off-list ref] wrote:quoted
Patch doesn't apply, tabs turned into spaces by your email client. As penance for test applying trivial patches by hand, here's a proper version. -J --- -Jay Vosburgh, IBM Linux Technology Center, fubar@us.ibm.comI guess I should be in the signed off list as the original author. sorry I'm an idiot and copied and pasted my original message (to the bonding-devel list) which screwed up the spacing. Signed-off-by: Eric Paris <redacted> Signed-off-by: Jay Vosburgh <redacted> diff -urN linux-2.6.11/drivers/net/bonding/bond_main.c linux-2.6.11-fix/drivers/net/bonding/bond_main.c--- linux-2.6.11/drivers/net/bonding/bond_main.c 2005-05-23 14:07:37.000000000 -0700 +++ linux-2.6.11-fix/drivers/net/bonding/bond_main.c 2005-05-23 14:08:13.000000000 -0700@@ -3046,7 +3046,7 @@ bond_set_slave_inactive_flags(bond->current_arp_slave); /* search for next candidate */ - bond_for_each_slave_from(bond, slave, i, bond->current_arp_slave) { + bond_for_each_slave_from(bond, slave, i, bond->current_arp_slave->next) { if (IS_UP(slave->dev)) { slave->link = BOND_LINK_BACK; bond_set_slave_active_flags(slave);