[RFC PATCH bpf-next 7/9] selftests/bpf: fib_lookup: Add IPv6 output route tests
From: Ido Schimmel <idosch@nvidia.com>
Date: 2026-02-26 08:01:50
Also in:
bpf
Subsystem:
bpf [general] (safe dynamic programs and tools), bpf [selftests] (test runners & infrastructure), kernel selftest framework, the rest · Maintainers:
Alexei Starovoitov, Daniel Borkmann, Andrii Nakryiko, Eduard Zingerman, Kumar Kartikeya Dwivedi, Shuah Khan, Shuah Khan, Linus Torvalds
In a similar fashion to the IPv4 output route tests, add equivalent IPv6 tests. Output without "bpf: fib_lookup: Honor oif in IPv6 output route lookups": # ./test_progs -a fib_lookup [...] Testing IPv6 output route, without source, via second device test_fib_lookup:PASS:bpf_prog_test_run_opts 0 nsec test_fib_lookup:PASS:fib_lookup_ret 0 nsec test_fib_lookup:FAIL:ifindex does not match unexpected ifindex does not match: actual 10010 != expected 10020 [...] Testing IPv6 output route, with source, via second device set_lookup_params:PASS:inet_pton(IPV6_IFACE_ADDR) 0 nsec test_fib_lookup:PASS:bpf_prog_test_run_opts 0 nsec test_fib_lookup:PASS:fib_lookup_ret 0 nsec test_fib_lookup:FAIL:ifindex does not match unexpected ifindex does not match: actual 10010 != expected 10020 [...] Testing IPv6 output route, oif mismatch set_lookup_params:PASS:inet_pton(IPV6_IFACE_ADDR) 0 nsec test_fib_lookup:PASS:bpf_prog_test_run_opts 0 nsec test_fib_lookup:FAIL:fib_lookup_ret unexpected fib_lookup_ret: actual 0 != expected 4 [...] #119 fib_lookup:FAIL Summary: 0/0 PASSED, 0 SKIPPED, 1 FAILED Output with "bpf: fib_lookup: Honor oif in IPv6 output route lookups": # ./test_progs -a fib_lookup #119 fib_lookup:OK Summary: 1/0 PASSED, 0 SKIPPED, 0 FAILED That is, without the specified patch, unlike IPv4, IPv6 would always resolve the most specific route with the lowest metric, even if its nexthop device does not match the specified oif. Reviewed-by: Petr Machata <petrm@nvidia.com> Signed-off-by: Ido Schimmel <idosch@nvidia.com> --- .../selftests/bpf/prog_tests/fib_lookup.c | 35 +++++++++++++++++++ 1 file changed, 35 insertions(+)
diff --git a/tools/testing/selftests/bpf/prog_tests/fib_lookup.c b/tools/testing/selftests/bpf/prog_tests/fib_lookup.c
index 6ceeecbdca43..9e933daf56d7 100644
--- a/tools/testing/selftests/bpf/prog_tests/fib_lookup.c
+++ b/tools/testing/selftests/bpf/prog_tests/fib_lookup.c@@ -45,6 +45,8 @@ #define IFINDEX_VETH2 10020 #define IPV4_OUTPUT_NET "192.0.2.0" #define IPV4_OUTPUT_DST "192.0.2.1" +#define IPV6_OUTPUT_NET "2001:db8:1::" +#define IPV6_OUTPUT_DST "2001:db8:1::1" struct fib_lookup_test { const char *desc;
@@ -177,6 +179,34 @@ static const struct fib_lookup_test tests[] = { .expected_ret = BPF_FIB_LKUP_RET_NOT_FWDED, .lookup_flags = BPF_FIB_LOOKUP_OUTPUT | BPF_FIB_LOOKUP_SKIP_NEIGH, .ifindex = IFINDEX_VETH2, }, + { .desc = "IPv6 output route, without source, via first device", + .daddr = IPV6_OUTPUT_DST, .expected_ret = BPF_FIB_LKUP_RET_SUCCESS, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT | BPF_FIB_LOOKUP_SRC, + .dmac = DMAC_INIT, .ifindex = IFINDEX_VETH1, + .expected_ifindex = IFINDEX_VETH1, }, + { .desc = "IPv6 output route, without source, via second device", + .daddr = IPV6_OUTPUT_DST, .expected_ret = BPF_FIB_LKUP_RET_SUCCESS, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT | BPF_FIB_LOOKUP_SRC, + .dmac = DMAC_INIT2, .ifindex = IFINDEX_VETH2, + .expected_ifindex = IFINDEX_VETH2, }, + { .desc = "IPv6 output route, with source, via first device", + .daddr = IPV6_OUTPUT_DST, .expected_ret = BPF_FIB_LKUP_RET_SUCCESS, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT, .dmac = DMAC_INIT, + .ifindex = IFINDEX_VETH1, .expected_ifindex = IFINDEX_VETH1, }, + { .desc = "IPv6 output route, with source, via second device", + .daddr = IPV6_OUTPUT_DST, .expected_ret = BPF_FIB_LKUP_RET_SUCCESS, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT, .dmac = DMAC_INIT2, + .ifindex = IFINDEX_VETH2, .expected_ifindex = IFINDEX_VETH2, }, + { .desc = "IPv6 output route, oif match", + .daddr = IPV6_NUD_STALE_ADDR, + .expected_ret = BPF_FIB_LKUP_RET_SUCCESS, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT | BPF_FIB_LOOKUP_SKIP_NEIGH, + .ifindex = IFINDEX_VETH1, .expected_ifindex = IFINDEX_VETH1, }, + { .desc = "IPv6 output route, oif mismatch", + .daddr = IPV6_NUD_STALE_ADDR, + .expected_ret = BPF_FIB_LKUP_RET_NOT_FWDED, + .lookup_flags = BPF_FIB_LOOKUP_OUTPUT | BPF_FIB_LOOKUP_SKIP_NEIGH, + .ifindex = IFINDEX_VETH2, }, }; static int setup_netns(void)
@@ -254,6 +284,11 @@ static int setup_netns(void) SYS(fail, "ip neigh add %s dev veth1 lladdr %s nud perm", IPV4_OUTPUT_DST, DMAC); SYS(fail, "ip neigh add %s dev veth2 lladdr %s nud perm", IPV4_OUTPUT_DST, DMAC2); + SYS(fail, "ip route add %s/64 dev veth1 metric 100", IPV6_OUTPUT_NET); + SYS(fail, "ip route add %s/64 dev veth2 metric 200", IPV6_OUTPUT_NET); + SYS(fail, "ip neigh add %s dev veth1 lladdr %s nud perm", IPV6_OUTPUT_DST, DMAC); + SYS(fail, "ip neigh add %s dev veth2 lladdr %s nud perm", IPV6_OUTPUT_DST, DMAC2); + return 0; fail: return -1;
--
2.53.0