scsi: ufs: core: Fix another deadlock during RTC update
authorPeter Wang <peter.wang@mediatek.com>
Thu, 24 Oct 2024 01:54:53 +0000 (09:54 +0800)
committerMartin K. Petersen <martin.petersen@oracle.com>
Fri, 25 Oct 2024 18:51:34 +0000 (14:51 -0400)
If ufshcd_rtc_work calls ufshcd_rpm_put_sync() and the pm's usage_count
is 0, we will enter the runtime suspend callback.  However, the runtime
suspend callback will wait to flush ufshcd_rtc_work, causing a deadlock.

Replace ufshcd_rpm_put_sync() with ufshcd_rpm_put() to avoid the
deadlock.

Fixes: 6bf999e0eb41 ("scsi: ufs: core: Add UFS RTC support")
Cc: stable@vger.kernel.org #6.11.x
Signed-off-by: Peter Wang <peter.wang@mediatek.com>
Link: https://lore.kernel.org/r/20241024015453.21684-1-peter.wang@mediatek.com
Reviewed-by: Bart Van Assche <bvanassche@acm.org>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/ufs/core/ufshcd.c

index 706dc81eb92454bcf53974fb38df11a29093a709..0e22bbb78239101dd952740fb8453d2c4e79839f 100644 (file)
@@ -8219,7 +8219,7 @@ static void ufshcd_update_rtc(struct ufs_hba *hba)
 
        err = ufshcd_query_attr(hba, UPIU_QUERY_OPCODE_WRITE_ATTR, QUERY_ATTR_IDN_SECONDS_PASSED,
                                0, 0, &val);
-       ufshcd_rpm_put_sync(hba);
+       ufshcd_rpm_put(hba);
 
        if (err)
                dev_err(hba->dev, "%s: Failed to update rtc %d\n", __func__, err);