linux-kernel - Re: [v2 PATCH 1/2] scsi:stex.c Support Pegasus 3 product

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
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 14 Jun 2016 14:49:42 +0800
From:	kbuild test robot <lkp@...el.com>
To:	Charles Chiou <ch1102chiou@...il.com>
Cc:	kbuild-all@...org, jejb@...ux.vnet.ibm.com,
	martin.petersen@...cle.com, linux-scsi@...r.kernel.org,
	linux-kernel@...r.kernel.org, linus.chen@...promise.com,
	grace.chang@...promise.com, victor.p@...mise.com,
	eva.cheng@...promise.com, charles.chiou@...promise.com,
	julian.calaby@...il.com, Paul <paul.lyu@...promise.com>
Subject: Re: [v2 PATCH 1/2] scsi:stex.c Support Pegasus 3 product

Hi,

[auto build test WARNING on scsi/for-next]
[also build test WARNING on v4.7-rc3 next-20160609]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Charles-Chiou/scsi-stex-c-Support-Pegasus-3-product/20160614-142621
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
config: sparc64-allmodconfig (attached as .config)
compiler: sparc64-linux-gnu-gcc (Debian 5.3.1-8) 5.3.1 20160205
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=sparc64 

Note: it may well be a FALSE warning. FWIW you are at least aware of it now.
http://gcc.gnu.org/wiki/Better_Uninitialized_Warnings

All warnings (new ones prefixed by >>):

   In file included from arch/sparc/include/asm/string.h:4:0,
                    from include/linux/string.h:18,
                    from include/linux/bitmap.h:8,
                    from include/linux/nodemask.h:94,
                    from include/linux/mmzone.h:16,
                    from include/linux/gfp.h:5,
                    from include/linux/slab.h:14,
                    from drivers/scsi/stex.c:20:
   drivers/scsi/stex.c: In function 'stex_handshake':
>> arch/sparc/include/asm/string_64.h:28:29: warning: 'scratch' may be used uninitialized in this function [-Wmaybe-uninitialized]
    #define memset(s, c, count) __builtin_memset(s, c, count)
                                ^
   drivers/scsi/stex.c:1111:10: note: 'scratch' was declared here
     __le32 *scratch;
             ^

vim +/scratch +28 arch/sparc/include/asm/string_64.h

f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  12  /* Really, userland/ksyms should not see any of this stuff. */
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  13  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  14  #ifdef __KERNEL__
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  15  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  16  #include <asm/asi.h>
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  17  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  18  #ifndef EXPORT_SYMTAB_STROPS
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  19  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  20  /* First the mem*() things. */
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  21  #define __HAVE_ARCH_MEMMOVE
f05a6865 arch/sparc/include/asm/string_64.h Sam Ravnborg    2014-05-16  22  void *memmove(void *, const void *, __kernel_size_t);
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  23  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  24  #define __HAVE_ARCH_MEMCPY
4d14a459 arch/sparc/include/asm/string_64.h David S. Miller 2009-12-10  25  #define memcpy(t, f, n) __builtin_memcpy(t, f, n)
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  26  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  27  #define __HAVE_ARCH_MEMSET
4d14a459 arch/sparc/include/asm/string_64.h David S. Miller 2009-12-10 @28  #define memset(s, c, count) __builtin_memset(s, c, count)
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  29  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  30  #define __HAVE_ARCH_MEMSCAN
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  31  
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  32  #undef memscan
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  33  #define memscan(__arg0, __char, __arg2)					\
f5e706ad include/asm-sparc/string_64.h      Sam Ravnborg    2008-07-17  34  ({									\
f05a6865 arch/sparc/include/asm/string_64.h Sam Ravnborg    2014-05-16  35  	void *__memscan_zero(void *, size_t);				\
f05a6865 arch/sparc/include/asm/string_64.h Sam Ravnborg    2014-05-16  36  	void *__memscan_generic(void *, int, size_t);			\

:::::: The code at line 28 was first introduced by commit
:::::: 4d14a459857bd151ecbd14bcd37b4628da00792b sparc: Stop trying to be so fancy and use __builtin_{memcpy,memset}()

:::::: TO: David S. Miller <davem@...emloft.net>
:::::: CC: David S. Miller <davem@...emloft.net>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Download attachment ".config.gz" of type "application/octet-stream" (46368 bytes)