On 16-01-26, 15:52, Frederic Weisbecker wrote:
> cpufreq calls get_cpu_idle_time_us() just to know if idle cputime
> accounting has a nanoseconds granularity.
> 
> Use the appropriate indicator instead to make that deduction.
> 
> Signed-off-by: Frederic Weisbecker <[email protected]>
> ---
>  drivers/cpufreq/cpufreq_ondemand.c | 7 +------
>  1 file changed, 1 insertion(+), 6 deletions(-)
> 
> diff --git a/drivers/cpufreq/cpufreq_ondemand.c 
> b/drivers/cpufreq/cpufreq_ondemand.c
> index a6ecc203f7b7..2d52ee035702 100644
> --- a/drivers/cpufreq/cpufreq_ondemand.c
> +++ b/drivers/cpufreq/cpufreq_ondemand.c
> @@ -334,17 +334,12 @@ static void od_free(struct policy_dbs_info *policy_dbs)
>  static int od_init(struct dbs_data *dbs_data)
>  {
>       struct od_dbs_tuners *tuners;
> -     u64 idle_time;
> -     int cpu;
>  
>       tuners = kzalloc(sizeof(*tuners), GFP_KERNEL);
>       if (!tuners)
>               return -ENOMEM;
>  
> -     cpu = get_cpu();
> -     idle_time = get_cpu_idle_time_us(cpu, NULL);
> -     put_cpu();
> -     if (idle_time != -1ULL) {
> +     if (tick_nohz_enabled) {
>               /* Idle micro accounting is supported. Use finer thresholds */
>               dbs_data->up_threshold = MICRO_FREQUENCY_UP_THRESHOLD;
>       } else {

Acked-by: Viresh Kumar <[email protected]>

-- 
viresh

Reply via email to