Linux kernel mirror (for testing) git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
kernel os linux

scsi: mpt3sas: Stop fw fault watchdog work item during system shutdown

During system shutdown or reboot, mpt3sas will reset the firmware back to
ready state. However, the driver leaves running a watchdog work item
intended to keep the firmware in operational state. This causes a second,
unneeded reset on shutdown and moves the firmware back to operational
instead of in ready state as intended. And if the mpt3sas_fwfault_debug
module parameter is set, this extra reset also panics the system.

mpt3sas's scsih_shutdown needs to stop the watchdog before resetting the
firmware back to ready state.

Link: https://lore.kernel.org/r/20220722142448.6289-1-djeffery@redhat.com
Fixes: fae21608c31c ("scsi: mpt3sas: Transition IOC to Ready state during shutdown")
Tested-by: Laurence Oberman <loberman@redhat.com>
Acked-by: Sreekanth Reddy <sreekanth.reddy@broadcom.com>
Signed-off-by: David Jeffery <djeffery@redhat.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>

authored by

David Jeffery and committed by
Martin K. Petersen
0fde22c5 355bf2e0

+1
+1
drivers/scsi/mpt3sas/mpt3sas_scsih.c
··· 11386 11386 _scsih_ir_shutdown(ioc); 11387 11387 _scsih_nvme_shutdown(ioc); 11388 11388 mpt3sas_base_mask_interrupts(ioc); 11389 + mpt3sas_base_stop_watchdog(ioc); 11389 11390 ioc->shost_recovery = 1; 11390 11391 mpt3sas_base_make_ioc_ready(ioc, SOFT_RESET); 11391 11392 ioc->shost_recovery = 0;