]> git.baikalelectronics.ru Git - kernel.git/commit
net/sched: fq_pie: re-factor fix for fq_pie endless loop
authorDavide Caratti <dcaratti@redhat.com>
Sat, 22 May 2021 13:14:45 +0000 (15:14 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 24 May 2021 00:16:09 +0000 (17:16 -0700)
commit74aae9fcbcba940ecc7c56b85a32dfb6b7f464e4
tree3e6a1db4a98b22e8bc060ddf280b895a2ed10a8e
parentaaec97800a49a9e583320f763640684086de8119
net/sched: fq_pie: re-factor fix for fq_pie endless loop

the patch that fixed an endless loop in_fq_pie_init() was not considering
that 65535 is a valid class id. The correct bugfix for this infinite loop
is to change 'idx' to become an u32, like Colin proposed in the past [1].

Fix this as follows:
 - restore 65536 as maximum possible values of 'flows_cnt'
 - use u32 'idx' when iterating on 'q->flows'
 - fix the TDC selftest

This reverts commit d17f5f1e9ca02a151505b6292f1f5cab8e11bf65.

[1] https://lore.kernel.org/netdev/20210407163808.499027-1-colin.king@canonical.com/

CC: Colin Ian King <colin.king@canonical.com>
CC: stable@vger.kernel.org
Fixes: d17f5f1e9ca0 ("net/sched: fix infinite loop in sch_fq_pie")
Fixes: 614b84da5c15 ("net: sched: add Flow Queue PIE packet scheduler")
Signed-off-by: Davide Caratti <dcaratti@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/sch_fq_pie.c
tools/testing/selftests/tc-testing/tc-tests/qdiscs/fq_pie.json