Hello, this is the v3 of the series aiming to reduce the flakyness of tc_tunnel/tc_edt tests in CI. This revision handles Ihor's feedback about the need to conform to the timeout value passed in opts->timeout. The first commit has been reorganized a bit more than what was needed for this change, to make sure that the code can be followed easily. There's also now a second commit to make sure that test_tc_edt benefits from this new connect() handling.
Signed-off-by: Alexis Lothoré (eBPF Foundation) <[email protected]> --- Changes in v3: - set errno before logging errors - respect time budget set by - respect opts->timeout_ms when polling: only poll for the remaining time not already consume by connect() - keep polling if poll returns with EINTR - reorder early returns and add intermediate variables to clarify code flow - Link to v2: https://patch.msgid.link/[email protected] Changes in v2: - drop unneeded initialization - add back error message for immediate connection failure, and slightly reword the async connection failure error message - Link to v1: https://patch.msgid.link/[email protected] To: Alexei Starovoitov <[email protected]> To: Daniel Borkmann <[email protected]> To: Andrii Nakryiko <[email protected]> To: Eduard Zingerman <[email protected]> To: Kumar Kartikeya Dwivedi <[email protected]> To: Martin KaFai Lau <[email protected]> To: Song Liu <[email protected]> To: Yonghong Song <[email protected]> To: Jiri Olsa <[email protected]> To: Emil Tsalapatis <[email protected]> To: Ihor Solodrai <[email protected]> To: Shuah Khan <[email protected]> Cc: [email protected] Cc: Bastien Curutchet <[email protected]> Cc: Thomas Petazzoni <[email protected]> Cc: [email protected] Cc: [email protected] Cc: [email protected] --- Alexis Lothoré (eBPF Foundation) (2): selftests/bpf: keep polling connection that is still in progress selftests/bpf: add connect timeout to test_tc_edt tools/testing/selftests/bpf/network_helpers.c | 70 ++++++++++++++++++++-- .../testing/selftests/bpf/prog_tests/test_tc_edt.c | 2 +- tools/testing/selftests/bpf/testing_helpers.h | 9 +++ 3 files changed, 75 insertions(+), 6 deletions(-) --- base-commit: 53cc65185a51fa9dd288cd1137a21467eced2df4 change-id: 20260710-tc_tunnel_flaky-27e9a191bd03 Best regards, -- Alexis Lothoré (eBPF Foundation) <[email protected]>
