There is no need to call synchronize_net() at swapping.

Ongoing add/del can happen to referenced sets and delete can be issued
to unreferenced sets. So the bogus call to synchronize_net() can safely
be removed.

diff --git a/kernel/ip_set_core.c b/kernel/ip_set_core.c
index 21cfc0f4b3dee6ded68f07b86b15449bb97be313..4634fd6df56e254d5bfed91def5683b7e87dcba7 100644 (file)
--- a/kernel/ip_set_core.c
+++ b/kernel/ip_set_core.c
@@ -825,10 +825,6 @@ ip_set_swap(struct sock *ctnl, struct sk_buff *skb,
        ip_set_list[from_id] = to;
        ip_set_list[to_id] = from;
 
-       /* Avoid possible race between ongoing slow add/del in kernel space
-        * and next destroy command. */
-       synchronize_net();
-
        return 0;
 }