Re: [PATCH v2 2/10] cpufreq: governor: Use common mutex for dbs_data protection

[Viresh Kumar]

On 05-02-16, 03:14, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> 
> Every governor relying on the common code in cpufreq_governor.c
> has to provide its own mutex in struct common_dbs_data.  However,
> there actually is no need to have a separate mutex per governor
> for this purpose, they may be using the same global mutex just
> fine.  Accordingly, introduce a single common mutex for that and
> drop the mutex field from struct common_dbs_data.
> 
> That at least will ensure that the mutex is always present and
> initialized regardless of what the particular governors do.
> 
> Another benefit is that the common code does not need a pointer to
> a governor-related structure to get to the mutex which sometimes
> helps.
> 
> Finally, it makes the code generally easier to follow.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> Acked-by: Saravana Kannan <skannan@xxxxxxxxxxxxxx>

Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

-- 
viresh