Re: [PATCH][NET_SCHED] sch_sfq: prevent unnecessary reordering

[Andy Furniss <lists@andyfurniss.entadsl.com>]

Jarek Poplawski wrote:
> On 27-04-2008 16:22, Jarek Poplawski wrote:
>> [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.