[PATCH v4 10/10] PM: EM: Use scope-based cleanup helper
From: Zihuan Zhang <hidden>
Date: 2025-09-03 13:18:38
Also in:
dri-devel, imx, intel-gfx, linux-acpi, linux-arm-kernel, linux-omap, linux-pm, lkml
Subsystem:
energy model, hibernation (aka software suspend, aka swsusp), suspend to ram, the rest · Maintainers:
Lukasz Luba, "Rafael J. Wysocki", Linus Torvalds
Replace the manual cpufreq_cpu_put() with __free(put_cpufreq_policy) annotation for policy references. This reduces the risk of reference counting mistakes and aligns the code with the latest kernel style. No functional change intended. Signed-off-by: Zihuan Zhang <redacted> --- kernel/power/energy_model.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/kernel/power/energy_model.c b/kernel/power/energy_model.c
index ea7995a25780..99401678e809 100644
--- a/kernel/power/energy_model.c
+++ b/kernel/power/energy_model.c@@ -451,7 +451,7 @@ static void em_cpufreq_update_efficiencies(struct device *dev, struct em_perf_state *table) { struct em_perf_domain *pd = dev->em_pd; - struct cpufreq_policy *policy; + struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL; int found = 0; int i, cpu;
@@ -479,8 +479,6 @@ em_cpufreq_update_efficiencies(struct device *dev, struct em_perf_state *table) found++; } - cpufreq_cpu_put(policy); - if (!found) return;
@@ -787,7 +785,7 @@ static void em_check_capacity_update(void) /* Check if CPUs capacity has changed than update EM */ for_each_possible_cpu(cpu) { - struct cpufreq_policy *policy; + struct cpufreq_policy *policy __free(put_cpufreq_policy) = NULL; struct em_perf_domain *pd; struct device *dev;
@@ -801,7 +799,6 @@ static void em_check_capacity_update(void) msecs_to_jiffies(1000)); break; } - cpufreq_cpu_put(policy); dev = get_cpu_device(cpu); pd = em_pd_get(dev);
--
2.25.1