From: Matt Kurz Date: Tue, 22 Jul 2014 20:10:33 +0000 (+1000) Subject: staging: lustre: Fix sparse warnings X-Git-Tag: v4.9.8~5976^2~356 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=c49d4b96ab6175fdcc3f858b0ef620f20ae16437;p=platform%2Fkernel%2Flinux-rpi3.git staging: lustre: Fix sparse warnings Declare several functions and variables as static in order to address the following sparse warnings drivers/staging/lustre/lnet/lnet/acceptor.c:49:3: warning: symbol 'lnet_acceptor_state' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/acceptor.c:79:1: warning: symbol 'lnet_acceptor_get_tunables' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/acceptor.c:211:1: warning: symbol 'lnet_accept' was not declared. Should it be static? drivers/staging/lustre/lnet/lnet/acceptor.c:333:1: warning: symbol 'lnet_acceptor' was not declared. Should it be static? Tested by compilation only. Signed-off-by: Matt Kurz Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/lustre/lnet/lnet/acceptor.c b/drivers/staging/lustre/lnet/lnet/acceptor.c index 001c4b7..5dfb887 100644 --- a/drivers/staging/lustre/lnet/lnet/acceptor.c +++ b/drivers/staging/lustre/lnet/lnet/acceptor.c @@ -42,7 +42,7 @@ static int accept_port = 988; static int accept_backlog = 127; static int accept_timeout = 5; -struct { +static struct { int pta_shutdown; struct socket *pta_sock; struct completion pta_signal; @@ -75,7 +75,7 @@ MODULE_PARM_DESC(accept_timeout, "Acceptor's timeout (seconds)"); static char *accept_type; -int +static int lnet_acceptor_get_tunables(void) { /* Userland acceptor uses 'accept_type' instead of 'accept', due to @@ -207,7 +207,7 @@ EXPORT_SYMBOL(lnet_connect); /* Below is the code common for both kernel and MT user-space */ -int +static int lnet_accept(struct socket *sock, __u32 magic) { lnet_acceptor_connreq_t cr; @@ -329,7 +329,7 @@ lnet_accept(struct socket *sock, __u32 magic) return rc; } -int +static int lnet_acceptor(void *arg) { struct socket *newsock;