linux-kernel - Re: [PATCH] blackfin: optimize ffz, __ffs, ffs, _

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]

Message-ID: <201604151759.vQnlZqFY%fengguang.wu@intel.com>
Date:	Fri, 15 Apr 2016 17:16:54 +0800
From:	kbuild test robot <lkp@...el.com>
To:	zengzhaoxiu@....com
Cc:	kbuild-all@...org, linux-kernel@...r.kernel.org,
	Zhaoxiu Zeng <zhaoxiu.zeng@...il.com>,
	Steven Miao <realmz6@...il.com>,
	adi-buildroot-devel@...ts.sourceforge.net
Subject: Re: [PATCH] blackfin: optimize ffz, __ffs, ffs, __fls, and fls
 functions

Hi Zhaoxiu,

[auto build test ERROR on v4.6-rc3]
[also build test ERROR on next-20160414]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/zengzhaoxiu-163-com/blackfin-optimize-ffz-__ffs-ffs-__fls-and-fls-functions/20160415-155549
config: blackfin-TCM-BF537_defconfig (attached as .config)
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=blackfin 

All errors (new ones prefixed by >>):

   In file included from arch/blackfin/include/asm/bitops.h:12:0,
                    from include/linux/bitops.h:36,
                    from include/linux/kernel.h:10,
                    from include/asm-generic/bug.h:13,
                    from arch/blackfin/include/asm/bug.h:71,
                    from include/linux/bug.h:4,
                    from include/linux/page-flags.h:9,
                    from kernel/bounds.c:9:
   include/asm-generic/bitops/fls64.h: In function 'fls64':
>> include/asm-generic/bitops/fls64.h:22:3: error: implicit declaration of function 'fls' [-Werror=implicit-function-declaration]
   In file included from arch/blackfin/include/asm/bitops.h:19:0,
                    from include/linux/bitops.h:36,
                    from include/linux/kernel.h:10,
                    from include/asm-generic/bug.h:13,
                    from arch/blackfin/include/asm/bug.h:71,
                    from include/linux/bug.h:4,
                    from include/linux/page-flags.h:9,
                    from kernel/bounds.c:9:
   include/asm-generic/bitops/sched.h: In function 'sched_find_first_bit':
>> include/asm-generic/bitops/sched.h:20:3: error: implicit declaration of function '__ffs' [-Werror=implicit-function-declaration]
   In file included from include/linux/bitops.h:36:0,
                    from include/linux/kernel.h:10,
                    from include/asm-generic/bug.h:13,
                    from arch/blackfin/include/asm/bug.h:71,
                    from include/linux/bug.h:4,
                    from include/linux/page-flags.h:9,
                    from kernel/bounds.c:9:
   arch/blackfin/include/asm/bitops.h: At top level:
>> arch/blackfin/include/asm/bitops.h:168:29: error: conflicting types for '__ffs'
   include/asm-generic/bitops/sched.h:20:10: note: previous implicit declaration of '__ffs' was here
>> arch/blackfin/include/asm/bitops.h:177:19: error: static declaration of 'fls' follows non-static declaration
   include/asm-generic/bitops/fls64.h:22:10: note: previous implicit declaration of 'fls' was here
   cc1: some warnings being treated as errors
   make[2]: *** [kernel/bounds.s] Error 1
   make[2]: Target '__build' not remade because of errors.
   make[1]: *** [prepare0] Error 2
   make[1]: Target 'prepare' not remade because of errors.
   make: *** [sub-make] Error 2

vim +/fls +22 include/asm-generic/bitops/fls64.h

d57594c20 Alexander van Heukelum 2008-03-15  16   */
d57594c20 Alexander van Heukelum 2008-03-15  17  #if BITS_PER_LONG == 32
c8399943b Andi Kleen             2009-01-12  18  static __always_inline int fls64(__u64 x)
2dfc383ad Akinobu Mita           2006-03-26  19  {
2dfc383ad Akinobu Mita           2006-03-26  20  	__u32 h = x >> 32;
2dfc383ad Akinobu Mita           2006-03-26  21  	if (h)
2dfc383ad Akinobu Mita           2006-03-26 @22  		return fls(h) + 32;
2dfc383ad Akinobu Mita           2006-03-26  23  	return fls(x);
2dfc383ad Akinobu Mita           2006-03-26  24  }
d57594c20 Alexander van Heukelum 2008-03-15  25  #elif BITS_PER_LONG == 64

:::::: The code at line 22 was first introduced by commit
:::::: 2dfc383ad587bbead84739a9ff9273df3eda983d [PATCH] bitops: generic fls64()

:::::: TO: Akinobu Mita <mita@...aclelinux.com>
:::::: CC: Linus Torvalds <torvalds@...osdl.org>

---
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" (10386 bytes)