lkml.org 
[lkml]   [2020]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH] cpufreq: Avoid configuring old governors as default with intel_pstate
    Date
    From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    Subject: [PATCH] cpufreq: Avoid configuring old governors as default with intel_pstate

    Commit 33aa46f252c7 ("cpufreq: intel_pstate: Use passive mode by
    default without HWP") was meant to cause intel_pstate without HWP
    to be used in the passive mode with the schedutil governor on top of
    it by default, but it missed the case in which either "ondemand" or
    "conservative" was selected as the default governor in the existing
    kernel config, in which case the previous old governor configuration
    would be used, causing the default legacy governor to be used on top
    of intel_pstate instead of schedutil.

    Address this by preventing "ondemand" and "conservative" from being
    configured as the default cpufreq governor in the case when schedutil
    is the default choice for the default governor setting.

    [Note that the default cpufreq governor can still be set via the
    kernel command line if need be and that choice is not limited,
    so if anyone really wants to use one of the legacy governors by
    default, it can be achieved this way.]

    Fixes: 33aa46f252c7 ("cpufreq: intel_pstate: Use passive mode by default without HWP")
    Cc: 5.8+ <stable@vger.kernel.org> # 5.8+
    Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
    ---
    drivers/cpufreq/Kconfig | 2 ++
    1 file changed, 2 insertions(+)

    Index: linux-pm/drivers/cpufreq/Kconfig
    ===================================================================
    --- linux-pm.orig/drivers/cpufreq/Kconfig
    +++ linux-pm/drivers/cpufreq/Kconfig
    @@ -71,6 +71,7 @@ config CPU_FREQ_DEFAULT_GOV_USERSPACE

    config CPU_FREQ_DEFAULT_GOV_ONDEMAND
    bool "ondemand"
    + depends on !SMP || !X86_INTEL_PSTATE
    select CPU_FREQ_GOV_ONDEMAND
    select CPU_FREQ_GOV_PERFORMANCE
    help
    @@ -83,6 +84,7 @@ config CPU_FREQ_DEFAULT_GOV_ONDEMAND

    config CPU_FREQ_DEFAULT_GOV_CONSERVATIVE
    bool "conservative"
    + depends on !SMP || !X86_INTEL_PSTATE
    select CPU_FREQ_GOV_CONSERVATIVE
    select CPU_FREQ_GOV_PERFORMANCE
    help


    \
     
     \ /
      Last update: 2020-10-22 18:24    [W:4.162 / U:1.108 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site