Messages in this thread |  | | From | Avri Altman <> | Subject | RE: [PATCH 1/1] scsi: ufs: make sure all interrupts are processed | Date | Mon, 5 Feb 2018 12:57:18 +0000 |
| |
> >>> -----Original Message----- > >>> From: linux-scsi-owner@vger.kernel.org [mailto:linux-scsi- > >>> owner@vger.kernel.org] On Behalf Of Asutosh Das > >>> Sent: Tuesday, January 30, 2018 6:54 AM > >>> To: subhashj@codeaurora.org; cang@codeaurora.org; > >>> vivek.gautam@codeaurora.org; rnayak@codeaurora.org; > >>> vinholikatti@gmail.com; jejb@linux.vnet.ibm.com; > >>> martin.petersen@oracle.com > >>> Cc: linux-scsi@vger.kernel.org; Venkat Gopalakrishnan > >>> <venkatg@codeaurora.org>; Asutosh Das <asutoshd@codeaurora.org>; > >>> open list <linux-kernel@vger.kernel.org> > >>> Subject: [PATCH 1/1] scsi: ufs: make sure all interrupts are > >>> processed > >>> > >>> From: Venkat Gopalakrishnan <venkatg@codeaurora.org> > >>> > >>> As multiple requests are submitted to the ufs host controller in > >>> parallel there could be instances where the command completion > >>> interrupt arrives later for a request that is already processed > >>> earlier as the corresponding doorbell was cleared when handling the > >>> previous interrupt. Read the interrupt status in a loop after > >>> processing the received interrupt to catch such interrupts and > >>> handle it. > >>> > >>> Signed-off-by: Venkat Gopalakrishnan <venkatg@codeaurora.org> > >>> Signed-off-by: Asutosh Das <asutoshd@codeaurora.org> Tested-by: avri.altman@wdc.com
Tested on kirin960 (mate9) and msm8998 (htc11), both where interrupt aggregation is not allowed.
As a side note, I noticed that this patch is part of several patches, fixing some qcom-staff. Maybe you want to put them all in a patchset?
Thanks, Avri
|  |