On Fri, Sep 06, 2024 at 04:45:35PM +0300, Ido Schimmel wrote: > On Fri, Sep 06, 2024 at 01:36:35PM +0200, Guillaume Nault wrote: > > On Thu, Sep 05, 2024 at 07:51:33PM +0300, Ido Schimmel wrote: > > > Unmask the upper DSCP bits when initializing an IPv4 flow key via > > > ip_tunnel_init_flow() before passing it to ip_route_output_key() so that > > > in the future we could perform the FIB lookup according to the full DSCP > > > value. > > > > > > Signed-off-by: Ido Schimmel <idosch@xxxxxxxxxx> > > > --- > > > net/ipv4/ip_tunnel.c | 2 +- > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > > > diff --git a/net/ipv4/ip_tunnel.c b/net/ipv4/ip_tunnel.c > > > index 18964394d6bd..b632c128ecb7 100644 > > > --- a/net/ipv4/ip_tunnel.c > > > +++ b/net/ipv4/ip_tunnel.c > > > @@ -293,7 +293,7 @@ static int ip_tunnel_bind_dev(struct net_device *dev) > > > > > > ip_tunnel_init_flow(&fl4, iph->protocol, iph->daddr, > > > iph->saddr, tunnel->parms.o_key, > > > - RT_TOS(iph->tos), dev_net(dev), > > > + iph->tos & INET_DSCP_MASK, dev_net(dev), > > > > The net/inet_dscp.h header file is only included in patch 6, while it's > > needed here in patch 5. > > Thanks. Probably happened when I reordered the patches. However, it > doesn't affect bisectability since the header is included via include/net/ip.h Okay, no need for a v2 then. I'll ack the remaining patches. And thanks again for your work! > > > tunnel->parms.link, tunnel->fwmark, 0, 0); > > > rt = ip_route_output_key(tunnel->net, &fl4); > > > > > > -- > > > 2.46.0 > > > > > >
