Add delays and drop 'retry' in the multiple subflows test.
Signed-off-by: Geliang Tang <geliang.tang@suse.com>
---
.../testing/selftests/net/mptcp/mptcp_join.sh | 25 ++++++++-----------
1 file changed, 11 insertions(+), 14 deletions(-)
diff --git a/tools/testing/selftests/net/mptcp/mptcp_join.sh b/tools/testing/selftests/net/mptcp/mptcp_join.sh
index dbc73e28cc06..d44508e5e1c3 100755
--- a/tools/testing/selftests/net/mptcp/mptcp_join.sh
+++ b/tools/testing/selftests/net/mptcp/mptcp_join.sh
@@ -219,6 +219,11 @@ reset_with_fail()
action pedit munge offset 148 u8 invert \
pipe csum tcp \
index 100 || exit 1
+
+ if [ $i -eq 2 ]; then
+ tc -n $ns2 qdisc add dev ns2eth1 root netem rate 20mbit delay 1
+ tc -n $ns2 qdisc add dev ns2eth3 root netem rate 20mbit delay 1
+ fi
}
ip -Version > /dev/null 2>&1
@@ -2291,20 +2296,12 @@ pedit_action_pkts()
fail_tests()
{
# multiple subflows
- # If the tc rules didn't produce the bit flips, retry three times.
- for n in `seq 1 3`; do
- reset_with_fail 2
- pm_nl_set_limits $ns1 0 2
- pm_nl_set_limits $ns2 0 2
- pm_nl_add_endpoint $ns2 10.0.2.2 dev ns2eth2 flags subflow
- pm_nl_add_endpoint $ns2 10.0.3.2 dev ns2eth3 flags subflow
- run_tests $ns1 $ns2 10.0.1.1 1024
- if [ $(pedit_action_pkts) -gt 0 ]; then
- break
- else
- printf "%03u %s\n" "$TEST_COUNT" "MP_FAIL MP_RST: retry $n"
- fi
- done
+ reset_with_fail 2
+ pm_nl_set_limits $ns1 0 2
+ pm_nl_set_limits $ns2 0 2
+ pm_nl_add_endpoint $ns2 10.0.2.2 dev ns2eth2 flags subflow
+ pm_nl_add_endpoint $ns2 10.0.3.2 dev ns2eth3 flags subflow
+ run_tests $ns1 $ns2 10.0.1.1 1024
chk_join_nr "MP_FAIL MP_RST: $(pedit_action_pkts) corrupted pkts" 2 2 2 1 1
# single subflow
--
2.34.1