This is a note to let you know that I've just added the patch titled selftests: mptcp: lib: skip if missing symbol to the 6.3-stable tree which can be found at: http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary The filename of the patch is: selftests-mptcp-lib-skip-if-missing-symbol.patch and it can be found in the queue-6.3 subdirectory. If you, or anyone else, feels it should not be added to the stable tree, please let <stable@xxxxxxxxxxxxxxx> know about it. >From 673004821ab98c6645bd21af56a290854e88f533 Mon Sep 17 00:00:00 2001 From: Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx> Date: Thu, 8 Jun 2023 18:38:43 +0200 Subject: selftests: mptcp: lib: skip if missing symbol From: Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx> commit 673004821ab98c6645bd21af56a290854e88f533 upstream. Selftests are supposed to run on any kernels, including the old ones not supporting all MPTCP features. New functions are now available to easily detect if a certain feature is missing by looking at kallsyms. These new helpers are going to be used in the following commits. In order to ease the backport of such future patches, it would be good if this patch is backported up to the introduction of MPTCP selftests, hence the Fixes tag below: this type of check was supposed to be done from the beginning. Link: https://github.com/multipath-tcp/mptcp_net-next/issues/368 Fixes: 048d19d444be ("mptcp: add basic kselftest for mptcp") Cc: stable@xxxxxxxxxxxxxxx Signed-off-by: Matthieu Baerts <matthieu.baerts@xxxxxxxxxxxx> Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx> Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> --- tools/testing/selftests/net/mptcp/config | 1 tools/testing/selftests/net/mptcp/mptcp_lib.sh | 38 +++++++++++++++++++++++++ 2 files changed, 39 insertions(+) --- a/tools/testing/selftests/net/mptcp/config +++ b/tools/testing/selftests/net/mptcp/config @@ -1,3 +1,4 @@ +CONFIG_KALLSYMS=y CONFIG_MPTCP=y CONFIG_IPV6=y CONFIG_MPTCP_IPV6=y --- a/tools/testing/selftests/net/mptcp/mptcp_lib.sh +++ b/tools/testing/selftests/net/mptcp/mptcp_lib.sh @@ -38,3 +38,41 @@ mptcp_lib_check_mptcp() { exit ${KSFT_SKIP} fi } + +mptcp_lib_check_kallsyms() { + if ! mptcp_lib_has_file "/proc/kallsyms"; then + echo "SKIP: CONFIG_KALLSYMS is missing" + exit ${KSFT_SKIP} + fi +} + +# Internal: use mptcp_lib_kallsyms_has() instead +__mptcp_lib_kallsyms_has() { + local sym="${1}" + + mptcp_lib_check_kallsyms + + grep -q " ${sym}" /proc/kallsyms +} + +# $1: part of a symbol to look at, add '$' at the end for full name +mptcp_lib_kallsyms_has() { + local sym="${1}" + + if __mptcp_lib_kallsyms_has "${sym}"; then + return 0 + fi + + mptcp_lib_fail_if_expected_feature "${sym} symbol not found" +} + +# $1: part of a symbol to look at, add '$' at the end for full name +mptcp_lib_kallsyms_doesnt_have() { + local sym="${1}" + + if ! __mptcp_lib_kallsyms_has "${sym}"; then + return 0 + fi + + mptcp_lib_fail_if_expected_feature "${sym} symbol has been found" +} Patches currently in stable-queue which might be from matthieu.baerts@xxxxxxxxxxxx are queue-6.3/selftests-mptcp-join-fix-shellcheck-warnings.patch queue-6.3/selftests-mptcp-join-support-rm_addr-for-used-endpoints-or-not.patch queue-6.3/selftests-mptcp-pm-nl-skip-fullmesh-flag-checks-if-not-supported.patch queue-6.3/selftests-mptcp-lib-skip-if-missing-symbol.patch queue-6.3/selftests-mptcp-join-skip-fullmesh-flag-tests-if-not-supported.patch queue-6.3/selftests-mptcp-connect-skip-tfo-tests-if-not-supported.patch queue-6.3/selftests-mptcp-join-skip-fastclose-tests-if-not-supported.patch queue-6.3/selftests-mptcp-remove-duplicated-entries-in-usage.patch queue-6.3/selftests-mptcp-sockopt-skip-getsockopt-checks-if-not-supported.patch queue-6.3/selftests-mptcp-join-skip-backup-if-set-flag-on-id-not-supported.patch queue-6.3/selftests-mptcp-join-skip-mixed-tests-if-not-supported.patch queue-6.3/selftests-mptcp-userspace-pm-skip-if-ip-tool-is-unavailable.patch queue-6.3/selftests-mptcp-join-helpers-to-skip-tests.patch queue-6.3/selftests-mptcp-lib-skip-if-not-below-kernel-version.patch queue-6.3/selftests-mptcp-userspace-pm-skip-if-not-supported.patch queue-6.3/selftests-mptcp-join-uniform-listener-tests.patch queue-6.3/selftests-mptcp-diag-skip-listen-tests-if-not-supported.patch queue-6.3/selftests-mptcp-sockopt-skip-tcp_inq-checks-if-not-supported.patch queue-6.3/selftests-mptcp-join-skip-test-if-iptables-tc-cmds-fail.patch queue-6.3/selftests-mptcp-join-skip-implicit-tests-if-not-supported.patch queue-6.3/selftests-mptcp-userspace-pm-skip-pm-listener-events-tests-if-unavailable.patch queue-6.3/selftests-mptcp-join-skip-fail-tests-if-not-supported.patch queue-6.3/selftests-mptcp-sockopt-relax-expected-returned-size.patch queue-6.3/selftests-mptcp-pm-nl-remove-hardcoded-default-limits.patch queue-6.3/selftests-mptcp-join-skip-check-if-mib-counter-not-supported.patch queue-6.3/selftests-mptcp-diag-skip-inuse-tests-if-not-supported.patch queue-6.3/selftests-mptcp-join-use-iptables-legacy-if-available.patch queue-6.3/selftests-mptcp-join-support-local-endpoint-being-tracked-or-not.patch queue-6.3/selftests-mptcp-join-skip-userspace-pm-tests-if-not-supported.patch queue-6.3/selftests-mptcp-connect-skip-transp-tests-if-not-supported.patch queue-6.3/selftests-mptcp-join-skip-mpc-backups-tests-if-not-supported.patch queue-6.3/selftests-mptcp-join-skip-pm-listener-tests-if-not-supported.patch queue-6.3/selftests-mptcp-connect-skip-disconnect-tests-if-not-supported.patch