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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Apr 16, 2022 at 11:13 AM Bagas Sanjaya <bagasdotme@xxxxxxxxx> wrote:
>
> On 4/15/22 15:34, Arun Ajith S wrote:
> > +accept_unsolicited_na - BOOLEAN
> > +     Add a new neighbour cache entry in STALE state for routers on receiving an
> > +     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
> > +      ====   ======  ======  ==============================================
> > +
> > +     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
> > +     buffer the initial return packets to do neighbour-solicitation.
> > +     The prerequisite is that the host is configured to send
> > +     unsolicited neighbour advertisements on interface bringup.
> > +     This setting should be used in conjunction with the ndisc_notify setting
> > +     on the host to satisfy this prerequisite.
> > +
> > +     By default this is turned off.
> > +
>
> Looks good. htmldocs builds successfully and the table displayed properly.
>
> Reviewed-by: Bagas Sanjaya <bagasdotme@xxxxxxxxx>
>
> However, I remind you the following:
>
> - The patch changelogs should be put between the dashes (---) and diffstat.
>   I don't see the changelogs when I hit reply-all because you put them as
>   message signature (at very bottom of patch message).
> - DON'T DO top-posting, DO configure your MUA to make reply text below
>   the quoted text instead.
>
> --
> An old man doll... just what I always wanted! - Clara
Thank you.
I will keep the tips for the Changelog in my mind for my next patch.



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux FS]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux