Netdev List
 help / color / mirror / Atom feed
From: Ido Schimmel <idosch@nvidia.com>
To: Lawrence Lee <lfqlee314@gmail.com>
Cc: David Ahern <dsahern@kernel.org>,
	"David S . Miller" <davem@davemloft.net>,
	Eric Dumazet <edumazet@google.com>,
	Jakub Kicinski <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,
	Simon Horman <horms@kernel.org>, Arun Ajith S <aajith@arista.com>,
	Roopa Prabhu <roopa@nvidia.com>,
	Jaehee Park <jhpark1013@gmail.com>,
	netdev@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [RFC net-next] ipv6: update NUD_FAILED neighbors from NA messages
Date: Thu, 27 Aug 2026 12:05:16 +0300	[thread overview]
Message-ID: <20260827090516.GA2014023@shredder> (raw)
In-Reply-To: <DKY84BEFKL1S.1BZWGQTGRZLMU@gmail.com>

On Tue, Aug 25, 2026 at 06:07:44PM +0000, Lawrence Lee wrote:
> Sorry I should have been more specific before. Maybe a better way to 
> phrase it is that this would be analogous to IPv4 in the sense that it 
> allows NUD_FAILED neighbors to change states upon receipt of a NA, but 
> by using NUD_STALE it remains consistent with RFC9131 Section 6.1.1.

I think that your setup is quite unique and I don't necessarily agree
with the statements regarding the CPU load, but it doesn't change my
conclusion: IMO it makes sense to transition from FAILED to STALE when
receiving NAs (most likely unsolicited given FAILED state).

Mention in the commit message that trying to resolve FAILED neighbours
by periodically probing them (e.g., using NTF_EXT_MANAGED) adds extra
work compared to having the kernel stop ignoring the information that
it's getting via NAs. Also mention that the approach taken here is more
conservative than IPv4, which by default transitions FAILED neighbours
to STALE upon receiving GARPs.

Note that you also need to update the RFC 9131 comment in
ndisc_recv_na() and the description of accept_untracked_na in
Documentation/networking/ip-sysctl.rst. In addition, please add a test
case in tools/testing/selftests/net/ndisc_unsolicited_na_test.sh

Before submitting, please read:
https://docs.kernel.org/next/process/maintainer-netdev.html

Run the CI checks locally:
https://github.com/linux-netdev/nipa#running-locally

And LLM review:
https://github.com/sashiko-dev/sashiko#1-local-review-recommended

Thanks

  reply	other threads:[~2026-08-27  9:05 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-08-13 23:33 [RFC net-next] ipv6: update NUD_FAILED neighbors from NA messages Lawrence Lee
2026-08-25 11:52 ` Ido Schimmel
2026-08-25 18:07   ` Lawrence Lee
2026-08-27  9:05     ` Ido Schimmel [this message]
2026-08-28 16:54       ` Lawrence Lee

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260827090516.GA2014023@shredder \
    --to=idosch@nvidia.com \
    --cc=aajith@arista.com \
    --cc=davem@davemloft.net \
    --cc=dsahern@kernel.org \
    --cc=edumazet@google.com \
    --cc=horms@kernel.org \
    --cc=jhpark1013@gmail.com \
    --cc=kuba@kernel.org \
    --cc=lfqlee314@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=pabeni@redhat.com \
    --cc=roopa@nvidia.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox