Re: [PATCH 1/1] scsi: ufs: core: Fix EH failure after wlun resume error

["Peter Wang (王信友)" <peter.wang@mediatek.com>]

On Wed, 2025-11-12 at 17:18 +0800, Brian Kao wrote:
> This log actually is triggered by fault injection when wlun is
> suspended
> echo 0xf > /sys/kernel/debug/ufshcd/3c2d0000.ufs/saved_uic_err
> 
> In a real-world scenario, this situation can occur when an error
> interrupt calls `ufshcd_check_errors()`, which schedules the error
> handler before the system reaches `ufshcd_set_dev_pwr_mode()`.

Hi Brian,

When ufshcd_suspend is called, it disables IRQs by calling
ufshcd_disable_irq. So, in a real-world scenario, it should
not be possible for the UFS to be suspended and the error
handler work to be triggered by an IRQ.

Fault injection using saved_uic_err could create this situation,
but if this cannot happen in reality, we should consider whether
this is a reasonable test case? If it is, then I think this patch
looks good to me. If not, maybe we should get the runtime PM state
in the fault injection and remove this redundant recovery code.
What do you think?

Thanks
Peter