[lkml]   [2011]   [Feb]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [RFC PATCH V3 2/3] cpuidle: list based cpuidle driver registration and selection
    * Peter Zijlstra <> [2011-02-09 12:17:43]:

    > On Tue, 2011-02-08 at 16:22 +0530, Trinabh Gupta wrote:
    > > A cpuidle_driver is global in nature as it provides routines
    > > for all the CPUS. Each CPU registered with the cpuidle subsystem is
    > > represented as a cpuidle_device. A cpuidle_device structure
    > > points to the low level idle routines for that CPU provided by
    > > a certain driver. In other words, a cpuidle driver creates a
    > > cpuidle_device structure for each CPU that it registers with the
    > > cpuidle subsystem. Whenever cpuidle idle loop is called, the cpuidle
    > > subsystem picks the cpuidle_device structure for that cpu and
    > > calls one of the low level idle routines through that structure.
    > Why all this per-cpu nonsense? I thought we all agreed that ACPI tables
    > specifying non-uniform C states were considered buggy and we should
    > simply use the intersection of all cpus.

    Hi Peter,

    We discussed this in the previous posts. On ppc64 we would like to
    have single global registration, but for x86 Arjan recommended that we
    keep the per-cpu registration or else we may break legacy or buggy


    One corner case that was against using lowest common C-State is that
    we could have cores/packages sporting a new lower C-State if they are
    at a thermal limit and want the OS to go to much lower C-State and
    sacrifice performance. Our global design will prevent that cpu from
    going to a state lower than the rest of the system.

    This is only a remote possibility, but could happen on battery
    operated devices, under low battery mode etc. Basically we would have
    to keep our design open to allow individual CPUs to goto 'their'
    deepest allowed sleep state even in a asymmetric case.

    Having a design to allow global registration as long as C-States are
    symmetric (non-x86 or boot param) and use per-cpu for asymmetric
    C-States (x86) case will be ideal.

    This patch series is only the first step to remove pm_idle() from x86
    and incrementally from all other architectures. Once we get this
    working, tested and accepted, we could remove the per-cpu data
    structures and have a global states table.

    Thanks for the review.


     \ /
      Last update: 2011-02-10 08:05    [W:0.024 / U:2.972 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site