From: Petr Machata Date: Wed, 21 Jan 2026 16:43:40 +0000 (+0100) Subject: net: core: neighbour: Reorder netlink & internal notification X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=795258891c944f37075ab8d3bc7415f5875e897a;p=thirdparty%2Fkernel%2Flinux.git net: core: neighbour: Reorder netlink & internal notification The netlink message needs to be send inside the critical section where the neighbor is changed, so that it reflects the notified-upon neighbor state. On the other hand, there is no such need in case of notifier chain: the listeners do not assume lock, and often in fact just schedule a delayed work to act on the neighbor later. At least one in fact also takes the neighbor lock. This requires that the netlink notification be done before the internal notifier chain message is sent. That is safe to do, because the current listeners, as well as __neigh_notify(), only read the updated neighbor fields, and never modify them. (Apart from locking.) Signed-off-by: Petr Machata Reviewed-by: Ido Schimmel Reviewed-by: Simon Horman Link: https://patch.msgid.link/f3ef74d5460f14c4d102b8a5857d4a6624da9a5a.1769012464.git.petrm@nvidia.com Signed-off-by: Jakub Kicinski --- diff --git a/net/core/neighbour.c b/net/core/neighbour.c index 0fbdaae7df99f..187615bcc1c39 100644 --- a/net/core/neighbour.c +++ b/net/core/neighbour.c @@ -1187,8 +1187,8 @@ out: } if (notify) { - call_netevent_notifiers(NETEVENT_NEIGH_UPDATE, neigh); neigh_notify(neigh, RTM_NEWNEIGH, 0, 0); + call_netevent_notifiers(NETEVENT_NEIGH_UPDATE, neigh); } trace_neigh_timer_handler(neigh, 0); @@ -1523,8 +1523,8 @@ out: neigh_update_managed_list(neigh); if (notify) { - call_netevent_notifiers(NETEVENT_NEIGH_UPDATE, neigh); neigh_notify(neigh, RTM_NEWNEIGH, 0, nlmsg_pid); + call_netevent_notifiers(NETEVENT_NEIGH_UPDATE, neigh); } trace_neigh_update_done(neigh, err);