All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] PCI: Mark Atheros AR9580 to avoid bus reset
@ 2016-08-09 14:41 Maik Broemme
  2016-08-18 20:39 ` Bjorn Helgaas
  0 siblings, 1 reply; 2+ messages in thread
From: Maik Broemme @ 2016-08-09 14:41 UTC (permalink / raw)
  To: linux-pci

Similar to the AR93xx and the AR94xx series, the AR95xx also have the
same quirk for the Bus Reset. It will lead to instant system reset if
the device is assigned via VFIO to a KVM VM. I've been able reproduce
this behavior with a MikroTik R11e-2HnD.

Fixes: c3e59ee4e766 ("PCI: Mark Atheros AR93xx to avoid bus reset")
Signed-off-by: Maik Broemme <mbroemme@libmpq.org>
CC: stable@vger.kernel.org
---
 drivers/pci/quirks.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
index 37ff015..7d82189 100644
--- a/drivers/pci/quirks.c
+++ b/drivers/pci/quirks.c
@@ -3198,6 +3198,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);
+DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0033, quirk_no_bus_reset);
 
 static void quirk_no_pm_reset(struct pci_dev *dev)
 {
-- 
2.9.2


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

* Re: [PATCH] PCI: Mark Atheros AR9580 to avoid bus reset
  2016-08-09 14:41 [PATCH] PCI: Mark Atheros AR9580 to avoid bus reset Maik Broemme
@ 2016-08-18 20:39 ` Bjorn Helgaas
  0 siblings, 0 replies; 2+ messages in thread
From: Bjorn Helgaas @ 2016-08-18 20:39 UTC (permalink / raw)
  To: Maik Broemme; +Cc: linux-pci, Chris Blake, Alex Williamson

[+cc Chris, Alex]

On Tue, Aug 09, 2016 at 04:41:31PM +0200, Maik Broemme wrote:
> Similar to the AR93xx and the AR94xx series, the AR95xx also have the
> same quirk for the Bus Reset. It will lead to instant system reset if
> the device is assigned via VFIO to a KVM VM. I've been able reproduce
> this behavior with a MikroTik R11e-2HnD.
> 
> Fixes: c3e59ee4e766 ("PCI: Mark Atheros AR93xx to avoid bus reset")
> Signed-off-by: Maik Broemme <mbroemme@libmpq.org>
> CC: stable@vger.kernel.org

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

> ---
>  drivers/pci/quirks.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/pci/quirks.c b/drivers/pci/quirks.c
> index 37ff015..7d82189 100644
> --- a/drivers/pci/quirks.c
> +++ b/drivers/pci/quirks.c
> @@ -3198,6 +3198,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);
> +DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_ATHEROS, 0x0033, quirk_no_bus_reset);
>  
>  static void quirk_no_pm_reset(struct pci_dev *dev)
>  {
> -- 
> 2.9.2
> 
> --
> 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] 2+ messages in thread

end of thread, other threads:[~2016-08-19  2:36 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-09 14:41 [PATCH] PCI: Mark Atheros AR9580 to avoid bus reset Maik Broemme
2016-08-18 20:39 ` Bjorn Helgaas

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.