Jarek Poplawski wrote:

On 27-04-2008 16:22, Jarek Poplawski wrote:

quoted

[NET_SCHED] sch_sfq: fix queue limiting while enqueuing

David, as Patrick noticed this patch was wrong and I withdraw it.

I still can't see why sfq should have such a low limit,

It's a useful param which I use - on slow egress I can't see the point 
of queuing many seconds worth of traffic. It may also help keep stronger 
connections in check.

When shaping ingress it's essential to drop early to get out of 
slowstart before the far buffer gets too full.

[NET_SCHED] sch_sfq: prevent unnecessary reordering

Current check of queue limit in sfq_enqueue() isn't optimal: there
is really not much more needed to prevent unnecessary dropping and
possible reordering.

Maybe it makes no difference in the long run, but if you drop and then 
enqueue a later packet the sender will get to know sooner so congestion 
control can do it's stuff.

Andy.