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'}

[Neal Liu <neal_liu@aspeedtech.com>]

> -----Original Message-----
> From: kernel test robot <lkp@intel.com>
> Sent: Saturday, September 3, 2022 12:38 AM
> To: Herbert Xu <herbert@gondor.apana.org.au>
> Cc: kbuild-all@lists.01.org; Ammar Faizi <ammarfaizi2@gnuweeb.org>;
> GNU/Weeb Mailing List <gwml@vger.gnuweeb.org>;
> linux-kernel@vger.kernel.org; Neal Liu <neal_liu@aspeedtech.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@intel.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@aspeedtech.com>
> :::::: CC: Herbert Xu <herbert@gondor.apana.org.au>
> 
> --
> 0-DAY CI Kernel Test Service
> https://01.org/lkp