Skip to content

Commit f2b492b

Browse files
matttbekuba-moo
authored andcommitted
selftests: mptcp: join: skip userspace PM tests if not supported
Selftests are supposed to run on any kernels, including the old ones not supporting all MPTCP features. One of them is the support of the userspace PM introduced by commit 4638de5 ("mptcp: handle local addrs announced by userspace PMs") and the following ones. It is possible to look for the MPTCP pm_type's sysctl knob to know in advance if the userspace PM is available. Link: multipath-tcp/mptcp_net-next#368 Fixes: 5ac1d2d ("selftests: mptcp: Add tests for userspace PM type") Cc: stable@vger.kernel.org Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net> Signed-off-by: Jakub Kicinski <kuba@kernel.org>
1 parent 9db34c4 commit f2b492b

1 file changed

Lines changed: 17 additions & 9 deletions

File tree

tools/testing/selftests/net/mptcp/mptcp_join.sh

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ init_partial()
8484
ip netns add $netns || exit $ksft_skip
8585
ip -net $netns link set lo up
8686
ip netns exec $netns sysctl -q net.mptcp.enabled=1
87-
ip netns exec $netns sysctl -q net.mptcp.pm_type=0
87+
ip netns exec $netns sysctl -q net.mptcp.pm_type=0 2>/dev/null || true
8888
ip netns exec $netns sysctl -q net.ipv4.conf.all.rp_filter=0
8989
ip netns exec $netns sysctl -q net.ipv4.conf.default.rp_filter=0
9090
if [ $checksum -eq 1 ]; then
@@ -3191,7 +3191,8 @@ fail_tests()
31913191
userspace_tests()
31923192
{
31933193
# userspace pm type prevents add_addr
3194-
if reset "userspace pm type prevents add_addr"; then
3194+
if reset "userspace pm type prevents add_addr" &&
3195+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
31953196
set_userspace_pm $ns1
31963197
pm_nl_set_limits $ns1 0 2
31973198
pm_nl_set_limits $ns2 0 2
@@ -3202,7 +3203,8 @@ userspace_tests()
32023203
fi
32033204

32043205
# userspace pm type does not echo add_addr without daemon
3205-
if reset "userspace pm no echo w/o daemon"; then
3206+
if reset "userspace pm no echo w/o daemon" &&
3207+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32063208
set_userspace_pm $ns2
32073209
pm_nl_set_limits $ns1 0 2
32083210
pm_nl_set_limits $ns2 0 2
@@ -3213,7 +3215,8 @@ userspace_tests()
32133215
fi
32143216

32153217
# userspace pm type rejects join
3216-
if reset "userspace pm type rejects join"; then
3218+
if reset "userspace pm type rejects join" &&
3219+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32173220
set_userspace_pm $ns1
32183221
pm_nl_set_limits $ns1 1 1
32193222
pm_nl_set_limits $ns2 1 1
@@ -3223,7 +3226,8 @@ userspace_tests()
32233226
fi
32243227

32253228
# userspace pm type does not send join
3226-
if reset "userspace pm type does not send join"; then
3229+
if reset "userspace pm type does not send join" &&
3230+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32273231
set_userspace_pm $ns2
32283232
pm_nl_set_limits $ns1 1 1
32293233
pm_nl_set_limits $ns2 1 1
@@ -3233,7 +3237,8 @@ userspace_tests()
32333237
fi
32343238

32353239
# userspace pm type prevents mp_prio
3236-
if reset "userspace pm type prevents mp_prio"; then
3240+
if reset "userspace pm type prevents mp_prio" &&
3241+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32373242
set_userspace_pm $ns1
32383243
pm_nl_set_limits $ns1 1 1
32393244
pm_nl_set_limits $ns2 1 1
@@ -3244,7 +3249,8 @@ userspace_tests()
32443249
fi
32453250

32463251
# userspace pm type prevents rm_addr
3247-
if reset "userspace pm type prevents rm_addr"; then
3252+
if reset "userspace pm type prevents rm_addr" &&
3253+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32483254
set_userspace_pm $ns1
32493255
set_userspace_pm $ns2
32503256
pm_nl_set_limits $ns1 0 1
@@ -3256,7 +3262,8 @@ userspace_tests()
32563262
fi
32573263

32583264
# userspace pm add & remove address
3259-
if reset_with_events "userspace pm add & remove address"; then
3265+
if reset_with_events "userspace pm add & remove address" &&
3266+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32603267
set_userspace_pm $ns1
32613268
pm_nl_set_limits $ns2 1 1
32623269
run_tests $ns1 $ns2 10.0.1.1 0 userspace_1 0 slow
@@ -3267,7 +3274,8 @@ userspace_tests()
32673274
fi
32683275

32693276
# userspace pm create destroy subflow
3270-
if reset_with_events "userspace pm create destroy subflow"; then
3277+
if reset_with_events "userspace pm create destroy subflow" &&
3278+
continue_if mptcp_lib_has_file '/proc/sys/net/mptcp/pm_type'; then
32713279
set_userspace_pm $ns2
32723280
pm_nl_set_limits $ns1 0 1
32733281
run_tests $ns1 $ns2 10.0.1.1 0 0 userspace_1 slow

0 commit comments

Comments
 (0)