]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
selftests: netconsole: print diagnostic on busywait timeout in netcons_basic
authorBreno Leitao <leitao@debian.org>
Mon, 2 Mar 2026 14:40:39 +0000 (06:40 -0800)
committerJakub Kicinski <kuba@kernel.org>
Wed, 4 Mar 2026 01:22:54 +0000 (17:22 -0800)
commitdfa77c0dd4ab267d3f1160617c95eab80181a76f
tree57921fae06e151831fceef377ea54dae7d078856
parent1085c258d88493f920789e1215b31f32031942ac
selftests: netconsole: print diagnostic on busywait timeout in netcons_basic

The script uses set -euo pipefail, so when busywait times out waiting
for the netconsole message to arrive, it returns 1 and the script exits
immediately without printing any error message. As reported by Jakub,
this makes failures hard to diagnose since the test reports exit=1 with
no explanation.

Handle the busywait failure explicitly so that a FAIL message is printed
before exiting. This is how it looks like now:

Running with target mode: basic (ipv6)
[  167.452561] netconsole selftest: netcons_QdMay
FAIL: Timed out waiting (20000 ms) for netconsole message in /tmp/netcons_QdMay

The remaining silent failures under set -e can only happen during the
setup phase (netdevsim creation, interface configuration, configfs
writes). So, it is not expected to have any silent failure once the test
starts.

Note that this issue might be less frequent now, since commit
a68a9bd086c28 ("selftests: netconsole: Increase port listening timeout")
increased the timeout that _might_ have been the root cause of these
random failures in NIPA.

Signed-off-by: Breno Leitao <leitao@debian.org>
Link: https://patch.msgid.link/20260302-netconsole_test_verbose-v1-1-b1be5d30cd7d@debian.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/drivers/net/netconsole/netcons_basic.sh