diff options
| author | Hangbin Liu <liuhangbin@gmail.com> | 2021-05-28 05:43:56 +0300 |
|---|---|---|
| committer | Daniel Borkmann <daniel@iogearbox.net> | 2021-05-28 23:14:18 +0300 |
| commit | e8e0f0f484780d7b90a63ea50020ac4bb027178d (patch) | |
| tree | 6c5c94ea17f63ff9a1b5738d042b0103ec34d19a /kernel/bpf/tnum.c | |
| parent | fc8c262e0eb5aa248af5051377e4ff3348841ac5 (diff) | |
| download | linux-e8e0f0f484780d7b90a63ea50020ac4bb027178d.tar.xz | |
bpf, devmap: Remove drops variable from bq_xmit_all()
As Colin pointed out, the first drops assignment after declaration will
be overwritten by the second drops assignment before using, which makes
it useless.
Since the drops variable will be used only once. Just remove it and
use "cnt - sent" in trace_xdp_devmap_xmit().
Fixes: cb261b594b41 ("bpf: Run devmap xdp_prog on flush instead of bulk enqueue")
Reported-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Hangbin Liu <liuhangbin@gmail.com>
Signed-off-by: Daniel Borkmann <daniel@iogearbox.net>
Acked-by: John Fastabend <john.fastabend@gmail.com>
Link: https://lore.kernel.org/bpf/20210528024356.24333-1-liuhangbin@gmail.com
Diffstat (limited to 'kernel/bpf/tnum.c')
0 files changed, 0 insertions, 0 deletions