lkml.org 
[lkml]   [2018]   [Feb]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 1/1] scsi: ufs: make sure all interrupts are processed
Date


> >>> -----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
\
 
 \ /
  Last update: 2018-02-05 13:57    [W:0.063 / U:1.244 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site