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@xxxxxxxxxxxxx -- 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
