Re: [PATCH] PCI: add NO_D3_DURING_SLEEP flag and revert 151b61284776be2

["Rafael J. Wysocki" <rjw@xxxxxxx>]

On Wednesday, June 13, 2012, Greg KH wrote:
> On Wed, Jun 13, 2012 at 11:20:19AM -0400, Alan Stern wrote:
> > This patch (as1558) fixes a problem affecting several ASUS computers:
> > The machine crashes or corrupts memory when going into suspend if the
> > ehci-hcd driver is bound to any controllers.  Users have been forced
> > to unbind or unload ehci-hcd before putting their systems to sleep.
> > 
> > After extensive testing, it was determined that the machines don't
> > like going into suspend when any EHCI controllers are in the PCI D3
> > power state.  Presumably this is a firmware bug, but there's nothing
> > we can do about it except to avoid putting the controllers in D3
> > during system sleep.
> > 
> > The patch adds a new flag to indicate whether the problem is present,
> > and avoids changing the controller's power state if the flag is set.
> > Runtime suspend is unaffected; this matters only for system suspend.
> > However as a side effect, the controller will not respond to remote
> > wakeup requests while the system is asleep.  Hence USB wakeup is not
> > functional -- but of course, this is already true in the current state
> > of affairs.
> > 
> > A similar patch has already been applied as commit
> > 151b61284776be2d6f02d48c23c3625678960b97 (USB: EHCI: fix crash during
> > suspend on ASUS computers).  The patch supersedes that one and reverts
> > it.  There are two differences:
> > 
> > 	The old patch added the flag at the USB level; this patch
> > 	adds it at the PCI level.
> > 
> > 	The old patch applied to all chipsets with the same vendor,
> > 	subsystem vendor, and product IDs; this patch makes an
> > 	exception for a known-good system (based on DMI information).
> > 
> > Signed-off-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
> > Tested-by: Dâniel Fraga <fragabr@xxxxxxxxx>
> > Tested-by: Andrey Rahmatullin <wrar@xxxxxxxxx>
> > Tested-by: Steven Rostedt <rostedt@xxxxxxxxxxx>
> > CC: Greg KH <greg@xxxxxxxxx>
> > CC: <stable@xxxxxxxxxxxxxxx>
> > 
> > ---
> > 
> > Greg, do you mind if this goes in through Rafael's tree?
> 
> Not at all, but I can also take it.
> 
> Rafael, if you want to take it, please add:
> 
> Acked-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> 
> to it, or let me know, and I can take this through my tree.

Please take it, if that's not a problem, I don't have any other
patches for 3.5 pending.  And please add:

Reviewed-by: Rafael J. Wysocki <rjw@xxxxxxx>

to it.

Thanks,
Rafael