Re: broken suspend in .2.6.25-rc3 on T61p (was Re: new regressionin 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p)

From: Dave Jones
Date: Tue Feb 26 2008 - 13:22:59 EST


On Tue, Feb 26, 2008 at 07:16:11PM +0100, Pavel Machek wrote:
> On Tue 2008-02-26 13:10:01, Dave Jones wrote:
> > On Tue, Feb 26, 2008 at 06:59:54PM +0100, Pavel Machek wrote:
> >
> > > > if by 'custom' you mean the solution everyone agreed to work
> > > > toward at the power management summit several years ago
> > > > (hal/pm-utils) then, yes.
> > >
> > > I must have been on different summit... I believe it is bad to tie
> > > s2ram to hal, because it makes testing on minimal system hard.
> > >
> > > Anyway, what is the "default" way to trigger s2ram for Andrew? Perhaps
> > > Fedora already has his machine whitelisted...
> >
> > There is no s2ram. pm-suspend uses the white/black-lists in pm-utils.
> > Remember that? The cross-distro package everyone agreed was a good idea
> > so that every distro didn't have their own magic utility ?
>
> Well, we have cross-distro package, it is at suspend.sf.net , and it
> can bring up video - which is kind of important. (It is single binary,
> so it can be pagelocked -- which is important for s2disk).
>
> Plus it does not depend on HAL.

Neither does pm-utils. Once again for the hard of thinking..

The mechanism belongs in pm-utils. HAL is just a fancy wrapper around that.
Don't want/like hal? fine, a smaller wrapper around pm-suspend and friends
is trivial (or even unnecessary if you're happy with running pm-suspend by hand)

Dave

--
http://www.codemonkey.org.uk
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/