net: push code from net notifier reg/unreg into helpers
authorJiri Pirko <jiri@mellanox.com>
Sat, 25 Jan 2020 11:17:07 +0000 (12:17 +0100)
committerDavid S. Miller <davem@davemloft.net>
Mon, 27 Jan 2020 10:03:44 +0000 (11:03 +0100)
Push the code which is done under rtnl lock in net notifier register and
unregister function into separate helpers.

Signed-off-by: Jiri Pirko <jiri@mellanox.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index ee4b1e64d66334e78f9ca8652a5188d82e3d1853..b521b509a6532cda9e934ba514cd6ec8d0b1deef 100644 (file)
@@ -1784,6 +1784,42 @@ unlock:
 }
 EXPORT_SYMBOL(unregister_netdevice_notifier);
 
+static int __register_netdevice_notifier_net(struct net *net,
+                                            struct notifier_block *nb,
+                                            bool ignore_call_fail)
+{
+       int err;
+
+       err = raw_notifier_chain_register(&net->netdev_chain, nb);
+       if (err)
+               return err;
+       if (dev_boot_phase)
+               return 0;
+
+       err = call_netdevice_register_net_notifiers(nb, net);
+       if (err && !ignore_call_fail)
+               goto chain_unregister;
+
+       return 0;
+
+chain_unregister:
+       raw_notifier_chain_unregister(&net->netdev_chain, nb);
+       return err;
+}
+
+static int __unregister_netdevice_notifier_net(struct net *net,
+                                              struct notifier_block *nb)
+{
+       int err;
+
+       err = raw_notifier_chain_unregister(&net->netdev_chain, nb);
+       if (err)
+               return err;
+
+       call_netdevice_unregister_net_notifiers(nb, net);
+       return 0;
+}
+
 /**
  * register_netdevice_notifier_net - register a per-netns network notifier block
  * @net: network namespace
@@ -1804,23 +1840,9 @@ int register_netdevice_notifier_net(struct net *net, struct notifier_block *nb)
        int err;
 
        rtnl_lock();
-       err = raw_notifier_chain_register(&net->netdev_chain, nb);
-       if (err)
-               goto unlock;
-       if (dev_boot_phase)
-               goto unlock;
-
-       err = call_netdevice_register_net_notifiers(nb, net);
-       if (err)
-               goto chain_unregister;
-
-unlock:
+       err = __register_netdevice_notifier_net(net, nb, false);
        rtnl_unlock();
        return err;
-
-chain_unregister:
-       raw_notifier_chain_unregister(&netdev_chain, nb);
-       goto unlock;
 }
 EXPORT_SYMBOL(register_netdevice_notifier_net);
 
@@ -1846,13 +1868,7 @@ int unregister_netdevice_notifier_net(struct net *net,
        int err;
 
        rtnl_lock();
-       err = raw_notifier_chain_unregister(&net->netdev_chain, nb);
-       if (err)
-               goto unlock;
-
-       call_netdevice_unregister_net_notifiers(nb, net);
-
-unlock:
+       err = __unregister_netdevice_notifier_net(net, nb);
        rtnl_unlock();
        return err;
 }