From e8d786bfa7cbc7dbf5b722e3df086dda3bf17072 Mon Sep 17 00:00:00 2001 From: Liang Chen Date: Thu, 26 Dec 2019 15:52:34 +0800 Subject: [PATCH] cpufreq: interactive: fix bug: do not set pcpu->policy to NULL Speedchange_task will access pcpu->policy randomly, and the policy will never be free, so just keep pcpu->policy will be ok even the governor is stopped. Fixes: a4524d29de6d ("cpufreq: interactive: fix race between input event and governor exit") Change-Id: Icac24a866c6f1aa57fe147396729b02c80798f2e Signed-off-by: Liang Chen --- drivers/cpufreq/cpufreq_interactive.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/cpufreq/cpufreq_interactive.c b/drivers/cpufreq/cpufreq_interactive.c index 2a26e181f4f5..f838ca01084d 100644 --- a/drivers/cpufreq/cpufreq_interactive.c +++ b/drivers/cpufreq/cpufreq_interactive.c @@ -1437,7 +1437,6 @@ static int cpufreq_governor_interactive(struct cpufreq_policy *policy, pcpu = &per_cpu(cpuinfo, j); down_write(&pcpu->enable_sem); pcpu->governor_enabled = 0; - pcpu->policy = NULL; del_timer_sync(&pcpu->cpu_timer); del_timer_sync(&pcpu->cpu_slack_timer); up_write(&pcpu->enable_sem);