From c49d4b96ab6175fdcc3f858b0ef620f20ae16437 Mon Sep 17 00:00:00 2001 From: Matt Kurz Date: Wed, 23 Jul 2014 06:10:33 +1000 Subject: [PATCH] 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 --- drivers/staging/lustre/lnet/lnet/acceptor.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/lustre/lnet/lnet/acceptor.c b/drivers/staging/lustre/lnet/lnet/acceptor.c index 001c4b7ed0ce..5dfb887a03ae 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; -- 2.30.2