Recent change to add more cases to XFAIL has a broken regex,
the matching needs a real regex not a glob pattern.

While at it add the cases Willem pointed out during review.

Fixes: 3030e3d57ba8 ("selftests/net: packetdrill: make tcp buf limited timing 
tests benign")
Signed-off-by: Jakub Kicinski <k...@kernel.org>
---
CC: sh...@kernel.org
CC: will...@google.com
CC: matt...@kernel.org
CC: linux-kselftest@vger.kernel.org
---
 tools/testing/selftests/net/packetdrill/ksft_runner.sh | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tools/testing/selftests/net/packetdrill/ksft_runner.sh 
b/tools/testing/selftests/net/packetdrill/ksft_runner.sh
index e15c43b7359b..ef8b25a606d8 100755
--- a/tools/testing/selftests/net/packetdrill/ksft_runner.sh
+++ b/tools/testing/selftests/net/packetdrill/ksft_runner.sh
@@ -39,11 +39,13 @@ if [[ -n "${KSFT_MACHINE_SLOW}" ]]; then
        # xfail tests that are known flaky with dbg config, not fixable.
        # still run them for coverage (and expect 100% pass without dbg).
        declare -ar xfail_list=(
+               "tcp_eor_no-coalesce-retrans.pkt"
                "tcp_fast_recovery_prr-ss.*.pkt"
+               "tcp_slow_start_slow-start-after-win-update.pkt"
                "tcp_timestamping.*.pkt"
                "tcp_user_timeout_user-timeout-probe.pkt"
                "tcp_zerocopy_epoll_.*.pkt"
-               "tcp_tcp_info_tcp-info-*-limited.pkt"
+               "tcp_tcp_info_tcp-info-.*-limited.pkt"
        )
        readonly xfail_regex="^($(printf '%s|' "${xfail_list[@]}"))$"
        [[ "$script" =~ ${xfail_regex} ]] && failfunc=ktap_test_xfail
-- 
2.48.1


Reply via email to