From: Jiri Kosina <jkosina@suse.cz>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
Ingo Molnar <mingo@elte.hu>,
x86@kernel.org, LKML <linux-kernel@vger.kernel.org>,
Andi Kleen <andi@firstfloor.org>,
Thomas Gleixner <tglx@linutronix.de>,
Henrique de Moraes Holschuh <hmh@hmh.eng.br>,
"H. Peter Anvin" <hpa@zytor.com>,
Ondrej Zary <linux@rainbow-software.org>,
Andrew Morton <akpm@linux-foundation.org>,
linux-pm@lists.linux-foundation.org
Subject: Re: [PATCH] apm: orphan the driver
Date: Mon, 28 Mar 2011 23:19:16 +0200 (CEST) [thread overview]
Message-ID: <alpine.LNX.2.00.1103282309290.26754__21969.6735417985$1301348314$gmane$org@pobox.suse.cz> (raw)
In-Reply-To: <AANLkTimqE2YZCH5GonnqeHGGBwzggTymauS1KYuTsqOX@mail.gmail.com>
On Mon, 28 Mar 2011, Linus Torvalds wrote:
> > Phasing out APM idle at least would be reasonable. resumably even if the old
> > laptops still work they are likely on AC because their batteries have
> > long died. So using a bit more power in idle shouldn't be a big
> > issue.
>
> Agreed. Especially since I'm not at all convinced that APM CPU idling
> ever really worked in the first place.
>
> And as you say, it's not a catastrophic failure even if we were to
> remove it, and even if it were to have mattered on those old laptops
> years ago. I think we can happily say "not worth worrying about" when
> it comes to APM_CPU_IDLE.
>
> And shutdown, suspend and screensaver should all be much easier to
> continue to support - they shouldn't have any serious infrastructure
> issues.
APM_CPU_IDLE removal doesn't really sound as a disaster, yes. I doubt
anyone would even notice.
So deprecating it altogether with planned Len's code reshuffling seems
like reasonable aproach to me.
_SUSPEND, poweroff and probably _STANDBY are the parts I would care much
more about.
--
Jiri Kosina
SUSE Labs, Novell Inc.
next prev parent reply other threads:[~2011-03-28 21:19 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-03-26 6:22 [PATCH] apm: orphan the driver Stephen Rothwell
2011-03-28 12:44 ` Jiri Kosina
2011-03-28 12:44 ` Jiri Kosina
2011-03-28 13:44 ` Stephen Rothwell
2011-03-28 13:44 ` Stephen Rothwell
2011-03-28 14:05 ` Jiri Kosina
2011-03-28 14:05 ` Jiri Kosina
2011-03-28 14:14 ` Joe Perches
2011-03-28 17:24 ` Jiri Kosina
2011-03-28 17:24 ` Jiri Kosina
2011-03-28 14:14 ` Joe Perches
2011-03-28 17:45 ` Andi Kleen
2011-03-28 17:45 ` Andi Kleen
2011-03-28 17:57 ` H. Peter Anvin
2011-03-28 18:21 ` Andi Kleen
2011-03-28 18:24 ` H. Peter Anvin
2011-03-28 18:24 ` H. Peter Anvin
2011-03-28 18:46 ` Linus Torvalds
2011-03-28 18:46 ` Linus Torvalds
2011-03-28 21:19 ` Jiri Kosina
2011-03-31 3:16 ` [PATCH] x86 idle APM: delete apm_cpu_idle(), and its use of pm_idle Len Brown
2011-03-31 3:16 ` Len Brown
2011-03-31 6:56 ` Ingo Molnar
2011-03-31 6:56 ` Ingo Molnar
2011-04-01 9:28 ` Jiri Kosina
2011-04-01 9:28 ` Jiri Kosina
2011-03-28 21:19 ` Jiri Kosina [this message]
2011-03-28 18:21 ` [PATCH] apm: orphan the driver Andi Kleen
2011-03-28 17:57 ` H. Peter Anvin
-- strict thread matches above, loose matches on Subject: below --
2011-03-26 6:22 Stephen Rothwell
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='alpine.LNX.2.00.1103282309290.26754__21969.6735417985$1301348314$gmane$org@pobox.suse.cz' \
--to=jkosina@suse.cz \
--cc=akpm@linux-foundation.org \
--cc=andi@firstfloor.org \
--cc=hmh@hmh.eng.br \
--cc=hpa@zytor.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=linux@rainbow-software.org \
--cc=mingo@elte.hu \
--cc=sfr@canb.auug.org.au \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=x86@kernel.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 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.