cpufreq: optimize policy_is_shared()

Yury Norov · rafaeljw · commit 16c1e8385b3b · 2026-03-18T21:27:13.000+01:00
The switch to cpumask_nth() over cpumask_weight(), as it may return earlier - as soon as the function counts the required number of CPUs. Signed-off-by: Yury Norov <ynorov@nvidia.com> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> Reviewed-by: Zhongqiu Han <zhongqiu.han@oss.qualcomm.com> Link: https://patch.msgid.link/20260314192544.605914-1-ynorov@nvidia.com Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
@@ -232,7 +232,7 @@ static inline bool policy_is_inactive(struct cpufreq_policy *policy)
 
 static inline bool policy_is_shared(struct cpufreq_policy *policy)
 {
-	return cpumask_weight(policy->cpus) > 1;
+	return cpumask_nth(1, policy->cpus) < nr_cpumask_bits;
 }
 
 #ifdef CONFIG_CPU_FREQ

Original file line number	Diff line number	Diff line change
`@@ -232,7 +232,7 @@ static inline bool policy_is_inactive(struct cpufreq_policy *policy)`
`232`	`232`
`233`	`233`	`static inline bool policy_is_shared(struct cpufreq_policy *policy)`
`234`	`234`	`{`
`235`		`- return cpumask_weight(policy->cpus) > 1;`
	`235`	`+ return cpumask_nth(1, policy->cpus) < nr_cpumask_bits;`
`236`	`236`	`}`
`237`	`237`
`238`	`238`	`#ifdef CONFIG_CPU_FREQ`