From: Pavel Machek <pavel@ucw.cz>
To: David Brownell <david-b@pacbell.net>
Cc: Linus Torvalds <torvalds@osdl.org>,
Linux-pm mailing list <linux-pm@lists.osdl.org>
Subject: Re: [patch 2.6.18-rc2-mm 1/2] PM: make /sys/devices/.../power/state optional
Date: Wed, 16 Aug 2006 13:57:29 +0200 [thread overview]
Message-ID: <20060816115729.GB10764@elf.ucw.cz> (raw)
In-Reply-To: <200607191337.02365.david-b@pacbell.net>
HI!
> There seems to be a consensus on linux-pm that the sysfs power/state files
> are the wrong approach to userspace having control over runtime PM. Not
> that there's agreement on what such controls _should_ look like, or even
> that there should be such controls for purposes other than testing...
>
> So here's a patch that just makes it easy to get rid of this misfeature.
> It'll help flush out any userspace software that really thinks it needs that
> mechanism; PCMCIA seems to no longer be in that camp, it has it's own
> sysfs attribute.
> Add a new PM_SYSFS_DEPRECATED config option to control whether or
> not the /sys/devices/.../power/state files are provided. This will
> make it easier to get rid of that mechanism when the time comes,
> and to verify that userspace tools work right without it.
>
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
Looks good to me, ACK.
(And sorry for long delay).
I guess this should go with it...? OTOH andrew wanted alternative,
first, before removing the feature... but it is unclear to me if we
want the feature at all on the production system, so...
Signed-off-by: Pavel Machek <pavel@suse.cz>
diff --git a/Documentation/feature-removal-schedule.txt b/Documentation/feature-removal-schedule.txt
index 87851ef..4902bef 100644
--- a/Documentation/feature-removal-schedule.txt
+++ b/Documentation/feature-removal-schedule.txt
@@ -6,6 +6,16 @@ be removed from this file.
---------------------------
+What: /sys/device/.../power/state
+When: July 2007
+Files:
+Why: Because it takes integers, and different userland applications
+ expect different numbers to mean different things.
+ (Pcmcia expect 2 for off, some other code expects 3 for off).
+Who: Pavel Machek <pavel@suse.cz>
+
+---------------------------
+
What: RAW driver (CONFIG_RAW_DRIVER)
When: December 2005
Why: declared obsolete since kernel 2.6.3
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
next prev parent reply other threads:[~2006-08-16 11:57 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2006-07-19 20:37 [patch 2.6.18-rc2-mm 1/2] PM: make /sys/devices/.../power/state optional David Brownell
2006-08-16 11:57 ` Pavel Machek [this message]
2006-08-16 18:40 ` David Brownell
2006-08-16 19:08 ` Pavel Machek
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=20060816115729.GB10764@elf.ucw.cz \
--to=pavel@ucw.cz \
--cc=david-b@pacbell.net \
--cc=linux-pm@lists.osdl.org \
--cc=torvalds@osdl.org \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox