From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752767AbdJPOJj (ORCPT ); Mon, 16 Oct 2017 10:09:39 -0400 Received: from mail-qt0-f193.google.com ([209.85.216.193]:44783 "EHLO mail-qt0-f193.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752290AbdJPOJg (ORCPT ); Mon, 16 Oct 2017 10:09:36 -0400 X-Google-Smtp-Source: ABhQp+R6meaxEcGYFhdUvxaIKuAGlFz3gALLMEbDpj+srlXu6bH37o4GOLDIz2n5jHsTC/+KD0FWJA== Message-ID: <1508162973.5023.11.camel@poochiereds.net> Subject: Re: [PATCH] sunrcp: make function _svc_create_xprt static From: Jeff Layton To: Colin King , "J . Bruce Fields" , Trond Myklebust , Anna Schumaker , "David S . Miller" , linux-nfs@vger.kernel.org, netdev@vger.kernel.org Cc: kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org Date: Mon, 16 Oct 2017 10:09:33 -0400 In-Reply-To: <20171016134021.15702-1-colin.king@canonical.com> References: <20171016134021.15702-1-colin.king@canonical.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.24.6 (3.24.6-1.fc26) Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2017-10-16 at 14:40 +0100, Colin King wrote: > From: Colin Ian King > > The function _svc_create_xprt is local to the source and > does not need to be in global scope, so make it static. > > Cleans up sparse warning: > symbol '_svc_create_xprt' was not declared. Should it be static? > > Signed-off-by: Colin Ian King > --- > 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 d16a8b423c20..18e87791350f 100644 > --- a/net/sunrpc/svc_xprt.c > +++ b/net/sunrpc/svc_xprt.c > @@ -250,9 +250,9 @@ void svc_add_new_perm_xprt(struct svc_serv *serv, struct svc_xprt *new) > svc_xprt_received(new); > } > > -int _svc_create_xprt(struct svc_serv *serv, const char *xprt_name, > - struct net *net, const int family, > - const unsigned short port, int flags) > +static int _svc_create_xprt(struct svc_serv *serv, const char *xprt_name, > + struct net *net, const int family, > + const unsigned short port, int flags) > { > struct svc_xprt_class *xcl; > Reviewed-by: Jeff Layton