From: James Prestwood <prestwoj@gmail.com>
To: netdev@vger.kernel.org
Cc: James Prestwood <prestwoj@gmail.com>,
"David S . Miller" <davem@davemloft.net>,
Jakub Kicinski <kuba@kernel.org>,
Jonathan Corbet <corbet@lwn.net>,
Hideaki YOSHIFUJI <yoshfuji@linux-ipv6.org>,
David Ahern <dsahern@kernel.org>, Roopa Prabhu <roopa@nvidia.com>,
Daniel Borkmann <daniel@iogearbox.net>,
Vladimir Oltean <vladimir.oltean@nxp.com>,
Ido Schimmel <idosch@nvidia.com>,
Nikolay Aleksandrov <nikolay@nvidia.com>,
Chinmay Agarwal <chinagar@codeaurora.org>,
Yajun Deng <yajun.deng@linux.dev>, Tong Zhu <zhutong@amazon.com>,
Johannes Berg <johannes@sipsolutions.net>,
Jouni Malinen <jouni@codeaurora.org>
Subject: [PATCH v5 0/2] Make neighbor eviction controllable by userspace
Date: Wed, 20 Oct 2021 17:32:10 -0700 [thread overview]
Message-ID: <20211021003212.878786-1-prestwoj@gmail.com> (raw)
v1 -> v2:
- It was suggested by Daniel Borkmann to extend the neighbor table settings
rather than adding IPv4/IPv6 options for ARP/NDISC separately. I agree
this way is much more concise since there is now only one place where the
option is checked and defined.
- Moved documentation/code into the same patch
- Explained in more detail the test scenario and results
v2 -> v3:
- Renamed 'skip_perm' to 'nocarrier'. The way this parameter is used
matches this naming.
- Changed logic to still flush if 'nocarrier' is false.
v3 -> v4:
- Moved NDTPA_EVICT_NOCARRIER after NDTPA_PAD
v4 -> v5:
- Went back to the original v1 patchset and changed:
- Used ANDCONF for IN_DEV macro
- Got RCU lock prior to __in_dev_get_rcu(). Do note that the logic
here was extended to handle if __in_dev_get_rcu() fails. If this
happens the existing behavior should be maintained and set the
carrier down. I'm unsure if get_rcu() can fail in this context
though. Similar logic was used for in6_dev_get.
- Changed ndisc_evict_nocarrier to use a u8, proper handler, and
set min/max values.
Cc: David S. Miller <davem@davemloft.net>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Jonathan Corbet <corbet@lwn.net>
Cc: Hideaki YOSHIFUJI <yoshfuji@linux-ipv6.org>
Cc: David Ahern <dsahern@kernel.org>
Cc: Roopa Prabhu <roopa@nvidia.com>
Cc: Daniel Borkmann <daniel@iogearbox.net>
Cc: Vladimir Oltean <vladimir.oltean@nxp.com>
Cc: Ido Schimmel <idosch@nvidia.com>
Cc: Nikolay Aleksandrov <nikolay@nvidia.com>
Cc: Chinmay Agarwal <chinagar@codeaurora.org>
Cc: Yajun Deng <yajun.deng@linux.dev>
Cc: Tong Zhu <zhutong@amazon.com>
Cc: Johannes Berg <johannes@sipsolutions.net>
Cc: Jouni Malinen <jouni@codeaurora.org>
James Prestwood (2):
net: arp: introduce arp_evict_nocarrier sysctl parameter
net: ndisc: introduce ndisc_evict_nocarrier sysctl parameter
Documentation/networking/ip-sysctl.rst | 18 ++++++++++++++++++
include/linux/inetdevice.h | 2 ++
include/linux/ipv6.h | 1 +
include/uapi/linux/ip.h | 1 +
include/uapi/linux/ipv6.h | 1 +
include/uapi/linux/sysctl.h | 1 +
net/ipv4/arp.c | 13 ++++++++++++-
net/ipv4/devinet.c | 4 ++++
net/ipv6/addrconf.c | 12 ++++++++++++
net/ipv6/ndisc.c | 5 ++++-
10 files changed, 56 insertions(+), 2 deletions(-)
--
2.31.1
next reply other threads:[~2021-10-21 0:37 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-21 0:32 James Prestwood [this message]
2021-10-21 0:32 ` [PATCH v5 1/2] net: arp: introduce arp_evict_nocarrier sysctl parameter James Prestwood
2021-10-21 14:08 ` David Ahern
2021-10-21 0:32 ` [PATCH v5 2/2] net: ndisc: introduce ndisc_evict_nocarrier " James Prestwood
2021-10-21 14:15 ` David Ahern
2021-10-21 2:33 ` [PATCH v5 0/2] Make neighbor eviction controllable by userspace David Ahern
2021-10-21 21:59 ` James Prestwood
2021-10-21 22:09 ` David Ahern
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=20211021003212.878786-1-prestwoj@gmail.com \
--to=prestwoj@gmail.com \
--cc=chinagar@codeaurora.org \
--cc=corbet@lwn.net \
--cc=daniel@iogearbox.net \
--cc=davem@davemloft.net \
--cc=dsahern@kernel.org \
--cc=idosch@nvidia.com \
--cc=johannes@sipsolutions.net \
--cc=jouni@codeaurora.org \
--cc=kuba@kernel.org \
--cc=netdev@vger.kernel.org \
--cc=nikolay@nvidia.com \
--cc=roopa@nvidia.com \
--cc=vladimir.oltean@nxp.com \
--cc=yajun.deng@linux.dev \
--cc=yoshfuji@linux-ipv6.org \
--cc=zhutong@amazon.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;
as well as URLs for NNTP newsgroup(s).