netdev - Re: [PATCH net-next v3 1/2] ipv6: introduce and uses route look hints for list input

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f3a53240733bee0322100e9747b6c2e1049b058c.camel@redhat.com>
Date:   Tue, 19 Nov 2019 17:00:21 +0100
From:   Paolo Abeni <pabeni@...hat.com>
To:     David Ahern <dsahern@...il.com>, netdev@...r.kernel.org
Cc:     "David S. Miller" <davem@...emloft.net>,
        Willem de Bruijn <willemdebruijn.kernel@...il.com>,
        Edward Cree <ecree@...arflare.com>
Subject: Re: [PATCH net-next v3 1/2] ipv6: introduce and uses route look
 hints for list input

Hi,

On Tue, 2019-11-19 at 08:39 -0700, David Ahern wrote:
> +static struct sk_buff *ip6_extract_route_hint(struct net *net,
> > +					      struct sk_buff *skb)
> > +{
> > +	if (IS_ENABLED(IPV6_SUBTREES) || fib6_has_custom_rules(net))
> 
> ... but basing on SUBTREES being disabled is going to limit its use. If
> no routes are source based (fib6_src is not set), you should be able to
> re-use the hint with SUBTREES enabled. e.g., track fib6_src use with a
> per-namespace counter - similar to fib6_rules_require_fldissect.

Thank you for the feedback! Would you consider this as an intermediate
step? e.g. get these patches in, and then I'll implement subtree
support? 
I'm asking because I don't have subtree setup handy, it will a little
time to get there.

Thanks!

Paolo