* Re: [PATCH] pci: designware: fix missing msi irqs
[not found] <20131212182903.409515738@gmail.com>
@ 2013-12-12 19:29 ` Matthias Mann
2013-12-13 8:42 ` Matthias Mann
2013-12-12 22:01 ` Marek Vasut
2013-12-16 0:54 ` Jingoo Han
2 siblings, 1 reply; 7+ messages in thread
From: Matthias Mann @ 2013-12-12 19:29 UTC (permalink / raw)
To: Harro Haan, Mohit Kumar, Jingoo Han
Cc: Marek Vasut, Pratyush Anand, Arnd Bergmann, Siva Reddy Kallam,
Richard Zhu, linux-pci, Tim Harvey, Srikanth T Shivanand,
Bjorn Helgaas, Sean Cross, Shawn Guo, linux-arm-kernel
Am 12.12.2013 19:29, schrieb Harro Haan:
> 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.
I discovered the same issue today. I'll test that patch tomorrow.
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] pci: designware: fix missing msi irqs
2013-12-12 19:29 ` [PATCH] pci: designware: fix missing msi irqs Matthias Mann
@ 2013-12-13 8:42 ` Matthias Mann
0 siblings, 0 replies; 7+ messages in thread
From: Matthias Mann @ 2013-12-13 8:42 UTC (permalink / raw)
To: Matthias Mann
Cc: Harro Haan, Mohit Kumar, Jingoo Han, Marek Vasut, Pratyush Anand,
Arnd Bergmann, Siva Reddy Kallam, Richard Zhu, linux-pci,
Tim Harvey, Srikanth T Shivanand, Bjorn Helgaas, Sean Cross,
Shawn Guo, linux-arm-kernel
Am 12.12.2013 20:29, schrieb Matthias Mann:
> Am 12.12.2013 19:29, schrieb Harro Haan:
>> 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.
>
> I discovered the same issue today. I'll test that patch tomorrow.
>
I tested this patch on our custom i.MX6 board with an Altera FPGA as PCIe
device and it fixed the missed MSI interrupts.
Tested-by: Matthias Mann <m.mann@arkona-technologies.de>
Regards
Matthias Mann
--
Matthias Mann
Research & Development
Phone: +49 (0) 6155 7802883
Fax: +49 (0) 6155 7802880
Email: M.Mann@arkona-technologies.de
<mailto:M.Mann@arkona-technologies.de>
Web: www.arkona-technologies.de <http://www.arkona-technologies.de>
arkona technologies GmbH
Im Leuschnerpark 4
64347 Griesheim
Germany
Amtsgericht / Commercial Register of Darmstadt, HRB 90080
USt-ID: DE273794666
Steuernummer / Tax-ID: 007 / 228 / 19331
Geschäftsführung / Managing Director: Rainer Sturm
This e-mail may contain confidential and/or privileged information. If you are
not the intended recipient (or have received this e-mail in error) please notify
the sender immediately and delete this message and any attachments from your
system. Any unauthorised copying, disclosure or distribution of the material in
this e-mail is strictly forbidden.
We cannot accept any responsibility for the accuracy or completeness of this
message as it has been transmitted over a public network. If, despite our use of
anti-virus software, a virus enters your systems in connection with the sending
of the e-mail, you may not hold us liable for any damages that may possibly
arise in that connection. We will accept liability which by law we cannot exclude.
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] pci: designware: fix missing msi irqs
[not found] <20131212182903.409515738@gmail.com>
2013-12-12 19:29 ` [PATCH] pci: designware: fix missing msi irqs Matthias Mann
@ 2013-12-12 22:01 ` Marek Vasut
2013-12-16 0:54 ` Jingoo Han
2 siblings, 0 replies; 7+ messages in thread
From: Marek Vasut @ 2013-12-12 22:01 UTC (permalink / raw)
To: Harro Haan
Cc: Mohit Kumar, Jingoo Han, Bjorn Helgaas, Richard Zhu, Shawn Guo,
Pratyush Anand, Tim Harvey, Juergen Beisert, Arnd Bergmann,
Siva Reddy Kallam, Srikanth T Shivanand, Sean Cross, linux-pci,
linux-arm-kernel
On Thursday, December 12, 2013 at 07:29:03 PM, 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@gmail.com>
> Cc: Mohit Kumar <mohit.kumar@st.com>
> Cc: Jingoo Han <jg1.han@samsung.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Richard Zhu <hong-xing.zhu@freescale.com>
> Cc: Shawn Guo <shawn.guo@linaro.org>
> Cc: Pratyush Anand <pratyush.anand@st.com>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: Juergen Beisert <jbe@pengutronix.de>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Siva Reddy Kallam <siva.kallam@samsung.com>
> Cc: Srikanth T Shivanand <ts.srikanth@samsung.com>
> Cc: Sean Cross <xobs@kosagi.com>
> Cc: linux-pci@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
> ---
> drivers/pci/host/pcie-designware.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/pci/host/pcie-designware.c
> b/drivers/pci/host/pcie-designware.c index 1ce0453..762f596 100644
> --- a/drivers/pci/host/pcie-designware.c
> +++ b/drivers/pci/host/pcie-designware.c
> @@ -167,11 +167,13 @@ void dw_handle_msi_irq(struct pcie_port *pp)
> while ((pos = find_next_bit(&val, 32, pos)) != 32) {
> irq = irq_find_mapping(pp->irq_domain,
> i * 32 + pos);
> + dw_pcie_wr_own_conf(pp,
> + PCIE_MSI_INTR0_STATUS + i * 12,
> + 4, 1 << pos);
> generic_handle_irq(irq);
> pos++;
> }
> }
> - dw_pcie_wr_own_conf(pp, PCIE_MSI_INTR0_STATUS + i * 12, 4, val);
> }
> }
On imx6-sabrelite:
Tested-by: Marek Vasut <marex@denx.de>
Best regards,
Marek Vasut
^ permalink raw reply [flat|nested] 7+ messages in thread* Re: [PATCH] pci: designware: fix missing msi irqs
[not found] <20131212182903.409515738@gmail.com>
2013-12-12 19:29 ` [PATCH] pci: designware: fix missing msi irqs Matthias Mann
2013-12-12 22:01 ` Marek Vasut
@ 2013-12-16 0:54 ` Jingoo Han
2013-12-16 6:48 ` Mohit KUMAR DCG
2 siblings, 1 reply; 7+ messages in thread
From: Jingoo Han @ 2013-12-16 0:54 UTC (permalink / raw)
To: 'Harro Haan', 'Mohit Kumar'
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, linux-arm-kernel,
'Jingoo Han'
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@gmail.com>
> Cc: Mohit Kumar <mohit.kumar@st.com>
> Cc: Jingoo Han <jg1.han@samsung.com>
> Cc: Bjorn Helgaas <bhelgaas@google.com>
> Cc: Marek Vasut <marex@denx.de>
> Cc: Richard Zhu <hong-xing.zhu@freescale.com>
> Cc: Shawn Guo <shawn.guo@linaro.org>
> Cc: Pratyush Anand <pratyush.anand@st.com>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: Juergen Beisert <jbe@pengutronix.de>
> Cc: Arnd Bergmann <arnd@arndb.de>
> Cc: Siva Reddy Kallam <siva.kallam@samsung.com>
> Cc: Srikanth T Shivanand <ts.srikanth@samsung.com>
> Cc: Sean Cross <xobs@kosagi.com>
> Cc: linux-pci@vger.kernel.org
> Cc: linux-arm-kernel@lists.infradead.org
Acked-by: Jingoo Han <jg1.han@samsung.com>
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.
Best regards,
Jingoo Han
> ---
> drivers/pci/host/pcie-designware.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
^ permalink raw reply [flat|nested] 7+ messages in thread* RE: [PATCH] pci: designware: fix missing msi irqs
2013-12-16 0:54 ` Jingoo Han
@ 2013-12-16 6:48 ` Mohit KUMAR DCG
2013-12-20 2:41 ` Jingoo Han
0 siblings, 1 reply; 7+ messages in thread
From: Mohit KUMAR DCG @ 2013-12-16 6:48 UTC (permalink / raw)
To: Jingoo Han, 'Harro Haan'
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@vger.kernel.org, linux-arm-kernel@lists.infradead.org
> -----Original Message-----
> From: Jingoo Han [mailto:jg1.han@samsung.com]
> 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@vger.kernel.org; linux-
> arm-kernel@lists.infradead.org; '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@gmail.com>
> > Cc: Mohit Kumar <mohit.kumar@st.com>
> > Cc: Jingoo Han <jg1.han@samsung.com>
> > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > Cc: Marek Vasut <marex@denx.de>
> > Cc: Richard Zhu <hong-xing.zhu@freescale.com>
> > Cc: Shawn Guo <shawn.guo@linaro.org>
> > Cc: Pratyush Anand <pratyush.anand@st.com>
> > Cc: Tim Harvey <tharvey@gateworks.com>
> > Cc: Juergen Beisert <jbe@pengutronix.de>
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Cc: Siva Reddy Kallam <siva.kallam@samsung.com>
> > Cc: Srikanth T Shivanand <ts.srikanth@samsung.com>
> > Cc: Sean Cross <xobs@kosagi.com>
> > Cc: linux-pci@vger.kernel.org
> > Cc: linux-arm-kernel@lists.infradead.org
>
> Acked-by: Jingoo Han <jg1.han@samsung.com>
>
> 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@st.com>
..
>
> > ---
> > drivers/pci/host/pcie-designware.c | 4 +++-
> > 1 file changed, 3 insertions(+), 1 deletion(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] pci: designware: fix missing msi irqs
2013-12-16 6:48 ` Mohit KUMAR DCG
@ 2013-12-20 2:41 ` Jingoo Han
2013-12-20 16:09 ` Bjorn Helgaas
0 siblings, 1 reply; 7+ messages in thread
From: Jingoo Han @ 2013-12-20 2:41 UTC (permalink / raw)
To: 'Bjorn Helgaas'
Cc: 'Mohit KUMAR DCG', 'Harro Haan',
'Marek Vasut', 'Richard Zhu', 'Shawn Guo',
'Pratyush ANAND', 'Tim Harvey',
'Juergen Beisert', 'Arnd Bergmann',
'Siva Reddy Kallam', 'Srikanth T Shivanand',
'Sean Cross', linux-pci, linux-arm-kernel,
'Jingoo Han'
On Monday, December 16, 2013 3:49 PM, Mohit KUMAR DCG wrote:
> On Monday, December 16, 2013 6:24 AM, Jingoo Han wrote:
> > 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@gmail.com>
> > > Cc: Mohit Kumar <mohit.kumar@st.com>
> > > Cc: Jingoo Han <jg1.han@samsung.com>
> > > Cc: Bjorn Helgaas <bhelgaas@google.com>
> > > Cc: Marek Vasut <marex@denx.de>
> > > Cc: Richard Zhu <hong-xing.zhu@freescale.com>
> > > Cc: Shawn Guo <shawn.guo@linaro.org>
> > > Cc: Pratyush Anand <pratyush.anand@st.com>
> > > Cc: Tim Harvey <tharvey@gateworks.com>
> > > Cc: Juergen Beisert <jbe@pengutronix.de>
> > > Cc: Arnd Bergmann <arnd@arndb.de>
> > > Cc: Siva Reddy Kallam <siva.kallam@samsung.com>
> > > Cc: Srikanth T Shivanand <ts.srikanth@samsung.com>
> > > Cc: Sean Cross <xobs@kosagi.com>
> > > Cc: linux-pci@vger.kernel.org
> > > Cc: linux-arm-kernel@lists.infradead.org
> >
> > Acked-by: Jingoo Han <jg1.han@samsung.com>
> >
> > 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@st.com>
Hi Bjorn Helgaas,
Would you apply this patch to your 'pci/host-designware' branch
with the following Tested-by and Acked-by?
Tested-by: Marek Vasut <marex@denx.de>
Tested-by: Matthias Mann <m.mann@arkona-technologies.de>
Acked-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Mohit Kumar <mohit.kumar@st.com>
> ..
> >
> > > ---
> > > drivers/pci/host/pcie-designware.c | 4 +++-
> > > 1 file changed, 3 insertions(+), 1 deletion(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] pci: designware: fix missing msi irqs
2013-12-20 2:41 ` Jingoo Han
@ 2013-12-20 16:09 ` Bjorn Helgaas
0 siblings, 0 replies; 7+ messages in thread
From: Bjorn Helgaas @ 2013-12-20 16:09 UTC (permalink / raw)
To: Jingoo Han
Cc: Mohit KUMAR DCG, Harro Haan, Marek Vasut, Richard Zhu, Shawn Guo,
Pratyush ANAND, Tim Harvey, Juergen Beisert, Arnd Bergmann,
Siva Reddy Kallam, Srikanth T Shivanand, Sean Cross,
linux-pci@vger.kernel.org, linux-arm
On Thu, Dec 19, 2013 at 7:41 PM, Jingoo Han <jg1.han@samsung.com> wrote:
> On Monday, December 16, 2013 3:49 PM, Mohit KUMAR DCG wrote:
>> On Monday, December 16, 2013 6:24 AM, Jingoo Han wrote:
>> > 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@gmail.com>
>> > > Cc: Mohit Kumar <mohit.kumar@st.com>
>> > > Cc: Jingoo Han <jg1.han@samsung.com>
>> > > Cc: Bjorn Helgaas <bhelgaas@google.com>
>> > > Cc: Marek Vasut <marex@denx.de>
>> > > Cc: Richard Zhu <hong-xing.zhu@freescale.com>
>> > > Cc: Shawn Guo <shawn.guo@linaro.org>
>> > > Cc: Pratyush Anand <pratyush.anand@st.com>
>> > > Cc: Tim Harvey <tharvey@gateworks.com>
>> > > Cc: Juergen Beisert <jbe@pengutronix.de>
>> > > Cc: Arnd Bergmann <arnd@arndb.de>
>> > > Cc: Siva Reddy Kallam <siva.kallam@samsung.com>
>> > > Cc: Srikanth T Shivanand <ts.srikanth@samsung.com>
>> > > Cc: Sean Cross <xobs@kosagi.com>
>> > > Cc: linux-pci@vger.kernel.org
>> > > Cc: linux-arm-kernel@lists.infradead.org
>> >
>> > Acked-by: Jingoo Han <jg1.han@samsung.com>
>> >
>> > 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@st.com>
>
> Hi Bjorn Helgaas,
>
> Would you apply this patch to your 'pci/host-designware' branch
> with the following Tested-by and Acked-by?
>
> Tested-by: Marek Vasut <marex@denx.de>
> Tested-by: Matthias Mann <m.mann@arkona-technologies.de>
> Acked-by: Jingoo Han <jg1.han@samsung.com>
> Acked-by: Mohit Kumar <mohit.kumar@st.com>
Applied to pci/host-designware for v3.14, thanks for the reminder!
Bjorn
^ permalink raw reply [flat|nested] 7+ messages in thread