Commit c49d4b96 authored Jul 23, 2014 by Matt Kurz Committed by Greg Kroah-Hartman Jul 22, 2014

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 <matt@ninezulu.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 137edbaa

drivers/staging/lustre/lnet/lnet/acceptor.c

+4 −4

Original line number	Diff line number	Diff line
		@@ -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;