Re: [PATCH] scsi: qedi: Add __printf to debug helpers and fix format strings

[kernel test robot <lkp@intel.com>]

Hi Adrián,

kernel test robot noticed the following build warnings:

[auto build test WARNING on jejb-scsi/for-next]
[also build test WARNING on mkp-scsi/for-next linus/master v7.4-rc2 next-20261109]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]

url:    https://github.com/intel-lab-lkp/linux/commits/Adri-n-Garc-a-Casado/scsi-qedi-Add-__printf-to-debug-helpers-and-fix-format-strings/20261109-151738
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jejb/scsi.git for-next
patch link:    https://lore.kernel.org/r/20261109141738.1624579-1-adriangarciacasado42%40gmail.com
patch subject: [PATCH] scsi: qedi: Add __printf to debug helpers and fix format strings
config: csky-allmodconfig (https://download.01.org/0day-ci/archive/20261110/202611100825.B5pebp4Q-lkp@intel.com/config)
compiler: csky-linux-gcc (GCC) 16.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20261110/202611100825.B5pebp4Q-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202611100825.B5pebp4Q-lkp@intel.com/

All warnings (new ones prefixed by >>):

   In file included from drivers/scsi/qedi/qedi.h:19,
                    from drivers/scsi/qedi/qedi_main.c:27:
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_alloc_nvm_iscsi_cfg':
>> drivers/scsi/qedi/qedi_main.c:1488:19: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type 'dma_addr_t' {aka 'unsigned int'} [-Wformat=]
    1488 |                   "NVM BUF addr=0x%p dma=0x%llx.\n", qedi->iscsi_image,
         |                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    1489 |                   qedi->nvm_buf_dma);
         |                   ~~~~~~~~~~~~~~~~~
         |                       |
         |                       dma_addr_t {aka unsigned int}
   drivers/scsi/qedi/qedi_dbg.h:78:64: note: in definition of macro 'QEDI_INFO'
      78 |                 qedi_dbg_info(pdev, __func__, __LINE__, level, fmt,     \
         |                                                                ^~~
   drivers/scsi/qedi/qedi_main.c:1488:47: note: format string is defined here
    1488 |                   "NVM BUF addr=0x%p dma=0x%llx.\n", qedi->iscsi_image,
         |                                            ~~~^
         |                                               |
         |                                               long long unsigned int
         |                                            %x
   drivers/scsi/qedi/qedi_main.c: In function 'qedi_alloc_global_queues':
   drivers/scsi/qedi/qedi_main.c:1754:1: error: could not split insn
    1754 | }
         | ^
   (insn 194 569 196 (set (reg:SI 3 a3 [orig:400 _253 ] [400])
           (xor:SI (reg:SI 2 a2 [orig:401 random_kmalloc_seed ] [401])
               (const:SI (plus:SI (symbol_ref:SI ("*.LANCHOR0") [flags 0x182])
                       (const_int 625 [0x271]))))) "include/linux/slab.h":762:52 288 {cskyv2_xorsi3}
        (expr_list:REG_DEAD (reg:SI 2 a2 [orig:401 random_kmalloc_seed ] [401])
           (nil)))
   during RTL pass: final
   drivers/scsi/qedi/qedi_main.c:1754:1: internal compiler error: in final_scan_insn_1, at final.cc:2823
   0x7f356d763d64 __libc_start_main
   	???:0
   ../lib/gcc/csky-linux/16.2.0/ -D __KERNEL__ -D KBUILD_EXTRA_WARN1 -D CSKYCPU_DEF_NAME="ck807f" -D MODULE -D KBUILD_BASENAME="qedi_main" -D KBUILD_MODNAME="qedi" -D __KBUILD_MODNAME=qedi -include include/linux/compiler-version.h -include include/linux/kconfig.h -include include/linux/compiler_types.h -MMD drivers/scsi/qedi/.qedi_main.o.d drivers/scsi/qedi/qedi_main.c -quiet -dumpdir drivers/scsi/qedi/ -dumpbase qedi_main.c -dumpbase-ext .c -mlittle-endian -mcpu=ck807 -mfloat-abi=soft -mdiv -mno-stack-size -O2 -Wall -Wextra -Werror=implicit-function-declaration -Werror=implicit-int -Werror=return-type -Werror=strict-prototypes -Wno-format-security -Wno-trigraphs -Wno-frame-address -Wno-address-of-packed-member -Wmissing-declarations -Wmissing-prototypes -Wframe-larger-than=1280 -Wno-main -Wno-type-limits -Wdangling-pointer=0 -Wvla-larger-than=1 -Wno-pointer-sign -Wcast-function-type -Wno-unterminated-string-initialization -Warray-bounds=0 -Wstringop-overflow=0 -Walloc-size-larger-than=18446744073709551615EiB -Wimplicit-fallthrough=5 -Werror=date-time -Werror=incompatible-pointer-types -Werror=designated-init -Wenum-conversion -Wunused -Wsuggest-attribute=format -Wmissing-include-dirs -Wunused-const-variable=2 -Wno-missing-field-initializers -Wno-shift-negative-value -Wno-maybe-uninitialized -Wno-sign-compare -Wno-unused-parameter -Wno-error=return-type -Wreturn-type -Wundef -std=gnu11 -p -fmacro-prefix-map== -fshort-wchar -fno-common -fno-PIE -fno-strict-aliasing -fms-extensions -fno-tree-vectorize -fno-delete-null-pointer-checks -fno-allow-store-data-races -fno-reorder-blocks -fno-ipa-cp-clone -fno-partial-inlining -fstack-protector-strong -ftrivial-auto-var-init=pattern -fzero-init-padding-bits=all -fno-stack-clash-protection -fdiagnostics-show-context=2 -fzero-call-used-regs=used-gpr -fno-inline-functions-called-once -fstrict-flex-arrays=3 -fno-strict-overflow -fstack-check=no -fconserve-stack -fno-builtin-wcslen -funsigned-char -falign-functions=64 -o -
   Please submit a full bug report, with preprocessed source (by using -freport-bug).
   Please include the complete backtrace with any bug report.
   See <https://gcc.gnu.org/bugs/> for instructions.


vim +1488 drivers/scsi/qedi/qedi_main.c

c57ec8fb7c02532 Nilesh Javali    2017-06-27  1477  
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1478  static int qedi_alloc_nvm_iscsi_cfg(struct qedi_ctx *qedi)
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1479  {
750afb08ca71310 Luis Chamberlain 2019-01-04  1480  	qedi->iscsi_image = dma_alloc_coherent(&qedi->pdev->dev,
872e192fab64388 Colin Ian King   2019-03-27  1481  					       sizeof(struct qedi_nvm_iscsi_image),
750afb08ca71310 Luis Chamberlain 2019-01-04  1482  					       &qedi->nvm_buf_dma, GFP_KERNEL);
c77a2fa3ff8f73d Nilesh Javali    2018-08-29  1483  	if (!qedi->iscsi_image) {
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1484  		QEDI_ERR(&qedi->dbg_ctx, "Could not allocate NVM BUF.\n");
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1485  		return -ENOMEM;
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1486  	}
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1487  	QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_INFO,
c77a2fa3ff8f73d Nilesh Javali    2018-08-29 @1488  		  "NVM BUF addr=0x%p dma=0x%llx.\n", qedi->iscsi_image,
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1489  		  qedi->nvm_buf_dma);
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1490  
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1491  	return 0;
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1492  }
c57ec8fb7c02532 Nilesh Javali    2017-06-27  1493  

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