On 23-06-20, 15:21, Quentin Perret wrote:
> @@ -2789,7 +2796,13 @@ static int __init cpufreq_core_init(void)
>       cpufreq_global_kobject = kobject_create_and_add("cpufreq", 
> &cpu_subsys.dev_root->kobj);
>       BUG_ON(!cpufreq_global_kobject);
>  
> +     mutex_lock(&cpufreq_governor_mutex);
> +     if (!default_governor)

Also is this check really required ? The pointer will always be NULL
at this point, isn't it ?

> +             default_governor = cpufreq_default_governor();
> +     mutex_unlock(&cpufreq_governor_mutex);
> +
>       return 0;
>  }
>  module_param(off, int, 0444);
> +module_param_string(default_governor, cpufreq_param_governor, 
> CPUFREQ_NAME_LEN, 0444);
>  core_initcall(cpufreq_core_init);
> -- 
> 2.27.0.111.gc72c7da667-goog

-- 
viresh

Reply via email to