On 27 December 2013 08:36, Jane Li <jiel@xxxxxxxxxxx> wrote: > When gov_queue_work(), governor_enabled may be modified. Following patch > can fix it by adding cpufreq_governor_lock in gov_queue_work. But in this > way, cpufreq_governor_lock also protects __gov_queue_work(). Do you think > this is a good idea? gov_queue_work() is called with timer_mutex on. Can you try placing this lock around gov_cancel_work() and see if it resolves issues you have reported ? -- viresh -- To unsubscribe from this list: send the line "unsubscribe cpufreq" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html