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-next>] [day] [month] [year] [list]
Message-ID: <202307080446.rpYEMqQ2-lkp@intel.com>
Date:   Sat, 8 Jul 2023 05:03:21 +0800
From:   kernel test robot <lkp@...el.com>
To:     Christophe Leroy <christophe.leroy@...roup.eu>
Cc:     llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev,
        linux-kernel@...r.kernel.org,
        Michael Ellerman <mpe@...erman.id.au>,
        Nathan Chancellor <nathan@...nel.org>
Subject: arch/powerpc/math-emu/fsel.c:14:2: warning: variable '_fex' set but
 not used

Hi Christophe,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   4f6b6c2b2f86b7878a770736bf478d8a263ff0bc
commit: 7245fc5bb7a966852d5bd7779d1f5855530b461a powerpc/math-emu: Remove -w build flag and fix warnings
date:   10 months ago
config: powerpc-randconfig-r033-20230708 (https://download.01.org/0day-ci/archive/20230708/202307080446.rpYEMqQ2-lkp@intel.com/config)
compiler: clang version 17.0.0 (https://github.com/llvm/llvm-project.git 4a5ac14ee968ff0ad5d2cc1ffa0299048db4c88a)
reproduce: (https://download.01.org/0day-ci/archive/20230708/202307080446.rpYEMqQ2-lkp@intel.com/reproduce)

If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202307080446.rpYEMqQ2-lkp@intel.com/

All warnings (new ones prefixed by >>):

>> arch/powerpc/math-emu/fsel.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable]
      14 |         FP_DECL_EX;
         |         ^
   include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX'
      94 | #define FP_DECL_EX int _fex = 0
         |                        ^
   arch/powerpc/math-emu/fsel.c:11:1: warning: no previous prototype for function 'fsel' [-Wmissing-prototypes]
      11 | fsel(u32 *frD, void *frA, u32 *frB, u32 *frC)
         | ^
   arch/powerpc/math-emu/fsel.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      10 | int
         | ^
         | static 
   2 warnings generated.
--
>> arch/powerpc/math-emu/fcmpu.c:13:2: warning: variable 'A_c' set but not used [-Wunused-but-set-variable]
      13 |         FP_DECL_D(A);
         |         ^
   include/math-emu/double.h:73:23: note: expanded from macro 'FP_DECL_D'
      73 | #define FP_DECL_D(X)            _FP_DECL(2,X)
         |                                 ^
   include/math-emu/op-common.h:28:14: note: expanded from macro '_FP_DECL'
      28 |   _FP_I_TYPE X##_c=0, X##_s=0, X##_e=0; \
         |              ^
   <scratch space>:171:1: note: expanded from here
     171 | A_c
         | ^
>> arch/powerpc/math-emu/fcmpu.c:14:2: warning: variable 'B_c' set but not used [-Wunused-but-set-variable]
      14 |         FP_DECL_D(B);
         |         ^
   include/math-emu/double.h:73:23: note: expanded from macro 'FP_DECL_D'
      73 | #define FP_DECL_D(X)            _FP_DECL(2,X)
         |                                 ^
   include/math-emu/op-common.h:28:14: note: expanded from macro '_FP_DECL'
      28 |   _FP_I_TYPE X##_c=0, X##_s=0, X##_e=0; \
         |              ^
   <scratch space>:177:1: note: expanded from here
     177 | B_c
         | ^
>> arch/powerpc/math-emu/fcmpu.c:15:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable]
      15 |         FP_DECL_EX;
         |         ^
   include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX'
      94 | #define FP_DECL_EX int _fex = 0
         |                        ^
   arch/powerpc/math-emu/fcmpu.c:11:1: warning: no previous prototype for function 'fcmpu' [-Wmissing-prototypes]
      11 | fcmpu(u32 *ccr, int crfD, void *frA, void *frB)
         | ^
   arch/powerpc/math-emu/fcmpu.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      10 | int
         | ^
         | static 
   4 warnings generated.
--
>> arch/powerpc/math-emu/fctiwz.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable]
      14 |         FP_DECL_EX;
         |         ^
   include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX'
      94 | #define FP_DECL_EX int _fex = 0
         |                        ^
   arch/powerpc/math-emu/fctiwz.c:11:1: warning: no previous prototype for function 'fctiwz' [-Wmissing-prototypes]
      11 | fctiwz(u32 *frD, void *frB)
         | ^
   arch/powerpc/math-emu/fctiwz.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      10 | int
         | ^
         | static 
   2 warnings generated.
--
>> arch/powerpc/math-emu/fctiw.c:14:2: warning: variable '_fex' set but not used [-Wunused-but-set-variable]
      14 |         FP_DECL_EX;
         |         ^
   include/math-emu/soft-fp.h:94:24: note: expanded from macro 'FP_DECL_EX'
      94 | #define FP_DECL_EX int _fex = 0
         |                        ^
   arch/powerpc/math-emu/fctiw.c:11:1: warning: no previous prototype for function 'fctiw' [-Wmissing-prototypes]
      11 | fctiw(u32 *frD, void *frB)
         | ^
   arch/powerpc/math-emu/fctiw.c:10:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
      10 | int
         | ^
         | static 
   2 warnings generated.


vim +/_fex +14 arch/powerpc/math-emu/fsel.c

^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16   9  
^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16  10  int
^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16  11  fsel(u32 *frD, void *frA, u32 *frB, u32 *frC)
^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16  12  {
^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16  13  	FP_DECL_D(A);
d2b194ed820880 arch/powerpc/math-emu/fsel.c Kumar Gala     2008-06-04 @14  	FP_DECL_EX;
^1da177e4c3f41 arch/ppc/math-emu/fsel.c     Linus Torvalds 2005-04-16  15  

:::::: The code at line 14 was first introduced by commit
:::::: d2b194ed820880eb19c43b9c10d9f5f30026ee54 powerpc/math-emu: Use kernel generic math-emu code

:::::: TO: Kumar Gala <galak@...nel.crashing.org>
:::::: CC: Kumar Gala <galak@...nel.crashing.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ