Re: [PATCH] PCI: Add Broadcom 4331 reset quirk to prevent IRQ storm

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, May 23, 2016 at 04:42:16PM +0200, Lukas Wunner wrote:
> 
> The reason why my previous patches didn't work on your machine is
> because you're using grub, and grub contains a patch by Matthew
> Garrett which puts the wireless card into power state D3hot. The
> card's mmio space isn't accessible in D3hot. Included below is a
> new patch which transitions the card to D0 before resetting it.
> Please let me know if this fixes the issue for you.

The grub patch is in Debian but didn't prevent the issue on my setup,
presumably because of the Ubuntu "linuxefi" fallback that Matthew
previously posted about.

> @Chris Bainbridge: Please test this as well, this is no longer a
> plain vanilla PCI quirk but an early quirk, it should prevent any
> kind of memory corruption by DMAed packets.

I tested a "no b43" kernel for 10 boots with and without the patch.

Without the patch: 7 boots had potential memory corruption (caught by
the IOMMU) and all had IRQ 17 disabled.

With the patch: no memory corruption or disabled IRQ 17.

With b43 built in to the kernel wifi worked as expected.

So the patch looks good to me. Thanks for the fix.

Tested-by: Chris Bainbridge <chris.bainbridge@xxxxxxxxx> [MacBookPro10,2]
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux