Re: [PATCH ver2] Avoid enqueuing skb for default qdiscs

[Maxime Bizon <mbizon@freebox.fr>]

On Mon, 2010-02-01 at 19:12 +0530, Krishna Kumar2 wrote:

Hi,

> sch_direct_xmit can be called from dev_queue_xmit for a
> stopped device only if the device had xmit the previous
> skb, stopped the device and returned OK. Then the next

Yes, that's what happen in my case.

> BTW, I don't think this patch would change the earlier
> behavior. The old code would have done the same thing.

Oh, my mistake then. I always thought the requeue counter was indicating
the driver returned TX_BUSY (and failed to stop the queue before), which
is not polite.

If the current behavior is the expected one, then there is no problem
here.

> As I explained, this should happen only once per stop event.
> Could you tell which driver is having this problem? Is it
> waking up too early, eg, it might be stopping when the hw tx
> descriptor is full but waking up when a few slots open up,
> and those will get filled up immediately on fast systems.
> Then you will see a lot of requeue's.

This is an atm driver which is not mainlined. The system is not that
fast, but the link is (very) slow (< 1 Mbit/s).

The driver is waking up the queue once the hardware queue is not full
anymore (at least one free tx desc), and the free slot gets filled
immediately since the link is slow.

Thanks for explaining !

-- 
Maxime