[Pm-utils] [PATCH 1/2] Enable autodetection and stacking of sleep methods.

[Danny Kukawka]

On Dienstag, 9. September 2008, Victor Lowther wrote:
> On Tue, 2008-09-09 at 11:24 +0200, Tim Dijkstra wrote:
> > Victor Lowther schreef:
> > > I also trimmed s2ram from uswsusp -- we just fall back to kernel
> > > suspend instead.  This way, we only have to handle video quirks in one
> > > place, and we control that place.
> >
> > So you are basically removing uswsusp support? That would be very
> > unfortunate, than we will have to add our own patches again...
>
> Just s2ram.  Handling quirks in s2ram and in 99video was just annoying,
> but with the recent "interesting" behaviour of the nvidia g80/g90 chips
> and their habit of only coming back from suspend if you posted from a
> saved copy of the BIOS, s2ram was causing resume failures that 99video
> handles correctly, so the old "disable 99video in favor of handling the
> quirks in uswsusp" bahaviour missed this case when running with s2ram.

That's not really a reason to drop s2ram support. There are good reasons, 
without starting any old discussion again, to use s2ram instead of 99video. 

If you don't have s2ram (which you obviously do), you have no trouble. And for 
the rest, which want to use s2ram, they should report it to the s2ram 
maintainers. But this isn't your problem at all. You make it only complicated 
for all these users. They have to revert your patch ....

> While I suppose I could set up some sort of framework where certian
> quirks are handled by s2ram and certian others are handled by 99video,
> that seems like a recipie for disaster.
>
> s2disk and s2both are still supported by this patch series, and now we
> force s2both to run quirkless.  I could have s2ram do the same thing,
> but since all it ends up doing the same as what we already do, it is
> just as easy to fall back to the kernel method.

That all is easy to say if you don't use s2ram. And not everyone is doing what 
you do. 

Danny