netdev - Re: [PATCH net v3] net/ipv6: Expand and rename accept_unsolicited_na to accept_untracked

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

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>