2014-06-03 19:40:14

by Jérôme Carretero

[permalink] [raw]
Subject: [PATCH] PCI: quirk dma_alias_devfn for HighPoint RocketRaid 642L (Marvell 9235)

This vendor has similar boards based on the same Marvell 88SE9235 chipset,
but this patch was only tested with the 642L.

Tested on ASUS Sabertooth 990FX (AMD).

If you listen to the vendor support stating:

Dear Customer,
Sorry about that.
Currently all our products' software package don't support IOMMU
and we don't have plan to support that.

... you'll think you also wasted your money, but it turns out that you
just lost a few hours of your valuable time talking to their support,
because the good people from the Internet made their board work,
with the regular AHCI driver, and even with IOMMU. Nice!


Signed-off-by: Jérôme Carretero <[email protected]>

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

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index f6a42bc..43c0ea0 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3380,6 +3380,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
/* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
quirk_dma_func1_alias);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
+ quirk_dma_func1_alias);
/* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
PCI_DEVICE_ID_JMICRON_JMB388_ESD,
--
2.0.0


2014-06-11 19:25:06

by Alex Williamson

[permalink] [raw]
Subject: Re: [PATCH] PCI: quirk dma_alias_devfn for HighPoint RocketRaid 642L (Marvell 9235)

On Tue, 2014-06-03 at 15:41 -0400, Jérôme Carretero wrote:
> This vendor has similar boards based on the same Marvell 88SE9235 chipset,
> but this patch was only tested with the 642L.
>
> Tested on ASUS Sabertooth 990FX (AMD).
>
> If you listen to the vendor support stating:
>
> Dear Customer,
> Sorry about that.
> Currently all our products' software package don't support IOMMU
> and we don't have plan to support that.
>
> ... you'll think you also wasted your money, but it turns out that you
> just lost a few hours of your valuable time talking to their support,
> because the good people from the Internet made their board work,
> with the regular AHCI driver, and even with IOMMU. Nice!
>
>
> Signed-off-by: Jérôme Carretero <[email protected]>

Acked-by: Alex Williamson <[email protected]>

> ---
> drivers/pci/quirks.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index f6a42bc..43c0ea0 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3380,6 +3380,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
> /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
> DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
> quirk_dma_func1_alias);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
> + quirk_dma_func1_alias);
> /* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
> DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
> PCI_DEVICE_ID_JMICRON_JMB388_ESD,


2014-06-11 19:54:08

by Bjorn Helgaas

[permalink] [raw]
Subject: Re: [PATCH] PCI: quirk dma_alias_devfn for HighPoint RocketRaid 642L (Marvell 9235)

On Tue, Jun 03, 2014 at 03:41:56PM -0400, J?r?me Carretero wrote:
> This vendor has similar boards based on the same Marvell 88SE9235 chipset,
> but this patch was only tested with the 642L.
>
> Tested on ASUS Sabertooth 990FX (AMD).
>
> If you listen to the vendor support stating:
>
> Dear Customer,
> Sorry about that.
> Currently all our products' software package don't support IOMMU
> and we don't have plan to support that.
>
> ... you'll think you also wasted your money, but it turns out that you
> just lost a few hours of your valuable time talking to their support,
> because the good people from the Internet made their board work,
> with the regular AHCI driver, and even with IOMMU. Nice!
>
>
> Signed-off-by: J?r?me Carretero <[email protected]>

Applied with Alex's ack to pci/iommu for v3.16, thanks!

Note that I only have the PCI parts here. There are several IOMMU patches
required in addition, and I haven't heard the fate of those.

> ---
> drivers/pci/quirks.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index f6a42bc..43c0ea0 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3380,6 +3380,8 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
> /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c49 */
> DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9230,
> quirk_dma_func1_alias);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_TTI, 0x0642,
> + quirk_dma_func1_alias);
> /* https://bugs.gentoo.org/show_bug.cgi?id=497630 */
> DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_JMICRON,
> PCI_DEVICE_ID_JMICRON_JMB388_ESD,
> --
> 2.0.0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/