selftests/net: packetdrill: make tcp buf limited timing tests benign

kuba-moo · kuba-moo · commit 3030e3d57ba8 · 2025-01-16T17:23:22.000-08:00
The following tests are failing on debug kernels: tcp_tcp_info_tcp-info-rwnd-limited.pkt tcp_tcp_info_tcp-info-sndbuf-limited.pkt with reports like: assert 19000 <= tcpi_sndbuf_limited <= 21000, tcpi_sndbuf_limited; \ AssertionError: 18000 and: assert 348000 <= tcpi_busy_time <= 360000, tcpi_busy_time AssertionError: 362000 Extend commit 912d6f6 ("selftests/net: packetdrill: report benign debug flakes as xfail") to cover them. Reviewed-by: Willem de Bruijn <willemb@google.com> Link: https://patch.msgid.link/20250115232129.845884-1-kuba@kernel.org Signed-off-by: Jakub Kicinski <kuba@kernel.org>
diff --git a/tools/testing/selftests/net/packetdrill/ksft_runner.sh b/tools/testing/selftests/net/packetdrill/ksft_runner.sh
@@ -43,6 +43,7 @@ if [[ -n "${KSFT_MACHINE_SLOW}" ]]; then
 		"tcp_timestamping.*.pkt"
 		"tcp_user_timeout_user-timeout-probe.pkt"
 		"tcp_zerocopy_epoll_.*.pkt"
+		"tcp_tcp_info_tcp-info-*-limited.pkt"
 	)
 	readonly xfail_regex="^($(printf '%s|' "${xfail_list[@]}"))$"
 	[[ "$script" =~ ${xfail_regex} ]] && failfunc=ktap_test_xfail

Original file line number	Diff line number	Diff line change
`@@ -43,6 +43,7 @@ if [[ -n "${KSFT_MACHINE_SLOW}" ]]; then`
`43`	`43`	`"tcp_timestamping.*.pkt"`
`44`	`44`	`"tcp_user_timeout_user-timeout-probe.pkt"`
`45`	`45`	`"tcp_zerocopy_epoll_.*.pkt"`
	`46`	`+ "tcp_tcp_info_tcp-info-*-limited.pkt"`
`46`	`47`	`)`
`47`	`48`	`readonly xfail_regex="^($(printf '%s\|' "${xfail_list[@]}"))$"`
`48`	`49`	`[[ "$script" =~ ${xfail_regex} ]] && failfunc=ktap_test_xfail`