From: kernel test robot <lkp@intel.com>
To: trix@redhat.com, njavali@marvell.com, mrangankar@marvell.com,
jejb@linux.ibm.com, martin.petersen@oracle.com
Cc: kbuild-all@lists.01.org, GR-QLogic-Storage-Upstream@marvell.com,
linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org,
Tom Rix <trix@redhat.com>
Subject: Re: [PATCH] scsi: qedi: add printf attribute to log function
Date: Mon, 28 Dec 2020 17:37:31 +0800 [thread overview]
Message-ID: <202012281717.UICvwqhL-lkp@intel.com> (raw)
In-Reply-To: <20201221162335.3756353-1-trix@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 13336 bytes --]
Hi,
I love your patch! Perhaps something to improve:
[auto build test WARNING on mkp-scsi/for-next]
[also build test WARNING on scsi/for-next v5.11-rc1 next-20201223]
[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]
url: https://github.com/0day-ci/linux/commits/trix-redhat-com/scsi-qedi-add-printf-attribute-to-log-function/20201222-002559
base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 9.3.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/0day-ci/linux/commit/d3e320669a713151990301b3ed5209a6b684869b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review trix-redhat-com/scsi-qedi-add-printf-attribute-to-log-function/20201222-002559
git checkout d3e320669a713151990301b3ed5209a6b684869b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <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_schedule_hw_err_handler':
>> drivers/scsi/qedi/qedi_main.c:1131:5: warning: format '%x' expects argument of type 'unsigned int', but argument 7 has type 'long unsigned int' [-Wformat=]
1131 | "HW error handler scheduled, err=%d err_flags=0x%x\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1132 | err_type, qedi->qedi_err_flags);
| ~~~~~~~~~~~~~~~~~~~~
| |
| long unsigned int
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1131:54: note: format string is defined here
1131 | "HW error handler scheduled, err=%d err_flags=0x%x\n",
| ~^
| |
| unsigned int
| %lx
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:5: 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:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1488:33: note: format string is defined here
1488 | "NVM BUF addr=0x%p dma=0x%llx.\n", qedi->iscsi_image,
| ~~~^
| |
| long long unsigned int
| %x
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_bdq':
>> drivers/scsi/qedi/qedi_main.c:1585:6: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type '__le32' {aka 'unsigned int'} [-Wformat=]
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1586 | pbl, pbl->address.hi, pbl->address.lo, i);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1585:40: note: format string is defined here
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_main.c:27:
drivers/scsi/qedi/qedi_main.c:1585:6: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type '__le32' {aka 'unsigned int'} [-Wformat=]
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1586 | pbl, pbl->address.hi, pbl->address.lo, i);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1585:52: note: format string is defined here
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
--
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c: In function 'qedi_get_rq_bdq_buf':
>> drivers/scsi/qedi/qedi_fw.c:341:5: warning: format '%p' expects argument of type 'void *', but argument 6 has type 'int' [-Wformat=]
341 | "rqe_opaque [0x%p], idx [%d]\n", cqe->rqe_opaque, idx);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~
| |
| int
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:341:21: note: format string is defined here
341 | "rqe_opaque [0x%p], idx [%d]\n", cqe->rqe_opaque, idx);
| ~^
| |
| void *
| %d
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c: In function 'qedi_put_rq_bdq_buf':
>> drivers/scsi/qedi/qedi_fw.c:380:5: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type '__le32' {aka 'unsigned int'} [-Wformat=]
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
381 | pbl, pbl->address.hi, pbl->address.lo, idx);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:380:39: note: format string is defined here
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c:380:5: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type '__le32' {aka 'unsigned int'} [-Wformat=]
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
381 | pbl, pbl->address.hi, pbl->address.lo, idx);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:380:51: note: format string is defined here
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
vim +1131 drivers/scsi/qedi/qedi_main.c
534bbdf8832ae48 Manish Rangankar 2018-05-22 1120
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1121 void qedi_schedule_hw_err_handler(void *dev,
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1122 enum qed_hw_err_type err_type)
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1123 {
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1124 struct qedi_ctx *qedi = (struct qedi_ctx *)dev;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1125 unsigned long override_flags = qedi_flags_override;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1126
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1127 if (override_flags && test_bit(QEDI_ERR_OVERRIDE_EN, &override_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1128 qedi->qedi_err_flags = qedi_flags_override;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1129
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1130 QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_INFO,
f4ba4e55db6db7e Manish Rangankar 2020-09-08 @1131 "HW error handler scheduled, err=%d err_flags=0x%x\n",
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1132 err_type, qedi->qedi_err_flags);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1133
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1134 switch (err_type) {
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1135 case QED_HW_ERR_FAN_FAIL:
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1136 schedule_delayed_work(&qedi->board_disable_work, 0);
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1137 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1138 case QED_HW_ERR_MFW_RESP_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1139 case QED_HW_ERR_HW_ATTN:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1140 case QED_HW_ERR_DMAE_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1141 case QED_HW_ERR_RAMROD_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1142 case QED_HW_ERR_FW_ASSERT:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1143 /* Prevent HW attentions from being reasserted */
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1144 if (test_bit(QEDI_ERR_ATTN_CLR_EN, &qedi->qedi_err_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1145 qedi_ops->common->attn_clr_enable(qedi->cdev, true);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1146
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1147 if (err_type == QED_HW_ERR_RAMROD_FAIL &&
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1148 test_bit(QEDI_ERR_IS_RECOVERABLE, &qedi->qedi_err_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1149 qedi_ops->common->recovery_process(qedi->cdev);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1150
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1151 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1152 default:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1153 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1154 }
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1155 }
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1156
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 66281 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH] scsi: qedi: add printf attribute to log function
Date: Mon, 28 Dec 2020 17:37:31 +0800 [thread overview]
Message-ID: <202012281717.UICvwqhL-lkp@intel.com> (raw)
In-Reply-To: <20201221162335.3756353-1-trix@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 13542 bytes --]
Hi,
I love your patch! Perhaps something to improve:
[auto build test WARNING on mkp-scsi/for-next]
[also build test WARNING on scsi/for-next v5.11-rc1 next-20201223]
[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]
url: https://github.com/0day-ci/linux/commits/trix-redhat-com/scsi-qedi-add-printf-attribute-to-log-function/20201222-002559
base: https://git.kernel.org/pub/scm/linux/kernel/git/mkp/scsi.git for-next
config: xtensa-allyesconfig (attached as .config)
compiler: xtensa-linux-gcc (GCC) 9.3.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/0day-ci/linux/commit/d3e320669a713151990301b3ed5209a6b684869b
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review trix-redhat-com/scsi-qedi-add-printf-attribute-to-log-function/20201222-002559
git checkout d3e320669a713151990301b3ed5209a6b684869b
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=xtensa
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <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_schedule_hw_err_handler':
>> drivers/scsi/qedi/qedi_main.c:1131:5: warning: format '%x' expects argument of type 'unsigned int', but argument 7 has type 'long unsigned int' [-Wformat=]
1131 | "HW error handler scheduled, err=%d err_flags=0x%x\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1132 | err_type, qedi->qedi_err_flags);
| ~~~~~~~~~~~~~~~~~~~~
| |
| long unsigned int
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1131:54: note: format string is defined here
1131 | "HW error handler scheduled, err=%d err_flags=0x%x\n",
| ~^
| |
| unsigned int
| %lx
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:5: 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:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1488:33: note: format string is defined here
1488 | "NVM BUF addr=0x%p dma=0x%llx.\n", qedi->iscsi_image,
| ~~~^
| |
| long long unsigned int
| %x
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_bdq':
>> drivers/scsi/qedi/qedi_main.c:1585:6: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type '__le32' {aka 'unsigned int'} [-Wformat=]
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1586 | pbl, pbl->address.hi, pbl->address.lo, i);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1585:40: note: format string is defined here
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_main.c:27:
drivers/scsi/qedi/qedi_main.c:1585:6: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type '__le32' {aka 'unsigned int'} [-Wformat=]
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
1586 | pbl, pbl->address.hi, pbl->address.lo, i);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_main.c:1585:52: note: format string is defined here
1585 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx], idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
--
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c: In function 'qedi_get_rq_bdq_buf':
>> drivers/scsi/qedi/qedi_fw.c:341:5: warning: format '%p' expects argument of type 'void *', but argument 6 has type 'int' [-Wformat=]
341 | "rqe_opaque [0x%p], idx [%d]\n", cqe->rqe_opaque, idx);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ~~~~~~~~~~~~~~~
| |
| int
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:341:21: note: format string is defined here
341 | "rqe_opaque [0x%p], idx [%d]\n", cqe->rqe_opaque, idx);
| ~^
| |
| void *
| %d
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c: In function 'qedi_put_rq_bdq_buf':
>> drivers/scsi/qedi/qedi_fw.c:380:5: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 7 has type '__le32' {aka 'unsigned int'} [-Wformat=]
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
381 | pbl, pbl->address.hi, pbl->address.lo, idx);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:380:39: note: format string is defined here
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
In file included from drivers/scsi/qedi/qedi.h:19,
from drivers/scsi/qedi/qedi_fw.c:11:
drivers/scsi/qedi/qedi_fw.c:380:5: warning: format '%llx' expects argument of type 'long long unsigned int', but argument 8 has type '__le32' {aka 'unsigned int'} [-Wformat=]
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
381 | pbl, pbl->address.hi, pbl->address.lo, idx);
| ~~~~~~~~~~~~~~~
| |
| __le32 {aka unsigned int}
drivers/scsi/qedi/qedi_dbg.h:79:50: note: in definition of macro 'QEDI_INFO'
79 | qedi_dbg_info(pdev, __func__, __LINE__, level, fmt, \
| ^~~
drivers/scsi/qedi/qedi_fw.c:380:51: note: format string is defined here
380 | "pbl [0x%p] pbl->address hi [0x%llx] lo [0x%llx] idx [%d]\n",
| ~~~^
| |
| long long unsigned int
| %x
vim +1131 drivers/scsi/qedi/qedi_main.c
534bbdf8832ae48 Manish Rangankar 2018-05-22 1120
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1121 void qedi_schedule_hw_err_handler(void *dev,
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1122 enum qed_hw_err_type err_type)
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1123 {
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1124 struct qedi_ctx *qedi = (struct qedi_ctx *)dev;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1125 unsigned long override_flags = qedi_flags_override;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1126
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1127 if (override_flags && test_bit(QEDI_ERR_OVERRIDE_EN, &override_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1128 qedi->qedi_err_flags = qedi_flags_override;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1129
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1130 QEDI_INFO(&qedi->dbg_ctx, QEDI_LOG_INFO,
f4ba4e55db6db7e Manish Rangankar 2020-09-08 @1131 "HW error handler scheduled, err=%d err_flags=0x%x\n",
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1132 err_type, qedi->qedi_err_flags);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1133
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1134 switch (err_type) {
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1135 case QED_HW_ERR_FAN_FAIL:
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1136 schedule_delayed_work(&qedi->board_disable_work, 0);
7dc71ac8eb0bcaa Manish Rangankar 2020-09-24 1137 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1138 case QED_HW_ERR_MFW_RESP_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1139 case QED_HW_ERR_HW_ATTN:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1140 case QED_HW_ERR_DMAE_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1141 case QED_HW_ERR_RAMROD_FAIL:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1142 case QED_HW_ERR_FW_ASSERT:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1143 /* Prevent HW attentions from being reasserted */
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1144 if (test_bit(QEDI_ERR_ATTN_CLR_EN, &qedi->qedi_err_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1145 qedi_ops->common->attn_clr_enable(qedi->cdev, true);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1146
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1147 if (err_type == QED_HW_ERR_RAMROD_FAIL &&
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1148 test_bit(QEDI_ERR_IS_RECOVERABLE, &qedi->qedi_err_flags))
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1149 qedi_ops->common->recovery_process(qedi->cdev);
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1150
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1151 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1152 default:
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1153 break;
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1154 }
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1155 }
f4ba4e55db6db7e Manish Rangankar 2020-09-08 1156
---
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: 66281 bytes --]
next prev parent reply other threads:[~2020-12-28 9:38 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-12-21 16:23 [PATCH] scsi: qedi: add printf attribute to log function trix
2020-12-28 6:14 ` Manish Rangankar
2020-12-28 9:37 ` kernel test robot [this message]
2020-12-28 9:37 ` kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202012281717.UICvwqhL-lkp@intel.com \
--to=lkp@intel.com \
--cc=GR-QLogic-Storage-Upstream@marvell.com \
--cc=jejb@linux.ibm.com \
--cc=kbuild-all@lists.01.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-scsi@vger.kernel.org \
--cc=martin.petersen@oracle.com \
--cc=mrangankar@marvell.com \
--cc=njavali@marvell.com \
--cc=trix@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.