]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
cpufreq: CPPC: Remove cpu_data_list
authorLifeng Zheng <zhenglifeng1@huawei.com>
Mon, 26 May 2025 11:30:55 +0000 (19:30 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 18 Jun 2025 19:03:52 +0000 (21:03 +0200)
After commit a28b2bfc099c ("cppc_cpufreq: replace per-cpu data array with a
list"), cpu_data can be got from policy->driver_data, so cpu_data_list is
not actually needed and can be removed.

Signed-off-by: Lifeng Zheng <zhenglifeng1@huawei.com>
Link: https://patch.msgid.link/20250526113057.3086513-2-zhenglifeng1@huawei.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cppc_cpufreq.c
include/acpi/cppc_acpi.h

index b7c688a5659c014b93c14621fa9c63eba628545d..f3b5ea9fcbf514384f4fa13fbb982c822ff69958 100644 (file)
 
 #include <acpi/cppc_acpi.h>
 
-/*
- * This list contains information parsed from per CPU ACPI _CPC and _PSD
- * structures: e.g. the highest and lowest supported performance, capabilities,
- * desired performance, level requested etc. Depending on the share_type, not
- * all CPUs will have an entry in the list.
- */
-static LIST_HEAD(cpu_data_list);
-
 static struct cpufreq_driver cppc_cpufreq_driver;
 
 #ifdef CONFIG_ACPI_CPPC_CPUFREQ_FIE
@@ -567,8 +559,6 @@ static struct cppc_cpudata *cppc_cpufreq_get_cpu_data(unsigned int cpu)
                goto free_mask;
        }
 
-       list_add(&cpu_data->node, &cpu_data_list);
-
        return cpu_data;
 
 free_mask:
@@ -583,7 +573,6 @@ static void cppc_cpufreq_put_cpu_data(struct cpufreq_policy *policy)
 {
        struct cppc_cpudata *cpu_data = policy->driver_data;
 
-       list_del(&cpu_data->node);
        free_cpumask_var(cpu_data->shared_cpu_map);
        kfree(cpu_data);
        policy->driver_data = NULL;
@@ -954,24 +943,10 @@ static int __init cppc_cpufreq_init(void)
        return ret;
 }
 
-static inline void free_cpu_data(void)
-{
-       struct cppc_cpudata *iter, *tmp;
-
-       list_for_each_entry_safe(iter, tmp, &cpu_data_list, node) {
-               free_cpumask_var(iter->shared_cpu_map);
-               list_del(&iter->node);
-               kfree(iter);
-       }
-
-}
-
 static void __exit cppc_cpufreq_exit(void)
 {
        cpufreq_unregister_driver(&cppc_cpufreq_driver);
        cppc_freq_invariance_exit();
-
-       free_cpu_data();
 }
 
 module_exit(cppc_cpufreq_exit);
index 325e9543e08f8b68d0b62504adf8a546be71a5a4..20f3d62e7a16a33d27ab18c2431dab9283699fde 100644 (file)
@@ -139,7 +139,6 @@ struct cppc_perf_fb_ctrs {
 
 /* Per CPU container for runtime CPPC management. */
 struct cppc_cpudata {
-       struct list_head node;
        struct cppc_perf_caps perf_caps;
        struct cppc_perf_ctrls perf_ctrls;
        struct cppc_perf_fb_ctrs perf_fb_ctrs;