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