Re: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare

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

To: hoyoung seo <hy50.seo@xxxxxxxxxxx>, linux-scsi@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx, alim.akhtar@xxxxxxxxxxx, avri.altman@xxxxxxx, jejb@xxxxxxxxxxxxx, martin.petersen@xxxxxxxxxx, beanhuo@xxxxxxxxxx, kwangwon.min@xxxxxxxxxxx, kwmad.kim@xxxxxxxxxxx, sh425.lee@xxxxxxxxxxx, sc.suh@xxxxxxxxxxx, quic_nguyenb@xxxxxxxxxxx, cpgs@xxxxxxxxxxx, grant.jung@xxxxxxxxxxx, junwoo80.lee@xxxxxxxxxxx
Subject: Re: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Wed, 24 Jan 2024 08:17:16 -0800
In-reply-to: <017501da4da5$405e5ec0$c11b1c40$@samsung.com>
User-agent: Mozilla Thunderbird

On 1/22/24 18:38, hoyoung seo wrote:

When err_handler is completed, active_reqs becomes negative because
ufshcd_release() is called again in ufshcd_err_handling_unprepare().
I tested it while printing the log, and if I misanalyzed it, let me know.


Please repeat your analysis. I think this patch is wrong.

Thanks,

Bart.

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

Follow-Ups:
- RE: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
  - From: hoyoung seo

References:
- [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
  - From: SEO HOYOUNG
- Re: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
  - From: Bart Van Assche
- RE: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
  - From: hoyoung seo

Prev by Date: Re: [PATCH v2] scsi: ufs: qcom: avoid re-init quirk when gears match
Next by Date: Re: [LSF/MM/BPF TOPIC] State Of The Page
Previous by thread: RE: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
Next by thread: RE: [PATCH v1] scsi: ufs: core: Remove the ufshcd_release in ufshcd_err_handling_prepare
Index(es):
- Date
- Thread

[Index of Archives] [SCSI Target Devel] [Linux SCSI Target Infrastructure] [Kernel Newbies] [IDE] [Security] [Git] [Netfilter] [Bugtraq] [Yosemite News] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux ATA RAID] [Linux IIO] [Samba] [Device Mapper]