> -----Original Message----- > From: Jingoo Han [mailto:jg1.han@xxxxxxxxxxx] > Sent: Monday, December 16, 2013 6:24 AM > To: 'Harro Haan'; Mohit KUMAR DCG > Cc: 'Bjorn Helgaas'; 'Marek Vasut'; 'Richard Zhu'; 'Shawn Guo'; Pratyush > ANAND; 'Tim Harvey'; 'Juergen Beisert'; 'Arnd Bergmann'; 'Siva Reddy > Kallam'; 'Srikanth T Shivanand'; 'Sean Cross'; linux-pci@xxxxxxxxxxxxxxx; linux- > arm-kernel@xxxxxxxxxxxxxxxxxxx; 'Jingoo Han' > Subject: Re: [PATCH] pci: designware: fix missing msi irqs > > On Friday, December 13, 2013 3:29 AM, Harro Haan wrote: > > > > The interrupts were cleared after the irq handler was called. > > This means that new interrupts that occur after the handler handled > > the previous irq but before the interrupt is cleared will be missed. > > > > Signed-off-by: Harro Haan <hrhaan@xxxxxxxxx> > > Cc: Mohit Kumar <mohit.kumar@xxxxxx> > > Cc: Jingoo Han <jg1.han@xxxxxxxxxxx> > > Cc: Bjorn Helgaas <bhelgaas@xxxxxxxxxx> > > Cc: Marek Vasut <marex@xxxxxxx> > > Cc: Richard Zhu <hong-xing.zhu@xxxxxxxxxxxxx> > > Cc: Shawn Guo <shawn.guo@xxxxxxxxxx> > > Cc: Pratyush Anand <pratyush.anand@xxxxxx> > > Cc: Tim Harvey <tharvey@xxxxxxxxxxxxx> > > Cc: Juergen Beisert <jbe@xxxxxxxxxxxxxx> > > Cc: Arnd Bergmann <arnd@xxxxxxxx> > > Cc: Siva Reddy Kallam <siva.kallam@xxxxxxxxxxx> > > Cc: Srikanth T Shivanand <ts.srikanth@xxxxxxxxxxx> > > Cc: Sean Cross <xobs@xxxxxxxxxx> > > Cc: linux-pci@xxxxxxxxxxxxxxx > > Cc: linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > > Acked-by: Jingoo Han <jg1.han@xxxxxxxxxxx> > > I was not able to reproduce the problem, however it looks good. Also, there > is no side effect on Exynos platform. > > Mohit, > If you have some comments, please let us know. - Quickly tested on SPEAr1310, working fine without any side effect. So is, Acked-by: Mohit Kumar <mohit.kumar@xxxxxx> .. > > > --- > > drivers/pci/host/pcie-designware.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html