On 2 April 2013 01:41, Nathan Zimmer <nzim...@sgi.com> wrote: > diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c
> +static struct cpufreq_driver __rcu *cpufreq_driver; > +static DEFINE_SPINLOCK(cpufreq_driver_lock); You really need this lock? This is only used in cpufreq_register_driver and unregister_driver... And it doesn't protect other routines at all. And because we are using rcu stuff now, probably this lock is just not required. > +static DEFINE_SPINLOCK(cpufreq_data_lock); Only this one is required and it can be the rwlock which is already pushed by rafael. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
