linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [RFC PATCH 0/3] add SW BOOST support for CPPC
@ 2020-05-08  9:11 Xiongfeng Wang
  2020-05-08  9:11 ` [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()' Xiongfeng Wang
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-08  9:11 UTC (permalink / raw)
  To: rjw, viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan
  Cc: Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel, wangxiongfeng2

ACPI spec 6.2 section 8.4.7.1 provide the following two CPC registers.

"Highest performance is the absolute maximum performance an individual
processor may reach, assuming ideal conditions. This performance level
may not be sustainable for long durations, and may only be achievable if
other platform components are in a specific state; for example, it may
require other processors be in an idle state.

Nominal Performance is the maximum sustained performance level of the
processor, assuming ideal operating conditions. In absence of an
external constraint (power, thermal, etc.) this is the performance level
the platform is expected to be able to maintain continuously. All
processors are expected to be able to sustain their nominal performance
state simultaneously."

We can use Highest Performance as the max performance in boost mode and
Nomial Performance as the max performance in non-boost mode. If the
Highest Performance is greater than the Nominal Performance, we assume
SW BOOST is supported.

Xiongfeng Wang (3):
  cpufreq: fix the return value in 'cpufreq_boost_set_sw()'
  cpufreq: Add SW BOOST support for drivers without frequency table
  CPPC: add support for SW BOOST

 drivers/cpufreq/cppc_cpufreq.c | 17 +++++++++++++++--
 drivers/cpufreq/cpufreq.c      | 25 +++++++++++++++++--------
 include/linux/cpufreq.h        |  2 ++
 3 files changed, 34 insertions(+), 10 deletions(-)

-- 
1.7.12.4


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()'
  2020-05-08  9:11 [RFC PATCH 0/3] add SW BOOST support for CPPC Xiongfeng Wang
@ 2020-05-08  9:11 ` Xiongfeng Wang
  2020-05-14 13:54   ` Rafael J. Wysocki
  2020-05-08  9:11 ` [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table Xiongfeng Wang
  2020-05-08  9:11 ` [RFC PATCH 3/3] CPPC: add support for SW BOOST Xiongfeng Wang
  2 siblings, 1 reply; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-08  9:11 UTC (permalink / raw)
  To: rjw, viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan
  Cc: Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel, wangxiongfeng2

When I try to add SW BOOST support for CPPC, I got the following error:
cpufreq: cpufreq_boost_trigger_state: Cannot enable BOOST
cpufreq: store_boost: Cannot enable BOOST!

It is because return value 1 of 'freq_qos_update_request()' means the
effective constraint value has changed, not a error code on failures.
But for 'cpufreq_driver.set_boost()', a nonzero return value means
failure. So change 'ret' to zero when 'freq_qos_update_request()'
returns a positive value.

Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
---
 drivers/cpufreq/cpufreq.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index 4adac3a..475fb1b 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -2522,6 +2522,8 @@ static int cpufreq_boost_set_sw(int state)
 		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
 		if (ret < 0)
 			break;
+		else
+			ret = 0;
 	}
 
 	return ret;
-- 
1.7.12.4


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table
  2020-05-08  9:11 [RFC PATCH 0/3] add SW BOOST support for CPPC Xiongfeng Wang
  2020-05-08  9:11 ` [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()' Xiongfeng Wang
@ 2020-05-08  9:11 ` Xiongfeng Wang
  2020-05-14 14:16   ` Rafael J. Wysocki
  2020-05-08  9:11 ` [RFC PATCH 3/3] CPPC: add support for SW BOOST Xiongfeng Wang
  2 siblings, 1 reply; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-08  9:11 UTC (permalink / raw)
  To: rjw, viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan
  Cc: Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel, wangxiongfeng2

Software-managed BOOST get the boost frequency by check the flag
CPUFREQ_BOOST_FREQ at driver's frequency table. But some cpufreq driver
don't have frequency table and use other methods to get the frequency
range, such CPPC cpufreq driver.

To add SW BOOST support for drivers without frequency table, we add
members in 'cpufreq_policy.cpufreq_cpuinfo' to record the max frequency
of boost mode and non-boost mode. The cpufreq driver initialize these two
members when probing.

Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
---
 drivers/cpufreq/cpufreq.c | 23 +++++++++++++++--------
 include/linux/cpufreq.h   |  2 ++
 2 files changed, 17 insertions(+), 8 deletions(-)

diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
index 475fb1b..a299426 100644
--- a/drivers/cpufreq/cpufreq.c
+++ b/drivers/cpufreq/cpufreq.c
@@ -2508,15 +2508,22 @@ static int cpufreq_boost_set_sw(int state)
 	int ret = -EINVAL;
 
 	for_each_active_policy(policy) {
-		if (!policy->freq_table)
-			continue;
-
-		ret = cpufreq_frequency_table_cpuinfo(policy,
+		if (policy->freq_table) {
+			ret = cpufreq_frequency_table_cpuinfo(policy,
 						      policy->freq_table);
-		if (ret) {
-			pr_err("%s: Policy frequency update failed\n",
-			       __func__);
-			break;
+			if (ret) {
+				pr_err("%s: Policy frequency update failed\n",
+				       __func__);
+				break;
+			}
+		} else if (policy->cpuinfo.boost_max_freq) {
+			if (state)
+				policy->max = policy->cpuinfo.boost_max_freq;
+			else
+				policy->max = policy->cpuinfo.nonboost_max_freq;
+			policy->cpuinfo.max_freq = policy->max;
+		} else {
+			continue;
 		}
 
 		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
index 018dce8..c3449e6 100644
--- a/include/linux/cpufreq.h
+++ b/include/linux/cpufreq.h
@@ -43,6 +43,8 @@ enum cpufreq_table_sorting {
 struct cpufreq_cpuinfo {
 	unsigned int		max_freq;
 	unsigned int		min_freq;
+	unsigned int		boost_max_freq;
+	unsigned int		nonboost_max_freq;
 
 	/* in 10^(-9) s = nanoseconds */
 	unsigned int		transition_latency;
-- 
1.7.12.4


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [RFC PATCH 3/3] CPPC: add support for SW BOOST
  2020-05-08  9:11 [RFC PATCH 0/3] add SW BOOST support for CPPC Xiongfeng Wang
  2020-05-08  9:11 ` [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()' Xiongfeng Wang
  2020-05-08  9:11 ` [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table Xiongfeng Wang
@ 2020-05-08  9:11 ` Xiongfeng Wang
  2 siblings, 0 replies; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-08  9:11 UTC (permalink / raw)
  To: rjw, viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan
  Cc: Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel, wangxiongfeng2

To add SW BOOST support for CPPC, we need to get the max frequency of
boost mode and non-boost mode. ACPI spec 6.2 section 8.4.7.1 describe
the following two CPC registers.

"Highest performance is the absolute maximum performance an individual
processor may reach, assuming ideal conditions. This performance level
may not be sustainable for long durations, and may only be achievable if
other platform components are in a specific state; for example, it may
require other processors be in an idle state.

Nominal Performance is the maximum sustained performance level of the
processor, assuming ideal operating conditions. In absence of an
external constraint (power, thermal, etc.) this is the performance level
the platform is expected to be able to maintain continuously. All
processors are expected to be able to sustain their nominal performance
state simultaneously."

To add SW BOOST support for CPPC, we can use Highest Performance as the
max performance in boost mode and Nominal Performance as the max
performance in non-boost mode. If the Highest Performance is greater
than the Nominal Performance, we assume SW BOOST is supported.

The current CPPC driver does not support SW BOOST and use 'Highest
Performance' as the max performance the CPU can achieve. 'Nominal
Performance' is used to convert 'performance' to 'frequency'. That
means, if firmware enable boost and provide a value for Highest
Performance which is greater than Nominal Performance, boost feature is
enabled by default.

Because SW BOOST is disabled by default, so, after this patch, boost
feature is disabled by default even if boost is enabled by firmware.

Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
---
 drivers/cpufreq/cppc_cpufreq.c | 17 +++++++++++++++--
 1 file changed, 15 insertions(+), 2 deletions(-)

diff --git a/drivers/cpufreq/cppc_cpufreq.c b/drivers/cpufreq/cppc_cpufreq.c
index bda0b24..6ae070f 100644
--- a/drivers/cpufreq/cppc_cpufreq.c
+++ b/drivers/cpufreq/cppc_cpufreq.c
@@ -310,7 +310,7 @@ static int cppc_cpufreq_cpu_init(struct cpufreq_policy *policy)
 	 * Section 8.4.7.1.1.5 of ACPI 6.1 spec)
 	 */
 	policy->min = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.lowest_nonlinear_perf);
-	policy->max = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.highest_perf);
+	policy->max = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.nominal_perf);
 
 	/*
 	 * Set cpuinfo.min_freq to Lowest to make the full range of performance
@@ -318,7 +318,7 @@ static int cppc_cpufreq_cpu_init(struct cpufreq_policy *policy)
 	 * nonlinear perf
 	 */
 	policy->cpuinfo.min_freq = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.lowest_perf);
-	policy->cpuinfo.max_freq = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.highest_perf);
+	policy->cpuinfo.max_freq = cppc_cpufreq_perf_to_khz(cpu, cpu->perf_caps.nominal_perf);
 
 	policy->transition_delay_us = cppc_cpufreq_get_transition_delay_us(cpu_num);
 	policy->shared_type = cpu->shared_type;
@@ -343,6 +343,19 @@ static int cppc_cpufreq_cpu_init(struct cpufreq_policy *policy)
 
 	cpu->cur_policy = policy;
 
+	/*
+	 * If 'highest_perf' is greater than 'nominal_perf', we assume CPU Boost
+	 * is supported.
+	 */
+	if (cpu->perf_caps.highest_perf > cpu->perf_caps.nominal_perf) {
+		policy->cpuinfo.boost_max_freq = cppc_cpufreq_perf_to_khz(cpu,
+						cpu->perf_caps.highest_perf);
+		policy->cpuinfo.nonboost_max_freq = cppc_cpufreq_perf_to_khz(cpu,
+						cpu->perf_caps.nominal_perf);
+
+		cpufreq_enable_boost_support();
+	}
+
 	/* Set policy->cur to max now. The governors will adjust later. */
 	policy->cur = cppc_cpufreq_perf_to_khz(cpu,
 					cpu->perf_caps.highest_perf);
-- 
1.7.12.4


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()'
  2020-05-08  9:11 ` [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()' Xiongfeng Wang
@ 2020-05-14 13:54   ` Rafael J. Wysocki
  2020-05-15  1:28     ` Xiongfeng Wang
  0 siblings, 1 reply; 10+ messages in thread
From: Rafael J. Wysocki @ 2020-05-14 13:54 UTC (permalink / raw)
  To: Xiongfeng Wang
  Cc: viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan, Sudeep.Holla,
	guohanjun, john.garry, jonathan.cameron, linux-pm, linux-kernel

On Friday, May 8, 2020 11:11:02 AM CEST Xiongfeng Wang wrote:
> When I try to add SW BOOST support for CPPC, I got the following error:
> cpufreq: cpufreq_boost_trigger_state: Cannot enable BOOST
> cpufreq: store_boost: Cannot enable BOOST!
> 
> It is because return value 1 of 'freq_qos_update_request()' means the
> effective constraint value has changed, not a error code on failures.
> But for 'cpufreq_driver.set_boost()', a nonzero return value means
> failure. So change 'ret' to zero when 'freq_qos_update_request()'
> returns a positive value.
> 
> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
> ---
>  drivers/cpufreq/cpufreq.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> index 4adac3a..475fb1b 100644
> --- a/drivers/cpufreq/cpufreq.c
> +++ b/drivers/cpufreq/cpufreq.c
> @@ -2522,6 +2522,8 @@ static int cpufreq_boost_set_sw(int state)
>  		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
>  		if (ret < 0)
>  			break;
> +		else
> +			ret = 0;
>  	}
>  
>  	return ret;
>

I would change cpufreq_boost_trigger_state() to take the 1 into account properly
instead.

Thanks!




^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table
  2020-05-08  9:11 ` [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table Xiongfeng Wang
@ 2020-05-14 14:16   ` Rafael J. Wysocki
  2020-05-15  1:49     ` Xiongfeng Wang
  0 siblings, 1 reply; 10+ messages in thread
From: Rafael J. Wysocki @ 2020-05-14 14:16 UTC (permalink / raw)
  To: Xiongfeng Wang
  Cc: viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan, Sudeep.Holla,
	guohanjun, john.garry, jonathan.cameron, linux-pm, linux-kernel

On Friday, May 8, 2020 11:11:03 AM CEST Xiongfeng Wang wrote:
> Software-managed BOOST get the boost frequency by check the flag
> CPUFREQ_BOOST_FREQ at driver's frequency table. But some cpufreq driver
> don't have frequency table and use other methods to get the frequency
> range, such CPPC cpufreq driver.
> 
> To add SW BOOST support for drivers without frequency table, we add
> members in 'cpufreq_policy.cpufreq_cpuinfo' to record the max frequency
> of boost mode and non-boost mode. The cpufreq driver initialize these two
> members when probing.
> 
> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
> ---
>  drivers/cpufreq/cpufreq.c | 23 +++++++++++++++--------
>  include/linux/cpufreq.h   |  2 ++
>  2 files changed, 17 insertions(+), 8 deletions(-)
> 
> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> index 475fb1b..a299426 100644
> --- a/drivers/cpufreq/cpufreq.c
> +++ b/drivers/cpufreq/cpufreq.c
> @@ -2508,15 +2508,22 @@ static int cpufreq_boost_set_sw(int state)
>  	int ret = -EINVAL;
>  
>  	for_each_active_policy(policy) {
> -		if (!policy->freq_table)
> -			continue;
> -
> -		ret = cpufreq_frequency_table_cpuinfo(policy,
> +		if (policy->freq_table) {
> +			ret = cpufreq_frequency_table_cpuinfo(policy,
>  						      policy->freq_table);
> -		if (ret) {
> -			pr_err("%s: Policy frequency update failed\n",
> -			       __func__);
> -			break;
> +			if (ret) {
> +				pr_err("%s: Policy frequency update failed\n",
> +				       __func__);
> +				break;
> +			}
> +		} else if (policy->cpuinfo.boost_max_freq) {
> +			if (state)
> +				policy->max = policy->cpuinfo.boost_max_freq;
> +			else
> +				policy->max = policy->cpuinfo.nonboost_max_freq;
> +			policy->cpuinfo.max_freq = policy->max;
> +		} else {
> +			continue;
>  		}

Why do you need to update this function?

The driver should be able to provide its own ->set_boost callback just fine,
shouldn't it?

>  
>  		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
> index 018dce8..c3449e6 100644
> --- a/include/linux/cpufreq.h
> +++ b/include/linux/cpufreq.h
> @@ -43,6 +43,8 @@ enum cpufreq_table_sorting {
>  struct cpufreq_cpuinfo {
>  	unsigned int		max_freq;
>  	unsigned int		min_freq;
> +	unsigned int		boost_max_freq;
> +	unsigned int		nonboost_max_freq;
>  
>  	/* in 10^(-9) s = nanoseconds */
>  	unsigned int		transition_latency;
> 





^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()'
  2020-05-14 13:54   ` Rafael J. Wysocki
@ 2020-05-15  1:28     ` Xiongfeng Wang
  0 siblings, 0 replies; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-15  1:28 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan, Sudeep.Holla,
	guohanjun, john.garry, jonathan.cameron, linux-pm, linux-kernel

Hi Rafael,

Thanks for your reply !

On 2020/5/14 21:54, Rafael J. Wysocki wrote:
> On Friday, May 8, 2020 11:11:02 AM CEST Xiongfeng Wang wrote:
>> When I try to add SW BOOST support for CPPC, I got the following error:
>> cpufreq: cpufreq_boost_trigger_state: Cannot enable BOOST
>> cpufreq: store_boost: Cannot enable BOOST!
>>
>> It is because return value 1 of 'freq_qos_update_request()' means the
>> effective constraint value has changed, not a error code on failures.
>> But for 'cpufreq_driver.set_boost()', a nonzero return value means
>> failure. So change 'ret' to zero when 'freq_qos_update_request()'
>> returns a positive value.
>>
>> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
>> ---
>>  drivers/cpufreq/cpufreq.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
>> index 4adac3a..475fb1b 100644
>> --- a/drivers/cpufreq/cpufreq.c
>> +++ b/drivers/cpufreq/cpufreq.c
>> @@ -2522,6 +2522,8 @@ static int cpufreq_boost_set_sw(int state)
>>  		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
>>  		if (ret < 0)
>>  			break;
>> +		else
>> +			ret = 0;
>>  	}
>>  
>>  	return ret;
>>
> 
> I would change cpufreq_boost_trigger_state() to take the 1 into account properly
> instead.

Thanks for your suggestion. I will change it in the next version.

Thanks,
Xiongfeng

> 
> Thanks!
> 
> 
> 
> 
> .
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table
  2020-05-14 14:16   ` Rafael J. Wysocki
@ 2020-05-15  1:49     ` Xiongfeng Wang
  2020-05-18  7:53       ` Viresh Kumar
  0 siblings, 1 reply; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-15  1:49 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: viresh.kumar, Souvik.Chakravarty, Thanu.Rangarajan, Sudeep.Holla,
	guohanjun, john.garry, jonathan.cameron, linux-pm, linux-kernel



On 2020/5/14 22:16, Rafael J. Wysocki wrote:
> On Friday, May 8, 2020 11:11:03 AM CEST Xiongfeng Wang wrote:
>> Software-managed BOOST get the boost frequency by check the flag
>> CPUFREQ_BOOST_FREQ at driver's frequency table. But some cpufreq driver
>> don't have frequency table and use other methods to get the frequency
>> range, such CPPC cpufreq driver.
>>
>> To add SW BOOST support for drivers without frequency table, we add
>> members in 'cpufreq_policy.cpufreq_cpuinfo' to record the max frequency
>> of boost mode and non-boost mode. The cpufreq driver initialize these two
>> members when probing.
>>
>> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
>> ---
>>  drivers/cpufreq/cpufreq.c | 23 +++++++++++++++--------
>>  include/linux/cpufreq.h   |  2 ++
>>  2 files changed, 17 insertions(+), 8 deletions(-)
>>
>> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
>> index 475fb1b..a299426 100644
>> --- a/drivers/cpufreq/cpufreq.c
>> +++ b/drivers/cpufreq/cpufreq.c
>> @@ -2508,15 +2508,22 @@ static int cpufreq_boost_set_sw(int state)
>>  	int ret = -EINVAL;
>>  
>>  	for_each_active_policy(policy) {
>> -		if (!policy->freq_table)
>> -			continue;
>> -
>> -		ret = cpufreq_frequency_table_cpuinfo(policy,
>> +		if (policy->freq_table) {
>> +			ret = cpufreq_frequency_table_cpuinfo(policy,
>>  						      policy->freq_table);
>> -		if (ret) {
>> -			pr_err("%s: Policy frequency update failed\n",
>> -			       __func__);
>> -			break;
>> +			if (ret) {
>> +				pr_err("%s: Policy frequency update failed\n",
>> +				       __func__);
>> +				break;
>> +			}
>> +		} else if (policy->cpuinfo.boost_max_freq) {
>> +			if (state)
>> +				policy->max = policy->cpuinfo.boost_max_freq;
>> +			else
>> +				policy->max = policy->cpuinfo.nonboost_max_freq;
>> +			policy->cpuinfo.max_freq = policy->max;
>> +		} else {
>> +			continue;
>>  		}
> 
> Why do you need to update this function?

My original thought is to reuse the current SW BOOST code as possible, but this
seems to change the cpufreq core too much.

> 
> The driver should be able to provide its own ->set_boost callback just fine,
> shouldn't it?

Thanks for your advice. This is better. I will provide a '->set_boost' callback
for CPPC driver. But I will need to export 'cpufreq_policy_list' and make the
macro 'for_each_active_policy' public.

Thanks,
Xiongfeng

> 
>>  
>>  		ret = freq_qos_update_request(policy->max_freq_req, policy->max);
>> diff --git a/include/linux/cpufreq.h b/include/linux/cpufreq.h
>> index 018dce8..c3449e6 100644
>> --- a/include/linux/cpufreq.h
>> +++ b/include/linux/cpufreq.h
>> @@ -43,6 +43,8 @@ enum cpufreq_table_sorting {
>>  struct cpufreq_cpuinfo {
>>  	unsigned int		max_freq;
>>  	unsigned int		min_freq;
>> +	unsigned int		boost_max_freq;
>> +	unsigned int		nonboost_max_freq;
>>  
>>  	/* in 10^(-9) s = nanoseconds */
>>  	unsigned int		transition_latency;
>>
> 
> 
> 
> 
> 
> .
> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table
  2020-05-15  1:49     ` Xiongfeng Wang
@ 2020-05-18  7:53       ` Viresh Kumar
  2020-05-19  1:04         ` Xiongfeng Wang
  0 siblings, 1 reply; 10+ messages in thread
From: Viresh Kumar @ 2020-05-18  7:53 UTC (permalink / raw)
  To: Xiongfeng Wang
  Cc: Rafael J. Wysocki, Souvik.Chakravarty, Thanu.Rangarajan,
	Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel

Sorry for the delay from my side in replying to this thread.

On 15-05-20, 09:49, Xiongfeng Wang wrote:
> On 2020/5/14 22:16, Rafael J. Wysocki wrote:
> > On Friday, May 8, 2020 11:11:03 AM CEST Xiongfeng Wang wrote:
> >> Software-managed BOOST get the boost frequency by check the flag
> >> CPUFREQ_BOOST_FREQ at driver's frequency table. But some cpufreq driver
> >> don't have frequency table and use other methods to get the frequency
> >> range, such CPPC cpufreq driver.
> >>
> >> To add SW BOOST support for drivers without frequency table, we add
> >> members in 'cpufreq_policy.cpufreq_cpuinfo' to record the max frequency
> >> of boost mode and non-boost mode. The cpufreq driver initialize these two
> >> members when probing.
> >>
> >> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
> >> ---
> >>  drivers/cpufreq/cpufreq.c | 23 +++++++++++++++--------
> >>  include/linux/cpufreq.h   |  2 ++
> >>  2 files changed, 17 insertions(+), 8 deletions(-)
> >>
> >> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> >> index 475fb1b..a299426 100644
> >> --- a/drivers/cpufreq/cpufreq.c
> >> +++ b/drivers/cpufreq/cpufreq.c
> >> @@ -2508,15 +2508,22 @@ static int cpufreq_boost_set_sw(int state)
> >>  	int ret = -EINVAL;
> >>  
> >>  	for_each_active_policy(policy) {
> >> -		if (!policy->freq_table)
> >> -			continue;
> >> -
> >> -		ret = cpufreq_frequency_table_cpuinfo(policy,
> >> +		if (policy->freq_table) {
> >> +			ret = cpufreq_frequency_table_cpuinfo(policy,
> >>  						      policy->freq_table);
> >> -		if (ret) {
> >> -			pr_err("%s: Policy frequency update failed\n",
> >> -			       __func__);
> >> -			break;
> >> +			if (ret) {
> >> +				pr_err("%s: Policy frequency update failed\n",
> >> +				       __func__);
> >> +				break;
> >> +			}
> >> +		} else if (policy->cpuinfo.boost_max_freq) {
> >> +			if (state)
> >> +				policy->max = policy->cpuinfo.boost_max_freq;
> >> +			else
> >> +				policy->max = policy->cpuinfo.nonboost_max_freq;
> >> +			policy->cpuinfo.max_freq = policy->max;
> >> +		} else {
> >> +			continue;
> >>  		}
> > 
> > Why do you need to update this function?
> 
> My original thought is to reuse the current SW BOOST code as possible, but this
> seems to change the cpufreq core too much.
> 
> Thanks for your advice. This is better. I will provide a '->set_boost' callback
> for CPPC driver. But I will need to export 'cpufreq_policy_list' and make the
> macro 'for_each_active_policy' public.

This can and should be avoided, I will rather move the for-each-policy
loop in cpufreq_boost_trigger_state() and call ->set_boost() for each
policy and pass policy as argument as well. You would be required to
update existing users of sw boost.

-- 
viresh

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table
  2020-05-18  7:53       ` Viresh Kumar
@ 2020-05-19  1:04         ` Xiongfeng Wang
  0 siblings, 0 replies; 10+ messages in thread
From: Xiongfeng Wang @ 2020-05-19  1:04 UTC (permalink / raw)
  To: Viresh Kumar
  Cc: Rafael J. Wysocki, Souvik.Chakravarty, Thanu.Rangarajan,
	Sudeep.Holla, guohanjun, john.garry, jonathan.cameron, linux-pm,
	linux-kernel

Hi Viresh,

Thanks for your reply !

On 2020/5/18 15:53, Viresh Kumar wrote:
> Sorry for the delay from my side in replying to this thread.
> 
> On 15-05-20, 09:49, Xiongfeng Wang wrote:
>> On 2020/5/14 22:16, Rafael J. Wysocki wrote:
>>> On Friday, May 8, 2020 11:11:03 AM CEST Xiongfeng Wang wrote:
>>>> Software-managed BOOST get the boost frequency by check the flag
>>>> CPUFREQ_BOOST_FREQ at driver's frequency table. But some cpufreq driver
>>>> don't have frequency table and use other methods to get the frequency
>>>> range, such CPPC cpufreq driver.
>>>>
>>>> To add SW BOOST support for drivers without frequency table, we add
>>>> members in 'cpufreq_policy.cpufreq_cpuinfo' to record the max frequency
>>>> of boost mode and non-boost mode. The cpufreq driver initialize these two
>>>> members when probing.
>>>>
>>>> Signed-off-by: Xiongfeng Wang <wangxiongfeng2@huawei.com>
>>>> ---
>>>>  drivers/cpufreq/cpufreq.c | 23 +++++++++++++++--------
>>>>  include/linux/cpufreq.h   |  2 ++
>>>>  2 files changed, 17 insertions(+), 8 deletions(-)
>>>>
>>>> diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
>>>> index 475fb1b..a299426 100644
>>>> --- a/drivers/cpufreq/cpufreq.c
>>>> +++ b/drivers/cpufreq/cpufreq.c
>>>> @@ -2508,15 +2508,22 @@ static int cpufreq_boost_set_sw(int state)
>>>>  	int ret = -EINVAL;
>>>>  
>>>>  	for_each_active_policy(policy) {
>>>> -		if (!policy->freq_table)
>>>> -			continue;
>>>> -
>>>> -		ret = cpufreq_frequency_table_cpuinfo(policy,
>>>> +		if (policy->freq_table) {
>>>> +			ret = cpufreq_frequency_table_cpuinfo(policy,
>>>>  						      policy->freq_table);
>>>> -		if (ret) {
>>>> -			pr_err("%s: Policy frequency update failed\n",
>>>> -			       __func__);
>>>> -			break;
>>>> +			if (ret) {
>>>> +				pr_err("%s: Policy frequency update failed\n",
>>>> +				       __func__);
>>>> +				break;
>>>> +			}
>>>> +		} else if (policy->cpuinfo.boost_max_freq) {
>>>> +			if (state)
>>>> +				policy->max = policy->cpuinfo.boost_max_freq;
>>>> +			else
>>>> +				policy->max = policy->cpuinfo.nonboost_max_freq;
>>>> +			policy->cpuinfo.max_freq = policy->max;
>>>> +		} else {
>>>> +			continue;
>>>>  		}
>>>
>>> Why do you need to update this function?
>>
>> My original thought is to reuse the current SW BOOST code as possible, but this
>> seems to change the cpufreq core too much.
>>
>> Thanks for your advice. This is better. I will provide a '->set_boost' callback
>> for CPPC driver. But I will need to export 'cpufreq_policy_list' and make the
>> macro 'for_each_active_policy' public.
> 
> This can and should be avoided, I will rather move the for-each-policy
> loop in cpufreq_boost_trigger_state() and call ->set_boost() for each
> policy and pass policy as argument as well. You would be required to
> update existing users of sw boost.

Thanks for your advice. It's a good idea. I will change it in the next version.

Thanks,
Xiongfeng

> 


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2020-05-19  1:04 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-05-08  9:11 [RFC PATCH 0/3] add SW BOOST support for CPPC Xiongfeng Wang
2020-05-08  9:11 ` [RFC PATCH 1/3] cpufreq: fix the return value in 'cpufreq_boost_set_sw()' Xiongfeng Wang
2020-05-14 13:54   ` Rafael J. Wysocki
2020-05-15  1:28     ` Xiongfeng Wang
2020-05-08  9:11 ` [RFC PATCH 2/3] cpufreq: Add SW BOOST support for drivers without frequency table Xiongfeng Wang
2020-05-14 14:16   ` Rafael J. Wysocki
2020-05-15  1:49     ` Xiongfeng Wang
2020-05-18  7:53       ` Viresh Kumar
2020-05-19  1:04         ` Xiongfeng Wang
2020-05-08  9:11 ` [RFC PATCH 3/3] CPPC: add support for SW BOOST Xiongfeng Wang

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).