Skip to content

neighbor: More gc_list changes

David Ahern netdev 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest5 patches 1 replyIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/5neighbor: Fix locking order for gc_list changesnet/core/neighbour.c+15 -120 replies
2/5neighbor: Fix state check in neigh_forced_gcnet/core/neighbour.c+2 -30 replies
3/5neighbor: Remove state and flags arguments to neigh_delnet/core/neighbour.c+4 -50 replies
4/5neighbor: Move neigh_update_ext_learned to core fileinclude/net/neighbour.h net/core/neighbour.c+18 -180 replies
5/5neighbor: Remove externally learned entries from gc_listnet/core/neighbour.c+28 -210 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH net-next 0/5] neighbor: More gc_list changes

From: David Ahern <[email protected]>

More gc_list changes and cleanups.

The first 2 patches are bug fixes from the first gc_list change. Specifically, fix the locking order to be consistent - table lock followed by neighbor lock, and then entries in the FAILED state should always be candidates for forced_gc without waiting for any time span (return to the eviction logic prior to the separate gc_list).

Patch 3 removes 2 now unnecessary arguments to neigh_del.

Patch 4 moves a helper from a header file to core code in preparation for Patch 5 which removes NTF_EXT_LEARNED entries from the gc_list. These entries are already exempt from forced_gc; patch 5 removes them from consideration and makes them on par with PERMANENT entries given that they are also managed by userspace.

David Ahern (5):
neighbor: Fix locking order for gc_list changes
neighbor: Fix state check in neigh_forced_gc
neighbor: Remove state and flags arguments to neigh_del
neighbor: Move neigh_update_ext_learned to core file
neighbor: Remove externally learned entries from gc_list

 include/net/neighbour.h | 18 ----------
 net/core/neighbour.c    | 94 +++++++++++++++++++++++++++++++------------------
 2 files changed, 60 insertions(+), 52 deletions(-)
Signature · 1 line
2.11.0