Commit 03e1d28e authored by Stanley Chu's avatar Stanley Chu Committed by Martin K. Petersen

scsi: ufs: unify scsi_block_requests usage

Currently UFS driver has ufshcd_scsi_block_requests() with reference
counter mechanism to avoid possible racing of blocking and unblocking
requests flow. Unify all users in UFS driver to use the same function.

Link: https://lore.kernel.org/r/1577192466-20762-2-git-send-email-stanley.chu@mediatek.comReviewed-by: default avatarCan Guo <cang@codeaurora.org>
Reviewed-by: default avatarBart Van Assche <bvanassche@acm.org>
Reviewed-by: default avatarAlim Akhtar <alim.akhtar@samsung.com>
Signed-off-by: default avatarStanley Chu <stanley.chu@mediatek.com>
Signed-off-by: default avatarMartin K. Petersen <martin.petersen@oracle.com>
parent dbfc5626
...@@ -5159,7 +5159,7 @@ static void ufshcd_exception_event_handler(struct work_struct *work) ...@@ -5159,7 +5159,7 @@ static void ufshcd_exception_event_handler(struct work_struct *work)
hba = container_of(work, struct ufs_hba, eeh_work); hba = container_of(work, struct ufs_hba, eeh_work);
pm_runtime_get_sync(hba->dev); pm_runtime_get_sync(hba->dev);
scsi_block_requests(hba->host); ufshcd_scsi_block_requests(hba);
err = ufshcd_get_ee_status(hba, &status); err = ufshcd_get_ee_status(hba, &status);
if (err) { if (err) {
dev_err(hba->dev, "%s: failed to get exception status %d\n", dev_err(hba->dev, "%s: failed to get exception status %d\n",
...@@ -5173,7 +5173,7 @@ static void ufshcd_exception_event_handler(struct work_struct *work) ...@@ -5173,7 +5173,7 @@ static void ufshcd_exception_event_handler(struct work_struct *work)
ufshcd_bkops_exception_event_handler(hba); ufshcd_bkops_exception_event_handler(hba);
out: out:
scsi_unblock_requests(hba->host); ufshcd_scsi_unblock_requests(hba);
pm_runtime_put_sync(hba->dev); pm_runtime_put_sync(hba->dev);
return; return;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment