Commit 4913af9b authored Feb 08, 2016 by Hanumath Prasad

cpufreq: interactive: use mult_frac in sl_busy_to_laf



Currently the load argument is taken as unsigned long in
sl_busy_to_laf. In case of 32-bit kernels, the use of
unsigned long results in overflows since it is only 32-bits.
And so the cpu_load calculation is going wrong and most of
the times getting reported very low values. Hence use
mult_frac call to avoid overflows when the final result
is expected to be within 32-bits.

Change-Id: Ib9e8bf6e777cd07b141761fb14c80840563b4cd5
Signed-off-by: Hanumath Prasad <hpprasad@codeaurora.org>

parent 1c10389d

drivers/cpufreq/cpufreq_interactive.c

+4 −2

Original line number	Diff line number	Diff line
		@@ -442,11 +442,13 @@ static u64 update_load(int cpu)
		static unsigned int sl_busy_to_laf(struct cpufreq_interactive_policyinfo *ppol,
		unsigned long busy)
		{
		int prev_load;
		struct cpufreq_interactive_tunables *tunables =
		ppol->policy->governor_data;

		busy *= ppol->policy->cpuinfo.max_freq;
		return div64_s64(busy, tunables->timer_rate) * 100;
		prev_load = mult_frac(ppol->policy->cpuinfo.max_freq * 100,
		busy, tunables->timer_rate);
		return prev_load;
		}

		#define NEW_TASK_RATIO 75