aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVenkatesh Pallipadi <venkatesh.pallipadi@intel.com>2006-11-13 20:47:44 -0500
committerDave Jones <davej@redhat.com>2006-12-12 17:20:50 -0500
commit3d4a7ef3d3f0c9f3cf8da724d5935253a713d8b6 (patch)
tree5ac8313702b6495171758763eb7bddd6d1b6f81a
parent8b9c6671f8dbedbd071a9a6c787d4086a8990db4 (diff)
[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>
-rw-r--r--arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c b/arch/i386/kernel/cpu/cpufreq/acpi-cpufreq.c
index 60d20cf427aa..18f4715c655d 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)
693 perf->states[i].core_frequency * 1000; 693 perf->states[i].core_frequency * 1000;
694 valid_states++; 694 valid_states++;
695 } 695 }
696 data->freq_table[perf->state_count].frequency = CPUFREQ_TABLE_END; 696 data->freq_table[valid_states].frequency = CPUFREQ_TABLE_END;
697 697
698 result = cpufreq_frequency_table_cpuinfo(policy, data->freq_table); 698 result = cpufreq_frequency_table_cpuinfo(policy, data->freq_table);
699 if (result) 699 if (result)