diff mbox series

[net-next] net: Convert can_pernet_ops

Message ID 152145954531.26024.3744883546191004693.stgit@localhost.localdomain
State Accepted, archived
Delegated to: David Miller
Headers show
Series [net-next] net: Convert can_pernet_ops | expand

Commit Message

Kirill Tkhai March 19, 2018, 11:39 a.m. UTC 
These pernet_operations create and destroy /proc entries
and cancel per-net timer.

Also, there are unneed iterations over empty list of net
devices, since all net devices must be already moved
to init_net or unregistered by default_device_ops. This
already was mentioned here:

https://marc.info/?l=linux-can&m=150169589119335&w=2

So, it looks safe to make them async.

Signed-off-by: Kirill Tkhai <ktkhai@virtuozzo.com>
---
 net/can/af_can.c |    1 +
 1 file changed, 1 insertion(+)

Comments

David Miller March 22, 2018, 3:14 p.m. UTC | #1 
From: Kirill Tkhai <ktkhai@virtuozzo.com>
Date: Mon, 19 Mar 2018 14:39:05 +0300

> These pernet_operations create and destroy /proc entries
> and cancel per-net timer.
> 
> Also, there are unneed iterations over empty list of net
> devices, since all net devices must be already moved
> to init_net or unregistered by default_device_ops. This
> already was mentioned here:
> 
> https://marc.info/?l=linux-can&m=150169589119335&w=2
> 
> So, it looks safe to make them async.
> 
> Signed-off-by: Kirill Tkhai <ktkhai@virtuozzo.com>

Applied.
diff mbox series

Patch

diff --git a/net/can/af_can.c b/net/can/af_can.c
index 6da324550eec..e899970398a1 100644
--- a/net/can/af_can.c
+++ b/net/can/af_can.c
@@ -954,6 +954,7 @@  static struct notifier_block can_netdev_notifier __read_mostly = {
 static struct pernet_operations can_pernet_ops __read_mostly = {
 	.init = can_pernet_init,
 	.exit = can_pernet_exit,
+	.async = true,
 };
 
 static __init int can_init(void)