From: Dave Jones <davej@codemonkey.org.uk>
To: Pavel Machek <pavel@ucw.cz>
Cc: Andrew Morton <akpm@linux-foundation.org>,
"Rafael J. Wysocki" <rjw@sisk.pl>,
kernel list <linux-kernel@vger.kernel.org>
Subject: Re: broken suspend in .2.6.25-rc3 on T61p (was Re: new regression in 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p)
Date: Tue, 26 Feb 2008 17:23:37 -0500 [thread overview]
Message-ID: <20080226222337.GA22172@codemonkey.org.uk> (raw)
In-Reply-To: <20080226215641.GE10280@elf.ucw.cz>
On Tue, Feb 26, 2008 at 10:56:41PM +0100, Pavel Machek wrote:
> Seems like pm-utils is just a thin wrapper around s2ram, at least in
> version debian ships. It does not seem to have its own whitelist.
The actual whitelists still live in hal (or specifically hal-data),
rather than pm-utils. /usr/share/hal/fdi/information/10freedesktop/20-video-quirk-pm-*
for example. This gets passed down to pm-utils by hal.
> /usr/lib/pm-utils/functions
>
> ...
>
> if [ -x /usr/sbin/s2ram ]; then
> if [ -n "$S2RAM_OPTS" ]; then
> # Trust HAL or the user to pass the correct
> options
> /usr/sbin/s2ram $S2RAM_OPTS
> elif /usr/sbin/s2ram --test > /dev/null ; then
> # Trust s2ram's internal whitelist
> /usr/sbin/s2ram
> else
> # Unknown machine
> echo "This machine is unkown, please try to
> find out how to suspend this machine. See s2ram(8)."
> fi
> else
> echo -n "mem" > /sys/power/state
> fi
Seems to be a debian specific change, the variant in Fedora, nor upstream
pm-utils doesn't have any of that. Possibly because it's a dumb idea
to have two separate sources of the same information.
> ...so it is ready to use s2ram, but will fall back to
> echo. Unfortunately, that will mean no video resume on _many_
> machines.
>
> To give some numbers: according to s2ram whitelist, we can restore
> video on 410 machines. On 74 of them, s2ram is not needed. So
> approximately 80% of machines need s2ram (at least in configuration
> without X running)....
>
> Pretty please, can we get s2ram for Fedora, so that video is restored
> there?
I'm not the gatekeeper of what goes into Fedora userspace, but I'm pretty
certain the path forward has already been decided. Running a modern
Fedora desktop installation without hal just isn't feasible.
Dave
--
http://www.codemonkey.org.uk
next prev parent reply other threads:[~2008-02-26 22:28 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-25 19:19 new regression in 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p Michael S. Tsirkin
2008-02-25 19:19 ` Michael S. Tsirkin
2008-02-25 19:42 ` Alexey Starikovskiy
2008-02-25 20:39 ` Michael S. Tsirkin
2008-02-25 20:50 ` Alexey Starikovskiy
2008-02-25 21:01 ` Michael S. Tsirkin
2008-02-25 22:33 ` Alexey Starikovskiy
2008-02-25 23:18 ` Michael S. Tsirkin
2008-02-25 19:46 ` Andrew Morton
2008-02-25 20:39 ` Michael S. Tsirkin
2008-02-25 20:45 ` Michael S. Tsirkin
2008-02-25 20:57 ` broken suspend in .2.6.25-rc3 on T61p (was Re: new regression in 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p) Rafael J. Wysocki
2008-02-25 21:26 ` Pavel Machek
2008-02-25 21:31 ` Michael S. Tsirkin
2008-02-25 21:50 ` Pavel Machek
2008-02-25 22:15 ` Michael S. Tsirkin
2008-02-25 22:20 ` Pavel Machek
2008-02-25 22:36 ` Michael S. Tsirkin
2008-02-25 22:39 ` Andrew Morton
2008-02-25 22:48 ` Michael S. Tsirkin
2008-02-25 22:53 ` Andrew Morton
2008-02-25 22:58 ` Michael S. Tsirkin
2008-02-25 22:58 ` Pavel Machek
[not found] ` <20080225155550.4a020dfb.akpm@linux-foundation.org>
[not found] ` <20080226000007.GB22102@atrey.karlin.mff.cuni.cz>
2008-02-26 10:11 ` Pavel Machek
2008-02-26 17:46 ` Dave Jones
2008-02-26 17:59 ` Pavel Machek
2008-02-26 18:10 ` Dave Jones
2008-02-26 18:16 ` Pavel Machek
2008-02-26 18:22 ` Dave Jones
2008-02-26 21:56 ` Pavel Machek
2008-02-26 22:23 ` Dave Jones [this message]
2008-02-26 22:31 ` Rafael J. Wysocki
2008-02-26 22:51 ` Pavel Machek
2008-02-27 13:12 ` Matthew Garrett
2008-02-27 13:23 ` Rafael J. Wysocki
2008-02-27 15:51 ` Stefan Seyfried
2008-02-27 15:55 ` Matthew Garrett
2008-02-26 18:56 ` Andrew Morton
2008-02-26 20:09 ` Len Brown
2008-02-26 23:31 ` Ray Lee
2008-02-27 0:12 ` Rafael J. Wysocki
2008-02-25 21:32 ` Jeff Garzik
2008-02-25 22:21 ` Pavel Machek
2008-02-26 8:54 ` new regression in 2.6.25-rc3: no keyboard/lid acpi events on thinkpad T61p Klaus S. Madsen
2008-02-26 21:32 ` Klaus S. Madsen
2008-02-26 21:44 ` Jiri Kosina
2008-02-27 22:13 ` Klaus S. Madsen
2008-02-27 23:20 ` Jiri Kosina
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080226222337.GA22172@codemonkey.org.uk \
--to=davej@codemonkey.org.uk \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=pavel@ucw.cz \
--cc=rjw@sisk.pl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.