From: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
Date: Tue, 14 Nov 2006 01:47:44 +0000 (-0800)
Subject: [CPUFREQ] Fix the bug in duplicate freq elimination code in acpi-cpufreq
X-Git-Tag: v3.12-rc1~32419
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3d4a7ef3d3f0c9f3cf8da724d5935253a713d8b6;p=kernel%2Fkernel-generic.git

[CPUFREQ] Fix the bug in duplicate freq elimination code in acpi-cpufreq

Fix the bug in duplicate states elimination in acpi-cpufreq.

Bug: Due to duplicate state elimiation in the loop earlier, the number
of valid_states can be less than perf->state_count, in which case
freq_table was ending up with some garbage/uninitialized entries
in the table.

Signed-off-by: Venkatesh Pallipadi <venkatesh.pallipadi@intel.com>
From:  Alexey Starikovskiy <alexey.y.starikovskiy@intel.com>
Signed-off-by: Dave Jones <davej@redhat.com>
---

diff --git a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c b/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
index 60d20cf..18f4715 100644
--- a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
+++ b/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
@@ -693,7 +693,7 @@ static int acpi_cpufreq_cpu_init(struct cpufreq_policy *policy)
 		    perf->states[i].core_frequency * 1000;
 		valid_states++;
 	}
-	data->freq_table[perf->state_count].frequency = CPUFREQ_TABLE_END;
+	data->freq_table[valid_states].frequency = CPUFREQ_TABLE_END;
 
 	result = cpufreq_frequency_table_cpuinfo(policy, data->freq_table);
 	if (result)