Commit 946d1a92 authored by Eric W. Biederman's avatar Eric W. Biederman Committed by David S. Miller

net: Simplify vlan pernet operations.

Take advantage of the new pernet automatic storage management,
and stop using compatibility network namespace functions.
Signed-off-by: default avatarEric W. Biederman <ebiederm@xmission.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 15449745
...@@ -686,47 +686,28 @@ static int vlan_ioctl_handler(struct net *net, void __user *arg) ...@@ -686,47 +686,28 @@ static int vlan_ioctl_handler(struct net *net, void __user *arg)
static int vlan_init_net(struct net *net) static int vlan_init_net(struct net *net)
{ {
struct vlan_net *vn = net_generic(net, vlan_net_id);
int err; int err;
struct vlan_net *vn;
err = -ENOMEM;
vn = kzalloc(sizeof(struct vlan_net), GFP_KERNEL);
if (vn == NULL)
goto err_alloc;
err = net_assign_generic(net, vlan_net_id, vn);
if (err < 0)
goto err_assign;
vn->name_type = VLAN_NAME_TYPE_RAW_PLUS_VID_NO_PAD; vn->name_type = VLAN_NAME_TYPE_RAW_PLUS_VID_NO_PAD;
err = vlan_proc_init(net); err = vlan_proc_init(net);
if (err < 0)
goto err_proc;
return 0;
err_proc:
/* nothing */
err_assign:
kfree(vn);
err_alloc:
return err; return err;
} }
static void vlan_exit_net(struct net *net) static void vlan_exit_net(struct net *net)
{ {
struct vlan_net *vn;
vn = net_generic(net, vlan_net_id);
rtnl_kill_links(net, &vlan_link_ops); rtnl_kill_links(net, &vlan_link_ops);
vlan_proc_cleanup(net); vlan_proc_cleanup(net);
kfree(vn);
} }
static struct pernet_operations vlan_net_ops = { static struct pernet_operations vlan_net_ops = {
.init = vlan_init_net, .init = vlan_init_net,
.exit = vlan_exit_net, .exit = vlan_exit_net,
.id = &vlan_net_id,
.size = sizeof(struct vlan_net),
}; };
static int __init vlan_proto_init(void) static int __init vlan_proto_init(void)
...@@ -736,7 +717,7 @@ static int __init vlan_proto_init(void) ...@@ -736,7 +717,7 @@ static int __init vlan_proto_init(void)
pr_info("%s v%s %s\n", vlan_fullname, vlan_version, vlan_copyright); pr_info("%s v%s %s\n", vlan_fullname, vlan_version, vlan_copyright);
pr_info("All bugs added by %s\n", vlan_buggyright); pr_info("All bugs added by %s\n", vlan_buggyright);
err = register_pernet_gen_device(&vlan_net_id, &vlan_net_ops); err = register_pernet_device(&vlan_net_ops);
if (err < 0) if (err < 0)
goto err0; goto err0;
...@@ -761,7 +742,7 @@ static int __init vlan_proto_init(void) ...@@ -761,7 +742,7 @@ static int __init vlan_proto_init(void)
err3: err3:
unregister_netdevice_notifier(&vlan_notifier_block); unregister_netdevice_notifier(&vlan_notifier_block);
err2: err2:
unregister_pernet_gen_device(vlan_net_id, &vlan_net_ops); unregister_pernet_device(&vlan_net_ops);
err0: err0:
return err; return err;
} }
...@@ -781,7 +762,7 @@ static void __exit vlan_cleanup_module(void) ...@@ -781,7 +762,7 @@ static void __exit vlan_cleanup_module(void)
for (i = 0; i < VLAN_GRP_HASH_SIZE; i++) for (i = 0; i < VLAN_GRP_HASH_SIZE; i++)
BUG_ON(!hlist_empty(&vlan_group_hash[i])); BUG_ON(!hlist_empty(&vlan_group_hash[i]));
unregister_pernet_gen_device(vlan_net_id, &vlan_net_ops); unregister_pernet_device(&vlan_net_ops);
rcu_barrier(); /* Wait for completion of call_rcu()'s */ rcu_barrier(); /* Wait for completion of call_rcu()'s */
vlan_gvrp_uninit(); vlan_gvrp_uninit();
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment