Re: [PATCH] bonding using arp_ip_target may stay down with active path

[Jay Vosburgh <fubar@xxxxxxxxxx>]

David S. Miller <davem@xxxxxxxxxxxxx> wrote:
>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@xxxxxxxxxx


Signed-off-by: Jay Vosburgh <fubar@xxxxxxxxxx>

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);