netdev - Re: [PATCH v3 net-next 0/4] fib: merge nl policies

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] [day] [month] [year] [list]

Message-ID: <20211215185121.1ffe768a@kicinski-fedora-pc1c0hjn.dhcp.thefacebook.com>
Date:   Wed, 15 Dec 2021 18:51:21 -0800
From:   Jakub Kicinski <kuba@...nel.org>
To:     Florian Westphal <fw@...len.de>
Cc:     <netdev@...r.kernel.org>, davem@...emloft.net, dsahern@...nel.org
Subject: Re: [PATCH v3 net-next 0/4] fib: merge nl policies

On Wed, 15 Dec 2021 12:32:40 +0100 Florian Westphal wrote:
> v3: drop first two patches, otherwise unchanged.
> 
> This series merges the different (largely identical) nla policies.
> 
> v2 also squashed the ->suppress() implementation, I've dropped this.
> Problem is that it needs ugly ifdef'ry to avoid build breakage
> with CONFIG_INET=n || IPV6=n.
> 
> Given that even microbenchmark doesn't show any noticeable improvement
> when ->suppress is inlined (it uses INDIRECT_CALLABLE) i decided to toss
> the patch instead of adding more ifdefs.

Would you mind resending? Patchwork thinks the series is incomplete
because the cover letter says 0/4 instead of 0/2.