powercap/dtpm_cpu: Reset per_cpu variable in the release function
authorDaniel Lezcano <daniel.lezcano@linaro.org>
Sun, 30 Jan 2022 21:02:04 +0000 (22:02 +0100)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Wed, 23 Feb 2022 18:45:22 +0000 (19:45 +0100)
The release function does not reset the per cpu variable when it is
called. That will prevent creation again as the variable will be
already from the previous creation.

Fix it by resetting them.

Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Ulf Hansson <ulf.hansson@linaro.org>
Link: https://lore.kernel.org/r/20220130210210.549877-2-daniel.lezcano@linaro.org
drivers/powercap/dtpm_cpu.c

index eed5ad688d46761043d5afe9c7381c514fc2527b..71f45d2f5a606f98605f1a686b4f42db30ab2db2 100644 (file)
@@ -151,10 +151,17 @@ static int update_pd_power_uw(struct dtpm *dtpm)
 static void pd_release(struct dtpm *dtpm)
 {
        struct dtpm_cpu *dtpm_cpu = to_dtpm_cpu(dtpm);
+       struct cpufreq_policy *policy;
 
        if (freq_qos_request_active(&dtpm_cpu->qos_req))
                freq_qos_remove_request(&dtpm_cpu->qos_req);
 
+       policy = cpufreq_cpu_get(dtpm_cpu->cpu);
+       if (policy) {
+               for_each_cpu(dtpm_cpu->cpu, policy->related_cpus)
+                       per_cpu(dtpm_per_cpu, dtpm_cpu->cpu) = NULL;
+       }
+       
        kfree(dtpm_cpu);
 }