On Thursday, September 03, 2015 10:14:54 AM Viresh Kumar wrote:
On 27-07-15, 17:58, Viresh Kumar wrote:
Hi Rafael,
This series fixes few more possible race conditions. Over that there is some non-trivial cleanup, in order to simplify code.
Preeti did review some of them before she left and shared concerns on others, all that is sorted out now.
V1->V2:
- Dropped 2/10 from V1 as it wasn't required
- 3/10 saw some changes due to above patch being dropped
- 7/10 changed a bit as we check for pending work items by looking at shared->policy, rather than calling delayed_work_pending. We wanted to check if governor is operational or not and the new check is enough for that.
Viresh Kumar (9): cpufreq: Use __func__ to print function's name cpufreq: conservative: remove 'enable' field cpufreq: ondemand: only queue canceled works from update_sampling_rate() cpufreq: governor: Drop __gov_queue_work() cpufreq: ondemand: Drop unnecessary locks from update_sampling_rate() cpufreq: ondemand: queue work for policy->cpus together cpufreq: ondemand: update sampling rate immidiately cpufreq: governor: Quit work-handlers early if governor is stopped cpufreq: Get rid of ->governor_enabled and its lock
I thought you had some review comment for this series?
Yes, I did. I'll send them later today if all goes well.
Thanks, Rafael