RE: [PATCH] ufs: core: Use link recovery when the h8 exit failure during runtime resume

["Seunghui Lee" <sh043.lee@samsung.com>]

> > > > Subject: Re: [PATCH] ufs: core: Use link recovery when the h8 exit
> > > > failure during runtime resume
> > > >
<snip> 
> 
> Based on my analysis, the tag might be:
> 
> commit 4db7a2360597 ("scsi: ufs: Fix concurrency of error handler and
> other error recovery paths"), This commit added the error handling logic
> that calls ufshcd_set_link_broken() and ufshcd_schedule_eh_work() when ret
> is non-zero in the ufshcd_uic_pwr_ctrl() function.
> 
> your patch adds a check for pm_op_in_progress to skip the error handler
> and use link recovery instead during PM operations.
> 
> Therefore, the appropriate Fixes tag for your patch would be:
> 
> Fixes: 4db7a2360597 ("scsi: ufs: Fix concurrency of error handler and
> other error recovery paths")
> 
> But, the problem is dd11376b9f1b ("scsi: ufs: Split the drivers/scsi/ufs
> directory") reorganzied the UFS code layout, it is true that not easy to
> add a proper tag,
> 
> Or we can use both fix tags, or just the last one.
> 
> Let Bart comment on this,
> 
> Kind regards,
> Bean


Thank you so much for sharing it.
I'll add it v2 as requested.

Fixes: 4db7a2360597 ("scsi: ufs: Fix concurrency of error handler and
other error recovery paths")