| Message ID | 20250304233204.1139251-1-kuba@kernel.org (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | [net-next,1/2] selftests: net: fix error message in bpf_offload | expand |
On 03/04, Jakub Kicinski wrote: > We hit a following exception on timeout, nmaps is never set: > > Test bpftool bound info reporting (own ns)... > Traceback (most recent call last): > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 1128, in <module> > check_dev_info(False, "") > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 583, in check_dev_info > maps = bpftool_map_list_wait(expected=2, ns=ns) > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 215, in bpftool_map_list_wait > raise Exception("Time out waiting for map counts to stabilize want %d, have %d" % (expected, nmaps)) > NameError: name 'nmaps' is not defined > > Signed-off-by: Jakub Kicinski <kuba@kernel.org> Acked-by: Stanislav Fomichev <sdf@fomichev.me> I've seen this a couple of times myself but was too lazy to send out a fix :-[
Hello: This series was applied to netdev/net-next.git (main) by Jakub Kicinski <kuba@kernel.org>: On Tue, 4 Mar 2025 15:32:03 -0800 you wrote: > We hit a following exception on timeout, nmaps is never set: > > Test bpftool bound info reporting (own ns)... > Traceback (most recent call last): > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 1128, in <module> > check_dev_info(False, "") > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 583, in check_dev_info > maps = bpftool_map_list_wait(expected=2, ns=ns) > File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 215, in bpftool_map_list_wait > raise Exception("Time out waiting for map counts to stabilize want %d, have %d" % (expected, nmaps)) > NameError: name 'nmaps' is not defined > > [...] Here is the summary with links: - [net-next,1/2] selftests: net: fix error message in bpf_offload https://git.kernel.org/netdev/net-next/c/f9d2f5ddd47c - [net-next,2/2] selftests: net: bpf_offload: add 'libbpf_global' to ignored maps https://git.kernel.org/netdev/net-next/c/56a586961bf8 You are awesome, thank you!
diff --git a/tools/testing/selftests/net/bpf_offload.py b/tools/testing/selftests/net/bpf_offload.py index fd0d959914e4..4a9be8c49561 100755 --- a/tools/testing/selftests/net/bpf_offload.py +++ b/tools/testing/selftests/net/bpf_offload.py @@ -207,9 +207,11 @@ netns = [] # net namespaces to be removed raise Exception("Time out waiting for program counts to stabilize want %d, have %d" % (expected, nprogs)) def bpftool_map_list_wait(expected=0, n_retry=20, ns=""): + nmaps = None for i in range(n_retry): maps = bpftool_map_list(ns=ns) - if len(maps) == expected: + nmaps = len(maps) + if nmaps == expected: return maps time.sleep(0.05) raise Exception("Time out waiting for map counts to stabilize want %d, have %d" % (expected, nmaps))
We hit a following exception on timeout, nmaps is never set: Test bpftool bound info reporting (own ns)... Traceback (most recent call last): File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 1128, in <module> check_dev_info(False, "") File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 583, in check_dev_info maps = bpftool_map_list_wait(expected=2, ns=ns) File "/home/virtme/testing-1/tools/testing/selftests/net/./bpf_offload.py", line 215, in bpftool_map_list_wait raise Exception("Time out waiting for map counts to stabilize want %d, have %d" % (expected, nmaps)) NameError: name 'nmaps' is not defined Signed-off-by: Jakub Kicinski <kuba@kernel.org> --- CC: shuah@kernel.org CC: linux-kselftest@vger.kernel.org CC: bpf@vger.kernel.org --- tools/testing/selftests/net/bpf_offload.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)