selftests: mptcp: lib: remove stats files args
authorMatthieu Baerts (NGI0) <matttbe@kernel.org>
Fri, 14 Nov 2025 18:12:06 +0000 (19:12 +0100)
committerJakub Kicinski <kuba@kernel.org>
Tue, 18 Nov 2025 03:27:46 +0000 (19:27 -0800)
Now that these files are written from MPTCP lib helpers, the stats file
paths are uniformed. Then, no need to specify them from the each
selftest.

No behavioural changes intended.

Acked-by: Paolo Abeni <pabeni@redhat.com>
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Link: https://patch.msgid.link/20251114-net-next-mptcp-sft-count-cache-stats-timeout-v1-2-863cb04e1b7b@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
tools/testing/selftests/net/mptcp/mptcp_connect.sh
tools/testing/selftests/net/mptcp/mptcp_join.sh
tools/testing/selftests/net/mptcp/mptcp_lib.sh
tools/testing/selftests/net/mptcp/mptcp_sockopt.sh
tools/testing/selftests/net/mptcp/simult_flows.sh

index e1f3450..4ff2d02 100755 (executable)
@@ -444,8 +444,7 @@ do_transfer()
        printf "(duration %05sms) " "${duration}"
        if [ ${rets} -ne 0 ] || [ ${retc} -ne 0 ]; then
                mptcp_lib_pr_fail "client exit code $retc, server $rets"
-               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}" \
-                       "/tmp/${listener_ns}.out" "/tmp/${connector_ns}.out"
+               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}"
 
                echo
                cat "$capout"
index f51c9bf..aee215d 100755 (executable)
@@ -1071,8 +1071,7 @@ do_transfer()
 
        if [ ${rets} -ne 0 ] || [ ${retc} -ne 0 ]; then
                fail_test "client exit code $retc, server $rets"
-               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}" \
-                       "/tmp/${listener_ns}.out" "/tmp/${connector_ns}.out"
+               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}"
                return 1
        fi
 
index 1f31746..4a26d41 100644 (file)
@@ -106,23 +106,28 @@ mptcp_lib_pr_info() {
        mptcp_lib_print_info "INFO: ${*}"
 }
 
-# $1-2: listener/connector ns ; $3 port ; $4-5 listener/connector stat file
+mptcp_lib_pr_nstat() {
+       local ns="${1}"
+       local hist="/tmp/${ns}.out"
+
+       cat "${hist}"
+}
+
+# $1-2: listener/connector ns ; $3 port
 mptcp_lib_pr_err_stats() {
        local lns="${1}"
        local cns="${2}"
        local port="${3}"
-       local lstat="${4}"
-       local cstat="${5}"
 
        echo -en "${MPTCP_LIB_COLOR_RED}"
        {
                printf "\nnetns %s (listener) socket stat for %d:\n" "${lns}" "${port}"
                ip netns exec "${lns}" ss -Menitam -o "sport = :${port}"
-               cat "${lstat}"
+               mptcp_lib_pr_nstat "${lns}"
 
                printf "\nnetns %s (connector) socket stat for %d:\n" "${cns}" "${port}"
                ip netns exec "${cns}" ss -Menitam -o "dport = :${port}"
-               [ "${lstat}" != "${cstat}" ] && cat "${cstat}"
+               [ "${lns}" != "${cns}" ] && mptcp_lib_pr_nstat "${cns}"
        } 1>&2
        echo -en "${MPTCP_LIB_COLOR_RESET}"
 }
index 3436159..42d533b 100755 (executable)
@@ -198,8 +198,7 @@ do_transfer()
        print_title "Transfer ${ip:2}"
        if [ ${rets} -ne 0 ] || [ ${retc} -ne 0 ]; then
                mptcp_lib_pr_fail "client exit code $retc, server $rets"
-               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}" \
-                       "/tmp/${listener_ns}.out" "/tmp/${connector_ns}.out"
+               mptcp_lib_pr_err_stats "${listener_ns}" "${connector_ns}" "${port}"
 
                mptcp_lib_result_fail "transfer ${ip}"
 
index fbd1685..503cb59 100755 (executable)
@@ -200,8 +200,7 @@ do_transfer()
        fi
 
        mptcp_lib_pr_fail "client exit code $retc, server $rets"
-       mptcp_lib_pr_err_stats "${ns3}" "${ns1}" "${port}" \
-               "/tmp/${ns3}.out" "/tmp/${ns1}.out"
+       mptcp_lib_pr_err_stats "${ns3}" "${ns1}" "${port}"
        ls -l $sin $cout
        ls -l $cin $sout