diff options
author | Lukas Bulwahn <[email protected]> | 2020-10-28 12:35:33 +0100 |
---|---|---|
committer | Jakub Kicinski <[email protected]> | 2020-10-30 14:05:55 -0700 |
commit | 0ad41b244ca0ca07af2a652014bc929cbdf91e87 (patch) | |
tree | 3b73d9f7a6cc649812dd0b48925773ac6c6ca063 /tools/perf/scripts/python/sctop.py | |
parent | 400490ac2ce56774cc941e99d637400a2d042537 (diff) |
net: cls_api: remove unneeded local variable in tc_dump_chain()
make clang-analyzer on x86_64 defconfig caught my attention with:
net/sched/cls_api.c:2964:3: warning: Value stored to 'parent' is never read
[clang-analyzer-deadcode.DeadStores]
parent = 0;
^
net/sched/cls_api.c:2977:4: warning: Value stored to 'parent' is never read
[clang-analyzer-deadcode.DeadStores]
parent = q->handle;
^
Commit 32a4f5ecd738 ("net: sched: introduce chain object to uapi")
introduced tc_dump_chain() and this initial implementation already
contained these unneeded dead stores.
Simplify the code to make clang-analyzer happy.
As compilers will detect these unneeded assignments and optimize this
anyway, the resulting binary is identical before and after this change.
No functional change. No change in object code.
Signed-off-by: Lukas Bulwahn <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/sctop.py')
0 files changed, 0 insertions, 0 deletions