scsi: ufs: core: Fix synchronization between scsi_unjam_host() and ufshcd_queuecommand()
authorAdrian Hunter <adrian.hunter@intel.com>
Fri, 8 Oct 2021 08:40:48 +0000 (11:40 +0300)
committerMartin K. Petersen <martin.petersen@oracle.com>
Tue, 12 Oct 2021 17:53:18 +0000 (13:53 -0400)
commitd489f18ad1fc33ab9b60dc40fe3851bb0d87de28
treee684b5afb96f38e62adf346cf0149aa5707a8082
parent9561f58442e48c3c71da250aa63bd02ec6bffcb5
scsi: ufs: core: Fix synchronization between scsi_unjam_host() and ufshcd_queuecommand()

The SCSI error handler calls scsi_unjam_host() which can call the queue
function ufshcd_queuecommand() indirectly. The error handler changes the
state to UFSHCD_STATE_RESET while running, but error interrupts that
happen while the error handler is running could change the state to
UFSHCD_STATE_EH_SCHEDULED_NON_FATAL which would allow requests to go
through ufshcd_queuecommand() even though the error handler is running.
Block that hole by checking whether the error handler is in progress.

Link: https://lore.kernel.org/r/20211008084048.257498-1-adrian.hunter@intel.com
Reviewed-by: Asutosh Das <asutoshd@codeaurora.org>
Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/scsi/ufs/ufshcd.c