* [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801
@ 2014-06-23 22:36 Alex Williamson
2014-07-05 16:31 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Alex Williamson @ 2014-06-23 22:36 UTC (permalink / raw)
To: linux-pci; +Cc: bhelgaas, linux-kernel, Alex Williamson
This bridge sometimes shows up as a root complex device and sometimes
as a discrete PCIe-to-PCI bridge. Testing indicates that in the
latter case, we need to enable the PCIe bridge DMA alias quirk.
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
Reported-by: Milos Kaurin <milos.kaurin@gmail.com>
Tested-by: Milos Kaurin <milos.kaurin@gmail.com>
---
drivers/pci/quirks.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 78a7df6..460c354 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3405,6 +3405,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ASMEDIA, 0x1080,
DECLARE_PCI_FIXUP_HEADER(0x10e3, 0x8113, quirk_use_pcie_bridge_dma_alias);
/* ITE 8892, https://bugzilla.kernel.org/show_bug.cgi?id=73551 */
DECLARE_PCI_FIXUP_HEADER(0x1283, 0x8892, quirk_use_pcie_bridge_dma_alias);
+/* Intel 82801, https://bugzilla.kernel.org/show_bug.cgi?id=44881#c49 */
+DECLARE_PCI_FIXUP_HEADER(0x8086, 0x244e, quirk_use_pcie_bridge_dma_alias);
/*
* AMD has indicated that the devices below do not support peer-to-peer
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801
2014-06-23 22:36 [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801 Alex Williamson
@ 2014-07-05 16:31 ` Bjorn Helgaas
2014-07-05 21:26 ` Bjorn Helgaas
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2014-07-05 16:31 UTC (permalink / raw)
To: Alex Williamson; +Cc: linux-pci, linux-kernel
On Mon, Jun 23, 2014 at 04:36:25PM -0600, Alex Williamson wrote:
> This bridge sometimes shows up as a root complex device and sometimes
> as a discrete PCIe-to-PCI bridge. Testing indicates that in the
> latter case, we need to enable the PCIe bridge DMA alias quirk.
>
> Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> Reported-by: Milos Kaurin <milos.kaurin@gmail.com>
> Tested-by: Milos Kaurin <milos.kaurin@gmail.com>
Applied to for-linus for v3.16, thanks!
> ---
>
> drivers/pci/quirks.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 78a7df6..460c354 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3405,6 +3405,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ASMEDIA, 0x1080,
> DECLARE_PCI_FIXUP_HEADER(0x10e3, 0x8113, quirk_use_pcie_bridge_dma_alias);
> /* ITE 8892, https://bugzilla.kernel.org/show_bug.cgi?id=73551 */
> DECLARE_PCI_FIXUP_HEADER(0x1283, 0x8892, quirk_use_pcie_bridge_dma_alias);
> +/* Intel 82801, https://bugzilla.kernel.org/show_bug.cgi?id=44881#c49 */
> +DECLARE_PCI_FIXUP_HEADER(0x8086, 0x244e, quirk_use_pcie_bridge_dma_alias);
>
> /*
> * AMD has indicated that the devices below do not support peer-to-peer
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801
2014-07-05 16:31 ` Bjorn Helgaas
@ 2014-07-05 21:26 ` Bjorn Helgaas
2014-07-05 21:56 ` Joerg Roedel
0 siblings, 1 reply; 4+ messages in thread
From: Bjorn Helgaas @ 2014-07-05 21:26 UTC (permalink / raw)
To: Alex Williamson; +Cc: linux-pci, linux-kernel, Joerg Roedel
[+cc Joerg]
On Sat, Jul 05, 2014 at 10:31:43AM -0600, Bjorn Helgaas wrote:
> On Mon, Jun 23, 2014 at 04:36:25PM -0600, Alex Williamson wrote:
> > This bridge sometimes shows up as a root complex device and sometimes
> > as a discrete PCIe-to-PCI bridge. Testing indicates that in the
> > latter case, we need to enable the PCIe bridge DMA alias quirk.
> >
> > Signed-off-by: Alex Williamson <alex.williamson@redhat.com>
> > Reported-by: Milos Kaurin <milos.kaurin@gmail.com>
> > Tested-by: Milos Kaurin <milos.kaurin@gmail.com>
>
> Applied to for-linus for v3.16, thanks!
Actually, this quirk doesn't make any difference until the iommu changes
are in, so there's no reason to have this in v3.16, is there?
I saw that Joerg applied those iommu changes to his core branch, which I
assume will be merged for v3.17. So I'll move this to a pci/iommu branch,
to be merged during the v3.17 merge window.
> > ---
> >
> > drivers/pci/quirks.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> > index 78a7df6..460c354 100644
> > --- a/drivers/pci/quirks.c
> > +++ b/drivers/pci/quirks.c
> > @@ -3405,6 +3405,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ASMEDIA, 0x1080,
> > DECLARE_PCI_FIXUP_HEADER(0x10e3, 0x8113, quirk_use_pcie_bridge_dma_alias);
> > /* ITE 8892, https://bugzilla.kernel.org/show_bug.cgi?id=73551 */
> > DECLARE_PCI_FIXUP_HEADER(0x1283, 0x8892, quirk_use_pcie_bridge_dma_alias);
> > +/* Intel 82801, https://bugzilla.kernel.org/show_bug.cgi?id=44881#c49 */
> > +DECLARE_PCI_FIXUP_HEADER(0x8086, 0x244e, quirk_use_pcie_bridge_dma_alias);
> >
> > /*
> > * AMD has indicated that the devices below do not support peer-to-peer
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801
2014-07-05 21:26 ` Bjorn Helgaas
@ 2014-07-05 21:56 ` Joerg Roedel
0 siblings, 0 replies; 4+ messages in thread
From: Joerg Roedel @ 2014-07-05 21:56 UTC (permalink / raw)
To: Bjorn Helgaas; +Cc: Alex Williamson, linux-pci, linux-kernel
On Sat, Jul 05, 2014 at 03:26:29PM -0600, Bjorn Helgaas wrote:
> I saw that Joerg applied those iommu changes to his core branch, which I
> assume will be merged for v3.17. So I'll move this to a pci/iommu branch,
> to be merged during the v3.17 merge window.
Yes, these changes are queued for v3.17.
Joerg
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2014-07-05 21:56 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-06-23 22:36 [PATCH] PCI: Add bridge DMA alias quirk for Intel 82801 Alex Williamson
2014-07-05 16:31 ` Bjorn Helgaas
2014-07-05 21:26 ` Bjorn Helgaas
2014-07-05 21:56 ` Joerg Roedel
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).