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
signature.asc
Description: Digital signature
------------------------------------------------------------------------------ The Planet: dedicated and managed hosting, cloud storage, colocation Stay online with enterprise data centers and the best network in the business Choose flexible plans and management services without long-term contracts Personal 24x7 support from experience hosting pros just a phone call away. http://p.sf.net/sfu/theplanet-com
-- _______________________________________________ Dri-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dri-devel
