Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751747AbaLaKkk (ORCPT ); Wed, 31 Dec 2014 05:40:40 -0500 Received: from mail-wi0-f175.google.com ([209.85.212.175]:63936 "EHLO mail-wi0-f175.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750760AbaLaKkj (ORCPT ); Wed, 31 Dec 2014 05:40:39 -0500 Date: Wed, 31 Dec 2014 11:40:29 +0100 From: Konrad Zapalowicz To: Serguey Parkhomovsky Cc: Oleg Drokin , Andreas Dilger , Greg Kroah-Hartman , HPDD-discuss@ml01.01.org, devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] staging: lustre: linux-prim.c: fix sparse warnings about static declaration Message-ID: <20141231104029.GA25819@t400> References: <20141231084833.GA6643@fedora.localdomain> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20141231084833.GA6643@fedora.localdomain> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 12/31, Serguey Parkhomovsky wrote: > Fixes the following sparse warnings: > > drivers/staging/lustre/lustre/libcfs/linux/linux-prim.c:198:1: warning: symbol 'libcfs_arch_init' was not declared. Should it be static? > drivers/staging/lustre/lustre/libcfs/linux/linux-prim.c:204:1: warning: symbol 'libcfs_arch_cleanup' was not declared. Should it be static? This lines are way too long. You should break the commit message line around 72nd column so that it looks nice when emailed. You can also omit the filename from the topic line as this is already in the diff. Thanks, Konrad > Signed-off-by: Serguey Parkhomovsky > --- > v2: Don't break the build, and fix the warnings by defining the functions in libcfs.h. > drivers/staging/lustre/include/linux/libcfs/libcfs.h | 3 +++ > drivers/staging/lustre/lustre/libcfs/module.c | 2 -- > 2 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/staging/lustre/include/linux/libcfs/libcfs.h b/drivers/staging/lustre/include/linux/libcfs/libcfs.h > index a6b2f90..4410d7f 100644 > --- a/drivers/staging/lustre/include/linux/libcfs/libcfs.h > +++ b/drivers/staging/lustre/include/linux/libcfs/libcfs.h > @@ -85,6 +85,9 @@ static inline int __is_po2(unsigned long long val) > > #include > > +int libcfs_arch_init(void); > +void libcfs_arch_cleanup(void); > + > /* libcfs tcpip */ > int libcfs_ipif_query(char *name, int *up, __u32 *ip, __u32 *mask); > int libcfs_ipif_enumerate(char ***names); > diff --git a/drivers/staging/lustre/lustre/libcfs/module.c b/drivers/staging/lustre/lustre/libcfs/module.c > index 2c4fc74..31a4d19 100644 > --- a/drivers/staging/lustre/lustre/libcfs/module.c > +++ b/drivers/staging/lustre/lustre/libcfs/module.c > @@ -334,8 +334,6 @@ extern struct mutex cfs_trace_thread_mutex; > extern struct cfs_wi_sched *cfs_sched_rehash; > > extern void libcfs_init_nidstrings(void); > -extern int libcfs_arch_init(void); > -extern void libcfs_arch_cleanup(void); > > static int init_libcfs_module(void) > { > -- > 1.9.3 > > _______________________________________________ > devel mailing list > devel@linuxdriverproject.org > http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/