diff options
author | Eric Dumazet <eric.dumazet@gmail.com> | 2010-11-18 09:33:19 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-11-18 09:33:19 -0800 |
commit | 866f3b25a2eb60d7529c227a0ecd80c3aba443fd (patch) | |
tree | b11b24cf158a232641548b6c652bf9e001ae1446 /drivers/net/bonding | |
parent | dda0b38692a7298f433b92b1329867b1ecabb4bb (diff) | |
download | op-kernel-dev-866f3b25a2eb60d7529c227a0ecd80c3aba443fd.zip op-kernel-dev-866f3b25a2eb60d7529c227a0ecd80c3aba443fd.tar.gz |
bonding: IGMP handling cleanup
Instead of iterating in_dev->mc_list from bonding driver, its better
to call a helper function provided by igmp.c
Details of implementation (locking) are private to igmp code.
ip_mc_rejoin_group(struct ip_mc_list *im) becomes
ip_mc_rejoin_groups(struct in_device *in_dev);
Signed-off-by: Eric Dumazet <eric.dumazet@gmail.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 | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/net/bonding/bond_main.c b/drivers/net/bonding/bond_main.c index 5188448..e588b2e 100644 --- a/drivers/net/bonding/bond_main.c +++ b/drivers/net/bonding/bond_main.c @@ -873,15 +873,11 @@ static void bond_mc_del(struct bonding *bond, void *addr) static void __bond_resend_igmp_join_requests(struct net_device *dev) { struct in_device *in_dev; - struct ip_mc_list *im; rcu_read_lock(); in_dev = __in_dev_get_rcu(dev); - if (in_dev) { - for (im = in_dev->mc_list; im; im = im->next) - ip_mc_rejoin_group(im); - } - + if (in_dev) + ip_mc_rejoin_groups(in_dev); rcu_read_unlock(); } |