cpufreq: Some optimizations for cpufreq.c

[PATCH v2 0/4] cpufreq: Some optimizations for cpufreq.c

Posted by Lifeng Zheng 3 months ago

This patch series makes some minor optimizations for cpufreq.c to make
codes cleaner.

Changelog:

v2:

 - drop patch 3 and 4 as they have been applied
 - drop patch 6
 - move invariance initialization instead of add a
   static_branch_disable_cpuslocked() in error path
 - add a Fixes tag in patch 2

---
Discussions of previous versions:
v1: https://lore.kernel.org/all/20250623133402.3120230-1-zhenglifeng1@huawei.com/

Lifeng Zheng (4):
  cpufreq: Move cpufreq-based invariance initialization to where all
    other initializations have been successfully done in
    cpufreq_register_driver()
  cpufreq: Init policy->rwsem before it may be possibly used
  cpufreq: Move the check of cpufreq_driver->get into
    cpufreq_verify_current_freq()
  cpufreq: Exit governor when failed to start old governor

 drivers/cpufreq/cpufreq.c | 40 ++++++++++++++++++++-------------------
 1 file changed, 21 insertions(+), 19 deletions(-)

-- 
2.33.0

Re: [PATCH v2 0/4] cpufreq: Some optimizations for cpufreq.c

Posted by Rafael J. Wysocki 2 months, 4 weeks ago

On Wed, Jul 9, 2025 at 12:42 PM Lifeng Zheng <zhenglifeng1@huawei.com> wrote:
>
> This patch series makes some minor optimizations for cpufreq.c to make
> codes cleaner.
>
> Changelog:
>
> v2:
>
>  - drop patch 3 and 4 as they have been applied
>  - drop patch 6
>  - move invariance initialization instead of add a
>    static_branch_disable_cpuslocked() in error path
>  - add a Fixes tag in patch 2
>
> ---
> Discussions of previous versions:
> v1: https://lore.kernel.org/all/20250623133402.3120230-1-zhenglifeng1@huawei.com/
>
> Lifeng Zheng (4):
>   cpufreq: Move cpufreq-based invariance initialization to where all
>     other initializations have been successfully done in
>     cpufreq_register_driver()
>   cpufreq: Init policy->rwsem before it may be possibly used
>   cpufreq: Move the check of cpufreq_driver->get into
>     cpufreq_verify_current_freq()
>   cpufreq: Exit governor when failed to start old governor
>
>  drivers/cpufreq/cpufreq.c | 40 ++++++++++++++++++++-------------------
>  1 file changed, 21 insertions(+), 19 deletions(-)
>
> --

All applied as 6.17 material, thanks!