svn commit: r248567 - head/sys/fs/nfsclient

[Peter Wemm]

On Thu, Mar 21, 2013 at 6:08 AM, Konstantin Belousov
<kostikbel at gmail.com> wrote:
> On Thu, Mar 21, 2013 at 01:32:36PM +0200, fidaj   wrote:
>>
>>
>>
>> that commit breaks kernel built
>>
>> ...
>> /usr/local/libexec/ccache/world/cc -O2 -pipe -fno-strict-aliasing -Werror
>> -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include
>> /usr/obj/usr/src/sys/GENERIC/opt_global.h -I. -I@ -I@/contrib/altq
>> -finline-limit=8000 --param inline-unit-growth=100 --param
>> large-function-growth=1000 -fno-common -g -fno-omit-frame-pointer
>> -I/usr/obj/usr/src/sys/GENERIC -mcmodel=kernel -mno-red-zone -mno-mmx
>> -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding
>> -fstack-protector -std=iso9899:1999 -fstack-protector -Wall
>> -Wredundant-decls -Wnested-externs -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef
>> -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs
>> -fdiagnostics-show-option -c
>> /usr/src/sys/modules/nfscl/../../fs/nfsclient/nfs_clvfsops.c
>> /usr/local/libexec/ccache/world/cc -O2 -pipe -fno-strict-aliasing -Werror
>> -D_KERNEL -DKLD_MODULE -nostdinc -DHAVE_KERNEL_OPTION_HEADERS -include
>> /usr/obj/usr/src/sys/GENERIC/opt_global.h -I. -I@ -I@/contrib/altq
>> -finline-limit=8000 --param inline-unit-growth=100 --param
>> large-function-growth=1000 -fno-common -g -fno-omit-frame-pointer
>> -I/usr/obj/usr/src/sys/GENERIC -mcmodel=kernel -mno-red-zone -mno-mmx
>> -mno-sse -msoft-float -fno-asynchronous-unwind-tables -ffreestanding
>> -fstack-protector -std=iso9899:1999 -fstack-protector -Wall
>> -Wredundant-decls -Wnested-externs -Wstrict-prototypes
>> -Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual -Wundef
>> -Wno-pointer-sign -fformat-extensions -Wmissing-include-dirs
>> -fdiagnostics-show-option -c
>> /usr/src/sys/modules/nfscl/../../fs/nfsclient/nfs_clvnops.c
>> ctfconvert -L VERSION -g nfs_diskless.o
>> ctfconvert -L VERSION -g nfs_clkrpc.o
>> ctfconvert -L VERSION -g nfs_clvfsops.o
>> ctfconvert -L VERSION -g nfs_clsubs.o
>> ctfconvert -L VERSION -g nfs_clrpcops.o
>> ctfconvert -L VERSION -g nfs_clnfsiod.o
>> ctfconvert -L VERSION -g nfs_clstate.o
>> ctfconvert -L VERSION -g nfs_clbio.o
>> ctfconvert -L VERSION -g nfs_clvnops.o
>> ctfconvert -L VERSION -g nfs_clcomsubs.o
>> ctfconvert -L VERSION -g nfs_clnode.o
>> cc1: warnings being treated as errors
>> /usr/src/sys/modules/nfscl/../../fs/nfsclient/nfs_clport.c: In function
>> 'nfscl_loadattrcache':
>> /usr/src/sys/modules/nfscl/../../fs/nfsclient/nfs_clport.c:364: warning:
>> 'nsize' may be used uninitialized in this function
>> *** [nfs_clport.o] Error code 1
>> 1 error
>> *** [all] Error code 2
>> 1 error
>> *** [modules-all] Error code 2
>> 2 errors
>> *** [buildkernel] Error code 2
>> 1 error
>> *** [buildkernel] Error code 2
>>
>> Stop in /usr/src.
>
> ccache ? What compiler is it ?
> Anyway, I committed the supposed fix in the r248581.

Looks like a gcc-style message.  Which version though is unclear.  It
is probably the usual thing where gcc can't tell if all the code paths
are complete for initializing something and a false warning.

-- 
Peter Wemm - peter at wemm.org; peter at FreeBSD.org; peter at yahoo-inc.com; KI6FJV
bitcoin:188ZjyYLFJiEheQZw4UtU27e2FMLmuRBUE