Re: Request for net guru help: waitqueue oops

From: Hans Grobler (grobh@sun.ac.za)
Date: Tue Oct 03 2000 - 10:37:58 EST


On Tue, 3 Oct 2000, Petko Manolov wrote:
> None of these can sleep. netif_*_queue routines are quite simple.
> They are all atomic so there is no need to protect them with locks.

Ok. I originally had them outside locks as they appeared to be atomic. I
moved them in incase they were the cause of the problem.

> It is not clear from the example above if it is needed to lock in
> the timer routine and what is locked inside. Anyway be careful
> about locking regions shared between interrupts/bottom halves and
> user context as this happens often.

The timer routines (there are 4) are used to switch hardware states and
must therefore be mutually exclusive with respect to the interrupt handler.
There are no bottom halves used in this driver. Andrew Morton suggested
that the problem could be in my use of the skb pointers, which seems
a likely candidate. I'll check that.

Thanks
-- Hans

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sat Oct 07 2000 - 21:00:12 EST