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 > > > tunnel->parms.link, tunnel->fwmark, 0, 0); > > rt = ip_route_output_key(tunnel->net, &fl4); > > > > -- > > 2.46.0 > > >
