]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
seg6: Allow End.X behavior to accept an oif
authorIdo Schimmel <idosch@nvidia.com>
Thu, 12 Jun 2025 12:23:22 +0000 (15:23 +0300)
committerJakub Kicinski <kuba@kernel.org>
Mon, 16 Jun 2025 22:31:14 +0000 (15:31 -0700)
Extend the End.X behavior to accept an output interface as an optional
attribute and make use of it when resolving a route. This is needed when
user space wants to use a link-local address as the nexthop address.

Before:

 # ip route add 2001:db8:1::/64 encap seg6local action End.X nh6 fe80::1 oif eth0 dev sr6
 # ip route add 2001:db8:2::/64 encap seg6local action End.X nh6 2001:db8:10::1 dev sr6
 $ ip -6 route show
 2001:db8:1::/64  encap seg6local action End.X nh6 fe80::1 dev sr6 metric 1024 pref medium
 2001:db8:2::/64  encap seg6local action End.X nh6 2001:db8:10::1 dev sr6 metric 1024 pref medium

After:

 # ip route add 2001:db8:1::/64 encap seg6local action End.X nh6 fe80::1 oif eth0 dev sr6
 # ip route add 2001:db8:2::/64 encap seg6local action End.X nh6 2001:db8:10::1 dev sr6
 $ ip -6 route show
 2001:db8:1::/64  encap seg6local action End.X nh6 fe80::1 oif eth0 dev sr6 metric 1024 pref medium
 2001:db8:2::/64  encap seg6local action End.X nh6 2001:db8:10::1 dev sr6 metric 1024 pref medium

Note that the oif attribute is not dumped to user space when it was not
specified (as an oif of 0) since each entry keeps track of the optional
attributes that it parsed during configuration (see struct
seg6_local_lwt::parsed_optattrs).

Signed-off-by: Ido Schimmel <idosch@nvidia.com>
Reviewed-by: Andrea Mayer <andrea.mayer@uniroma2.it>
Link: https://patch.msgid.link/20250612122323.584113-4-idosch@nvidia.com
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
net/ipv6/seg6_local.c

index c00b78f3abad05f1f8b91783e34a7d0c82091b33..dfa825ee870e6b6c75b688d9aa9512c5bd0f39d8 100644 (file)
@@ -421,7 +421,7 @@ static int end_next_csid_core(struct sk_buff *skb, struct seg6_local_lwt *slwt)
 static int input_action_end_x_finish(struct sk_buff *skb,
                                     struct seg6_local_lwt *slwt)
 {
-       seg6_lookup_any_nexthop(skb, &slwt->nh6, 0, false, 0);
+       seg6_lookup_any_nexthop(skb, &slwt->nh6, 0, false, slwt->oif);
 
        return dst_input(skb);
 }
@@ -1480,7 +1480,8 @@ static struct seg6_action_desc seg6_action_table[] = {
                .action         = SEG6_LOCAL_ACTION_END_X,
                .attrs          = SEG6_F_ATTR(SEG6_LOCAL_NH6),
                .optattrs       = SEG6_F_LOCAL_COUNTERS |
-                                 SEG6_F_LOCAL_FLAVORS,
+                                 SEG6_F_LOCAL_FLAVORS |
+                                 SEG6_F_ATTR(SEG6_LOCAL_OIF),
                .input          = input_action_end_x,
        },
        {