* [XFRM]: xfrm_state_clone() should be static, not exported
@ 2008-01-08 8:00 Eric Dumazet
2008-01-08 9:36 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Eric Dumazet @ 2008-01-08 8:00 UTC (permalink / raw)
To: David S. Miller; +Cc: Linux Netdev List
[-- Attachment #1: Type: text/plain, Size: 313 bytes --]
xfrm_state_clone() is not used outside of net/xfrm/xfrm_state.c
There is no need to export it.
Spoted by sparse checker.
CHECK net/xfrm/xfrm_state.c
net/xfrm/xfrm_state.c:1103:19: warning: symbol 'xfrm_state_clone' was not
declared. Should it be static?
Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
[-- Attachment #2: xfrm_state.patch --]
[-- Type: text/plain, Size: 716 bytes --]
diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c
index 81b0fce..5fc8217 100644
--- a/net/xfrm/xfrm_state.c
+++ b/net/xfrm/xfrm_state.c
@@ -1100,7 +1100,7 @@ out:
EXPORT_SYMBOL(xfrm_state_add);
#ifdef CONFIG_XFRM_MIGRATE
-struct xfrm_state *xfrm_state_clone(struct xfrm_state *orig, int *errp)
+static struct xfrm_state *xfrm_state_clone(struct xfrm_state *orig, int *errp)
{
int err = -ENOMEM;
struct xfrm_state *x = xfrm_state_alloc();
@@ -1175,7 +1175,6 @@ struct xfrm_state *xfrm_state_clone(struct xfrm_state *orig, int *errp)
kfree(x);
return NULL;
}
-EXPORT_SYMBOL(xfrm_state_clone);
/* xfrm_state_lock is held */
struct xfrm_state * xfrm_migrate_state_find(struct xfrm_migrate *m)
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [XFRM]: xfrm_state_clone() should be static, not exported
2008-01-08 8:00 [XFRM]: xfrm_state_clone() should be static, not exported Eric Dumazet
@ 2008-01-08 9:36 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2008-01-08 9:36 UTC (permalink / raw)
To: dada1; +Cc: netdev
From: Eric Dumazet <dada1@cosmosbay.com>
Date: Tue, 08 Jan 2008 09:00:20 +0100
> xfrm_state_clone() is not used outside of net/xfrm/xfrm_state.c
> There is no need to export it.
>
> Spoted by sparse checker.
> CHECK net/xfrm/xfrm_state.c
> net/xfrm/xfrm_state.c:1103:19: warning: symbol 'xfrm_state_clone' was not
> declared. Should it be static?
>
> Signed-off-by: Eric Dumazet <dada1@cosmosbay.com>
Applied to net-2.6.25, thanks Eric.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-01-08 9:36 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-08 8:00 [XFRM]: xfrm_state_clone() should be static, not exported Eric Dumazet
2008-01-08 9:36 ` David Miller
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).