All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
@ 2015-05-21 21:03 Sakari Ailus
  2015-05-22 14:52 ` Bjorn Helgaas
  0 siblings, 1 reply; 6+ messages in thread
From: Sakari Ailus @ 2015-05-21 21:03 UTC (permalink / raw)
  To: linux-pci; +Cc: alex.williamson

Marvell 9120 SATA controller has the same issue as a number of others, use
the same quirk for this one. The other quirks were added by patch
"PCI: Add function 1 DMA alias quirk for Marvell devices"
(commit id cc346a4714a59d08c118e8f33fd86692d3563133).

Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Acked-by: Alex Williamson <alex.williamson@redhat.com>
---
(Resending to linux-pci, was originally sent to linux-iommu list.)

Hi,

I recently booted the v4.0.1 kernel with intel-iommu enabled and had similar
issues as in the original bug report with Marvell 88SE9120 (rev 12). The
same quirk appears to work for that one as well.

 drivers/pci/quirks.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 85f247e..d4af5a8 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
  * SKUs this function is not present, making this a ghost requester.
  * https://bugzilla.kernel.org/show_bug.cgi?id=42679
  */
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
+			 quirk_dma_func1_alias);
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
 			 quirk_dma_func1_alias);
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
-- 
1.7.10.4


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
  2015-05-21 21:03 [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120 Sakari Ailus
@ 2015-05-22 14:52 ` Bjorn Helgaas
  0 siblings, 0 replies; 6+ messages in thread
From: Bjorn Helgaas @ 2015-05-22 14:52 UTC (permalink / raw)
  To: Sakari Ailus; +Cc: linux-pci, alex.williamson

On Fri, May 22, 2015 at 12:03:38AM +0300, Sakari Ailus wrote:
> Marvell 9120 SATA controller has the same issue as a number of others, use
> the same quirk for this one. The other quirks were added by patch
> "PCI: Add function 1 DMA alias quirk for Marvell devices"
> (commit id cc346a4714a59d08c118e8f33fd86692d3563133).
> 
> Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> Acked-by: Alex Williamson <alex.williamson@redhat.com>

Applied to pci/virtualization for v4.2, thanks!

> ---
> (Resending to linux-pci, was originally sent to linux-iommu list.)
> 
> Hi,
> 
> I recently booted the v4.0.1 kernel with intel-iommu enabled and had similar
> issues as in the original bug report with Marvell 88SE9120 (rev 12). The
> same quirk appears to work for that one as well.
> 
>  drivers/pci/quirks.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 85f247e..d4af5a8 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
>   * SKUs this function is not present, making this a ghost requester.
>   * https://bugzilla.kernel.org/show_bug.cgi?id=42679
>   */
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
> +			 quirk_dma_func1_alias);
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
>  			 quirk_dma_func1_alias);
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
> -- 
> 1.7.10.4
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-pci" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
  2015-05-21 14:12     ` Alex Williamson
@ 2015-05-21 14:14       ` Sakari Ailus
  0 siblings, 0 replies; 6+ messages in thread
From: Sakari Ailus @ 2015-05-21 14:14 UTC (permalink / raw)
  To: Alex Williamson; +Cc: iommu, linux-pci

Hi Alex,

Alex Williamson wrote:
> On Thu, 2015-05-21 at 12:54 +0300, Sakari Ailus wrote:
>> Ping.
>>
>> (Cc Alex.)
> 
> You can add my ack, but the linux-pci list is the proper list to get
> this upstream.  I suggest resending it there.  Thanks,

Thank you for your reply and ack; I'll resend it there.

-- 
Regards,

Sakari Ailus
sakari.ailus@linux.intel.com

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
  2015-05-21  9:54   ` Sakari Ailus
@ 2015-05-21 14:12     ` Alex Williamson
  2015-05-21 14:14       ` Sakari Ailus
  0 siblings, 1 reply; 6+ messages in thread
From: Alex Williamson @ 2015-05-21 14:12 UTC (permalink / raw)
  To: Sakari Ailus; +Cc: iommu, linux-pci

On Thu, 2015-05-21 at 12:54 +0300, Sakari Ailus wrote:
> Ping.
> 
> (Cc Alex.)

You can add my ack, but the linux-pci list is the proper list to get
this upstream.  I suggest resending it there.  Thanks,

Ale

> Sakari Ailus wrote:
> > Marvell 9120 SATA controller has the same issue as a number of others, use
> > the same quirk for this one. The other quirks were added by patch
> > "PCI: Add function 1 DMA alias quirk for Marvell devices"
> > (commit id cc346a4714a59d08c118e8f33fd86692d3563133).
> > 
> > Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>


Acked-by: Alex Williamson <alex.williamson@redhat.com>


> > ---
> > Hi,
> > 
> > I just booted the v4.0.1 kernel with intel-iommu enabled and had similar
> > issues as in the original bug report with Marvell 88SE9120 (rev 12). The
> > same quirk appears to work for that one as well.
> > 
> >  drivers/pci/quirks.c |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> > index 85f247e..d4af5a8 100644
> > --- a/drivers/pci/quirks.c
> > +++ b/drivers/pci/quirks.c
> > @@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
> >   * SKUs this function is not present, making this a ghost requester.
> >   * https://bugzilla.kernel.org/show_bug.cgi?id=42679
> >   */
> > +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
> > +			 quirk_dma_func1_alias);
> >  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
> >  			 quirk_dma_func1_alias);
> >  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
> > 
> 
> 




^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
       [not found] ` <1430409392-32479-1-git-send-email-sakari.ailus-X3B1VOXEql0@public.gmane.org>
@ 2015-05-21  9:54   ` Sakari Ailus
  2015-05-21 14:12     ` Alex Williamson
  0 siblings, 1 reply; 6+ messages in thread
From: Sakari Ailus @ 2015-05-21  9:54 UTC (permalink / raw)
  To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA

Ping.

(Cc Alex.)

Sakari Ailus wrote:
> Marvell 9120 SATA controller has the same issue as a number of others, use
> the same quirk for this one. The other quirks were added by patch
> "PCI: Add function 1 DMA alias quirk for Marvell devices"
> (commit id cc346a4714a59d08c118e8f33fd86692d3563133).
> 
> Signed-off-by: Sakari Ailus <sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
> ---
> Hi,
> 
> I just booted the v4.0.1 kernel with intel-iommu enabled and had similar
> issues as in the original bug report with Marvell 88SE9120 (rev 12). The
> same quirk appears to work for that one as well.
> 
>  drivers/pci/quirks.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 85f247e..d4af5a8 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
>   * SKUs this function is not present, making this a ghost requester.
>   * https://bugzilla.kernel.org/show_bug.cgi?id=42679
>   */
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
> +			 quirk_dma_func1_alias);
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
>  			 quirk_dma_func1_alias);
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
> 


-- 
Regards,

Sakari Ailus
sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120
@ 2015-04-30 15:56 Sakari Ailus
       [not found] ` <1430409392-32479-1-git-send-email-sakari.ailus-X3B1VOXEql0@public.gmane.org>
  0 siblings, 1 reply; 6+ messages in thread
From: Sakari Ailus @ 2015-04-30 15:56 UTC (permalink / raw)
  To: iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA

Marvell 9120 SATA controller has the same issue as a number of others, use
the same quirk for this one. The other quirks were added by patch
"PCI: Add function 1 DMA alias quirk for Marvell devices"
(commit id cc346a4714a59d08c118e8f33fd86692d3563133).

Signed-off-by: Sakari Ailus <sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
---
Hi,

I just booted the v4.0.1 kernel with intel-iommu enabled and had similar
issues as in the original bug report with Marvell 88SE9120 (rev 12). The
same quirk appears to work for that one as well.

 drivers/pci/quirks.c |    2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 85f247e..d4af5a8 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3572,6 +3572,8 @@ static void quirk_dma_func1_alias(struct pci_dev *dev)
  * SKUs this function is not present, making this a ghost requester.
  * https://bugzilla.kernel.org/show_bug.cgi?id=42679
  */
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9120,
+			 quirk_dma_func1_alias);
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9123,
 			 quirk_dma_func1_alias);
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c14 */
-- 
1.7.10.4

^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2015-05-22 14:52 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-05-21 21:03 [PATCH 1/1] PCI: Add function 1 DMA alias quirk for Marvell 9120 Sakari Ailus
2015-05-22 14:52 ` Bjorn Helgaas
  -- strict thread matches above, loose matches on Subject: below --
2015-04-30 15:56 Sakari Ailus
     [not found] ` <1430409392-32479-1-git-send-email-sakari.ailus-X3B1VOXEql0@public.gmane.org>
2015-05-21  9:54   ` Sakari Ailus
2015-05-21 14:12     ` Alex Williamson
2015-05-21 14:14       ` Sakari Ailus

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.