On 2010.02.03 23:44:41 +0100, Rafael J. Wysocki wrote: > On Wednesday 03 February 2010, Alan Jenkins wrote: > > Hi > > > > I found this regression on my EeePC 701 with modesetting enabled. When > > I hibernate using s2disk, I can abort the hibernation by pressing the > > backspace key. Doing so breaks X on 2.6.32-rc6 (but not 2.6.32). > > Yeah. > > To be honest, I knew that's going to happen, but didn't have the time to take > care of it. > > The problem is that i915 does literally _nothing_ in its .thaw() callback, > although it should at least reverse whatever .freeze() did to the hardware > (and memory allocations and so on), so that the adapter is functional > after creating the image. > > Fixing this requires some thought, though, because at the moment .freeze() > thinks it's .suspend(), which is not the case as this report clearly shows. > So, in fact i915_pci_suspend() has to be split into the .freeze() part and > the poweroff part cleanly and that's not so simple (at least to me). > Right, I think that'll be more clean, stuff in i915_save/restore_state() need to be splited too, especially isolate stuff for mode setting and other device state, as what my original purpose for this is to remove extra mode setting cycle in old behavior so not waste time for hibernate. -- Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
