selftests: mptcp: connect: skip if MPTCP is not supported
commitd83013bdf9upstream. Selftests are supposed to run on any kernels, including the old ones not supporting MPTCP. A new check is then added to make sure MPTCP is supported. If not, the test stops and is marked as "skipped". Note that this check can also mark the test as failed if 'SELFTESTS_MPTCP_LIB_EXPECT_ALL_FEATURES' env var is set to 1: by doing that, we can make sure a test is not being skipped by mistake. A new shared file is added here to be able to re-used the same check in the different selftests we have. Link: https://github.com/multipath-tcp/mptcp_net-next/issues/368 Fixes:048d19d444("mptcp: add basic kselftest for mptcp") Cc: stable@vger.kernel.org Acked-by: Paolo Abeni <pabeni@redhat.com> Signed-off-by: Matthieu Baerts <matthieu.baerts@tessares.net> Signed-off-by: Paolo Abeni <pabeni@redhat.com> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
57eb824b8c
commit
54dea0aa6b
3 changed files with 45 additions and 1 deletions
|
|
@ -10,7 +10,7 @@ TEST_PROGS := mptcp_connect.sh pm_netlink.sh mptcp_join.sh diag.sh \
|
|||
|
||||
TEST_GEN_FILES = mptcp_connect pm_nl_ctl
|
||||
|
||||
TEST_FILES := settings
|
||||
TEST_FILES := mptcp_lib.sh settings
|
||||
|
||||
EXTRA_CLEAN := *.pcap
|
||||
|
||||
|
|
|
|||
|
|
@ -1,6 +1,8 @@
|
|||
#!/bin/bash
|
||||
# SPDX-License-Identifier: GPL-2.0
|
||||
|
||||
. "$(dirname "${0}")/mptcp_lib.sh"
|
||||
|
||||
time_start=$(date +%s)
|
||||
|
||||
optstring="S:R:d:e:l:r:h4cm:f:t"
|
||||
|
|
@ -131,6 +133,8 @@ cleanup()
|
|||
done
|
||||
}
|
||||
|
||||
mptcp_lib_check_mptcp
|
||||
|
||||
ip -Version > /dev/null 2>&1
|
||||
if [ $? -ne 0 ];then
|
||||
echo "SKIP: Could not run test without ip tool"
|
||||
|
|
|
|||
40
tools/testing/selftests/net/mptcp/mptcp_lib.sh
Normal file
40
tools/testing/selftests/net/mptcp/mptcp_lib.sh
Normal file
|
|
@ -0,0 +1,40 @@
|
|||
#! /bin/bash
|
||||
# SPDX-License-Identifier: GPL-2.0
|
||||
|
||||
readonly KSFT_FAIL=1
|
||||
readonly KSFT_SKIP=4
|
||||
|
||||
# SELFTESTS_MPTCP_LIB_EXPECT_ALL_FEATURES env var can be set when validating all
|
||||
# features using the last version of the kernel and the selftests to make sure
|
||||
# a test is not being skipped by mistake.
|
||||
mptcp_lib_expect_all_features() {
|
||||
[ "${SELFTESTS_MPTCP_LIB_EXPECT_ALL_FEATURES:-}" = "1" ]
|
||||
}
|
||||
|
||||
# $1: msg
|
||||
mptcp_lib_fail_if_expected_feature() {
|
||||
if mptcp_lib_expect_all_features; then
|
||||
echo "ERROR: missing feature: ${*}"
|
||||
exit ${KSFT_FAIL}
|
||||
fi
|
||||
|
||||
return 1
|
||||
}
|
||||
|
||||
# $1: file
|
||||
mptcp_lib_has_file() {
|
||||
local f="${1}"
|
||||
|
||||
if [ -f "${f}" ]; then
|
||||
return 0
|
||||
fi
|
||||
|
||||
mptcp_lib_fail_if_expected_feature "${f} file not found"
|
||||
}
|
||||
|
||||
mptcp_lib_check_mptcp() {
|
||||
if ! mptcp_lib_has_file "/proc/sys/net/mptcp/enabled"; then
|
||||
echo "SKIP: MPTCP support is not available"
|
||||
exit ${KSFT_SKIP}
|
||||
fi
|
||||
}
|
||||
Loading…
Add table
Add a link
Reference in a new issue