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

net: Simplify pppoe 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 741a6fa2
...@@ -1139,59 +1139,37 @@ static struct pppox_proto pppoe_proto = { ...@@ -1139,59 +1139,37 @@ static struct pppox_proto pppoe_proto = {
static __net_init int pppoe_init_net(struct net *net) static __net_init int pppoe_init_net(struct net *net)
{ {
struct pppoe_net *pn; struct pppoe_net *pn = pppoe_pernet(net);
struct proc_dir_entry *pde; struct proc_dir_entry *pde;
int err;
pn = kzalloc(sizeof(*pn), GFP_KERNEL);
if (!pn)
return -ENOMEM;
rwlock_init(&pn->hash_lock); rwlock_init(&pn->hash_lock);
err = net_assign_generic(net, pppoe_net_id, pn);
if (err)
goto out;
pde = proc_net_fops_create(net, "pppoe", S_IRUGO, &pppoe_seq_fops); pde = proc_net_fops_create(net, "pppoe", S_IRUGO, &pppoe_seq_fops);
#ifdef CONFIG_PROC_FS #ifdef CONFIG_PROC_FS
if (!pde) { if (!pde)
err = -ENOMEM; return -ENOMEM;
goto out;
}
#endif #endif
return 0; return 0;
out:
kfree(pn);
return err;
} }
static __net_exit void pppoe_exit_net(struct net *net) static __net_exit void pppoe_exit_net(struct net *net)
{ {
struct pppoe_net *pn;
proc_net_remove(net, "pppoe"); proc_net_remove(net, "pppoe");
pn = net_generic(net, pppoe_net_id);
/*
* if someone has cached our net then
* further net_generic call will return NULL
*/
net_assign_generic(net, pppoe_net_id, NULL);
kfree(pn);
} }
static struct pernet_operations pppoe_net_ops = { static struct pernet_operations pppoe_net_ops = {
.init = pppoe_init_net, .init = pppoe_init_net,
.exit = pppoe_exit_net, .exit = pppoe_exit_net,
.id = &pppoe_net_id,
.size = sizeof(struct pppoe_net),
}; };
static int __init pppoe_init(void) static int __init pppoe_init(void)
{ {
int err; int err;
err = register_pernet_gen_device(&pppoe_net_id, &pppoe_net_ops); err = register_pernet_device(&pppoe_net_ops);
if (err) if (err)
goto out; goto out;
...@@ -1212,7 +1190,7 @@ static int __init pppoe_init(void) ...@@ -1212,7 +1190,7 @@ static int __init pppoe_init(void)
out_unregister_pppoe_proto: out_unregister_pppoe_proto:
proto_unregister(&pppoe_sk_proto); proto_unregister(&pppoe_sk_proto);
out_unregister_net_ops: out_unregister_net_ops:
unregister_pernet_gen_device(pppoe_net_id, &pppoe_net_ops); unregister_pernet_device(&pppoe_net_ops);
out: out:
return err; return err;
} }
...@@ -1224,7 +1202,7 @@ static void __exit pppoe_exit(void) ...@@ -1224,7 +1202,7 @@ static void __exit pppoe_exit(void)
dev_remove_pack(&pppoes_ptype); dev_remove_pack(&pppoes_ptype);
unregister_pppox_proto(PX_PROTO_OE); unregister_pppox_proto(PX_PROTO_OE);
proto_unregister(&pppoe_sk_proto); proto_unregister(&pppoe_sk_proto);
unregister_pernet_gen_device(pppoe_net_id, &pppoe_net_ops); unregister_pernet_device(&pppoe_net_ops);
} }
module_init(pppoe_init); module_init(pppoe_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