2021-10-04 17:28:30

by Hector Yuan

[permalink] [raw]
Subject: [PATCH v1] cpufreq: Fix parameter in parse_perf_domain

Should pass per-cpu id to map perf_domain for related_cpus
rather than map policy_cpu several times.


Hector.Yuan (1):
cpufreq:Fix parameter in parse_perf_domain

include/linux/cpufreq.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)


2021-10-04 21:58:42

by Hector Yuan

[permalink] [raw]
Subject: [PATCH v1 1/1] cpufreq:Fix parameter in parse_perf_domain

From: "Hector.Yuan" <[email protected]>

Should pass per-cpu id to map perf_domain for related_cpus
rather than map policy_cpu several times.

Signed-off-by: Hector.Yuan <[email protected]>
---
include/linux/cpufreq.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
index ff88bb3..66a1f49 100644
--- a/include/linux/cpufreq.h
+++ b/include/linux/cpufreq.h
@@ -1041,7 +1041,7 @@ static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_
if (cpu == pcpu)
continue;

- ret = parse_perf_domain(pcpu, list_name, cell_name);
+ ret = parse_perf_domain(cpu, list_name, cell_name);
if (ret < 0)
continue;

--
1.7.9.5

2021-10-05 04:22:35

by Viresh Kumar

[permalink] [raw]
Subject: Re: [PATCH v1 1/1] cpufreq:Fix parameter in parse_perf_domain

On 04-10-21, 22:42, Hector Yuan wrote:
> From: "Hector.Yuan" <[email protected]>
>
> Should pass per-cpu id to map perf_domain for related_cpus
> rather than map policy_cpu several times.
>
> Signed-off-by: Hector.Yuan <[email protected]>
> ---
> include/linux/cpufreq.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
> index ff88bb3..66a1f49 100644
> --- a/include/linux/cpufreq.h
> +++ b/include/linux/cpufreq.h
> @@ -1041,7 +1041,7 @@ static inline int of_perf_domain_get_sharing_cpumask(int pcpu, const char *list_
> if (cpu == pcpu)
> continue;
>
> - ret = parse_perf_domain(pcpu, list_name, cell_name);
> + ret = parse_perf_domain(cpu, list_name, cell_name);
> if (ret < 0)
> continue;

Applied. Thanks.

--
viresh