From: kernel test robot <lkp@intel.com>
To: Brian King <brking@linux.vnet.ibm.com>
Cc: llvm@lists.linux.dev, oe-kbuild-all@lists.linux.dev,
linux-kernel@vger.kernel.org,
"Martin K. Petersen" <martin.petersen@oracle.com>,
Damien Le Moal <dlemoal@kernel.org>,
John Garry <john.g.garry@oracle.com>
Subject: drivers/scsi/ipr.c:5430:6: warning: variable 'ioasc' set but not used
Date: Wed, 10 Jan 2024 11:06:04 +0800 [thread overview]
Message-ID: <202401101004.fcxlPdWI-lkp@intel.com> (raw)
Hi Brian,
FYI, the error/warning still remains.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: ab27740f76654ed58dd32ac0ba0031c18a6dea3b
commit: 65a15d6560dff065339f6cac91de4beded60fe3f scsi: ipr: Remove SATA support
date: 9 months ago
config: i386-buildonly-randconfig-002-20240107 (https://download.01.org/0day-ci/archive/20240110/202401101004.fcxlPdWI-lkp@intel.com/config)
compiler: ClangBuiltLinux clang version 17.0.6 (https://github.com/llvm/llvm-project 6009708b4367171ccdbf4b5905cb6a803753fe18)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240110/202401101004.fcxlPdWI-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/202401101004.fcxlPdWI-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/scsi/ipr.c:1104:15: warning: variable 'proto' set but not used [-Wunused-but-set-variable]
1104 | unsigned int proto;
| ^
drivers/scsi/ipr.c:1261:15: warning: variable 'proto' set but not used [-Wunused-but-set-variable]
1261 | unsigned int proto;
| ^
drivers/scsi/ipr.c:4417:29: warning: variable 'res' set but not used [-Wunused-but-set-variable]
4417 | struct ipr_resource_entry *res;
| ^
>> drivers/scsi/ipr.c:5430:6: warning: variable 'ioasc' set but not used [-Wunused-but-set-variable]
5430 | u32 ioasc;
| ^
4 warnings generated.
vim +/ioasc +5430 drivers/scsi/ipr.c
3feeb89d40cc0ab Wayne Boyer 2009-10-20 5426
b53d124a154d002 wenxiong@linux.vnet.ibm.com 2013-01-11 5427 static int ipr_process_hrrq(struct ipr_hrr_queue *hrr_queue, int budget,
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5428 struct list_head *doneq)
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5429 {
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 @5430 u32 ioasc;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5431 u16 cmd_index;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5432 struct ipr_cmnd *ipr_cmd;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5433 struct ipr_ioa_cfg *ioa_cfg = hrr_queue->ioa_cfg;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5434 int num_hrrq = 0;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5435
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5436 /* If interrupts are disabled, ignore the interrupt */
56d6aa33d3f6847 wenxiong@linux.vnet.ibm.com 2013-01-11 5437 if (!hrr_queue->allow_interrupts)
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5438 return 0;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5439
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5440 while ((be32_to_cpu(*hrr_queue->hrrq_curr) & IPR_HRRQ_TOGGLE_BIT) ==
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5441 hrr_queue->toggle_bit) {
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5442
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5443 cmd_index = (be32_to_cpu(*hrr_queue->hrrq_curr) &
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5444 IPR_HRRQ_REQ_RESP_HANDLE_MASK) >>
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5445 IPR_HRRQ_REQ_RESP_HANDLE_SHIFT;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5446
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5447 if (unlikely(cmd_index > hrr_queue->max_cmd_id ||
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5448 cmd_index < hrr_queue->min_cmd_id)) {
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5449 ipr_isr_eh(ioa_cfg,
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5450 "Invalid response handle from IOA: ",
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5451 cmd_index);
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5452 break;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5453 }
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5454
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5455 ipr_cmd = ioa_cfg->ipr_cmnd_list[cmd_index];
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5456 ioasc = be32_to_cpu(ipr_cmd->s.ioasa.hdr.ioasc);
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5457
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5458 ipr_trc_hook(ipr_cmd, IPR_TRACE_FINISH, ioasc);
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5459
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5460 list_move_tail(&ipr_cmd->queue, doneq);
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5461
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5462 if (hrr_queue->hrrq_curr < hrr_queue->hrrq_end) {
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5463 hrr_queue->hrrq_curr++;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5464 } else {
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5465 hrr_queue->hrrq_curr = hrr_queue->hrrq_start;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5466 hrr_queue->toggle_bit ^= 1u;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5467 }
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5468 num_hrrq++;
b53d124a154d002 wenxiong@linux.vnet.ibm.com 2013-01-11 5469 if (budget > 0 && num_hrrq >= budget)
b53d124a154d002 wenxiong@linux.vnet.ibm.com 2013-01-11 5470 break;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5471 }
b53d124a154d002 wenxiong@linux.vnet.ibm.com 2013-01-11 5472
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5473 return num_hrrq;
05a6538a9a20499 wenxiong@linux.vnet.ibm.com 2013-01-11 5474 }
b53d124a154d002 wenxiong@linux.vnet.ibm.com 2013-01-11 5475
:::::: The code at line 5430 was first introduced by commit
:::::: 05a6538a9a204999e0c0f7faee00b81b334f4fc7 [SCSI] ipr: Add support for MSI-X and distributed completion
:::::: TO: wenxiong@linux.vnet.ibm.com <wenxiong@linux.vnet.ibm.com>
:::::: CC: James Bottomley <JBottomley@Parallels.com>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
next reply other threads:[~2024-01-10 3:06 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-01-10 3:06 kernel test robot [this message]
-- strict thread matches above, loose matches on Subject: below --
2023-09-09 7:42 drivers/scsi/ipr.c:5430:6: warning: variable 'ioasc' set but not used 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=202401101004.fcxlPdWI-lkp@intel.com \
--to=lkp@intel.com \
--cc=brking@linux.vnet.ibm.com \
--cc=dlemoal@kernel.org \
--cc=john.g.garry@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=llvm@lists.linux.dev \
--cc=martin.petersen@oracle.com \
--cc=oe-kbuild-all@lists.linux.dev \
/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.