Patch "scsi: ufs: core: Fix task management request completion timeout" has been added to the 5.4-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    scsi: ufs: core: Fix task management request completion timeout

to the 5.4-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     scsi-ufs-core-fix-task-management-request-completion.patch
and it can be found in the queue-5.4 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit ffce6f17239e5ffe05034bfb7c4c7e6c8dd6ae52
Author: Can Guo <cang@xxxxxxxxxxxxxx>
Date:   Thu Apr 1 00:39:08 2021 -0700

    scsi: ufs: core: Fix task management request completion timeout
    
    [ Upstream commit 1235fc569e0bf541ddda0a1224d4c6fa6d914890 ]
    
    ufshcd_tmc_handler() calls blk_mq_tagset_busy_iter(fn = ufshcd_compl_tm()),
    but since blk_mq_tagset_busy_iter() only iterates over all reserved tags
    and requests which are not in IDLE state, ufshcd_compl_tm() never gets a
    chance to run. Thus, TMR always ends up with completion timeout. Fix it by
    calling blk_mq_start_request() in __ufshcd_issue_tm_cmd().
    
    Link: https://lore.kernel.org/r/1617262750-4864-2-git-send-email-cang@xxxxxxxxxxxxxx
    Fixes: 69a6c269c097 ("scsi: ufs: Use blk_{get,put}_request() to allocate and free TMFs")
    Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>
    Signed-off-by: Can Guo <cang@xxxxxxxxxxxxxx>
    Signed-off-by: Martin K. Petersen <martin.petersen@xxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/drivers/scsi/ufs/ufshcd.c b/drivers/scsi/ufs/ufshcd.c
index e7e6405401dd..3f20270f0ca0 100644
--- a/drivers/scsi/ufs/ufshcd.c
+++ b/drivers/scsi/ufs/ufshcd.c
@@ -5710,6 +5710,7 @@ static int __ufshcd_issue_tm_cmd(struct ufs_hba *hba,
 
 	spin_lock_irqsave(host->host_lock, flags);
 	task_tag = hba->nutrs + free_slot;
+	blk_mq_start_request(req);
 
 	treq->req_header.dword_0 |= cpu_to_be32(task_tag);
 



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux