selftests: netconsole: print diagnostic on busywait timeout in netcons_basicThe script uses set -euo pipefail, so when busywait times out waitingfor the netconsole message to arrive, it returns 1
selftests: netconsole: print diagnostic on busywait timeout in netcons_basicThe script uses set -euo pipefail, so when busywait times out waitingfor the netconsole message to arrive, it returns 1 and the script exitsimmediately without printing any error message. As reported by Jakub,this makes failures hard to diagnose since the test reports exit=1 withno explanation.Handle the busywait failure explicitly so that a FAIL message is printedbefore 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_QdMayThe remaining silent failures under set -e can only happen during thesetup phase (netdevsim creation, interface configuration, configfswrites). So, it is not expected to have any silent failure once the teststarts.Note that this issue might be less frequent now, since commita68a9bd086c28 ("selftests: netconsole: Increase port listening timeout")increased the timeout that _might_ have been the root cause of theserandom failures in NIPA.Signed-off-by: Breno Leitao <leitao@debian.org>Link: https://patch.msgid.link/20260302-netconsole_test_verbose-v1-1-b1be5d30cd7d@debian.orgSigned-off-by: Jakub Kicinski <kuba@kernel.org>
show more ...
netconsole: selftests: Move netconsole selftests to separate targetThis patch moves netconsole selftests from drivers/net to its own targetin drivers/net/netconsole.This change helps saving some
netconsole: selftests: Move netconsole selftests to separate targetThis patch moves netconsole selftests from drivers/net to its own targetin drivers/net/netconsole.This change helps saving some resources from CI since tests indrivers/net automatically run against real hardware which are not usedby netconsole tests as they rely solely on netdevsim.lib_netcons.sh is kept under drivers/net/lib since it is also used bybonding selftests. Finally, drivers/net config remains unchanged asnetpoll_basic.py requires netconsole (and does leverage real HW testing).Reviewed-by: Breno Leitao <leitao@debian.org>Signed-off-by: Andre Carvalho <asantostc@gmail.com>Link: https://patch.msgid.link/20260127-netcons-selftest-target-v2-1-f509ab65b3bc@gmail.comSigned-off-by: Jakub Kicinski <kuba@kernel.org>