Re: [PATCH] IPv6: sr: Fix End.X nexthop to use oif.

Jakub Kicinski
2020-10-13T17:01:45Z

On Tue, 13 Oct 2020 17:31:51 +0530 Reji Thomas wrote:
> Currently End.X action doesn't consider the outgoing interface
> while looking up the nexthop.This breaks packet path functionality
> specifically while using link local address as the End.X nexthop.
> The patch fixes this by enforcing End.X action to have both nh6 and
> oif and using oif in lookup.It seems this is a day one issue.
>
> Fixes: 140f04c33bbc ("implement several seg6local actions")
>
> Signed-off-by: Reji Thomas <rejithomas@juniper.net>

You need to respin to add the missing 'static' kbuild bot pointed out.

When you do please also edit the fixes tag to include the full subject,
it should look like this:

Fixes: 140f04c33bbc ("ipv6: sr: implement several seg6local actions")

and remove the empty line between the fixes tag and your signoff.

> @@ -239,6 +249,8 @@ static int input_action_end(struct sk_buff *skb, struct seg6_local_lwt *slwt)
> static int input_action_end_x(struct sk_buff *skb, struct seg6_local_lwt *slwt)
> {
> struct ipv6_sr_hdr *srh;
> + struct net *net = dev_net(skb->dev);
> + struct net_device *odev;

Please sort the variable declarations longest to shortest.