Subject: Re: [PATCH net-next-2.6] bonding: IGMP handling cleanup

From: David Miller <[email-masked]>
Date: Thu, 18 Nov 2010 09:33:30 -0800 (PST)

From: Eric Dumazet <[email-masked]>
Date: Thu, 18 Nov 2010 11:49:12 +0100

> Le jeudi 18 novembre 2010 à 11:26 +0100, Eric Dumazet a écrit :
> 
>> Actually this raises an interesting case for bonding as well.
>> 
>> Before my RCU conversion __bond_resend_igmp_join_requests() was unsafe.
>> 
>> For net-next-2.6, it is now safe (RCU is held), but needs a cleanup
>> patch to avoid sparse errors.
> 
> [PATCH net-next-2.6] 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 <[email-masked]>
> Cc: Jay Vosburgh <[email-masked]>
> Cc: Sachin Sant <[email-masked]>

Applied, thanks Eric.