diff options
| author | Eric Dumazet <edumazet@google.com> | 2014-11-25 08:57:29 -0800 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2016-09-18 12:47:04 +0200 |
| commit | 93c0237e2bbe6e8c48c9411be54adcdf33dd1489 (patch) | |
| tree | 71ac21856ba424cfc0ce76e37fd8503863735937 | |
| parent | cee5735b7ae6b2d42be45eed21f2ce7c13295205 (diff) | |
pkt_sched: fq: increase max delay from 125 ms to one second
FQ/pacing has a clamp of delay of 125 ms, to avoid some possible harm.
It turns out this delay is too small to allow pacing low rates :
Some ISP setup very aggressive policers as low as 16kbit.
Now TCP stack has spurious rtx prevention, it seems safe to increase
this fixed parameter, without adding a qdisc attribute.
Signed-off-by: Eric Dumazet <edumazet@google.com>
Cc: Yang Yingliang <yangyingliang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | net/sched/sch_fq.c | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/net/sched/sch_fq.c b/net/sched/sch_fq.c index f81e480c6..5e592a51d 100644 --- a/net/sched/sch_fq.c +++ b/net/sched/sch_fq.c @@ -482,12 +482,11 @@ begin: if (likely(rate)) do_div(len, rate); /* Since socket rate can change later, - * clamp the delay to 125 ms. - * TODO: maybe segment the too big skb, as in commit - * e43ac79a4bc ("sch_tbf: segment too big GSO packets") + * clamp the delay to 1 second. + * Really, providers of too big packets should be fixed ! */ - if (unlikely(len > 125 * NSEC_PER_MSEC)) { - len = 125 * NSEC_PER_MSEC; + if (unlikely(len > NSEC_PER_SEC)) { + len = NSEC_PER_SEC; q->stat_pkts_too_long++; } |