Re: [PATCH] staging: lustre: lustre: lmv: fix sparse warnings about static declarations

From: Sudip Mukherjee
Date: Tue Feb 03 2015 - 12:29:41 EST


On Tue, Feb 03, 2015 at 08:54:45PM +0530, Mohammad Jamal wrote:
> This patch adds a static keyword to lprocfs_lmv_init_vars and
> lprocfs_lmv_module_vars to suppress the sparse warnings about
> static declaration
have you build tested your patch?

After your patch:
drivers/staging/lustre/lustre/lmv/lproc_lmv.c:218:31: warning: âlmv_proc_target_fopsâ defined but not used [-Wunused-variable]
drivers/staging/lustre/lustre/lmv/lproc_lmv.c:226:13: warning: âlprocfs_lmv_init_varsâ defined but not used [-Wunused-function]

WARNING: "lprocfs_lmv_init_vars" [drivers/staging/lustre/lustre/lmv/lmv.ko] undefined!
WARNING: "lmv_proc_target_fops" [drivers/staging/lustre/lustre/lmv/lmv.ko] undefined!

regards
sudip

>
> Signed-off-by: Mohammad Jamal <md.jamalmohiuddin@xxxxxxxxx>
> ---
<snip>
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/