[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <f79c9fc6-9635-50a6-8b71-f280bf2a192b@kernel.org>
Date: Mon, 30 May 2022 10:22:12 -0600
From: David Ahern <dsahern@...nel.org>
To: Arun Ajith S <aajith@...sta.com>, netdev@...r.kernel.org
Cc: davem@...emloft.net, linux-kernel@...r.kernel.org,
linux-doc@...r.kernel.org, linux-kselftest@...r.kernel.org,
bagasdotme@...il.com, yoshfuji@...ux-ipv6.org, kuba@...nel.org,
pabeni@...hat.com, prestwoj@...il.com, corbet@....net,
justin.iurman@...ege.be, edumazet@...gle.com, shuah@...nel.org,
gilligan@...sta.com, noureddine@...sta.com, gk@...sta.com
Subject: Re: [PATCH net v3] net/ipv6: Expand and rename accept_unsolicited_na
to accept_untracked_na
On 5/30/22 4:14 AM, Arun Ajith S wrote:
> RFC 9131 changes default behaviour of handling RX of NA messages when the
> corresponding entry is absent in the neighbour cache. The current
> implementation is limited to accept just unsolicited NAs. However, the
> RFC is more generic where it also accepts solicited NAs. Both types
> should result in adding a STALE entry for this case.
>
> Expand accept_untracked_na behaviour to also accept solicited NAs to
> be compliant with the RFC and rename the sysctl knob to
> accept_untracked_na.
>
> Fixes: f9a2fb73318e ("net/ipv6: Introduce accept_unsolicited_na knob to implement router-side changes for RFC9131")
> Signed-off-by: Arun Ajith S <aajith@...sta.com>
> ---
> This change updates the accept_unsolicited_na feature that merged to net-next
> for v5.19 to be better compliant with the RFC. It also involves renaming the sysctl
> knob to accept_untracked_na before shipping in a release.
>
> Note that the behaviour table has been modifed in the code comments,
> but dropped from the Documentation. This is because the table
> documents behaviour that is not unique to the knob, and it is more
> relevant to understanding the code. The documentation has been updated
> to be unambiguous even without the table.
>
> v2:
> 1. Changed commit message and subject as suggested.
> 2. Added Fixes tag.
> 3. Used en-uk spellings consistently.
> 4. Added a couple of missing comments.
> 5. Refactored patch to be smaller by avoiding early return.
> 6. Made the documentation more clearer.
>
> v3:
> 1. Fixed build issue. (Verified make defconfig && make && make htmldocs SPHINXDIRS=networking)
>
> Documentation/networking/ip-sysctl.rst | 23 ++++------
> include/linux/ipv6.h | 2 +-
> include/uapi/linux/ipv6.h | 2 +-
> net/ipv6/addrconf.c | 6 +--
> net/ipv6/ndisc.c | 42 +++++++++++--------
> .../net/ndisc_unsolicited_na_test.sh | 23 +++++-----
> 6 files changed, 50 insertions(+), 48 deletions(-)
>
Reviewed-by: David Ahern <dsahern@...nel.org>
Powered by blists - more mailing lists