netdev - Re: [PATCH net-next v2 0/2] net: Speedup some nexthop handling when having A LOT of nexthops

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: <a9ae4ab6-bc93-4a0c-8018-d8422568b09b@kernel.org>
Date: Mon, 18 Aug 2025 07:33:19 -0600
From: David Ahern <dsahern@...nel.org>
To: cpaasch@...nai.com, Nikolay Aleksandrov <razor@...ckwall.org>,
 "David S. Miller" <davem@...emloft.net>, Eric Dumazet <edumazet@...gle.com>,
 Jakub Kicinski <kuba@...nel.org>, Paolo Abeni <pabeni@...hat.com>,
 Simon Horman <horms@...nel.org>, Ido Schimmel <idosch@...sch.org>
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH net-next v2 0/2] net: Speedup some nexthop handling when
 having A LOT of nexthops

On 8/16/25 5:12 PM, Christoph Paasch via B4 Relay wrote:
> Configuring a very large number of nexthops is fairly possible within a
> reasonable time-frame. But, certain netlink commands can become
> extremely slow.
> 
> This series addresses some of these, namely dumping and removing
> nexthops.
> 
> Signed-off-by: Christoph Paasch <cpaasch@...nai.com>
> ---
> Changes in v2:
> - Added another improvement to the series "net: When removing nexthops,
>   don't call synchronize_net if it is not necessary"
> - Fixed typos, made comments within 80-character limit and unified
>   comment-style. (Ido Schimmel)
> - Removed if (nh->id < s_idx) in the for-loop as it is no more needed.
>   (Ido Schimmel)
> - Link to v1: https://lore.kernel.org/r/20250724-nexthop_dump-v1-1-6b43fffd5bac@openai.com
> 
> ---
> Christoph Paasch (2):
>       net: Make nexthop-dumps scale linearly with the number of nexthops
>       net: When removing nexthops, don't call synchronize_net if it is not necessary
> 
>  net/ipv4/nexthop.c | 42 +++++++++++++++++++++++++++++++++++++++---
>  1 file changed, 39 insertions(+), 3 deletions(-)
> ---
> base-commit: bab3ce404553de56242d7b09ad7ea5b70441ea41
> change-id: 20250724-nexthop_dump-f6c32472bcdf
> 
> Best regards,

For the set:
Reviewed-by: David Ahern <dsahern@...nel.org>