diff options
author | Davide Caratti <dcaratti@redhat.com> | 2023-04-20 17:59:46 +0300 |
---|---|---|
committer | Jakub Kicinski <kuba@kernel.org> | 2023-04-22 06:24:29 +0300 |
commit | 7041101ff6c3073fd8f2e99920f535b111c929cb (patch) | |
tree | 1c1b6f7d442e35aa823ce1b53372f4f6472a16b1 /tools/perf/scripts/python/stackcollapse.py | |
parent | 63cfd210034c772fad047afa13dd5a4664b0a72e (diff) | |
download | linux-7041101ff6c3073fd8f2e99920f535b111c929cb.tar.xz |
net/sched: sch_fq: fix integer overflow of "credit"
if sch_fq is configured with "initial quantum" having values greater than
INT_MAX, the first assignment of "credit" does signed integer overflow to
a very negative value.
In this situation, the syzkaller script provided by Cristoph triggers the
CPU soft-lockup warning even with few sockets. It's not an infinite loop,
but "credit" wasn't probably meant to be minus 2Gb for each new flow.
Capping "initial quantum" to INT_MAX proved to fix the issue.
v2: validation of "initial quantum" is done in fq_policy, instead of open
coding in fq_change() _ suggested by Jakub Kicinski
Reported-by: Christoph Paasch <cpaasch@apple.com>
Link: https://github.com/multipath-tcp/mptcp_net-next/issues/377
Fixes: afe4fd062416 ("pkt_sched: fq: Fair Queue packet scheduler")
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: Davide Caratti <dcaratti@redhat.com>
Link: https://lore.kernel.org/r/7b3a3c7e36d03068707a021760a194a8eb5ad41a.1682002300.git.dcaratti@redhat.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python/stackcollapse.py')
0 files changed, 0 insertions, 0 deletions