Skip to content

Commit

Permalink
[IPV6]: create route6 proc init-fini functions
Browse files Browse the repository at this point in the history
Make the proc creation/destruction to be a separate function. That
allows to remove the #ifdef CONFIG_PROC_FS in the init/fini function
and make them more readable.

Signed-off-by: Daniel Lezcano <dlezcano@fr.ibm.com>
Acked-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Daniel Lezcano authored and David S. Miller committed Jan 28, 2008
1 parent b8e1f9b commit 75314fb
Showing 1 changed file with 40 additions and 18 deletions.
58 changes: 40 additions & 18 deletions net/ipv6/route.c
Original file line number Diff line number Diff line change
Expand Up @@ -2358,6 +2358,40 @@ static const struct file_operations rt6_stats_seq_fops = {
.llseek = seq_lseek,
.release = single_release,
};

static int ipv6_route_proc_init(struct net *net)
{
int ret = -ENOMEM;
if (!proc_net_fops_create(net, "ipv6_route",
0, &ipv6_route_proc_fops))
goto out;

if (!proc_net_fops_create(net, "rt6_stats",
S_IRUGO, &rt6_stats_seq_fops))
goto out_ipv6_route;

ret = 0;
out:
return ret;
out_ipv6_route:
proc_net_remove(net, "ipv6_route");
goto out;
}

static void ipv6_route_proc_fini(struct net *net)
{
proc_net_remove(net, "ipv6_route");
proc_net_remove(net, "rt6_stats");
}
#else
static inline int ipv6_route_proc_init(struct net *net)
{
return 0;
}
static inline void ipv6_route_proc_fini(struct net *net)
{
return ;
}
#endif /* CONFIG_PROC_FS */

#ifdef CONFIG_SYSCTL
Expand Down Expand Up @@ -2484,21 +2518,14 @@ int __init ip6_route_init(void)
if (ret)
goto out_kmem_cache;

#ifdef CONFIG_PROC_FS
ret = -ENOMEM;
if (!proc_net_fops_create(&init_net, "ipv6_route",
0, &ipv6_route_proc_fops))
ret = ipv6_route_proc_init(&init_net);
if (ret)
goto out_fib6_init;

if (!proc_net_fops_create(&init_net, "rt6_stats",
S_IRUGO, &rt6_stats_seq_fops))
goto out_proc_ipv6_route;
#endif

#ifdef CONFIG_XFRM
ret = xfrm6_init();
if (ret)
goto out_proc_rt6_stats;
goto out_proc_init;
#endif
#ifdef CONFIG_IPV6_MULTIPLE_TABLES
ret = fib6_rules_init();
Expand All @@ -2522,14 +2549,10 @@ int __init ip6_route_init(void)
#endif
#ifdef CONFIG_XFRM
xfrm6_fini();
out_proc_rt6_stats:
#endif
#ifdef CONFIG_PROC_FS
proc_net_remove(&init_net, "rt6_stats");
out_proc_ipv6_route:
proc_net_remove(&init_net, "ipv6_route");
out_proc_init:
ipv6_route_proc_fini(&init_net);
out_fib6_init:
#endif
rt6_ifdown(NULL);
fib6_gc_cleanup();
out_kmem_cache:
Expand All @@ -2542,8 +2565,7 @@ void ip6_route_cleanup(void)
#ifdef CONFIG_IPV6_MULTIPLE_TABLES
fib6_rules_cleanup();
#endif
proc_net_remove(&init_net, "ipv6_route");
proc_net_remove(&init_net, "rt6_stats");
ipv6_route_proc_fini(&init_net);
#ifdef CONFIG_XFRM
xfrm6_fini();
#endif
Expand Down

0 comments on commit 75314fb

Please sign in to comment.