cpufreq.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@linux.intel.com>
To: David C Niemi <dniemi@verisign.com>
Cc: dirk.brandewie@gmail.com, cpufreq@vger.kernel.org, rjw@sisk.pl,
	deneen.t.dock@intel.com
Subject: Re: [PATCH RFC 0/1] cpufreq/x86: Add P-state driver for sandy bridge.
Date: Wed, 05 Dec 2012 13:01:27 -0800	[thread overview]
Message-ID: <50BFB627.5080106@linux.intel.com> (raw)
In-Reply-To: <50BFAE5A.1010809@verisign.com>

On 12/5/2012 12:28 PM, David C Niemi wrote:
>
> Dirk,
>
> I applaud the work you are doing.  In general I believe it is important to separate policy (governor and its settings) from the driver, particularly so as different end-users have very different goals for power management.  Not everyone is trying to maximize performance per watt per se (in fact probably rather few end users are doing so literally).  In server applications, for example, the first priority is typically maximum performance when under heavy load, and the second priority is minimum power consumption at idle.  There may not ever be a benefit for choosing one of the middle clock states.  The OnDemand governor with the sampling_down_factor set to ~100 can do quite well at this, at least compared to implementations prior to yours.  Another consideration is that just blindly tryin
 g to run flat out all the time (e.g. the old performance governor approach) bumps you up against your thermal limits and can actually slow you down, vs. intelligently powersaving idle hardware
> threads -- so a user who totally aims for performance with no regard for power savings cannot avoid must paying some attention to power management.
>

the idea that you can have separate policy and hardware is a big fallacy though.
A good policy ends up very hardware specific, and policies of the past work poorly on todays hardware
("ondemand" is one of the worst case behaviors you can have on modern Intel cpus for example).

While I appreciate the desire for some level of "preference" control, the split of policy and hardware in the
way cpufreq did that really isn't the way to go forward...


  reply	other threads:[~2012-12-05 21:01 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-05 19:01 [PATCH RFC 0/1] cpufreq/x86: Add P-state driver for sandy bridge dirk.brandewie
2012-12-05 19:01 ` [PATCH RFC 1/1] " dirk.brandewie
2012-12-05 20:28 ` [PATCH RFC 0/1] " David C Niemi
2012-12-05 21:01   ` Arjan van de Ven [this message]
2012-12-05 21:40     ` David C Niemi
2012-12-05 21:54       ` Arjan van de Ven
2012-12-06 15:01         ` David C Niemi
2012-12-06 16:27           ` Arjan van de Ven
2012-12-06 17:30             ` David C Niemi
2012-12-06 17:41               ` Arjan van de Ven
2012-12-06 18:25               ` Dirk Brandewie
2012-12-06 18:41                 ` David C Niemi
2012-12-06 21:35                   ` Dirk Brandewie
2012-12-06 22:23                     ` David C Niemi
2012-12-06 20:45             ` Rafael J. Wysocki
2012-12-06 21:15               ` Arjan van de Ven
2012-12-06 21:26                 ` Rafael J. Wysocki
2012-12-06 21:34                   ` Rafael J. Wysocki
2012-12-06 22:08                     ` Arjan van de Ven
2012-12-06 22:53                       ` Rafael J. Wysocki
2012-12-06 16:35   ` Dirk Brandewie
2012-12-06 16:49     ` Arjan van de Ven
2012-12-06 18:16     ` David C Niemi

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=50BFB627.5080106@linux.intel.com \
    --to=arjan@linux.intel.com \
    --cc=cpufreq@vger.kernel.org \
    --cc=deneen.t.dock@intel.com \
    --cc=dirk.brandewie@gmail.com \
    --cc=dniemi@verisign.com \
    --cc=rjw@sisk.pl \
    /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;
as well as URLs for NNTP newsgroup(s).