Subject: Re: [patch net-2.6] bonding: fix rx_handler locking

From: Nicolas de Pesloüan<[email-masked]>
Date: Tue, 22 Mar 2011 21:35:37 +0100

Le 22/03/2011 21:26, Andy Gospodarek a écrit :
> On Tue, Mar 22, 2011 at 01:38:12PM +0100, Jiri Pirko wrote:
>> This prevents possible race between bond_enslave and bond_handle_frame
>> as reported by Nicolas by moving rx_handler register/unregister.
>> slave->bond is added to hold pointer to master bonding sructure. That
>> way dev->master is no longer used in bond_handler_frame.
>> Also, this removes "BUG: scheduling while atomic" message
>>
>> Reported-by: Nicolas de Pesloüan<[email-masked]>
>> Signed-off-by: Jiri Pirko<[email-masked]>
>
> Signed-off-by: Andy Gospodarek<[email-masked]>
>
> This seems reasonable.  I presume Nicolas was able to test it and verify
> it resolved his issue?

Unfortunately, not yet. I plan to give it a try tonight... or tomorrow on the worst case.

	Nicolas.