Re: [PATCH 2/3] opp: Allocate the OPP table outside of opp_table_lock

2020-11-05 Thread Viresh Kumar
On 02-11-20, 10:51, Viresh Kumar wrote: > diff --git a/drivers/opp/core.c b/drivers/opp/core.c > +/* > + * We need to make sure that the OPP table for a device doesn't get added > twice, > + * if this routine gets called in parallel with the same device pointer. > + * > + * The simplest way to enf

[PATCH 2/3] opp: Allocate the OPP table outside of opp_table_lock

2020-11-01 Thread Viresh Kumar
There is no critical section which needs protection with locks while allocating an OPP table, except while adding it to the opp_tables list. And taking the opp_table_lock for the entire duration causes circular dependency issues like the one mentioned below. This patch takes another approach to re