1. 07 7月, 2009 2 次提交
  2. 15 6月, 2009 2 次提交
  3. 27 5月, 2009 1 次提交
    • M
      [CPUFREQ] fix timer teardown in ondemand governor · b14893a6
      Mathieu Desnoyers 提交于
      * Rafael J. Wysocki (rjw@sisk.pl) wrote:
      > This message has been generated automatically as a part of a report
      > of regressions introduced between 2.6.28 and 2.6.29.
      >
      > The following bug entry is on the current list of known regressions
      > introduced between 2.6.28 and 2.6.29.  Please verify if it still should
      > be listed and let me know (either way).
      >
      >
      > Bug-Entry	: http://bugzilla.kernel.org/show_bug.cgi?id=13186
      > Subject		: cpufreq timer teardown problem
      > Submitter	: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
      > Date		: 2009-04-23 14:00 (24 days old)
      > References	: http://marc.info/?l=linux-kernel&m=124049523515036&w=4
      > Handled-By	: Mathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
      > Patch		: http://patchwork.kernel.org/patch/19754/
      > 		  http://patchwork.kernel.org/patch/19753/
      >
      
      (updated changelog)
      
      cpufreq fix timer teardown in ondemand governor
      
      The problem is that dbs_timer_exit() uses cancel_delayed_work() when it should
      use cancel_delayed_work_sync(). cancel_delayed_work() does not wait for the
      workqueue handler to exit.
      
      The ondemand governor does not seem to be affected because the
      "if (!dbs_info->enable)" check at the beginning of the workqueue handler returns
      immediately without rescheduling the work. The conservative governor in
      2.6.30-rc has the same check as the ondemand governor, which makes things
      usually run smoothly. However, if the governor is quickly stopped and then
      started, this could lead to the following race :
      
      dbs_enable could be reenabled and multiple do_dbs_timer handlers would run.
      This is why a synchronized teardown is required.
      
      The following patch applies to, at least, 2.6.28.x, 2.6.29.1, 2.6.30-rc2.
      
      Depends on patch
      cpufreq: remove rwsem lock from CPUFREQ_GOV_STOP call
      Signed-off-by: NMathieu Desnoyers <mathieu.desnoyers@polymtl.ca>
      CC: Andrew Morton <akpm@linux-foundation.org>
      CC: gregkh@suse.de
      CC: stable@kernel.org
      CC: cpufreq@vger.kernel.org
      CC: Ingo Molnar <mingo@elte.hu>
      CC: rjw@sisk.pl
      CC: Ben Slusky <sluskyb@paranoiacs.org>
      Signed-off-by: NDave Jones <davej@redhat.com>
      b14893a6
  4. 25 2月, 2009 3 次提交
  5. 06 2月, 2009 1 次提交
  6. 06 1月, 2009 1 次提交
  7. 10 10月, 2008 8 次提交
  8. 24 5月, 2008 1 次提交
  9. 18 1月, 2008 1 次提交
    • J
      cpufreq: Initialise default governor before use · 6915719b
      Johannes Weiner 提交于
      When the cpufreq driver starts up at boot time, it calls into the default
      governor which might not be initialised yet.  This hurts when the
      governor's worker function relies on memory that is not yet set up by its
      init function.
      
      This migrates all governors from module_init() to fs_initcall() when being
      the default, as was already done in cpufreq_performance when it was the
      only possible choice.  The performance governor is always initialized early
      because it might be used as fallback even when not being the default.
      
      Fixes at least one actual oops where ondemand is the default governor and
      cpufreq_governor_dbs() uses the uninitialised kondemand_wq work-queue
      during boot-time.
      Signed-off-by: NJohannes Weiner <hannes@saeurebad.de>
      Cc: Dave Jones <davej@codemonkey.org.uk>
      Cc: "Rafael J. Wysocki" <rjw@sisk.pl>
      Cc: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
      Acked-by: NIngo Molnar <mingo@elte.hu>
      Cc: Thomas Gleixner <tglx@linutronix.de>
      Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: NLinus Torvalds <torvalds@linux-foundation.org>
      6915719b
  10. 05 10月, 2007 1 次提交
    • T
      [CPUFREQ] allow ondemand and conservative cpufreq governors to be used as default · 1c256245
      Thomas Renninger 提交于
      Depending on the transition latency of the HW for cpufreq switches, the
      ondemand or conservative governor cannot be used with certain cpufreq
      drivers.  Still the ondemand should be the default governor on a wide range
      of systems.  This patch allows this and lets the governor fallback to the
      performance governor at cpufreq driver load time, if the driver does not
      support fast enough frequency switching.
      
      Main benefit is that on e.g.  installation or other systems without
      userspace support a working dynamic cpufreq support can be achieved on most
      systems by simply loading the cpufreq driver.  This is especially essential
      for recent x86(_64) laptop hardware which may rely on working dynamic
      cpufreq OS support.
      Signed-off-by: NThomas Renninger <trenn@suse.de>
      Signed-off-by: NVenkatesh Pallipadi <venkatesh.pallipadi@intel.com>
      Cc: Russell King <rmk@arm.linux.org.uk>
      Cc: Bryan Wu <bryan.wu@analog.com>
      Cc: Andi Kleen <ak@suse.de>
      Cc: "Luck, Tony" <tony.luck@intel.com>
      Cc: Paul Mackerras <paulus@samba.org>
      Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>
      Cc: Paul Mundt <lethal@linux-sh.org>
      Cc: "David S. Miller" <davem@davemloft.net>
      Signed-off-by: NAndrew Morton <akpm@linux-foundation.org>
      Signed-off-by: NDave Jones <davej@redhat.com>
      1c256245
  11. 22 6月, 2007 2 次提交
  12. 09 5月, 2007 1 次提交
  13. 21 2月, 2007 1 次提交
  14. 11 2月, 2007 4 次提交
  15. 22 11月, 2006 1 次提交
  16. 07 11月, 2006 1 次提交
  17. 21 10月, 2006 1 次提交
  18. 16 10月, 2006 1 次提交
    • V
      [CPUFREQ][8/8] acpi-cpufreq: Add support for freq feedback from hardware · dfde5d62
      Venkatesh Pallipadi 提交于
      Enable ondemand governor and acpi-cpufreq to use IA32_APERF and IA32_MPERF MSR
      to get active frequency feedback for the last sampling interval. This will
      make ondemand take right frequency decisions when hardware coordination of
      frequency is going on.
      
      Without APERF/MPERF, ondemand can take wrong decision at times due
      to underlying hardware coordination or TM2.
      Example:
      * CPU 0 and CPU 1 are hardware cooridnated.
      * CPU 1 running at highest frequency.
      * CPU 0 was running at highest freq. Now ondemand reduces it to
        some intermediate frequency based on utilization.
      * Due to underlying hardware coordination with other CPU 1, CPU 0 continues to
        run at highest frequency (as long as other CPU is at highest).
      * When ondemand samples CPU 0 again next time, without actual frequency
        feedback from APERF/MPERF, it will think that previous frequency change
        was successful and can go to wrong target frequency. This is because it
        thinks that utilization it has got this sampling interval is when running at
        intermediate frequency, rather than actual highest frequency.
      
      More information about IA32_APERF IA32_MPERF MSR:
      Refer to IA-32 Intel® Architecture Software Developer's Manual at
      http://developer.intel.comSigned-off-by: NVenkatesh Pallipadi <venkatesh.pallipadi@intel.com>
      Signed-off-by: NDave Jones <davej@redhat.com>
      dfde5d62
  19. 06 9月, 2006 1 次提交
  20. 14 8月, 2006 1 次提交
  21. 12 8月, 2006 2 次提交
    • A
      [CPUFREQ][2/2] ondemand: updated add powersave_bias tunable · 05ca0350
      Alexey Starikovskiy 提交于
      ondemand selects the minimum frequency that can retire
      a workload with negligible idle time -- ideally resulting in the highest
      performance/power efficiency with negligible performance impact.
      
      But on some systems and some workloads, this algorithm
      is more performance biased than necessary, and
      de-tuning it a bit to allow some performance impact
      can save measurable power.
      
      This patch adds a "powersave_bias" tunable to ondemand
      to allow it to reduce its target frequency by a specified percent.
      
      By default, the powersave_bias is 0 and has no effect.
      powersave_bias is in units of 0.1%, so it has an effective range
      of 1 through 1000, resulting in 0.1% to 100% impact.
      
      In practice, users will not be able to detect a difference between
      0.1% increments, but 1.0% increments turned out to be too large.
      Also, the max value of 1000 (100%) would simply peg the system
      in its deepest power saving P-state, unless the processor really has
      a hardware P-state at 0Hz:-)
      
      For example, If ondemand requests 2.0GHz based on utilization,
      and powersave_bias=100, this code will knock 10% off the target
      and seek  a target of 1.8GHz instead of 2.0GHz until the
      next sampling.  If 1.8 is an exact match with an hardware frequency
      we use it, otherwise we average our time between the frequency
      next higher than 1.8 and next lower than 1.8.
      
      Note that a user or administrative program can change powersave_bias
      at run-time depending on how they expect the system to be used.
      
      Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi at intel.com>
      Signed-off-by: Alexey Starikovskiy <alexey.y.starikovskiy at intel.com>
      Signed-off-by: NDave Jones <davej@redhat.com>
      05ca0350
    • A
      [CPUFREQ][1/2] ondemand: updated tune for hardware coordination · 1ce28d6b
      Alexey Starikovskiy 提交于
      Try to make dbs_check_cpu() call on all CPUs at the same jiffy.
      This will help when multiple cores share P-states via Hardware Coordination.
      
      Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi at intel.com>
      Signed-off-by: Alexey Starikovskiy <alexey.y.starikovskiy at intel.com>
      Signed-off-by: NDave Jones <davej@redhat.com>
      1ce28d6b
  22. 26 7月, 2006 1 次提交
    • A
      [PATCH] Reorganize the cpufreq cpu hotplug locking to not be totally bizare · 153d7f3f
      Arjan van de Ven 提交于
      The patch below moves the cpu hotplugging higher up in the cpufreq
      layering; this is needed to avoid recursive taking of the cpu hotplug
      lock and to otherwise detangle the mess.
      
      The new rules are:
      1. you must do lock_cpu_hotplug() around the following functions:
         __cpufreq_driver_target
         __cpufreq_governor (for CPUFREQ_GOV_LIMITS operation only)
         __cpufreq_set_policy
      2. governer methods (.governer) must NOT take the lock_cpu_hotplug()
         lock in any way; they are called with the lock taken already
      3. if your governer spawns a thread that does things, like calling
         __cpufreq_driver_target, your thread must honor rule #1.
      4. the policy lock and other cpufreq internal locks nest within
         the lock_cpu_hotplug() lock.
      
      I'm not entirely happy about how the __cpufreq_governor rule ended up
      (conditional locking rule depending on the argument) but basically all
      callers pass this as a constant so it's not too horrible.
      
      The patch also removes the cpufreq_governor() function since during the
      locking audit it turned out to be entirely unused (so no need to fix it)
      
      The patch works on my testbox, but it could use more testing
      (otoh... it can't be much worse than the current code)
      Signed-off-by: NArjan van de Ven <arjan@linux.intel.com>
      Signed-off-by: NLinus Torvalds <torvalds@osdl.org>
      153d7f3f
  23. 24 7月, 2006 1 次提交
  24. 30 6月, 2006 1 次提交