Andrew Morton wrote:
> A number of drivers do this:
> if (room for another packet)
Which ones? They need fixing..
> For devices which have a Tx packet ring or decent FIFO I don't expect
> this to be a problem, because the Tx ISR will call netif_wake_queue()
> and the subsequent BH run will keep stuffing packets into the Tx ring
> until it's full. But for devices which have very limited Tx buffering
> there may be a lost opportunity to refill the Tx buffer earlier. Seems
> unlikely to me.
If you do something like that, it seems like it should be dependent on
certain thresholds, not necessarily occurring all the time. And you'd
want to sync with the Tx reaper called from the interrupt handler, too.
Jeff Garzik | Liberty is always dangerous, but
Building 1024 | it is the safest thing we have.
MandrakeSoft, Inc. | -- Harry Emerson Fosdick