lkml.org 
[lkml]   [2015]   [Feb]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 5/6] intel_idle: Add ->enter_freeze callbacks
Date
On Thursday, February 12, 2015 02:26:43 PM Peter Zijlstra wrote:
> On Wed, Feb 11, 2015 at 05:04:17AM +0100, Rafael J. Wysocki wrote:
> > @@ -131,28 +133,32 @@ static struct cpuidle_state nehalem_csta
> > .flags = MWAIT2flg(0x00),
> > .exit_latency = 3,
> > .target_residency = 6,
> > - .enter = &intel_idle },
> > + .enter = &intel_idle,
> > + .enter_freeze = intel_idle_freeze, },
> > {
> > .name = "C1E-NHM",
> > .desc = "MWAIT 0x01",
> > .flags = MWAIT2flg(0x01),
> > .exit_latency = 10,
> > .target_residency = 20,
> > - .enter = &intel_idle },
> > + .enter = &intel_idle,
> > + .enter_freeze = intel_idle_freeze, },
> > {
> > .name = "C3-NHM",
> > .desc = "MWAIT 0x10",
> > .flags = MWAIT2flg(0x10) | CPUIDLE_FLAG_TLB_FLUSHED,
> > .exit_latency = 20,
> > .target_residency = 80,
> > - .enter = &intel_idle },
> > + .enter = &intel_idle,
> > + .enter_freeze = intel_idle_freeze, },
> > {
> > .name = "C6-NHM",
> > .desc = "MWAIT 0x20",
> > .flags = MWAIT2flg(0x20) | CPUIDLE_FLAG_TLB_FLUSHED,
> > .exit_latency = 200,
> > .target_residency = 800,
> > - .enter = &intel_idle },
> > + .enter = &intel_idle,
> > + .enter_freeze = intel_idle_freeze, },
> > {
> > .enter = NULL }
> > };
>
> Why bother with enter_freeze() for any but the deepest state (C6 in this
> case)?

User space may disable the deepest one (and any of them in general) via sysfs
and there's no good reason to ignore its choice in this particular case while
we're honoring it otherwise.

So the logic is basically "find the deepest one which isn't disabled" and
setting the pointers costs us nothing really.

> Also, should we ignore things like intel_idle.max_cstate for this
> selection?

No, we shouldn't. The deeper ones may just not work then.



\
 
 \ /
  Last update: 2015-02-12 17:21    [W:0.071 / U:0.264 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site