Re: [PATCH] Disable Bus Master on PCI device shutdown

[Matthew Garrett <mjg@redhat.com>]

On Fri, Apr 27, 2012 at 01:00:33PM -0600, Khalid Aziz wrote:
> Disable Bus Master bit on the device in
> pci_device_shutdown() to ensure PCI devices do not continue
> to DMA data after shutdown. This can cause memory
> corruption in case of a kexec where the current kernel
> shuts down and transfers control to a new kernel while a
> PCI device continues to DMA to memory that does not belong
> to it any more in the new kernel.

This protects against the case where a piece of hardware is continuing 
to DMA even after the driver shutdown method has been called? I'm not 
convinced this is safe. Some Broadcom parts will crash if busmastering 
is disabled while they're still performing DMA, and they'll then hang 
the bus if reenabled. There's also the risk that the hardware will start 
DMAing again if it's reenabled after being shut down. It seems like 
you're covering over the case where the driver didn't correctly quiesce 
the hardware, but you risk triggering other bugs instead.

-- 
Matthew Garrett | mjg59@srcf.ucam.org