diff mbox series

PCI: Add function 1 DMA alias quirk for Marvell 88SS9183

Message ID 20180813194123.GM113140@bhelgaas-glaptop.roam.corp.google.com (mailing list archive)
State New, archived
Delegated to: Bjorn Helgaas
Headers show
Series PCI: Add function 1 DMA alias quirk for Marvell 88SS9183 | expand

Commit Message

Bjorn Helgaas Aug. 13, 2018, 7:41 p.m. UTC
Hi,

Thanks a lot for your new report
(https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134).

Can you confirm that the patch below is equivalent to what you tested and
it resolves the problem?  If so, I'll try to include it for v4.19.


commit dd4e0ad485e12c9c04eb7a21ba69c24f12cb1918
Author: Bjorn Helgaas <bhelgaas@google.com>
Date:   Mon Aug 13 14:30:41 2018 -0500

    PCI: Add function 1 DMA alias quirk for Marvell 88SS9183
    
    Add function 1 DMA alias quirk for Marvell 88SS9183 PCIe SSD Controller.
    
    Link: https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134
    Reported-by: <f.bluethner@mailbox.org>
    Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>

Comments

Bjorn Helgaas Aug. 14, 2018, 2:19 p.m. UTC | #1
On Mon, Aug 13, 2018 at 02:41:23PM -0500, Bjorn Helgaas wrote:
> Hi,
> 
> Thanks a lot for your new report
> (https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134).
> 
> Can you confirm that the patch below is equivalent to what you tested and
> it resolves the problem?  If so, I'll try to include it for v4.19.

I applied this to pci/virtualization for v4.19.  Please let me know if you
see any issues.

> commit dd4e0ad485e12c9c04eb7a21ba69c24f12cb1918
> Author: Bjorn Helgaas <bhelgaas@google.com>
> Date:   Mon Aug 13 14:30:41 2018 -0500
> 
>     PCI: Add function 1 DMA alias quirk for Marvell 88SS9183
>     
>     Add function 1 DMA alias quirk for Marvell 88SS9183 PCIe SSD Controller.
>     
>     Link: https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134
>     Reported-by: <f.bluethner@mailbox.org>
>     Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index deb051583eda..6994544125ff 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3845,6 +3845,9 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x917a,
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c78 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9182,
>  			 quirk_dma_func1_alias);
> +/* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134 */
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9183,
> +			 quirk_dma_func1_alias);
>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c46 */
>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
>  			 quirk_dma_func1_alias);
Felix Blüthner Aug. 14, 2018, 6:47 p.m. UTC | #2
Yes, I can confirm that patch. It is the same change I already compiled
and installed and is running on my system successfully.

> On Mon, Aug 13, 2018 at 02:41:23PM -0500, Bjorn Helgaas wrote:
>> Hi,
>>
>> Thanks a lot for your new report
>> (https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134).
>>
>> Can you confirm that the patch below is equivalent to what you tested and
>> it resolves the problem?  If so, I'll try to include it for v4.19.
> I applied this to pci/virtualization for v4.19.  Please let me know if you
> see any issues.
>
>> commit dd4e0ad485e12c9c04eb7a21ba69c24f12cb1918
>> Author: Bjorn Helgaas <bhelgaas@google.com>
>> Date:   Mon Aug 13 14:30:41 2018 -0500
>>
>>     PCI: Add function 1 DMA alias quirk for Marvell 88SS9183
>>     
>>     Add function 1 DMA alias quirk for Marvell 88SS9183 PCIe SSD Controller.
>>     
>>     Link: https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134
>>     Reported-by: <f.bluethner@mailbox.org>
>>     Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
>>
>> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
>> index deb051583eda..6994544125ff 100644
>> --- a/drivers/pci/quirks.c
>> +++ b/drivers/pci/quirks.c
>> @@ -3845,6 +3845,9 @@ DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x917a,
>>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c78 */
>>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9182,
>>  			 quirk_dma_func1_alias);
>> +/* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134 */
>> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9183,
>> +			 quirk_dma_func1_alias);
>>  /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c46 */
>>  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
>>  			 quirk_dma_func1_alias);
diff mbox series

Patch

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index deb051583eda..6994544125ff 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3845,6 +3845,9 @@  DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x917a,
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c78 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9182,
 			 quirk_dma_func1_alias);
+/* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c134 */
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x9183,
+			 quirk_dma_func1_alias);
 /* https://bugzilla.kernel.org/show_bug.cgi?id=42679#c46 */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_MARVELL_EXT, 0x91a0,
 			 quirk_dma_func1_alias);