netfilter: nf_ct_timeout: move initialization out of pernet_operations

Move the global initial codes to the module_init/exit context.

Signed-off-by: Gao feng <gaofeng@cn.fujitsu.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
This commit is contained in:
Gao feng 2013-01-21 22:10:29 +00:00 committed by Pablo Neira Ayuso
parent 3fe0f943d4
commit 8684094cf1
3 changed files with 19 additions and 27 deletions

View File

@ -76,15 +76,15 @@ nf_ct_timeout_lookup(struct net *net, struct nf_conn *ct,
} }
#ifdef CONFIG_NF_CONNTRACK_TIMEOUT #ifdef CONFIG_NF_CONNTRACK_TIMEOUT
extern int nf_conntrack_timeout_init(struct net *net); extern int nf_conntrack_timeout_init(void);
extern void nf_conntrack_timeout_fini(struct net *net); extern void nf_conntrack_timeout_fini(void);
#else #else
static inline int nf_conntrack_timeout_init(struct net *net) static inline int nf_conntrack_timeout_init(void)
{ {
return 0; return 0;
} }
static inline void nf_conntrack_timeout_fini(struct net *net) static inline void nf_conntrack_timeout_fini(void)
{ {
return; return;
} }

View File

@ -1348,6 +1348,7 @@ void nf_conntrack_cleanup_end(void)
#ifdef CONFIG_NF_CONNTRACK_ZONES #ifdef CONFIG_NF_CONNTRACK_ZONES
nf_ct_extend_unregister(&nf_ct_zone_extend); nf_ct_extend_unregister(&nf_ct_zone_extend);
#endif #endif
nf_conntrack_timeout_fini();
nf_conntrack_ecache_fini(); nf_conntrack_ecache_fini();
nf_conntrack_tstamp_fini(); nf_conntrack_tstamp_fini();
nf_conntrack_acct_fini(); nf_conntrack_acct_fini();
@ -1378,7 +1379,6 @@ void nf_conntrack_cleanup_net(struct net *net)
nf_conntrack_proto_fini(net); nf_conntrack_proto_fini(net);
nf_conntrack_labels_fini(net); nf_conntrack_labels_fini(net);
nf_conntrack_helper_fini(net); nf_conntrack_helper_fini(net);
nf_conntrack_timeout_fini(net);
nf_conntrack_ecache_pernet_fini(net); nf_conntrack_ecache_pernet_fini(net);
nf_conntrack_tstamp_pernet_fini(net); nf_conntrack_tstamp_pernet_fini(net);
nf_conntrack_acct_pernet_fini(net); nf_conntrack_acct_pernet_fini(net);
@ -1522,6 +1522,10 @@ int nf_conntrack_init_start(void)
if (ret < 0) if (ret < 0)
goto err_ecache; goto err_ecache;
ret = nf_conntrack_timeout_init();
if (ret < 0)
goto err_timeout;
#ifdef CONFIG_NF_CONNTRACK_ZONES #ifdef CONFIG_NF_CONNTRACK_ZONES
ret = nf_ct_extend_register(&nf_ct_zone_extend); ret = nf_ct_extend_register(&nf_ct_zone_extend);
if (ret < 0) if (ret < 0)
@ -1539,8 +1543,10 @@ int nf_conntrack_init_start(void)
#ifdef CONFIG_NF_CONNTRACK_ZONES #ifdef CONFIG_NF_CONNTRACK_ZONES
err_extend: err_extend:
nf_conntrack_ecache_fini(); nf_conntrack_timeout_fini();
#endif #endif
err_timeout:
nf_conntrack_ecache_fini();
err_ecache: err_ecache:
nf_conntrack_tstamp_fini(); nf_conntrack_tstamp_fini();
err_tstamp: err_tstamp:
@ -1616,9 +1622,6 @@ int nf_conntrack_init_net(struct net *net)
ret = nf_conntrack_ecache_pernet_init(net); ret = nf_conntrack_ecache_pernet_init(net);
if (ret < 0) if (ret < 0)
goto err_ecache; goto err_ecache;
ret = nf_conntrack_timeout_init(net);
if (ret < 0)
goto err_timeout;
ret = nf_conntrack_helper_init(net); ret = nf_conntrack_helper_init(net);
if (ret < 0) if (ret < 0)
goto err_helper; goto err_helper;
@ -1637,8 +1640,6 @@ err_proto:
err_labels: err_labels:
nf_conntrack_helper_fini(net); nf_conntrack_helper_fini(net);
err_helper: err_helper:
nf_conntrack_timeout_fini(net);
err_timeout:
nf_conntrack_ecache_pernet_fini(net); nf_conntrack_ecache_pernet_fini(net);
err_ecache: err_ecache:
nf_conntrack_tstamp_pernet_fini(net); nf_conntrack_tstamp_pernet_fini(net);

View File

@ -37,24 +37,15 @@ static struct nf_ct_ext_type timeout_extend __read_mostly = {
.id = NF_CT_EXT_TIMEOUT, .id = NF_CT_EXT_TIMEOUT,
}; };
int nf_conntrack_timeout_init(struct net *net) int nf_conntrack_timeout_init(void)
{ {
int ret = 0; int ret = nf_ct_extend_register(&timeout_extend);
if (ret < 0)
if (net_eq(net, &init_net)) { pr_err("nf_ct_timeout: Unable to register timeout extension.\n");
ret = nf_ct_extend_register(&timeout_extend); return ret;
if (ret < 0) {
printk(KERN_ERR "nf_ct_timeout: Unable to register "
"timeout extension.\n");
return ret;
}
}
return 0;
} }
void nf_conntrack_timeout_fini(struct net *net) void nf_conntrack_timeout_fini(void)
{ {
if (net_eq(net, &init_net)) nf_ct_extend_unregister(&timeout_extend);
nf_ct_extend_unregister(&timeout_extend);
} }