CC: [email protected] BCC: [email protected] CC: [email protected] TO: Bart Van Assche <[email protected]> CC: "Martin K. Petersen" <[email protected]> CC: Bean Huo <[email protected]>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: 34f4335c16a5f4bb7da6c8d2d5e780b6a163846a commit: dd11376b9f1b73aca3f8c6eb541486bbb6996f05 scsi: ufs: Split the drivers/scsi/ufs directory date: 3 weeks ago :::::: branch date: 3 hours ago :::::: commit date: 3 weeks ago config: arm-randconfig-m031-20220608 (https://download.01.org/0day-ci/archive/20220609/[email protected]/config) compiler: arm-linux-gnueabi-gcc (GCC) 11.3.0 If you fix the issue, kindly add following tag where applicable Reported-by: kernel test robot <[email protected]> Reported-by: Dan Carpenter <[email protected]> smatch warnings: drivers/ufs/core/ufshcd.c:5283 ufshcd_uic_cmd_compl() error: we previously assumed 'hba->active_uic_cmd' could be null (see line 5271) vim +5283 drivers/ufs/core/ufshcd.c a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5253 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5254 /** 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5255 * ufshcd_uic_cmd_compl - handle completion of uic command 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5256 * @hba: per adapter instance 53b3d9c3fdda94 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-08-31 5257 * @intr_status: interrupt status generated by the controller 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5258 * 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5259 * Returns 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5260 * IRQ_HANDLED - If interrupt is valid 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5261 * IRQ_NONE - If invalid interrupt 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5262 */ 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5263 static irqreturn_t ufshcd_uic_cmd_compl(struct ufs_hba *hba, u32 intr_status) 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5264 { 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5265 irqreturn_t retval = IRQ_NONE; 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5266 a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5267 spin_lock(hba->host->host_lock); a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5268 if (ufshcd_is_auto_hibern8_error(hba, intr_status)) a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5269 hba->errors |= (UFSHCD_UIC_HIBERN8_MASK & intr_status); a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5270 53b3d9c3fdda94 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-08-31 @5271 if ((intr_status & UIC_COMMAND_COMPL) && hba->active_uic_cmd) { 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5272 hba->active_uic_cmd->argument2 |= 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5273 ufshcd_get_uic_cmd_result(hba); 12b4fdb4f6bccb drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-08-31 5274 hba->active_uic_cmd->argument3 = 12b4fdb4f6bccb drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-08-31 5275 ufshcd_get_dme_attr_val(hba); 0f52fcb99ea273 drivers/scsi/ufs/ufshcd.c Can Guo 2020-11-02 5276 if (!hba->uic_async_done) 0f52fcb99ea273 drivers/scsi/ufs/ufshcd.c Can Guo 2020-11-02 5277 hba->active_uic_cmd->cmd_active = 0; 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5278 complete(&hba->active_uic_cmd->done); 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5279 retval = IRQ_HANDLED; 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5280 } 53b3d9c3fdda94 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-08-31 5281 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5282 if ((intr_status & UFSHCD_UIC_PWR_MASK) && hba->uic_async_done) { 0f52fcb99ea273 drivers/scsi/ufs/ufshcd.c Can Guo 2020-11-02 @5283 hba->active_uic_cmd->cmd_active = 0; 57d104c153d3d6 drivers/scsi/ufs/ufshcd.c Subhash Jadavani 2014-09-25 5284 complete(hba->uic_async_done); 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5285 retval = IRQ_HANDLED; 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5286 } aa5c697988b4c7 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-06-15 5287 aa5c697988b4c7 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-06-15 5288 if (retval == IRQ_HANDLED) aa5c697988b4c7 drivers/scsi/ufs/ufshcd.c Stanley Chu 2020-06-15 5289 ufshcd_add_uic_command_trace(hba, hba->active_uic_cmd, 28fa68fc557a7e drivers/scsi/ufs/ufshcd.c Bean Huo 2021-01-05 5290 UFS_CMD_COMP); a45f937110fa6b drivers/scsi/ufs/ufshcd.c Can Guo 2021-05-24 5291 spin_unlock(hba->host->host_lock); 9333d77573485c drivers/scsi/ufs/ufshcd.c Venkat Gopalakrishnan 2019-11-14 5292 return retval; 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5293 } 6ccf44fe4cd7c4 drivers/scsi/ufs/ufshcd.c Seungwon Jeon 2013-06-26 5294 :::::: The code at line 5283 was first introduced by commit :::::: 0f52fcb99ea2738a0a0f28e12cf4dd427069dd2a scsi: ufs: Try to save power mode change and UIC cmd completion timeout :::::: TO: Can Guo <[email protected]> :::::: CC: Martin K. Petersen <[email protected]> -- 0-DAY CI Kernel Test Service https://01.org/lkp _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
