selftests: mptcp: add ms units for tc-netem delay
authorGeliang Tang <tanggeliang@kylinos.cn>
Fri, 5 Apr 2024 10:52:06 +0000 (12:52 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 8 Apr 2024 10:53:20 +0000 (11:53 +0100)
'delay 1' in tc-netem is confusing, not sure if it's a delay of 1 second or
1 millisecond. This patch explicitly adds millisecond units to make these
commands clearer.

Signed-off-by: Geliang Tang <tanggeliang@kylinos.cn>
Reviewed-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/mptcp/mptcp_join.sh
tools/testing/selftests/net/mptcp/simult_flows.sh

index 5a95798eb40a1f47643828eaa71226e1632d8cdd..73a2131e6da2eaae0f99973e02213b26399d0c81 100755 (executable)
@@ -125,8 +125,8 @@ init_shapers()
 {
        local i
        for i in $(seq 1 4); do
-               tc -n $ns1 qdisc add dev ns1eth$i root netem rate 20mbit delay 1
-               tc -n $ns2 qdisc add dev ns2eth$i root netem rate 20mbit delay 1
+               tc -n $ns1 qdisc add dev ns1eth$i root netem rate 20mbit delay 1ms
+               tc -n $ns2 qdisc add dev ns2eth$i root netem rate 20mbit delay 1ms
        done
 }
 
@@ -3212,7 +3212,7 @@ fail_tests()
 
        # multiple subflows
        if reset_with_fail "MP_FAIL MP_RST" 2; then
-               tc -n $ns2 qdisc add dev ns2eth1 root netem rate 1mbit delay 5
+               tc -n $ns2 qdisc add dev ns2eth1 root netem rate 1mbit delay 5ms
                pm_nl_set_limits $ns1 0 1
                pm_nl_set_limits $ns2 0 1
                pm_nl_add_endpoint $ns2 10.0.2.2 dev ns2eth2 flags subflow
index 497141c49ccd4f1e710f40ff4bd6901356a806af..4e6d8fc56b380083fdef5ad1594725e6f738981d 100755 (executable)
@@ -216,8 +216,8 @@ run_test()
        shift 4
        local msg=$*
 
-       [ $delay1 -gt 0 ] && delay1="delay $delay1" || delay1=""
-       [ $delay2 -gt 0 ] && delay2="delay $delay2" || delay2=""
+       [ $delay1 -gt 0 ] && delay1="delay ${delay1}ms" || delay1=""
+       [ $delay2 -gt 0 ] && delay2="delay ${delay2}ms" || delay2=""
 
        for dev in ns1eth1 ns1eth2; do
                tc -n $ns1 qdisc del dev $dev root >/dev/null 2>&1