LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: William Heimbigner <icxcnika@mar.tar.cc>
To: Dave Jones <davej@redhat.com>
Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [RFC] [PATCH] cpufreq: allow full selection of default governors
Date: Tue, 24 Apr 2007 22:35:54 +0000 (GMT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0704242232280.25153@server.thyself> (raw)
In-Reply-To: <20070424211253.GG23598@redhat.com>

On Tue, 24 Apr 2007, Dave Jones wrote:
> On Tue, Apr 24, 2007 at 09:03:23PM +0000, William Heimbigner wrote:
> > The following patches should allow selection of conservative, powersave, and
> > ondemand in the kernel configuration.
>
> This has been rejected several times already.
> Ondemand and conservative isn't a viable governor for all cpufreq implementations
> (ie, ones with high switching latencies).  Also, see the comment in the Kconfig
> a few lines above where you are adding this.
>
> 	Dave

It may be wise to disable the selection of ondemand, however, conservative was 
designed to fix the very problem created by ondemand (the need for high 
switching latencies), and this also says nothing about powersave.

Additionally, it seems odd that the kernel would be left in an undefined state 
if a governor failed to initalize - rather, shouldn't the governor fail in such 
a way as to not cause errors?

William Heimbigner
icxcnika@mar.tar.cc

  parent reply	other threads:[~2007-04-24 22:35 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-24 21:03 William Heimbigner
2007-04-24 21:12 ` Dave Jones
2007-04-24 21:25   ` Chuck Ebbert
2007-04-24 22:05   ` Nish Aravamudan
2007-04-25  0:03     ` Dave Jones
2007-04-27  1:54       ` Dominik Brodowski
2007-04-27  6:09         ` Dave Jones
2007-04-27 12:24           ` Dominik Brodowski
2007-04-27 13:40             ` William Heimbigner
2007-04-27 17:06             ` Gautham Shenoy
2007-04-27 18:17               ` William Heimbigner
2007-04-24 22:35   ` William Heimbigner [this message]
2007-04-24 21:13 ` Robert P. J. Day

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=Pine.LNX.4.64.0704242232280.25153@server.thyself \
    --to=icxcnika@mar.tar.cc \
    --cc=akpm@linux-foundation.org \
    --cc=davej@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --subject='Re: [RFC] [PATCH] cpufreq: allow full selection of default governors' \
    /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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).