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

Chris Bainbridge chris.bainbridge at gmail.com
Tue May 24 16:38:58 PDT 2016


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 at gmail.com> [MacBookPro10,2]



More information about the b43-dev mailing list