diff options
author | Eric Dumazet <edumazet@google.com> | 2014-11-25 11:57:29 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2014-11-26 12:08:04 -0500 |
commit | ced7a04e394fe1df41d13b833baab5cfcb1d03b3 (patch) | |
tree | d7b4cd32ef3b882498b3578ee14ed73db8ba231d /net/sched | |
parent | d3fc6b3fddd54c2220a075aefc4e5e5ca25cff34 (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>
Diffstat (limited to 'net/sched')
-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 cbd7e1fd23b4..9b05924cc386 100644 --- a/net/sched/sch_fq.c +++ b/net/sched/sch_fq.c | |||
@@ -481,12 +481,11 @@ begin: | |||
481 | if (likely(rate)) | 481 | if (likely(rate)) |
482 | do_div(len, rate); | 482 | do_div(len, rate); |
483 | /* Since socket rate can change later, | 483 | /* Since socket rate can change later, |
484 | * clamp the delay to 125 ms. | 484 | * clamp the delay to 1 second. |
485 | * TODO: maybe segment the too big skb, as in commit | 485 | * Really, providers of too big packets should be fixed ! |
486 | * e43ac79a4bc ("sch_tbf: segment too big GSO packets") | ||
487 | */ | 486 | */ |
488 | if (unlikely(len > 125 * NSEC_PER_MSEC)) { | 487 | if (unlikely(len > NSEC_PER_SEC)) { |
489 | len = 125 * NSEC_PER_MSEC; | 488 | len = NSEC_PER_SEC; |
490 | q->stat_pkts_too_long++; | 489 | q->stat_pkts_too_long++; |
491 | } | 490 | } |
492 | 491 | ||