On Monday, 30 July 2007 08:56, Stefan Seyfried wrote: > On Fri, Jul 27, 2007 at 08:45:41PM +0200, Rafael J. Wysocki wrote: > > On Friday, 27 July 2007 18:33, Stefan Seyfried wrote: > > > On Fri, Jul 27, 2007 at 05:18:45PM +0200, Rafael J. Wysocki wrote: > > > > > > > > (yes, i think i know what it does; writing 1 into the state of the > > > > > framebuffer device just disables any drawing - and thus any access > > > > > of possibly not really initialized hardware before running vbe_post > > > > > etc...) > > > > > At least it seemed to do no harm in my (limited) tests. > > > > > > > > I think it can go if > > > > (1) it doesn't break any setups that currently work > > > > > > We will know after the next openSUSE alpha release, where i will have > > > unleashed that patch onto the unexpecting guinea pigs^W^WopenSUSE users > > > ;-) > > > If i don't get a wave of bugreports, than i'll assume that it is safe. > > > > > > Theoretically we could "protect" it by another quirk-flag, but i'd rather > > > try to avoid that. > > > > Agreed. > > > > > I will also go through the kernel code and check if anything can go wrong. > > > > Yes, please do that if you can. > > I did try to go through the kernel code, but i'd not put my hand into the > fire and guarantee that it does not do anything stupid :-) > > Some drivers (radeonfb and i810fb for example) are already doing this in > their suspend method and it seems to work.
That sounds good. > That said, it should long-term be fixed in the kernel properly, but if we > can prove from userspace that it helps some machines, then we know that it > is worth the trouble. Yes. > > > Echoing "1" into the state file simply makes the text console "freeze" > > > (but > > > apparently it is still doing stuff in the background, you can still output > > > text etc, you just won't see it). Echoing "0" will make it paint the > > > current > > > screen contents. At least that was what happened in my experiments. > > > > > > > (2) it fixes at least one box that currently doesn't work > > > > > > Yes, the user in the bugreport is a happy camper now :-) (he does not have > > > the s2ram code right now, but uses the script equivalent, but unless i > > > have > > > put a bug into the C code, it should be equivalent). > > > > Can you ask the user to test s2ram with your patch, please? > > yes, i will do that once i have built and pushed a package. OK -- "Premature optimization is the root of all evil." - Donald Knuth ------------------------------------------------------------------------- This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Suspend-devel mailing list Suspend-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/suspend-devel