netdev - Re: [PATCH net-next v4] net/ipv6: Introduce accept_unsolicited_na knob to implement router-side changes for RFC9131

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:   Thu, 14 Apr 2022 14:29:01 +0530
From:   Arun Ajith S <aajith@...sta.com>
To:     Bagas Sanjaya <bagasdotme@...il.com>
Cc:     netdev@...r.kernel.org, davem@...emloft.net,
        linux-kernel@...r.kernel.org, linux-doc@...r.kernel.org,
        dsahern@...nel.org, yoshfuji@...ux-ipv6.org, kuba@...nel.org,
        pabeni@...hat.com, corbet@....net, prestwoj@...il.com,
        gilligan@...sta.com, noureddine@...sta.com, gk@...sta.com
Subject: Re: [PATCH net-next v4] net/ipv6: Introduce accept_unsolicited_na
 knob to implement router-side changes for RFC9131

Thank you.
Do I have to post a v5 with the fixup ?

-Arun

-Arun


On Thu, Apr 14, 2022 at 11:42 AM Bagas Sanjaya <bagasdotme@...il.com> wrote:
>
> On 4/14/22 09:56, Arun Ajith S wrote:
> > Add a new neighbour cache entry in STALE state for routers on receiving
> > an unsolicited (gratuitous) neighbour advertisement with
> > target link-layer-address option specified.
> > This is similar to the arp_accept configuration for IPv4.
> > A new sysctl endpoint is created to turn on this behaviour:
> > /proc/sys/net/ipv6/conf/interface/accept_unsolicited_na.
> >
>
> Hi,
>
> Building the documentation (htmldocs) with this patch, I got:
>
> /home/bagas/repo/linux-stable/Documentation/networking/ip-sysctl.rst:2475:
> WARNING: Unexpected indentation.
> /home/bagas/repo/linux-stable/Documentation/networking/ip-sysctl.rst:2477:
> WARNING: Unexpected indentation.
> /home/bagas/repo/linux-stable/Documentation/networking/ip-sysctl.rst:2481:
> WARNING: Unexpected indentation.
> /home/bagas/repo/linux-stable/Documentation/networking/ip-sysctl.rst:2482:
> WARNING: Block quote ends without a blank line; unexpected unindent.
>
> I have applied following fixup.
>
> ---- 8> ----
> From 304846b43a9f962f53f3841afabfd597b3b80951 Mon Sep 17 00:00:00 2001
> From: Bagas Sanjaya <bagasdotme@...il.com>
> Date: Thu, 14 Apr 2022 12:59:46 +0700
> Subject: [PATCH] fixup for "net/ipv6: Introduce accept_unsolicited_na knob to
>  implement router-side changes for RFC9131"
>
> Fix the simple table syntax.
>
> Signed-off-by: Bagas Sanjaya <bagasdotme@...il.com>
> ---
>  Documentation/networking/ip-sysctl.rst | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/networking/ip-sysctl.rst b/Documentation/networking/ip-sysctl.rst
> index 9e17efe343a..433f2e4a5fe 100644
> --- a/Documentation/networking/ip-sysctl.rst
> +++ b/Documentation/networking/ip-sysctl.rst
> @@ -2472,13 +2472,17 @@ accept_unsolicited_na - BOOLEAN
>         unsolicited neighbour advertisement with target link-layer address option
>         specified. This is as per router-side behavior documented in RFC9131.
>         This has lower precedence than drop_unsolicited_na.
> +
> +        ====   ======  ======  ==============================================
>          drop   accept  fwding                   behaviour
>          ----   ------  ------  ----------------------------------------------
>             1        X       X  Drop NA packet and don't pass up the stack
>             0        0       X  Pass NA packet up the stack, don't update NC
>             0        1       0  Pass NA packet up the stack, don't update NC
>             0        1       1  Pass NA packet up the stack, and add a STALE
> -                                 NC entry
> +                               NC entry
> +        ====   ======  ======  ==============================================
> +
>         This will optimize the return path for the initial off-link communication
>         that is initiated by a directly connected host, by ensuring that
>         the first-hop router which turns on this setting doesn't have to
>
> base-commit: 38e01f46e0e7f88b92ca0b3f52ac6b9909ed413b
> --
> An old man doll... just what I always wanted! - Clara
>
> Thanks.
>
> --
> An old man doll... just what I always wanted! - Clara