netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "J. Bruce Fields" <bfields@fieldses.org>
To: Rashika Kheria <rashika.kheria@gmail.com>
Cc: linux-kernel@vger.kernel.org,
	Trond Myklebust <trond.myklebust@primarydata.com>,
	"David S. Miller" <davem@davemloft.net>,
	linux-nfs@vger.kernel.org, netdev@vger.kernel.org,
	josh@joshtriplett.org
Subject: Re: [PATCH 13/13] net: Mark functions as static in net/sunrpc/svc_xprt.c
Date: Tue, 11 Feb 2014 12:12:37 -0500	[thread overview]
Message-ID: <20140211171237.GF19599@fieldses.org> (raw)
In-Reply-To: <259f3dc38d7115eccf3aaa2aa7de2414a47f90a2.1391888654.git.rashika.kheria@gmail.com>

On Sun, Feb 09, 2014 at 01:40:54AM +0530, Rashika Kheria wrote:
> Mark functions as static in net/sunrpc/svc_xprt.c because they are not
> used outside this file.
> 
> This eliminates the following warning in net/sunrpc/svc_xprt.c:
> net/sunrpc/svc_xprt.c:574:5: warning: no previous prototype for ‘svc_alloc_arg’ [-Wmissing-prototypes]
> net/sunrpc/svc_xprt.c:615:18: warning: no previous prototype for ‘svc_get_next_xprt’ [-Wmissing-prototypes]
> net/sunrpc/svc_xprt.c:694:6: warning: no previous prototype for ‘svc_add_new_temp_xprt’ [-Wmissing-prototypes]

Thanks, applying.--b.

> 
> Signed-off-by: Rashika Kheria <rashika.kheria@gmail.com>
> ---
>  net/sunrpc/svc_xprt.c |    6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/net/sunrpc/svc_xprt.c b/net/sunrpc/svc_xprt.c
> index 80a6640..06c6ff0 100644
> --- a/net/sunrpc/svc_xprt.c
> +++ b/net/sunrpc/svc_xprt.c
> @@ -571,7 +571,7 @@ static void svc_check_conn_limits(struct svc_serv *serv)
>  	}
>  }
>  
> -int svc_alloc_arg(struct svc_rqst *rqstp)
> +static int svc_alloc_arg(struct svc_rqst *rqstp)
>  {
>  	struct svc_serv *serv = rqstp->rq_server;
>  	struct xdr_buf *arg;
> @@ -612,7 +612,7 @@ int svc_alloc_arg(struct svc_rqst *rqstp)
>  	return 0;
>  }
>  
> -struct svc_xprt *svc_get_next_xprt(struct svc_rqst *rqstp, long timeout)
> +static struct svc_xprt *svc_get_next_xprt(struct svc_rqst *rqstp, long timeout)
>  {
>  	struct svc_xprt *xprt;
>  	struct svc_pool		*pool = rqstp->rq_pool;
> @@ -691,7 +691,7 @@ struct svc_xprt *svc_get_next_xprt(struct svc_rqst *rqstp, long timeout)
>  	return xprt;
>  }
>  
> -void svc_add_new_temp_xprt(struct svc_serv *serv, struct svc_xprt *newxpt)
> +static void svc_add_new_temp_xprt(struct svc_serv *serv, struct svc_xprt *newxpt)
>  {
>  	spin_lock_bh(&serv->sv_lock);
>  	set_bit(XPT_TEMP, &newxpt->xpt_flags);
> -- 
> 1.7.9.5
> 

  parent reply	other threads:[~2014-02-11 17:12 UTC|newest]

Thread overview: 41+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-08 19:47 [PATCH 01/13] net: Mark function as static in 9p/client.c Rashika Kheria
2014-02-08 19:49 ` [PATCH 02/13] net: Include appropriate header file in caif/caif_dev.c Rashika Kheria
2014-02-08 20:04   ` Josh Triplett
2014-02-08 19:51 ` [PATCH 03/13] net: Include appropriate header file in caif/cfsrvl.c Rashika Kheria
2014-02-08 20:04   ` Josh Triplett
2014-02-08 19:53 ` [PATCH 04/13] net: Mark functions as static in core/dev.c Rashika Kheria
2014-02-08 20:07   ` Josh Triplett
2014-02-08 21:18   ` Veaceslav Falico
2014-02-08 19:55 ` [PATCH 05/13] net: Mark function as static in core/filter.c Rashika Kheria
2014-02-08 20:09   ` Josh Triplett
2014-02-08 19:58 ` [PATCH 06/13] net: Move prototype declaration to appropriate header file from decnet/af_decnet.c Rashika Kheria
2014-02-08 20:10   ` Josh Triplett
2014-02-08 19:59 ` [PATCH 07/13] net: Move prototype declaration to header file include/net/dn.h from net/decnet/af_decnet.c Rashika Kheria
2014-02-08 20:11   ` Josh Triplett
2014-02-08 20:02 ` [PATCH 08/13] net: Move prototype declaration to include/net/ipx.h from net/ipx/ipx_route.c Rashika Kheria
2014-02-08 20:11   ` Josh Triplett
2014-02-08 20:03 ` [PATCH 09/13] net: Move prototype declaration to header file include/net/ipx.h from net/ipx/af_ipx.c Rashika Kheria
2014-02-08 20:12   ` Josh Triplett
2014-02-08 20:04 ` [PATCH 01/13] net: Mark function as static in 9p/client.c Josh Triplett
2014-02-08 20:05 ` [PATCH 10/13] net: Move prototype declaration to header file include/net/datalink.h from net/ipx/af_ipx.c Rashika Kheria
2014-02-08 20:12   ` Josh Triplett
2014-02-08 20:07 ` [PATCH 11/13] net: Move prototype declaration to header file include/net/net_namespace.h " Rashika Kheria
2014-02-08 20:14   ` Josh Triplett
2014-02-08 20:09 ` [PATCH 12/13] net: Include appropriate header file in netfilter/nft_lookup.c Rashika Kheria
2014-02-08 20:14   ` Josh Triplett
2014-02-08 20:10 ` [PATCH 13/13] net: Mark functions as static in net/sunrpc/svc_xprt.c Rashika Kheria
     [not found]   ` <259f3dc38d7115eccf3aaa2aa7de2414a47f90a2.1391888654.git.rashika.kheria-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-02-08 20:23     ` Josh Triplett
2014-02-11 17:12   ` J. Bruce Fields [this message]
2014-02-09 14:27 ` [PATCH v2 01/13] net: Mark function as static in 9p/client.c Rashika Kheria
2014-02-09 14:29   ` [PATCH v2 02/13] net: Include appropriate header file in caif/caif_dev.c Rashika Kheria
2014-02-09 14:32   ` [PATCH v2 03/13] net: Include appropriate header file in caif/cfsrvl.c Rashika Kheria
2014-02-09 14:56   ` [PATCH v2 04/13] net: Mark functions as static in core/dev.c Rashika Kheria
2014-02-09 16:50   ` [PATCH v2 06/13] net: Move prototype declaration to appropriate header file from decnet/af_decnet.c Rashika Kheria
2014-02-09 16:52   ` [PATCH v2 07/13] net: Move prototype declaration to header file include/net/dn.h from net/decnet/af_decnet.c Rashika Kheria
2014-02-09 16:54   ` [PATCH v2 08/13] net: Move prototype declaration to include/net/ipx.h from net/ipx/ipx_route.c Rashika Kheria
2014-02-09 16:56   ` [PATCH v2 09/13] net: Move prototype declaration to header file include/net/ipx.h from net/ipx/af_ipx.c Rashika Kheria
2014-02-09 16:57   ` [PATCH v2 10/13] net: Move prototype declaration to header file include/net/datalink.h " Rashika Kheria
2014-02-09 16:59   ` [PATCH v2 11/13] net: Move prototype declaration to header file include/net/net_namespace.h " Rashika Kheria
2014-02-09 17:01   ` [PATCH v2 12/13] net: Include appropriate header file in netfilter/nft_lookup.c Rashika Kheria
     [not found]   ` <6f029c895035908595957fb16ab445c82793c77d.1391955924.git.rashika.kheria-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2014-02-09 17:03     ` [PATCH v2 13/13] net: Mark functions as static in net/sunrpc/svc_xprt.c Rashika Kheria
2014-02-10  2:02   ` [PATCH v2 01/13] net: Mark function as static in 9p/client.c David Miller

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20140211171237.GF19599@fieldses.org \
    --to=bfields@fieldses.org \
    --cc=davem@davemloft.net \
    --cc=josh@joshtriplett.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-nfs@vger.kernel.org \
    --cc=netdev@vger.kernel.org \
    --cc=rashika.kheria@gmail.com \
    --cc=trond.myklebust@primarydata.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).