Re: [PATCH 2/2] Fix console handling during suspend/resume

[Greg KH <greg@kroah.com>]

On Wed, Jun 14, 2006 at 08:39:26PM -0400, Adam Belay wrote:
> On Wed, Jun 14, 2006 at 03:37:51PM -0700, Linus Torvalds wrote:
> > And btw, the reason it didn't resume originally was because _we_ did 
> > things wrong. The PCI command word mustn't be writen before the rest of 
> > the config space has been restored (one of the things I used my debugging 
> > patches for, until I noticed that -mm had the same fix independently, so 
> > that's the one that is merged right now ;)
> 
> I was hoping to see a more complete fix merged.  This patch still writes to a
> large number of read-only registers, touches BIST (which can be dangerous on
> some hardware), and isn't careful about the initial state of the PCI command
> word.
> 
> I attempted to rework pci_save/restore_state() a couple weeks ago:
> http://marc.theaimsgroup.com/?l=linux-kernel&m=114949711413176&w=2
> 
> Any comments would be appreciated.

Your patches are still in my queue, so don't worry, they aren't being
ignored (the other restore patch had been in my tree, and in -mm for a
long time, and deserved to be merged already.)

In other email threads, the idea came up that we should probably be
restoring more than just the "basic" configuration.  PCI-E and PCI-X 2.0
devices have a much bigger config space, and there's the "new
capabilities list" that we should also probably restore in the proper
manner if present.

thanks,

greg k-h