mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Ingo Molnar <mingo@elte.hu>,
	Stephen Rothwell <sfr@canb.auug.org.au>,
	x86@kernel.org, linux-pm@lists.linux-foundation.org,
	linux-kernel@vger.kernel.org,
	Linus Torvalds <torvalds@linux-foundation.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	"H. Peter Anvin" <hpa@zytor.com>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] x86 APM: delete Linux kernel APM support
Date: Sat, 26 Mar 2011 00:35:51 -0400 (EDT)	[thread overview]
Message-ID: <alpine.LFD.2.02.1103260021410.28928@x980> (raw)
In-Reply-To: <201103252333.23880.rjw@sisk.pl>

> >  > > Beyond the lack of a upstream-visible feature-removal-schedule entry, we 
> >  > > still have an Arcnet driver which hardware was obsoleted by Ethernet in the 
> >  > > late 80s, and we still have i486 support and those are *much* older than 
> >  > > APM.

> > So how does your reasoning not apply to those drivers? There's several which 
> > are older than APM support.

If we follow-through with the proposed cpuidle changes,
then we'll have to cut APM code.   The problem isn't cutting
the code, it is testing it.  I do have a couple of 15-year old
laptops which include APM support.  However, with ACPI disabled
to enable APM, they don't even boot the upstream kernel today.

> > We had this really big battle about x86/Voyager two years ago, which x86 
> > subarchitecture literally had just a single user left, and the code was more 
> > intrusive than APM. Even there after much flaming the eventual consensus was 
> > that we'd accept it back if it was done cleanly, as part of the new-style 
> > x86_platform code.
> > 
> > Given that APM fits into the current PM frameworks there's no such problem here 
> > that i can see.

Somebody was pushing for Voyager support in the kernel
and was energetic about maintaining it.

cheers,
-Len


  reply	other threads:[~2011-03-26  4:36 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <alpine.LFD.2.02.1103232321070.3848@x980>
     [not found] ` <20110324154505.934a56a0.sfr@canb.auug.org.au>
2011-03-24  7:39   ` [PATCH] APM: delete APM in Linux-2.6.40 Len Brown
2011-03-24  8:16     ` [PATCH] x86 APM: delete Linux kernel APM support Len Brown
2011-03-24  8:31       ` [PATCH] x86 APM: delete Linux kernel APM support (v2) Len Brown
2011-03-24 16:01         ` Andi Kleen
2011-03-24  8:39       ` [PATCH] x86 APM: delete Linux kernel APM support Ingo Molnar
2011-03-24 23:49         ` Len Brown
2011-03-25 11:30           ` Ingo Molnar
2011-03-25 12:38             ` Ingo Molnar
2011-03-25 12:41               ` Ingo Molnar
2011-03-25 22:33             ` Rafael J. Wysocki
2011-03-26  4:35               ` Len Brown [this message]
2011-04-08  6:25               ` Pavel Machek
2011-04-08 20:55                 ` H. Peter Anvin
2011-04-11 13:05                   ` Pavel Machek
2011-04-11 18:19                     ` H. Peter Anvin
2011-04-13 13:30                       ` Pavel Machek
2011-03-26  5:01             ` Len Brown
2011-03-26  9:31               ` Ingo Molnar
2011-03-27 21:40               ` Ondrej Zary
2011-03-28  4:51                 ` Ingo Molnar
2011-03-28  5:43                   ` H. Peter Anvin
2011-03-28 12:57                 ` H. Peter Anvin
2011-03-28  5:18               ` Ingo Molnar
2011-03-25 15:41           ` Calvin Walton
2011-03-24  8:18     ` [PATCH] APM: delete APM in Linux-2.6.40 Stephen Rothwell
2011-03-24  8:38     ` Alexander Stein
2011-03-24 10:21     ` Henrique de Moraes Holschuh
2011-03-24 23:05       ` Len Brown
2011-03-25  1:07         ` Henrique de Moraes Holschuh
2011-03-25  1:34           ` Len Brown
2011-04-04 16:44           ` Pavel Machek
2011-04-04 20:18             ` Len Brown
2011-03-24 12:15     ` Ondrej Zary
2011-03-24 23:29       ` Len Brown
2011-03-25  8:07         ` Ondrej Zary
2011-03-26  5:09           ` Len Brown
2011-04-02 21:40     ` Yuhong Bao

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.LFD.2.02.1103260021410.28928@x980 \
    --to=lenb@kernel.org \
    --cc=akpm@linux-foundation.org \
    --cc=hpa@zytor.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@lists.linux-foundation.org \
    --cc=mingo@elte.hu \
    --cc=rjw@sisk.pl \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome