netdev
[Top] [All Lists]

Re: PATCH: bonding might sleep with lock held

To: netdev@xxxxxxxxxxx
Subject: Re: PATCH: bonding might sleep with lock held
From: Jay Vosburgh <fubar@xxxxxxxxxx>
Date: Thu, 13 May 2004 12:34:49 -0700
In-reply-to: Your message of "Thu, 13 May 2004 21:04:45 +0200." <20040513210445.3e08c1f4.ak@xxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
Andi Kleen <ak@xxxxxxx> wrote:
>On Thu, 13 May 2004 11:44:45 -0700
>Jay Vosburgh <fubar@xxxxxxxxxx> wrote:
>
>>      It will race against the timers, although I don't see anything
>> in there offhand that would conflict.  It just feels somehow unclean to
>> do it without the lock.
>
>Why should the timers care if there can be a packet received from the 
>network or not?  (this is all what dev_remove_pack prevents) 

        The concern I had was that the timers might modify the fields of
the bonding structure accessed outside the lock, but they don't.

        -J

---
        -Jay Vosburgh, IBM Linux Technology Center, fubar@xxxxxxxxxx

<Prev in Thread] Current Thread [Next in Thread>