Re: [PATCH v1] ufs: core: fix racing issue between ufshcd_mcq_abort and ISR

To: peter.wang@xxxxxxxxxxxx, stanley.chu@xxxxxxxxxxxx, linux-scsi@xxxxxxxxxxxxxxx, martin.petersen@xxxxxxxxxx, avri.altman@xxxxxxx, alim.akhtar@xxxxxxxxxxx, jejb@xxxxxxxxxxxxx
Subject: Re: [PATCH v1] ufs: core: fix racing issue between ufshcd_mcq_abort and ISR
From: Bart Van Assche <bvanassche@xxxxxxx>
Date: Fri, 3 Nov 2023 07:57:20 -0700
Cc: wsd_upstream@xxxxxxxxxxxx, linux-mediatek@xxxxxxxxxxxxxxxxxxx, chun-hung.wu@xxxxxxxxxxxx, alice.chao@xxxxxxxxxxxx, cc.chou@xxxxxxxxxxxx, chaotian.jing@xxxxxxxxxxxx, jiajie.hao@xxxxxxxxxxxx, powen.kao@xxxxxxxxxxxx, qilin.tan@xxxxxxxxxxxx, lin.gui@xxxxxxxxxxxx, tun-yu.yu@xxxxxxxxxxxx, eddie.huang@xxxxxxxxxxxx, naomi.chu@xxxxxxxxxxxx
In-reply-to: <20231027084329.4067-1-peter.wang@mediatek.com>
User-agent: Mozilla Thunderbird

On 10/27/23 01:43, peter.wang@xxxxxxxxxxxx wrote:

If command timeout happen and cq complete irq raise at the same time,
ufshcd_mcq_abort null the lprb->cmd and NULL poiner KE in ISR.


Please add a Fixes: tag. Otherwise this patch looks good to me.

Thanks,

Bart.

Follow-Ups:
- Re: [PATCH v1] ufs: core: fix racing issue between ufshcd_mcq_abort and ISR
  - From: Peter Wang (王信友)

References:
- [PATCH v1] ufs: core: fix racing issue between ufshcd_mcq_abort and ISR
  - From: peter.wang