'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>
 {
        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
 }
 
 
        # 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
 
        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