On 23-10-20, 17:15, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <[email protected]> > > Commit 33aa46f252c7 ("cpufreq: intel_pstate: Use passive mode by > default without HWP") was meant to cause intel_pstate to be used > in the passive mode with the schedutil governor on top of it, 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") > Reported-by: Julia Lawall <[email protected]> > Cc: 5.8+ <[email protected]> # 5.8+ > Signed-off-by: Rafael J. Wysocki <[email protected]> > --- > > The v2 addresses a review comment from Viresh regarding of the expression > format > and adds a missing Reported-by for Julia. > > --- > drivers/cpufreq/Kconfig | 2 ++ > 1 file changed, 2 insertions(+)
Acked-by: Viresh Kumar <[email protected]> -- viresh
