linux-kernel - Re: [PATCH v2] staging: lustre: linux-prim.c: fix sparse warnings about static declaration

Open Source and information security mailing list archives

Message-ID: <20141231104029.GA25819@t400>
Date:	Wed, 31 Dec 2014 11:40:29 +0100
From:	Konrad Zapalowicz <bergo.torino@...il.com>
To:	Serguey Parkhomovsky <sergueyparkhomovsky@...il.com>
Cc:	Oleg Drokin <oleg.drokin@...el.com>,
	Andreas Dilger <andreas.dilger@...el.com>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	HPDD-discuss@...ts.01.org, devel@...verdev.osuosl.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2] staging: lustre: linux-prim.c: fix sparse warnings
 about static declaration

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 <sergueyparkhomovsky@...il.com>
> ---
> 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 <linux/list.h>
>  
> +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@...uxdriverproject.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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives