[PATCH 0/2] Add AR9485 & QCA988X to no_bus_reset quirk

[PATCH 0/2] Add AR9485 & QCA988X to no_bus_reset quirk

[Chris Blake]

Both the AR9485 and QCA988x require the no_bus_reset quirk to be set for them
to work correctly with PCI Passthrough. Both of these have been tested on
Linux 4.4.8 and work as intended.

Chris Blake (2):
  Add Quirk for Atheros AR9485 Bus Reset
  Add Quirk for Qualcom Atheros QCA9882 Bus Reset

 drivers/pci/quirks.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

--
1.9.1

[PATCH 1/2] Add Quirk for Atheros AR9485 No Bus Reset

[Chris Blake]

Similar to the AR93xx series, the AR94xx also has the same quirk for the
Bus Reset. Adding this will resolve the issue, and help in cases where
PCI Passthrough is used.

Signed-off-by: Chris Blake <chrisrblake93@gmail.com>
---
 drivers/pci/quirks.c | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index ee72ebe..35c824d 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3189,13 +3189,14 @@ static void quirk_no_bus_reset(struct pci_dev *dev)
 }

 /*
- * Atheros AR93xx chips do not behave after a bus reset.  The device will
- * throw a Link Down error on AER-capable systems and regardless of AER,
- * config space of the device is never accessible again and typically
- * causes the system to hang or reset when access is attempted.
+ * Some Atheros AR9xxx and QCA988x chips do not behave after a bus reset.
+ * The device will throw a Link Down error on AER-capable systems and
+ * regardless of AER, config space of the device is never accessible again
+ * and typically causes the system to hang or reset when access is attempted.
  * http://www.spinics.net/lists/linux-pci/msg34797.html
  */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0030, quirk_no_bus_reset);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0032, quirk_no_bus_reset);

 static void quirk_no_pm_reset(struct pci_dev *dev)
 {
--
1.9.1

[PATCH 2/2] Add Quirk for Qualcom Atheros QCA9882 No Bus Reset

[Chris Blake]

Similar to the AR93xx series, the Qualcom QCA988x also has the same
quirk for the Bus Reset. Adding this will resolve the issue, and help in
cases where PCI Passthrough is used.

Signed-off-by: Chris Blake <chrisrblake93@gmail.com>
---
 drivers/pci/quirks.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 35c824d..7902fbf 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3197,6 +3197,7 @@ static void quirk_no_bus_reset(struct pci_dev *dev)
  */
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0030, quirk_no_bus_reset);
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0032, quirk_no_bus_reset);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x003c, quirk_no_bus_reset);

 static void quirk_no_pm_reset(struct pci_dev *dev)
 {
--
1.9.1

Re: [PATCH 0/2] Add AR9485 & QCA988X to no_bus_reset quirk

[Bjorn Helgaas]

On Mon, May 30, 2016 at 07:26:36AM -0500, Chris Blake wrote:
> Both the AR9485 and QCA988x require the no_bus_reset quirk to be set for them
> to work correctly with PCI Passthrough. Both of these have been tested on
> Linux 4.4.8 and work as intended.
> 
> Chris Blake (2):
>   Add Quirk for Atheros AR9485 Bus Reset
>   Add Quirk for Qualcom Atheros QCA9882 Bus Reset

I squashed these into a single patch, marked it for stable just like
we did for the original, c3e59ee4e766 ("PCI: Mark Atheros AR93xx to
avoid bus reset"), and applied it to pci/virtualization for v4.8.

Thanks, Chris!