]> granicus.if.org Git - ipset/commitdiff
net: Convert ip_set_net_ops
authorKirill Tkhai <ktkhai@virtuozzo.com>
Mon, 22 Oct 2018 18:46:53 +0000 (20:46 +0200)
committerJozsef Kadlecsik <kadlec@blackhole.kfki.hu>
Mon, 22 Oct 2018 18:46:53 +0000 (20:46 +0200)
These pernet_operations initialize and destroy
net_generic(net, ip_set_net_id)-related data.
Since ip_set is under CONFIG_IP_SET, it's easy
to watch drivers, which depend on this config.
All of them are in net/netfilter/ipset directory,
except of net/netfilter/xt_set.c. There are no
more drivers, which use ip_set, and all of
the above don't register another pernet_operations.
Also, there are is no indirect users, as header
file include/linux/netfilter/ipset/ip_set.h does
not define indirect users by something like this:

        #ifdef CONFIG_IP_SET
        extern func(void);
        #else
        static inline func(void);
        #endif

So, there are no more pernet operations, dereferencing
net_generic(net, ip_set_net_id).

ip_set_net_ops are OK to be executed in parallel
for several net, so we mark them as async.

Signed-off-by: Kirill Tkhai <ktkhai@virtuozzo.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Jozsef Kadlecsik <kadlec@blackhole.kfki.hu>
kernel/net/netfilter/ipset/ip_set_core.c

index 9dccdf38027bf96bbb11706971d4e4e07b43acd5..65628ae8c4c678fa92df2b3b0b3fdb18738d6c0a 100644 (file)
@@ -2159,7 +2159,10 @@ static struct pernet_operations ip_set_net_ops = {
        .exit   = ip_set_net_exit,
 #ifdef HAVE_NET_OPS_ID
        .id     = &ip_set_net_id,
-       .size   = sizeof(struct ip_set_net)
+       .size   = sizeof(struct ip_set_net),
+#ifdef HAVE_NET_OPS_ASYNC
+       .async  = true,
+#endif
 #endif
 };