[VLAN]: Add a net argument to proc init and cleanup calls.
authorPavel Emelyanov <xemul@openvz.org>
Wed, 16 Apr 2008 07:51:12 +0000 (00:51 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 16 Apr 2008 07:51:12 +0000 (00:51 -0700)
All proc files will be created in each net, so prepare them for
this change now, not to mess it with real creation patch.

The net != &init_net checks in them are for git-bisect sanity,
but I will drop them soon.

Signed-off-by: Pavel Emelyanov <xemul@openvz.org>
Acked-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/8021q/vlan.c
net/8021q/vlanproc.c
net/8021q/vlanproc.h

index 92966016a221fe0a3b51d6249e6cd61601c369b6..541542e2a2c1383171b08216a4ca0ce55d1c9ba0 100644 (file)
@@ -642,8 +642,14 @@ static int vlan_init_net(struct net *net)
        if (err < 0)
                goto err_assign;
 
+       err = vlan_proc_init(net);
+       if (err < 0)
+               goto err_proc;
+
        return 0;
 
+err_proc:
+       /* nothing */
 err_assign:
        kfree(vn);
 err_alloc:
@@ -655,6 +661,7 @@ static void vlan_exit_net(struct net *net)
        struct vlan_net *vn;
 
        vn = net_generic(net, vlan_net_id);
+       vlan_proc_cleanup(net);
        kfree(vn);
 }
 
@@ -674,10 +681,6 @@ static int __init vlan_proto_init(void)
        if (err < 0)
                goto err0;
 
-       err = vlan_proc_init();
-       if (err < 0)
-               goto err1;
-
        err = register_netdevice_notifier(&vlan_notifier_block);
        if (err < 0)
                goto err2;
@@ -693,8 +696,6 @@ static int __init vlan_proto_init(void)
 err3:
        unregister_netdevice_notifier(&vlan_notifier_block);
 err2:
-       vlan_proc_cleanup();
-err1:
        unregister_pernet_gen_device(vlan_net_id, &vlan_net_ops);
 err0:
        return err;
@@ -715,8 +716,6 @@ static void __exit vlan_cleanup_module(void)
        for (i = 0; i < VLAN_GRP_HASH_SIZE; i++)
                BUG_ON(!hlist_empty(&vlan_group_hash[i]));
 
-       vlan_proc_cleanup();
-
        unregister_pernet_gen_device(vlan_net_id, &vlan_net_ops);
 
        synchronize_net();
index 24cd96ee8109d6d675ef2ea776af2a87575d461c..4d13aeb885842dcf243b0d9ca6cb894cfdb0e291 100644 (file)
@@ -138,8 +138,11 @@ static const char *vlan_name_type_str[VLAN_NAME_TYPE_HIGHEST] = {
  *     Clean up /proc/net/vlan entries
  */
 
-void vlan_proc_cleanup(void)
+void vlan_proc_cleanup(struct net *net)
 {
+       if (net != &init_net)
+               return;
+
        if (proc_vlan_conf)
                remove_proc_entry(name_conf, proc_vlan_dir);
 
@@ -155,8 +158,11 @@ void vlan_proc_cleanup(void)
  *     Create /proc/net/vlan entries
  */
 
-int __init vlan_proc_init(void)
+int vlan_proc_init(struct net *net)
 {
+       if (net != &init_net)
+               return 0;
+
        proc_vlan_dir = proc_mkdir(name_root, init_net.proc_net);
        if (!proc_vlan_dir)
                goto err;
@@ -169,7 +175,7 @@ int __init vlan_proc_init(void)
 
 err:
        pr_err("%s: can't create entry in proc filesystem!\n", __func__);
-       vlan_proc_cleanup();
+       vlan_proc_cleanup(net);
        return -ENOBUFS;
 }
 
index da542cacc5a5ea3980a80286a96d79d1828da2f0..063f60a3d5cc2ee05c724dd76e647596d677fddf 100644 (file)
@@ -2,15 +2,17 @@
 #define __BEN_VLAN_PROC_INC__
 
 #ifdef CONFIG_PROC_FS
-int vlan_proc_init(void);
+struct net;
+
+int vlan_proc_init(struct net *net);
 int vlan_proc_rem_dev(struct net_device *vlandev);
 int vlan_proc_add_dev(struct net_device *vlandev);
-void vlan_proc_cleanup(void);
+void vlan_proc_cleanup(struct net *net);
 
 #else /* No CONFIG_PROC_FS */
 
-#define vlan_proc_init()       (0)
-#define vlan_proc_cleanup()    do {} while (0)
+#define vlan_proc_init(net)    (0)
+#define vlan_proc_cleanup(net) do {} while (0)
 #define vlan_proc_add_dev(dev) ({(void)(dev), 0; })
 #define vlan_proc_rem_dev(dev) ({(void)(dev), 0; })
 #endif