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

From: Rafael J. Wysocki
Date: Thu Feb 04 2016 - 22:33:22 EST


On Fri, Feb 5, 2016 at 4:24 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> On 05-02-16, 04:17, Rafael J. Wysocki wrote:
>> And don't we switch governors under policy->rwsem anyway?
>
> So ? That is blocking only a single policy only, but with the new
> change, we will block all policies from doing that concurrently.

No, it won't. Again: one lock instead of two. How much of a
difference this makes performance-wise?

And the price is the stupid dance we need to do to even get to those
locks! Come on.

Thanks,
Rafael