2023-04-06 10:38:37

by kernel test robot

[permalink] [raw]
Subject: [herbert-cryptodev-2.6:master 99/105] include/linux/compiler_types.h:397:45: error: call to '__compiletime_assert_385' declared with attribute error: BUILD_BUG_ON failed: sizeof(struct crypt_ctl) != 64

tree: https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git master
head: a2216e1874715a8b4a6f4da2ddbe9277e5613c49
commit: 1bc7fdbf2677cc1866c025e5a393811ea8e25486 [99/105] crypto: ixp4xx - Move driver to drivers/crypto/intel/ixp4xx
config: riscv-allmodconfig (https://download.01.org/0day-ci/archive/20230406/[email protected]/config)
compiler: riscv64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git/commit/?id=1bc7fdbf2677cc1866c025e5a393811ea8e25486
git remote add herbert-cryptodev-2.6 https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git
git fetch --no-tags herbert-cryptodev-2.6 master
git checkout 1bc7fdbf2677cc1866c025e5a393811ea8e25486
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=riscv olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash drivers/crypto/intel/ixp4xx/

If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <[email protected]>
| Link: https://lore.kernel.org/oe-kbuild-all/[email protected]/

All errors (new ones prefixed by >>):

In file included from <command-line>:
In function 'setup_crypt_desc',
inlined from 'get_crypt_desc' at drivers/crypto/intel/ixp4xx/ixp4xx_crypto.c:285:3:
>> include/linux/compiler_types.h:397:45: error: call to '__compiletime_assert_385' declared with attribute error: BUILD_BUG_ON failed: sizeof(struct crypt_ctl) != 64
397 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^
include/linux/compiler_types.h:378:25: note: in definition of macro '__compiletime_assert'
378 | prefix ## suffix(); \
| ^~~~~~
include/linux/compiler_types.h:397:9: note: in expansion of macro '_compiletime_assert'
397 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
| ^~~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:39:37: note: in expansion of macro 'compiletime_assert'
39 | #define BUILD_BUG_ON_MSG(cond, msg) compiletime_assert(!(cond), msg)
| ^~~~~~~~~~~~~~~~~~
include/linux/build_bug.h:50:9: note: in expansion of macro 'BUILD_BUG_ON_MSG'
50 | BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed: " #condition)
| ^~~~~~~~~~~~~~~~
drivers/crypto/intel/ixp4xx/ixp4xx_crypto.c:266:9: note: in expansion of macro 'BUILD_BUG_ON'
266 | BUILD_BUG_ON(sizeof(struct crypt_ctl) != 64);
| ^~~~~~~~~~~~


vim +/__compiletime_assert_385 +397 include/linux/compiler_types.h

eb5c2d4b45e3d2 Will Deacon 2020-07-21 383
eb5c2d4b45e3d2 Will Deacon 2020-07-21 384 #define _compiletime_assert(condition, msg, prefix, suffix) \
eb5c2d4b45e3d2 Will Deacon 2020-07-21 385 __compiletime_assert(condition, msg, prefix, suffix)
eb5c2d4b45e3d2 Will Deacon 2020-07-21 386
eb5c2d4b45e3d2 Will Deacon 2020-07-21 387 /**
eb5c2d4b45e3d2 Will Deacon 2020-07-21 388 * compiletime_assert - break build and emit msg if condition is false
eb5c2d4b45e3d2 Will Deacon 2020-07-21 389 * @condition: a compile-time constant condition to check
eb5c2d4b45e3d2 Will Deacon 2020-07-21 390 * @msg: a message to emit if condition is false
eb5c2d4b45e3d2 Will Deacon 2020-07-21 391 *
eb5c2d4b45e3d2 Will Deacon 2020-07-21 392 * In tradition of POSIX assert, this macro will break the build if the
eb5c2d4b45e3d2 Will Deacon 2020-07-21 393 * supplied condition is *false*, emitting the supplied error message if the
eb5c2d4b45e3d2 Will Deacon 2020-07-21 394 * compiler has support to do so.
eb5c2d4b45e3d2 Will Deacon 2020-07-21 395 */
eb5c2d4b45e3d2 Will Deacon 2020-07-21 396 #define compiletime_assert(condition, msg) \
eb5c2d4b45e3d2 Will Deacon 2020-07-21 @397 _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
eb5c2d4b45e3d2 Will Deacon 2020-07-21 398

:::::: The code at line 397 was first introduced by commit
:::::: eb5c2d4b45e3d2d5d052ea6b8f1463976b1020d5 compiler.h: Move compiletime_assert() macros into compiler_types.h

:::::: TO: Will Deacon <[email protected]>
:::::: CC: Will Deacon <[email protected]>

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


2023-04-07 19:47:21

by Tom Zanussi

[permalink] [raw]
Subject: Re: [herbert-cryptodev-2.6:master 99/105] include/linux/compiler_types.h:397:45: error: call to '__compiletime_assert_385' declared with attribute error: BUILD_BUG_ON failed: sizeof(struct crypt_ctl) != 64

On Thu, 2023-04-06 at 18:28 +0800, kernel test robot wrote:
> tree:  
> https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git
>  master
> head:   a2216e1874715a8b4a6f4da2ddbe9277e5613c49
> commit: 1bc7fdbf2677cc1866c025e5a393811ea8e25486 [99/105] crypto:
> ixp4xx - Move driver to drivers/crypto/intel/ixp4xx
> config: riscv-allmodconfig
> (https://download.01.org/0day-ci/archive/20230406/202304061846.G6cpPX
> [email protected]/config)
> compiler: riscv64-linux-gcc (GCC) 12.1.0
> reproduce (this is a W=1 build):
>         wget
> https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross
>  -O ~/bin/make.cross
>         chmod +x ~/bin/make.cross
>         #
> https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git/commit/?id=1bc7fdbf2677cc1866c025e5a393811ea8e25486
>         git remote add herbert-cryptodev-2.6
> https://git.kernel.org/pub/scm/linux/kernel/git/herbert/cryptodev-2.6.git
>         git fetch --no-tags herbert-cryptodev-2.6 master
>         git checkout 1bc7fdbf2677cc1866c025e5a393811ea8e25486
>         # save the config file
>         mkdir build_dir && cp config build_dir/.config
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0
> make.cross W=1 O=build_dir ARCH=riscv olddefconfig
>         COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0
> make.cross W=1 O=build_dir ARCH=riscv SHELL=/bin/bash
> drivers/crypto/intel/ixp4xx/
>
> If you fix the issue, kindly add following tag where applicable
> > Reported-by: kernel test robot <[email protected]>
> > Link:
> > https://lore.kernel.org/oe-kbuild-all/[email protected]/
>
> All errors (new ones prefixed by >>):
>
>    In file included from <command-line>:
>    In function 'setup_crypt_desc',
>        inlined from 'get_crypt_desc' at
> drivers/crypto/intel/ixp4xx/ixp4xx_crypto.c:285:3:
> > > include/linux/compiler_types.h:397:45: error: call to
> > > '__compiletime_assert_385' declared with attribute error:
> > > BUILD_BUG_ON failed: sizeof(struct crypt_ctl) != 64
>      397 |         _compiletime_assert(condition, msg,
> __compiletime_assert_, __COUNTER__)
>          |                                             ^
>    include/linux/compiler_types.h:378:25: note: in definition of
> macro '__compiletime_assert'
>      378 |                         prefix ##
> suffix();                             \
>          |                         ^~~~~~
>    include/linux/compiler_types.h:397:9: note: in expansion of macro
> '_compiletime_assert'
>      397 |         _compiletime_assert(condition, msg,
> __compiletime_assert_, __COUNTER__)
>          |         ^~~~~~~~~~~~~~~~~~~
>    include/linux/build_bug.h:39:37: note: in expansion of macro
> 'compiletime_assert'
>       39 | #define BUILD_BUG_ON_MSG(cond, msg)
> compiletime_assert(!(cond), msg)
>          |                                     ^~~~~~~~~~~~~~~~~~
>    include/linux/build_bug.h:50:9: note: in expansion of macro
> 'BUILD_BUG_ON_MSG'
>       50 |         BUILD_BUG_ON_MSG(condition, "BUILD_BUG_ON failed:
> " #condition)
>          |         ^~~~~~~~~~~~~~~~
>    drivers/crypto/intel/ixp4xx/ixp4xx_crypto.c:266:9: note: in
> expansion of macro 'BUILD_BUG_ON'
>      266 |         BUILD_BUG_ON(sizeof(struct crypt_ctl) != 64);
>          |         ^~~~~~~~~~~~
>
>

This happened because I added COMPILE_TEST as a dependency and
obviously shouldn't have.

Will send a patch to remove it.

Tom