Skip to content
This repository was archived by the owner on Oct 31, 2024. It is now read-only.

Commit 371efc2

Browse files
geliangtanggregkh
authored andcommitted
selftests: mptcp: add mptcp_lib_is_v6
commit b850f2c upstream. To avoid duplicated code in different MPTCP selftests, we can add and use helpers defined in mptcp_lib.sh. is_v6() helper is defined in mptcp_connect.sh, mptcp_join.sh and mptcp_sockopt.sh, so export it into mptcp_lib.sh and rename it as mptcp_lib_is_v6(). Use this new helper in all scripts. Reviewed-by: Matthieu Baerts <matttbe@kernel.org> Signed-off-by: Geliang Tang <geliang.tang@suse.com> Signed-off-by: Mat Martineau <martineau@kernel.org> Link: https://lore.kernel.org/r/20231128-send-net-next-2023107-v4-10-8d6b94150f6b@kernel.org Signed-off-by: Jakub Kicinski <kuba@kernel.org> Signed-off-by: Matthieu Baerts (NGI0) <matttbe@kernel.org> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
1 parent 42f4277 commit 371efc2

4 files changed

Lines changed: 15 additions & 28 deletions

File tree

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

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -310,12 +310,6 @@ check_mptcp_disabled()
310310
return 0
311311
}
312312

313-
# $1: IP address
314-
is_v6()
315-
{
316-
[ -z "${1##*:*}" ]
317-
}
318-
319313
do_ping()
320314
{
321315
local listener_ns="$1"
@@ -324,7 +318,7 @@ do_ping()
324318
local ping_args="-q -c 1"
325319
local rc=0
326320

327-
if is_v6 "${connect_addr}"; then
321+
if mptcp_lib_is_v6 "${connect_addr}"; then
328322
$ipv6 || return 0
329323
ping_args="${ping_args} -6"
330324
fi
@@ -620,12 +614,12 @@ run_tests_lo()
620614
fi
621615

622616
# skip if we don't want v6
623-
if ! $ipv6 && is_v6 "${connect_addr}"; then
617+
if ! $ipv6 && mptcp_lib_is_v6 "${connect_addr}"; then
624618
return 0
625619
fi
626620

627621
local local_addr
628-
if is_v6 "${connect_addr}"; then
622+
if mptcp_lib_is_v6 "${connect_addr}"; then
629623
local_addr="::"
630624
else
631625
local_addr="0.0.0.0"
@@ -693,7 +687,7 @@ run_test_transparent()
693687
TEST_GROUP="${msg}"
694688

695689
# skip if we don't want v6
696-
if ! $ipv6 && is_v6 "${connect_addr}"; then
690+
if ! $ipv6 && mptcp_lib_is_v6 "${connect_addr}"; then
697691
return 0
698692
fi
699693

@@ -726,7 +720,7 @@ EOF
726720
fi
727721

728722
local local_addr
729-
if is_v6 "${connect_addr}"; then
723+
if mptcp_lib_is_v6 "${connect_addr}"; then
730724
local_addr="::"
731725
r6flag="-6"
732726
else

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

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -592,12 +592,6 @@ link_failure()
592592
done
593593
}
594594

595-
# $1: IP address
596-
is_v6()
597-
{
598-
[ -z "${1##*:*}" ]
599-
}
600-
601595
# $1: ns, $2: port
602596
wait_local_port_listen()
603597
{
@@ -877,7 +871,7 @@ pm_nl_set_endpoint()
877871
local id=10
878872
while [ $add_nr_ns1 -gt 0 ]; do
879873
local addr
880-
if is_v6 "${connect_addr}"; then
874+
if mptcp_lib_is_v6 "${connect_addr}"; then
881875
addr="dead:beef:$counter::1"
882876
else
883877
addr="10.0.$counter.1"
@@ -929,7 +923,7 @@ pm_nl_set_endpoint()
929923
local id=20
930924
while [ $add_nr_ns2 -gt 0 ]; do
931925
local addr
932-
if is_v6 "${connect_addr}"; then
926+
if mptcp_lib_is_v6 "${connect_addr}"; then
933927
addr="dead:beef:$counter::2"
934928
else
935929
addr="10.0.$counter.2"
@@ -971,7 +965,7 @@ pm_nl_set_endpoint()
971965
pm_nl_flush_endpoint ${connector_ns}
972966
elif [ $rm_nr_ns2 -eq 9 ]; then
973967
local addr
974-
if is_v6 "${connect_addr}"; then
968+
if mptcp_lib_is_v6 "${connect_addr}"; then
975969
addr="dead:beef:1::2"
976970
else
977971
addr="10.0.1.2"
@@ -3339,7 +3333,7 @@ userspace_pm_rm_sf()
33393333
local cnt
33403334

33413335
[ "$1" == "$ns2" ] && evts=$evts_ns2
3342-
if is_v6 $2; then ip=6; fi
3336+
if mptcp_lib_is_v6 $2; then ip=6; fi
33433337
tk=$(mptcp_lib_evts_get_info token "$evts")
33443338
da=$(mptcp_lib_evts_get_info "daddr$ip" "$evts" $t)
33453339
dp=$(mptcp_lib_evts_get_info dport "$evts" $t)

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

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -227,6 +227,11 @@ mptcp_lib_kill_wait() {
227227
wait "${1}" 2>/dev/null
228228
}
229229

230+
# $1: IP address
231+
mptcp_lib_is_v6() {
232+
[ -z "${1##*:*}" ]
233+
}
234+
230235
# $1: ns, $2: MIB counter
231236
mptcp_lib_get_counter() {
232237
local ns="${1}"

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

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -161,12 +161,6 @@ check_transfer()
161161
return 0
162162
}
163163

164-
# $1: IP address
165-
is_v6()
166-
{
167-
[ -z "${1##*:*}" ]
168-
}
169-
170164
do_transfer()
171165
{
172166
local listener_ns="$1"
@@ -183,7 +177,7 @@ do_transfer()
183177
local mptcp_connect="./mptcp_connect -r 20"
184178

185179
local local_addr ip
186-
if is_v6 "${connect_addr}"; then
180+
if mptcp_lib_is_v6 "${connect_addr}"; then
187181
local_addr="::"
188182
ip=ipv6
189183
else

0 commit comments

Comments
 (0)