mbox series

[v2,0/4] cpufreq: Fix some boost errors related to CPU online and offline.

Message ID 20250117101457.1530653-1-zhenglifeng1@huawei.com (mailing list archive)
Headers show
Series cpufreq: Fix some boost errors related to CPU online and offline. | expand

Message

zhenglifeng (A) Jan. 17, 2025, 10:14 a.m. UTC
This patch series fix some boost errors related to CPU online and offline:

 - patch 1 fix an error that causes the CPU stay on base frequency after a
   specific operation

 - patch 2 introduce a more generic way to set default per-policy boost
   flag and fix a error that causes the per-policy boost flag remians true
   when cpufreq_driver boost disabled

 - patch 3 fix an error in cppc_cpufreq that causes the CPU stay on base
   frequency when boost flag is true

 - patch 4 remove the set_boost in acpi_cpufreq_cpu_init(), since it will
   be executed in cpufreq_online

Change since v1:
 - remove update of min_freq_req
 - optimize the conditions for executing set_boost in cpufreq_online
 - fix another error in cppc_cpufreq
 - remove set_boost in acpi_cpufreq_cpu_init()

Lifeng Zheng (4):
  cpufreq: Fix re-boost issue after hotplugging a cpu
  cpufreq: Introduce a more generic way to set default per-policy boost
    flag
  cpufreq: CPPC: Fix wrong max_freq in policy initialization
  cpufreq: ACPI: Remove set_boost in acpi_cpufreq_cpu_init()

 drivers/cpufreq/acpi-cpufreq.c |  5 -----
 drivers/cpufreq/cppc_cpufreq.c |  5 +++--
 drivers/cpufreq/cpufreq.c      | 20 ++++++++++++++++----
 3 files changed, 19 insertions(+), 11 deletions(-)

Comments

Viresh Kumar Jan. 21, 2025, 6:38 a.m. UTC | #1
On 17-01-25, 18:14, Lifeng Zheng wrote:
> This patch series fix some boost errors related to CPU online and offline:
> 
>  - patch 1 fix an error that causes the CPU stay on base frequency after a
>    specific operation
> 
>  - patch 2 introduce a more generic way to set default per-policy boost
>    flag and fix a error that causes the per-policy boost flag remians true
>    when cpufreq_driver boost disabled
> 
>  - patch 3 fix an error in cppc_cpufreq that causes the CPU stay on base
>    frequency when boost flag is true
> 
>  - patch 4 remove the set_boost in acpi_cpufreq_cpu_init(), since it will
>    be executed in cpufreq_online
> 
> Change since v1:
>  - remove update of min_freq_req
>  - optimize the conditions for executing set_boost in cpufreq_online
>  - fix another error in cppc_cpufreq
>  - remove set_boost in acpi_cpufreq_cpu_init()

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Rafael J. Wysocki Jan. 23, 2025, 8:07 p.m. UTC | #2
On Tue, Jan 21, 2025 at 7:38 AM Viresh Kumar <viresh.kumar@linaro.org> wrote:
>
> On 17-01-25, 18:14, Lifeng Zheng wrote:
> > This patch series fix some boost errors related to CPU online and offline:
> >
> >  - patch 1 fix an error that causes the CPU stay on base frequency after a
> >    specific operation
> >
> >  - patch 2 introduce a more generic way to set default per-policy boost
> >    flag and fix a error that causes the per-policy boost flag remians true
> >    when cpufreq_driver boost disabled
> >
> >  - patch 3 fix an error in cppc_cpufreq that causes the CPU stay on base
> >    frequency when boost flag is true
> >
> >  - patch 4 remove the set_boost in acpi_cpufreq_cpu_init(), since it will
> >    be executed in cpufreq_online
> >
> > Change since v1:
> >  - remove update of min_freq_req
> >  - optimize the conditions for executing set_boost in cpufreq_online
> >  - fix another error in cppc_cpufreq
> >  - remove set_boost in acpi_cpufreq_cpu_init()
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

All patches in the series applied as 6.14-rc material, thanks!