| 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
| ||
|
Message-ID: <HK0PR06MB3202DA604E2FD79066C9E8CB807F9@HK0PR06MB3202.apcprd06.prod.outlook.com>
Date: Mon, 5 Sep 2022 03:04:18 +0000
From: Neal Liu <neal_liu@...eedtech.com>
To: kernel test robot <lkp@...el.com>,
Herbert Xu <herbert@...dor.apana.org.au>
CC: "kbuild-all@...ts.01.org" <kbuild-all@...ts.01.org>,
Ammar Faizi <ammarfaizi2@...weeb.org>,
GNU/Weeb Mailing List <gwml@...r.gnuweeb.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: RE: [ammarfaizi2-block:herbert/cryptodev-2.6/master 73/76]
drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x' expects
argument of type 'unsigned int', but argument 7 has type 'size_t' {aka 'long
unsigned int'}
> -----Original Message-----
> From: kernel test robot <lkp@...el.com>
> Sent: Saturday, September 3, 2022 12:38 AM
> To: Herbert Xu <herbert@...dor.apana.org.au>
> Cc: kbuild-all@...ts.01.org; Ammar Faizi <ammarfaizi2@...weeb.org>;
> GNU/Weeb Mailing List <gwml@...r.gnuweeb.org>;
> linux-kernel@...r.kernel.org; Neal Liu <neal_liu@...eedtech.com>
> Subject: [ammarfaizi2-block:herbert/cryptodev-2.6/master 73/76]
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x'
> expects argument of type 'unsigned int', but argument 7 has type 'size_t' {aka
> 'long unsigned int'}
>
> Hi Herbert,
>
> FYI, the error/warning was bisected to this commit, please ignore it if it's
> irrelevant.
I'll send patch to fix this build warning, thanks.
>
> tree: https://github.com/ammarfaizi2/linux-block
> herbert/cryptodev-2.6/master
> head: fa2bf6e35091e66fc83af1aebea06a78a5a2fde4
> commit: 31b39755e32568b43c80814c5e13d7b1ab796d73 [73/76] crypto:
> aspeed - Enable compile testing
> config: ia64-allyesconfig
> compiler: ia64-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://github.com/ammarfaizi2/linux-block/commit/31b39755e32568b43c80
> 814c5e13d7b1ab796d73
> git remote add ammarfaizi2-block
> https://github.com/ammarfaizi2/linux-block
> git fetch --no-tags ammarfaizi2-block herbert/cryptodev-2.6/master
> git checkout 31b39755e32568b43c80814c5e13d7b1ab796d73
> # 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=ia64 SHELL=/bin/bash
> drivers/crypto/aspeed/
>
> If you fix the issue, kindly add following tag where applicable
> Reported-by: kernel test robot <lkp@...el.com>
>
> All warnings (new ones prefixed by >>):
>
> In file included from include/linux/device.h:15,
> from include/linux/dma-mapping.h:7,
> from drivers/crypto/aspeed/aspeed-hace.h:9,
> from drivers/crypto/aspeed/aspeed-hace-hash.c:6:
> drivers/crypto/aspeed/aspeed-hace-hash.c: In function
> 'aspeed_ahash_dma_prepare_sg':
> >> drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x'
> expects argument of type 'unsigned int', but argument 7 has type 'size_t' {aka
> 'long unsigned int'} [-Wformat=]
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~
> include/linux/dev_printk.h:110:30: note: in definition of macro
> 'dev_printk_index_wrap'
> 110 | _p_func(dev, fmt, ##__VA_ARGS__);
> \
> | ^~~
> include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt'
> 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev,
> dev_fmt(fmt), ##__VA_ARGS__)
> |
> ^~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:9: note: in expansion of
> macro 'dev_info'
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:188:9: note: in expansion of
> macro 'AHASH_DBG'
> 188 | AHASH_DBG(hace_dev, "%s:0x%x, %s:0x%x, %s:0x%x,
> %s:0x%x\n",
> | ^~~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c: In function
> 'aspeed_hace_ahash_trigger':
> >> drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x'
> expects argument of type 'unsigned int', but argument 4 has type
> 'dma_addr_t' {aka 'long long unsigned int'} [-Wformat=]
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~
> include/linux/dev_printk.h:110:30: note: in definition of macro
> 'dev_printk_index_wrap'
> 110 | _p_func(dev, fmt, ##__VA_ARGS__);
> \
> | ^~~
> include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt'
> 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev,
> dev_fmt(fmt), ##__VA_ARGS__)
> |
> ^~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:9: note: in expansion of
> macro 'dev_info'
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:327:9: note: in expansion of
> macro 'AHASH_DBG'
> 327 | AHASH_DBG(hace_dev, "src_dma:0x%x,
> digest_dma:0x%x, length:0x%x\n",
> | ^~~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x'
> expects argument of type 'unsigned int', but argument 5 has type
> 'dma_addr_t' {aka 'long long unsigned int'} [-Wformat=]
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~
> include/linux/dev_printk.h:110:30: note: in definition of macro
> 'dev_printk_index_wrap'
> 110 | _p_func(dev, fmt, ##__VA_ARGS__);
> \
> | ^~~
> include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt'
> 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev,
> dev_fmt(fmt), ##__VA_ARGS__)
> |
> ^~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:9: note: in expansion of
> macro 'dev_info'
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:327:9: note: in expansion of
> macro 'AHASH_DBG'
> 327 | AHASH_DBG(hace_dev, "src_dma:0x%x,
> digest_dma:0x%x, length:0x%x\n",
> | ^~~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:28: warning: format '%x'
> expects argument of type 'unsigned int', but argument 6 has type 'size_t' {aka
> 'long unsigned int'} [-Wformat=]
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~
> include/linux/dev_printk.h:110:30: note: in definition of macro
> 'dev_printk_index_wrap'
> 110 | _p_func(dev, fmt, ##__VA_ARGS__);
> \
> | ^~~
> include/linux/dev_printk.h:150:58: note: in expansion of macro 'dev_fmt'
> 150 | dev_printk_index_wrap(_dev_info, KERN_INFO, dev,
> dev_fmt(fmt), ##__VA_ARGS__)
> |
> ^~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:10:9: note: in expansion of
> macro 'dev_info'
> 10 | dev_info((h)->dev, "%s() " fmt, __func__,
> ##__VA_ARGS__)
> | ^~~~~~~~
> drivers/crypto/aspeed/aspeed-hace-hash.c:327:9: note: in expansion of
> macro 'AHASH_DBG'
> 327 | AHASH_DBG(hace_dev, "src_dma:0x%x,
> digest_dma:0x%x, length:0x%x\n",
> | ^~~~~~~~~
> {standard input}: Assembler messages:
> {standard input}:489: Error: Register number out of range 0..3
> {standard input}:489: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:484: Warning: This is the location of the conflicting usage
> {standard input}:490: Error: Register number out of range 0..3
> {standard input}:490: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:484: Warning: This is the location of the conflicting usage
> {standard input}:490: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:489: Warning: This is the location of the conflicting usage
> {standard input}:493: Error: Register number out of range 0..3
> {standard input}:494: Error: Register number out of range 0..3
> {standard input}:494: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:493: Warning: This is the location of the conflicting usage
> {standard input}:2109: Error: Register number out of range 0..0
> {standard input}:2110: Error: Register number out of range 0..0
> {standard input}:2110: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2109: Warning: This is the location of the conflicting usage
> {standard input}:2111: Error: Register number out of range 0..0
> {standard input}:2111: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2109: Warning: This is the location of the conflicting usage
> {standard input}:2111: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2110: Warning: This is the location of the conflicting usage
> {standard input}:2113: Error: Register number out of range 0..0
> {standard input}:2113: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2109: Warning: This is the location of the conflicting usage
> {standard input}:2113: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2110: Warning: This is the location of the conflicting usage
> {standard input}:2113: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2111: Warning: This is the location of the conflicting usage
> {standard input}:2114: Error: Register number out of range 0..0
> {standard input}:2114: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2109: Warning: This is the location of the conflicting usage
> {standard input}:2114: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2110: Warning: This is the location of the conflicting usage
> {standard input}:2114: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2111: Warning: This is the location of the conflicting usage
> {standard input}:2114: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2113: Warning: This is the location of the conflicting usage
> {standard input}:2115: Error: Register number out of range 0..0
> {standard input}:2115: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2109: Warning: This is the location of the conflicting usage
> {standard input}:2115: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2110: Warning: This is the location of the conflicting usage
> {standard input}:2115: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2111: Warning: This is the location of the conflicting usage
> {standard input}:2115: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2113: Warning: This is the location of the conflicting usage
> {standard input}:2115: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 36
> {standard input}:2114: Warning: This is the location of the conflicting usage
> {standard input}:2119: Error: Register number out of range 0..0
> {standard input}:3353: Error: Register number out of range 0..4
> {standard input}:3354: Error: Register number out of range 0..4
> {standard input}:3354: Warning: Use of 'mov' violates WAW dependency
> 'GR%, % in 1 - 127' (impliedf), specific resource number is 45
> {standard input}:3354: Warning: Only the first path encountering the
> conflict is reported
> {standard input}:3353: Warning: This is the location of the conflicting usage
> {standard input}:3358: Error: Register number out of range 0..4
>
>
> vim +10 drivers/crypto/aspeed/aspeed-hace-hash.c
>
> 108713a713c7e4 Neal Liu 2022-08-18 7
> 108713a713c7e4 Neal Liu 2022-08-18 8 #ifdef
> CONFIG_CRYPTO_DEV_ASPEED_DEBUG
> 108713a713c7e4 Neal Liu 2022-08-18 9 #define AHASH_DBG(h, fmt, ...)
> \
> 108713a713c7e4 Neal Liu 2022-08-18 @10 dev_info((h)->dev, "%s() " fmt,
> __func__, ##__VA_ARGS__)
> 108713a713c7e4 Neal Liu 2022-08-18 11 #else
> 108713a713c7e4 Neal Liu 2022-08-18 12 #define AHASH_DBG(h, fmt, ...)
> \
> 108713a713c7e4 Neal Liu 2022-08-18 13 dev_dbg((h)->dev, "%s() " fmt,
> __func__, ##__VA_ARGS__)
> 108713a713c7e4 Neal Liu 2022-08-18 14 #endif
> 108713a713c7e4 Neal Liu 2022-08-18 15
>
> :::::: The code at line 10 was first introduced by commit
> :::::: 108713a713c7e4b7d07e6cd9b808503d5bb7089b crypto: aspeed - Add
> HACE hash driver
>
> :::::: TO: Neal Liu <neal_liu@...eedtech.com>
> :::::: CC: Herbert Xu <herbert@...dor.apana.org.au>
>
> --
> 0-DAY CI Kernel Test Service
> https://01.org/lkp
Powered by blists - more mailing lists