Re: [PATCH v3 1/1] scsi: ufs: core: Flush exception handling work when RPM level is zero

[Thomas Yen]

Hi Bart,

Thanks for the tip regarding the tag ordering. I will ensure the Cc
tag is placed above the Signed-off-by tag in future submissions.

I had just sent v4 (to add the missing Fixes tag) before seeing this
message. Since the code logic in v4 is identical to v3, I hope that is
acceptable. Thanks.

Thomas


On Fri, Jan 30, 2026 at 1:04 AM Bart Van Assche <bvanassche@xxxxxxx> wrote:
>
> On 1/28/26 11:06 PM, Thomas Yen wrote:
> > Ensure that the exception event handling work is explicitly flushed
> > during suspend when the runtime power management level is set to
> > UFS_PM_LVL_0.
> >
> > When the RPM level is zero, the device power mode and link state both
> > remain active. Previously, the UFS core driver bypassed flushing
> > exception event handling jobs in this configuration. This created a race
> > condition where the driver could attempt to access the host controller
> > to handle an exception after the system had already entered a deep
> > power-down state, resulting in a system crash.
> >
> > Explicitly flush this work and disable auto BKOPs before the suspend
> > callback proceeds. This guarantees that pending exception tasks complete
> > and prevents illegal hardware access during the power-down sequence.
> >
> > Signed-off-by: Thomas Yen <thomasyen@xxxxxxxxxx>
> > Cc: Stable Tree <stable@xxxxxxxxxxxxxxx>
> For future patch submissions, please place the Cc: tag above the
> Signed-off-by tag. I think that is a widely used convention in the Linux
> kernel community. Anyway:
>
> Reviewed-by: Bart Van Assche <bvanassche@xxxxxxx>
>