cpufreq.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Arjan van de Ven <arjan@linux.intel.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: David C Niemi <dniemi@verisign.com>,
	dirk.brandewie@gmail.com, cpufreq@vger.kernel.org,
	deneen.t.dock@intel.com
Subject: Re: [PATCH RFC 0/1] cpufreq/x86: Add P-state driver for sandy bridge.
Date: Thu, 06 Dec 2012 13:15:13 -0800	[thread overview]
Message-ID: <50C10AE1.4060608@linux.intel.com> (raw)
In-Reply-To: <1881660.ZpA1SQTrgA@vostro.rjw.lan>

On 12/6/2012 12:45 PM, Rafael J. Wysocki wrote:

>> My idea for a policy "dial" is mostly
>>
>> * Uncompromised performance
>> * Balanced - biased towards performance     (say, defined to be lowest power at most a 2 1/2% perf hit)
>> * Balanced                                  (say, at most a 5% perf hit)
>> * Balanced - biased towards lower power     (sat, at most a 10% perf hit)
>> * Uncompromised lowest power
>>
>> we can argue about the exact %ages, but the idea is to give at least some reasonably definition that people can understand,
>> but that also can be measured
>
> It looks like you'd like a tunable setting the maximum allowed performance hit
> due to power management.  Is that correct?

basically yes, but not as a continuous dial (that's not practical), but as a
certain number of sensible steps.... I'm not sure it makes sense to have more than 5 steps.

My key interest is to have something that is both understandable by the sysadmin as to what it means,
but also something that you can actually measure (and thus test)...
while still describing a desire/preference, not a specific implementation of an algorithm.

A sysadmin understands "willing to give up at most 5% of max performance to save power"... and he can reason about it,
and think about it, take his own situation into account and then make a decision on it as a result.

The policy side can then measure algorithms and tunables to make sure they stay within that 5%
(of course on "reasonable" or "realistic" workloads.. not theoretical foofoo stuff).

You must put something like this in place, because if you just call it "balanced", that means 101 different things
to 100 people.... and as a result neither the algorithm side can ever do anything (SOMEONE somewhere will regress),
nor can the sysadmin side make a reasonable decision, since it means something else on each machine as well.





  reply	other threads:[~2012-12-06 21:15 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
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 [this message]
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=50C10AE1.4060608@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).