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

net: Simplify ip6_tunnel 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 86de8a63
...@@ -1409,17 +1409,8 @@ static void ip6_tnl_destroy_tunnels(struct ip6_tnl_net *ip6n) ...@@ -1409,17 +1409,8 @@ static void ip6_tnl_destroy_tunnels(struct ip6_tnl_net *ip6n)
static int ip6_tnl_init_net(struct net *net) static int ip6_tnl_init_net(struct net *net)
{ {
struct ip6_tnl_net *ip6n = net_generic(net, ip6_tnl_net_id);
int err; int err;
struct ip6_tnl_net *ip6n;
err = -ENOMEM;
ip6n = kzalloc(sizeof(struct ip6_tnl_net), GFP_KERNEL);
if (ip6n == NULL)
goto err_alloc;
err = net_assign_generic(net, ip6_tnl_net_id, ip6n);
if (err < 0)
goto err_assign;
ip6n->tnls[0] = ip6n->tnls_wc; ip6n->tnls[0] = ip6n->tnls_wc;
ip6n->tnls[1] = ip6n->tnls_r_l; ip6n->tnls[1] = ip6n->tnls_r_l;
...@@ -1442,27 +1433,23 @@ static int ip6_tnl_init_net(struct net *net) ...@@ -1442,27 +1433,23 @@ static int ip6_tnl_init_net(struct net *net)
err_register: err_register:
free_netdev(ip6n->fb_tnl_dev); free_netdev(ip6n->fb_tnl_dev);
err_alloc_dev: err_alloc_dev:
/* nothing */
err_assign:
kfree(ip6n);
err_alloc:
return err; return err;
} }
static void ip6_tnl_exit_net(struct net *net) static void ip6_tnl_exit_net(struct net *net)
{ {
struct ip6_tnl_net *ip6n; struct ip6_tnl_net *ip6n = net_generic(net, ip6_tnl_net_id);
ip6n = net_generic(net, ip6_tnl_net_id);
rtnl_lock(); rtnl_lock();
ip6_tnl_destroy_tunnels(ip6n); ip6_tnl_destroy_tunnels(ip6n);
rtnl_unlock(); rtnl_unlock();
kfree(ip6n);
} }
static struct pernet_operations ip6_tnl_net_ops = { static struct pernet_operations ip6_tnl_net_ops = {
.init = ip6_tnl_init_net, .init = ip6_tnl_init_net,
.exit = ip6_tnl_exit_net, .exit = ip6_tnl_exit_net,
.id = &ip6_tnl_net_id,
.size = sizeof(struct ip6_tnl_net),
}; };
/** /**
...@@ -1487,7 +1474,7 @@ static int __init ip6_tunnel_init(void) ...@@ -1487,7 +1474,7 @@ static int __init ip6_tunnel_init(void)
goto unreg_ip4ip6; goto unreg_ip4ip6;
} }
err = register_pernet_gen_device(&ip6_tnl_net_id, &ip6_tnl_net_ops); err = register_pernet_device(&ip6_tnl_net_ops);
if (err < 0) if (err < 0)
goto err_pernet; goto err_pernet;
return 0; return 0;
...@@ -1511,7 +1498,7 @@ static void __exit ip6_tunnel_cleanup(void) ...@@ -1511,7 +1498,7 @@ static void __exit ip6_tunnel_cleanup(void)
if (xfrm6_tunnel_deregister(&ip6ip6_handler, AF_INET6)) if (xfrm6_tunnel_deregister(&ip6ip6_handler, AF_INET6))
printk(KERN_INFO "ip6_tunnel close: can't deregister ip6ip6\n"); printk(KERN_INFO "ip6_tunnel close: can't deregister ip6ip6\n");
unregister_pernet_gen_device(ip6_tnl_net_id, &ip6_tnl_net_ops); unregister_pernet_device(&ip6_tnl_net_ops);
} }
module_init(ip6_tunnel_init); module_init(ip6_tunnel_init);
......
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