All of lore.kernel.org
 help / color / mirror / Atom feed
From: bugzilla-daemon@bugzilla.kernel.org
To: cpufreq@vger.kernel.org
Subject: [Bug 64261] Intel Pstate driver truncates to pstate instead of rounding to nearest pstate
Date: Mon, 04 Nov 2013 18:39:41 +0000	[thread overview]
Message-ID: <bug-64261-12968-Q00B9fqyCz@https.bugzilla.kernel.org/> (raw)
In-Reply-To: <bug-64261-12968@https.bugzilla.kernel.org/>

https://bugzilla.kernel.org/show_bug.cgi?id=64261

--- Comment #4 from Dirk Brandewie <dirk.brandewie@gmail.com> ---
So you are trying to emulate the userspace governor here, not really to goal of
intel_pstate but OK :-)

The ability to select a single P state was the intended usage of
{min,max}_pct_perf but allow users to select a floor and ceiling for the range
of available P states.  The absolute meaning percent available performance
changes based on the SKU of the part (p states available).

The driver can only select integer values 16->38 in your turbo-on test case.

The p states are 2.6315789 percent wide in terms of turbo frequency and
2.9411765 percent wide in terms of the non-turbo max on your CPU

.42 * 38 = 15.96
.43 * 38 = 16.34
.44 * 38 = 16.72
.45 * 38 = 17.1

45% is where your test goes from 16->17.

For the 3 percent shelves in the turbo-off case. 
.50 * 34 = 17
.51 * 34 = 17.34
.52 * 34 = 17.68
.53 * 34 = 18.03 (so we are off by 3 percent of a pstate due to truncation)

-- 
You are receiving this mail because:
You are the assignee for the bug.

  parent reply	other threads:[~2013-11-04 18:39 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-11-02 21:52 [Bug 64261] New: Intel Pstate driver truncates to pstate instead of rounding to nearest pstate bugzilla-daemon
2013-11-04  7:17 ` [Bug 64261] " bugzilla-daemon
2013-11-04 16:14 ` bugzilla-daemon
2013-11-04 16:54 ` bugzilla-daemon
2013-11-04 18:39 ` bugzilla-daemon [this message]
2013-11-04 22:19 ` bugzilla-daemon
2013-11-05  1:28 ` bugzilla-daemon
2013-11-05  7:21 ` bugzilla-daemon
2013-11-05  7:35 ` bugzilla-daemon
2013-11-05  8:12 ` bugzilla-daemon
2013-11-05 15:06 ` bugzilla-daemon
2013-11-06  5:34 ` bugzilla-daemon
2013-11-06  5:38 ` bugzilla-daemon
2013-11-06  5:43 ` bugzilla-daemon
2013-11-06  5:48 ` bugzilla-daemon
2013-11-06  6:04 ` bugzilla-daemon
2013-11-06 18:58 ` bugzilla-daemon
2013-11-06 19:44 ` bugzilla-daemon
2014-04-18  3:02 ` bugzilla-daemon

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=bug-64261-12968-Q00B9fqyCz@https.bugzilla.kernel.org/ \
    --to=bugzilla-daemon@bugzilla.kernel.org \
    --cc=cpufreq@vger.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 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.