All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dominik Brodowski <linux@brodo.de>
To: Arjan van de Ven <arjanv@redhat.com>
Cc: "Mallick, Asit K" <asit.k.mallick@intel.com>,
	linux-acpi <linux-acpi@intel.com>,
	cpufreq@www.linux.org.uk, "Nakajima,
	Jun" <jun.nakajima@intel.com>,
	"Pallipadi, Venkatesh" <venkatesh.pallipadi@intel.com>
Subject: Re: [PATCH] 3/3 Dynamic cpufreq governor and updates to ACPI P-state driver
Date: Tue, 21 Oct 2003 22:44:10 +0200	[thread overview]
Message-ID: <20031021204410.GH26971@brodo.de> (raw)
In-Reply-To: <1066725533.5237.3.camel@laptop.fenrus.com>

On Tue, Oct 21, 2003 at 10:38:53AM +0200, Arjan van de Ven wrote:
> Note that I'm not totally against doing some of this in the kernel; I
> can well see the point of say, detecting an IRQ overload and based on
> that, go to max speed in the kernel because it's a situation where
> userspace doesn't even run; but the patch as is doesn't do any such
> advanced things...

Indeed, this driver doesn't do anything like that. However,  I want to point
out two [of more] reasons why this is the right thing to do:

1.) I'm aware of one embedded system running linux which only has two 
userspace programs - busybox and cardmgr. All other stuff is handled 
in the kernel. An effort to remove the dependency on cardmgr is going on
[ <linux-pcmcia@lists.infradead.org> ]. If this embedded system was
supporting cpufreq, would you want to do dynamic scaling in the kernel, or
in userspace?

2.) It's a starting point for more advanced governors which really cannot be
done in userspace as this would be much too expensive. Having to choose
between different kernel and different userspace governors seems a bit
complicated for me. The _cost_ of doing this specific governor in the kernel
is near zero.

	Dominik

  parent reply	other threads:[~2003-10-21 20:44 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-10-21  2:56 [PATCH] 3/3 Dynamic cpufreq governor and updates to ACPI P-state driver Pallipadi, Venkatesh
2003-10-21  2:56 ` Pallipadi, Venkatesh
2003-10-21  8:38 ` Arjan van de Ven
2003-10-21  8:38   ` Arjan van de Ven
2003-10-21  9:59   ` Mattia Dongili
2003-10-21  9:59     ` Mattia Dongili
2003-10-21 10:17     ` Wichert Akkerman
2003-10-21 10:52       ` Mattia Dongili
2003-10-21 12:59         ` Wichert Akkerman
2003-10-21 15:36         ` Daniel Thor Kristjansson
2003-10-21 20:32           ` Dominik Brodowski
2003-10-21 20:32             ` Dominik Brodowski
2003-10-22 15:48             ` Daniel Thor Kristjansson
2003-10-22 15:48               ` Daniel Thor Kristjansson
2003-10-23 14:32             ` Pavel Machek
2003-10-23 14:32               ` Pavel Machek
2003-10-24 18:22               ` Dominik Brodowski
2003-10-24 18:22                 ` Dominik Brodowski
2003-10-21 19:23         ` Carl Thompson
2003-10-21 20:37           ` Dominik Brodowski
2003-10-21 20:37             ` Dominik Brodowski
2003-10-21 20:44   ` Dominik Brodowski [this message]
2003-10-21 20:59 ` Dominik Brodowski
2003-10-23 14:17 ` Pavel Machek
2003-10-23 14:17   ` Pavel Machek
2003-10-23 20:47   ` Moore, Robert
2003-10-23 20:47     ` Moore, Robert
2003-10-23 21:50     ` Nakajima, Jun
2003-10-23 21:50       ` Nakajima, Jun
2003-10-24 18:38       ` Dominik Brodowski
2003-10-24 18:38         ` Dominik Brodowski
2003-10-24 11:27     ` Ducrot Bruno
2003-10-24 11:27       ` Ducrot Bruno
  -- strict thread matches above, loose matches on Subject: below --
2003-10-22  5:50 Pallipadi, Venkatesh
2003-10-24 18:52 Pallipadi, Venkatesh
2003-10-24 18:52 ` Pallipadi, Venkatesh

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=20031021204410.GH26971@brodo.de \
    --to=linux@brodo.de \
    --cc=arjanv@redhat.com \
    --cc=asit.k.mallick@intel.com \
    --cc=cpufreq@www.linux.org.uk \
    --cc=jun.nakajima@intel.com \
    --cc=linux-acpi@intel.com \
    --cc=venkatesh.pallipadi@intel.com \
    /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.