diff options
author | Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk> | 2016-05-07 09:57:18 +0100 |
---|---|---|
committer | John Crispin <john@phrozen.org> | 2016-05-12 03:29:36 +0200 |
commit | e288e1bd3210c2025f6fc0dd9c338ea9dbf04bca (patch) | |
tree | cab1c29f4f8b5925a07bf6709f5114a7baadc82b /target/linux/adm8668 | |
parent | 374cae9e6fcaf4cf4de4f2095f3fec558da51df6 (diff) | |
download | upstream-e288e1bd3210c2025f6fc0dd9c338ea9dbf04bca.tar.gz upstream-e288e1bd3210c2025f6fc0dd9c338ea9dbf04bca.tar.bz2 upstream-e288e1bd3210c2025f6fc0dd9c338ea9dbf04bca.zip |
kernel: fq_codel match flows_cnt to limit sizing
OpenWRT changed the default fq_codel sch->limit from 10240 to 1024,
without also adjusting q->flows_cnt. Eric Dumazet explains below that
you must also adjust the buckets (q->flows_cnt) for this not to break.
Eric explains: Limit of 1024 packets and 1024 flows is not wise I think.
(If all buckets are in use, each bucket has a virtual queue of 1 packet,
which is almost the same than having no queue at all)
I suggest to have at least 8 packets per bucket, to let Codel have a
chance to trigger. So you could either reduce number of buckets to 128
(if memory is tight), or increase limit to 8192.
flows_cnt is now set to 1024/8=128
Signed-off-by: Kevin Darbyshire-Bryant <kevin@darbyshire-bryant.me.uk>
Diffstat (limited to 'target/linux/adm8668')
0 files changed, 0 insertions, 0 deletions