Re: [PATCH net-next] selftests/net: Skip srv6_end_dt46_l3vpn_test::host_vpn_tests if iproute2 too old
From: Andrea Mayer <hidden>
Date: 2026-07-10 13:30:13
Also in:
linux-kselftest
On Wed, 8 Jul 2026 17:27:45 +0200 Alessio Faina [off-list ref] wrote:
quoted hunk ↗ jump to hunk
In case iproute2 is older than version 5.14.0, released ~Sept 1, 2021, the End.DT46 support is not available and the host_vpn_tests test contained in the srv6_end_dt46_l3vpn_test.sh file is failing in some kernel backports. This is the result of those tests: ################################################################################ TEST SECTION: SRv6 VPN connectivity test among hosts in the same tenant ################################################################################ TEST: IPv6 Hosts connectivity: hs-t100-1 -> hs-t100-2 (tenant 100) [ FAIL ] TEST: IPv4 Hosts connectivity: hs-t100-1 -> hs-t100-2 (tenant 100) [ FAIL ] TEST: IPv6 Hosts connectivity: hs-t100-2 -> hs-t100-1 (tenant 100) [ FAIL ] TEST: IPv4 Hosts connectivity: hs-t100-2 -> hs-t100-1 (tenant 100) [ FAIL ] TEST: IPv6 Hosts connectivity: hs-t200-3 -> hs-t200-4 (tenant 200) [ FAIL ] TEST: IPv4 Hosts connectivity: hs-t200-3 -> hs-t200-4 (tenant 200) [ FAIL ] TEST: IPv6 Hosts connectivity: hs-t200-4 -> hs-t200-3 (tenant 200) [ FAIL ] TEST: IPv4 Hosts connectivity: hs-t200-4 -> hs-t200-3 (tenant 200) [ FAIL ] To amend this, check the current running iproute2 supports the required feature and, if not, just skip the test to avoid a failure. Signed-off-by: Alessio Faina <redacted> --- .../selftests/net/srv6_end_dt46_l3vpn_test.sh | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-)diff --git a/tools/testing/selftests/net/srv6_end_dt46_l3vpn_test.sh b/tools/testing/selftests/net/srv6_end_dt46_l3vpn_test.sh index a5e959a080bb..406c6a3d253e 100755 --- a/tools/testing/selftests/net/srv6_end_dt46_l3vpn_test.sh +++ b/tools/testing/selftests/net/srv6_end_dt46_l3vpn_test.sh@@ -510,12 +510,15 @@ host2gateway_tests() host_vpn_tests() { log_section "SRv6 VPN connectivity test among hosts in the same tenant" + if ip route add help 2>&1 | grep -q -i End.DT46; then + check_and_log_hs_connectivity 1 2 100 + check_and_log_hs_connectivity 2 1 100 - check_and_log_hs_connectivity 1 2 100 - check_and_log_hs_connectivity 2 1 100 - - check_and_log_hs_connectivity 3 4 200 - check_and_log_hs_connectivity 4 3 200 + check_and_log_hs_connectivity 3 4 200 + check_and_log_hs_connectivity 4 3 200 + else + echo "SKIP: SRv6 End.DT46 is unsupported by the installed iproute2 version" + fi }
Hi Alessio, Rather than inlining the check inside host_vpn_tests(), I think it would be better to follow the pattern already used in srv6_hencap_red_l3vpn_test.sh, where test_iproute2_supp_or_ksft_skip() is called before setup(). A similar function here could exit the whole script with $ksft_skip if End.DT46 is not supported by the installed iproute2. Thanks, Ciao Andrea