[linux-stable-rc:linux-4.14.y 9768/9999] lib/raid6/recov_neon_inner.c:13:31: warning: vector initializers are not compatible with NEON intrinsics in big endian mode

[linux-stable-rc:linux-4.14.y 9768/9999] lib/raid6/recov_neon_inner.c:13:31: warning: vector initializers are not compatible with NEON intrinsics in big endian mode

[kbuild test robot]

[-- Attachment #1: Type: text/plain, Size: 3191 bytes --]

Hi Dirk,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.14.y
head:   e60eb60a661c0738dafb0907de42ff3ff8ac91d0
commit: 88f7a6aa7fb9aa5076b65489146045dac865f1d3 [9768/9999] scripts/dtc: Remove redundant YYLOC global declaration
config: arm64-randconfig-a001-20200421 (attached as .config)
compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project a9b137f9ffba8cb25dfd7dd1fb613e8aac121b37)
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        git checkout 88f7a6aa7fb9aa5076b65489146045dac865f1d3
        # save the attached .config to linux build tree
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@intel.com>

All warnings (new ones prefixed by >>):

>> lib/raid6/recov_neon_inner.c:13:31: warning: vector initializers are not compatible with NEON intrinsics in big endian mode [-Wnonportable-vector-initialization]
   static const uint8x16_t x0f = {
                                 ^
   lib/raid6/recov_neon_inner.c:13:31: note: consider using vld1q_u8() to initialize a vector from memory, or vcombine_u8(vcreate_u8(), vcreate_u8()) to initialize from integer constants
   1 warning generated.
--
>> drivers/video/fbdev/sis/init301.c:525:42: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
         } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
                    ~~~~~~~~~~~~~~~~~~~~~~~~^~~~
   drivers/video/fbdev/sis/init301.c:525:42: note: remove extraneous parentheses around the comparison to silence this warning
         } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
                   ~                        ^   ~
   drivers/video/fbdev/sis/init301.c:525:42: note: use '=' to turn this equality comparison into an assignment
         } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
                                            ^~
                                            =
   1 warning generated.

vim +13 lib/raid6/recov_neon_inner.c

6ec4e2514decd6 Ard Biesheuvel 2017-07-13  12  
6ec4e2514decd6 Ard Biesheuvel 2017-07-13 @13  static const uint8x16_t x0f = {
6ec4e2514decd6 Ard Biesheuvel 2017-07-13  14  	0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f,
6ec4e2514decd6 Ard Biesheuvel 2017-07-13  15  	0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f, 0x0f,
6ec4e2514decd6 Ard Biesheuvel 2017-07-13  16  };
6ec4e2514decd6 Ard Biesheuvel 2017-07-13  17  

:::::: The code at line 13 was first introduced by commit
:::::: 6ec4e2514decd6fb4782a9364fa71d6244d05af4 md/raid6: implement recovery using ARM NEON intrinsics

:::::: TO: Ard Biesheuvel <ard.biesheuvel@linaro.org>
:::::: CC: Catalin Marinas <catalin.marinas@arm.com>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org

[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 33582 bytes --]

Re: [linux-stable-rc:linux-4.14.y 9768/9999] lib/raid6/recov_neon_inner.c:13:31: warning: vector initializers are not compatible with NEON intrinsics in big endian mode

[Greg Kroah-Hartman]

[-- Attachment #1: Type: text/plain, Size: 3107 bytes --]

On Tue, Apr 21, 2020 at 09:02:57AM -0700, Nathan Chancellor wrote:
> Hi all,
> 
> Sorry for converging on this commit yet again... hopefully it does not
> continue to happen for much longer.
> 
> On Tue, Apr 21, 2020 at 11:35:07PM +0800, kbuild test robot wrote:
> > Hi Dirk,
> > 
> > First bad commit (maybe != root cause):
> > 
> > tree:   https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-4.14.y
> > head:   e60eb60a661c0738dafb0907de42ff3ff8ac91d0
> > commit: 88f7a6aa7fb9aa5076b65489146045dac865f1d3 [9768/9999] scripts/dtc: Remove redundant YYLOC global declaration
> > config: arm64-randconfig-a001-20200421 (attached as .config)
> > compiler: clang version 11.0.0 (https://github.com/llvm/llvm-project a9b137f9ffba8cb25dfd7dd1fb613e8aac121b37)
> > reproduce:
> >         wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
> >         chmod +x ~/bin/make.cross
> >         # install arm64 cross compiling tool for clang build
> >         # apt-get install binutils-aarch64-linux-gnu
> >         git checkout 88f7a6aa7fb9aa5076b65489146045dac865f1d3
> >         # save the attached .config to linux build tree
> >         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross ARCH=arm64 
> > 
> > If you fix the issue, kindly add following tag as appropriate
> > Reported-by: kbuild test robot <lkp@intel.com>
> > 
> > All warnings (new ones prefixed by >>):
> > 
> > >> lib/raid6/recov_neon_inner.c:13:31: warning: vector initializers are not compatible with NEON intrinsics in big endian mode [-Wnonportable-vector-initialization]
> >    static const uint8x16_t x0f = {
> >                                  ^
> >    lib/raid6/recov_neon_inner.c:13:31: note: consider using vld1q_u8() to initialize a vector from memory, or vcombine_u8(vcreate_u8(), vcreate_u8()) to initialize from integer constants
> >    1 warning generated.
> 
> Already fixed upstream:
> 
> 1ad3935b39da7 ("lib/raid6: use vdupq_n_u8 to avoid endianness warnings")

Now queued up for 4.19.y and 4.14.y

> > >> drivers/video/fbdev/sis/init301.c:525:42: warning: equality comparison with extraneous parentheses [-Wparentheses-equality]
> >          } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
> >                     ~~~~~~~~~~~~~~~~~~~~~~~~^~~~
> >    drivers/video/fbdev/sis/init301.c:525:42: note: remove extraneous parentheses around the comparison to silence this warning
> >          } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
> >                    ~                        ^   ~
> >    drivers/video/fbdev/sis/init301.c:525:42: note: use '=' to turn this equality comparison into an assignment
> >          } else if((SiS_Pr->SiS_IF_DEF_LVDS == 1) /* ||
> >                                             ^~
> >                                             =
> >    1 warning generated.
> 
> Already fixed upstream:
> 
> 864eb1afc60cb ("video: fbdev: sis: Remove unnecessary parentheses and commented code")

Queued up for 4.4.y, 4.9.y, 4.14.y, and 4.19.y.

thanks!

greg k-h