tree ce42ca70ca770e81794054c507d1837b0810cd18
parent 994a8f2514e91c16616c4a1b53e9eb2b24de97b7
author Viresh Kumar <viresh.kumar@linaro.org> 1489032294 +0530
committer Rafael J. Wysocki <rafael.j.wysocki@intel.com> 1489356693 +0100

cpufreq: schedutil: Refactor sugov_next_freq_shared()

The loop in sugov_next_freq_shared() contains an if block to skip the
loop for the current CPU. This turns out to be an unnecessary
conditional in the scheduler's hot-path for every CPU in the policy.

It would be better to drop the conditional and make the loop treat all
the CPUs in the same way. That would eliminate the need of calling
sugov_iowait_boost() at the top of the routine.

To keep the code optimized to return early if the current CPU has RT/DL
flags set, move the flags check to sugov_update_shared() instead in
order to avoid the function call entirely.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>