[RFC PATCH v2] powerpc/pseries/mobility: notify network peers after migration
From: Thomas Falcon <hidden>
Date: 2019-11-06 17:55:27
Also in:
netdev
Subsystem:
linux for powerpc (32-bit and 64-bit), the rest · Maintainers:
Madhavan Srinivasan, Linus Torvalds
After a migration, it is necessary to send a gratuitous ARP from all running interfaces so that the rest of the network is aware of its new location. However, some supported network devices are unaware that they have been migrated. To avoid network interruptions and other unwanted behavior, force a GARP on all valid, running interfaces as part of the post_mobility_fixup routine. Signed-off-by: Thomas Falcon <redacted> --- v2: fix missing brackets caught by Russell Currey --- arch/powerpc/platforms/pseries/mobility.c | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+)
diff --git a/arch/powerpc/platforms/pseries/mobility.c b/arch/powerpc/platforms/pseries/mobility.c
index b571285..dddc3c1 100644
--- a/arch/powerpc/platforms/pseries/mobility.c
+++ b/arch/powerpc/platforms/pseries/mobility.c@@ -17,6 +17,9 @@ #include <linux/delay.h> #include <linux/slab.h> #include <linux/stringify.h> +#include <linux/netdevice.h> +#include <linux/rtnetlink.h> +#include <net/net_namespace.h> #include <asm/machdep.h> #include <asm/rtas.h>
@@ -331,6 +334,8 @@ void post_mobility_fixup(void) { int rc; int activate_fw_token; + struct net_device *netdev; + struct net *net; activate_fw_token = rtas_token("ibm,activate-firmware"); if (activate_fw_token == RTAS_UNKNOWN_SERVICE) {
@@ -371,6 +376,22 @@ void post_mobility_fixup(void) /* Possibly switch to a new RFI flush type */ pseries_setup_rfi_flush(); + /* need to force a gratuitous ARP on running interfaces */ + rtnl_lock(); + for_each_net(net) { + for_each_netdev(net, netdev) { + if (netif_device_present(netdev) && + netif_running(netdev) && + !(netdev->flags & (IFF_NOARP | IFF_LOOPBACK))) { + call_netdevice_notifiers(NETDEV_NOTIFY_PEERS, + netdev); + call_netdevice_notifiers(NETDEV_RESEND_IGMP, + netdev); + } + } + } + rtnl_unlock(); + return; }
--
1.8.3.1