netdev - Re: [PATCH net-next v2] neighbour: add support for NUD

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: <20250617230627.31793-1-kuni1840@gmail.com>
Date: Tue, 17 Jun 2025 16:06:03 -0700
From: Kuniyuki Iwashima <kuni1840@...il.com>
To: nico.escande@...il.com
Cc: davem@...emloft.net,
	decot+git@...gle.com,
	edumazet@...gle.com,
	gnaaman@...venets.com,
	horms@...nel.org,
	kuba@...nel.org,
	kuniyu@...gle.com,
	netdev@...r.kernel.org,
	pabeni@...hat.com
Subject: Re: [PATCH net-next v2] neighbour: add support for NUD_PERMANENT proxy entries

From: Nicolas Escande <nico.escande@...il.com>
Date: Tue, 17 Jun 2025 16:13:34 +0200
> As discussesd before in [0] proxy entries (which are more configuration
> than runtime data) should stay when the link (carrier) goes does down.
> This is what happens for regular neighbour entries.
> 
> So lets fix this by:
>   - storing in proxy entries the fact that it was added as NUD_PERMANENT
>   - not removing NUD_PERMANENT proxy entries when the carrier goes down
>     (same as how it's done in neigh_flush_dev() for regular neigh entries)
> 
> [0]: https://lore.kernel.org/netdev/c584ef7e-6897-01f3-5b80-12b53f7b4bf4@kernel.org/
> 
> Signed-off-by: Nicolas Escande <nico.escande@...il.com>

Reviewed-by: Kuniyuki Iwashima <kuniyu@...gle.com>