Commit f3b05bb8 authored by Petr Machata's avatar Petr Machata Committed by David S. Miller

selftests: mlxsw: Update ping limits

The current ping intervals are too short for running mirroring tests in
simulator. This leads to ping sending a follow-up ping before the reply
arrives, thus sending more than the requested 10 ICMP requests. This
traffic is seen at the counters, and causes spurious failures.

Bump interval and timeout numbers 5x in mirroring tests to address the
spurious failures.
Signed-off-by: default avatarPetr Machata <petrm@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 0175cb59
...@@ -148,9 +148,10 @@ dscp_ping_test() ...@@ -148,9 +148,10 @@ dscp_ping_test()
eval "t0s=($(dscp_fetch_stats $dev_10 10) eval "t0s=($(dscp_fetch_stats $dev_10 10)
$(dscp_fetch_stats $dev_20 20))" $(dscp_fetch_stats $dev_20 20))"
local ping_timeout=$((PING_TIMEOUT * 5))
ip vrf exec $vrf_name \ ip vrf exec $vrf_name \
${PING} -Q $dscp_10 ${sip:+-I $sip} $dip \ ${PING} -Q $dscp_10 ${sip:+-I $sip} $dip \
-c 10 -i 0.1 -w $PING_TIMEOUT &> /dev/null -c 10 -i 0.5 -w $ping_timeout &> /dev/null
local -A t1s local -A t1s
eval "t1s=($(dscp_fetch_stats $dev_10 10) eval "t1s=($(dscp_fetch_stats $dev_10 10)
......
...@@ -169,9 +169,10 @@ dscp_ping_test() ...@@ -169,9 +169,10 @@ dscp_ping_test()
eval "local -A dev1_t0s=($(dscp_fetch_stats $dev1 0))" eval "local -A dev1_t0s=($(dscp_fetch_stats $dev1 0))"
eval "local -A dev2_t0s=($(dscp_fetch_stats $dev2 0))" eval "local -A dev2_t0s=($(dscp_fetch_stats $dev2 0))"
local ping_timeout=$((PING_TIMEOUT * 5))
ip vrf exec $vrf_name \ ip vrf exec $vrf_name \
${PING} -Q $dscp ${sip:+-I $sip} $dip \ ${PING} -Q $dscp ${sip:+-I $sip} $dip \
-c 10 -i 0.1 -w $PING_TIMEOUT &> /dev/null -c 10 -i 0.5 -w $ping_timeout &> /dev/null
eval "local -A dev1_t1s=($(dscp_fetch_stats $dev1 0))" eval "local -A dev1_t1s=($(dscp_fetch_stats $dev1 0))"
eval "local -A dev2_t1s=($(dscp_fetch_stats $dev2 0))" eval "local -A dev2_t1s=($(dscp_fetch_stats $dev2 0))"
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment