On Tue, Apr 17, 2012 at 11:11:04AM -0400, Alan Stern wrote:
> Just to make sure, did you test the patch with the script installed 
> (that is, with ehci-hcd unbound)?  I assume that will work normally.
It works normally, yes.

> 
> Moving on, the next thing is to remove changes from the patch, one at a 
> time, until we find one that prevents the system from resuming.  So, 
> testing at each step, please remove from the patch:
> 
> 	1. This change in hcd-pci.c:
> +	pci_dev->current_state = PCI_UNKNOWN;
It locks up if this line is removed.

> 	2. This change in ehci-pci.c:
> +	ehci_silence_controller(ehci);
> 
> 	3. This change in hcd-pci.c:
> +	pci_disable_device(pci_dev);
> 
> 	4. This change in hcd-pci.c:
> +	iounmap(hcd->regs);
> 
> 	5. This change in hcd-pci.c:
> +	free_irq(hcd->irq, hcd);
> 
> 	6. This change in hcd-pci.c:
> -	pci_disable_device(pci_dev);
> +	pci_disable_enabled_device(pci_dev);
> 
> 	7. This change in ehci-pci.c:
> -	ehci_prepare_ports_for_controller_suspend(ehci, do_wakeup);
> 
> Once all those things have been removed, the patch should be the same 
> as one you tried earlier, which did crash the machine.

-- 
WBR, wRAR