From 5ec55931d64b899de2c202ee7fa901c76925a5c8 Mon Sep 17 00:00:00 2001 From: Alexander Motin Date: Sat, 19 Jun 2010 13:09:42 +0000 Subject: [PATCH] Core i5, same as previously Core2Duo, found to not set P-state for single core lower then set on other cores. Do not try to test P-states on attach on SMP systems. It is hopeless now and will just pollute verbose logs. If needed, check still can be forced via loader tunable. --- sys/x86/cpufreq/est.c | 8 ++------ 1 file changed, 2 insertions(+), 6 deletions(-) diff --git a/sys/x86/cpufreq/est.c b/sys/x86/cpufreq/est.c index 6a7b514a2c37..144858c94517 100644 --- a/sys/x86/cpufreq/est.c +++ b/sys/x86/cpufreq/est.c @@ -1123,16 +1123,12 @@ est_acpi_info(device_t dev, freq_info **freqs) * Confirm id16 value is correct. */ if (sets[i].freq > 0) { - error = est_set_id16(dev, sets[i].spec[0], 1); - if (error != 0 && strict) { + error = est_set_id16(dev, sets[i].spec[0], strict); + if (error != 0) { if (bootverbose) device_printf(dev, "Invalid freq %u, " "ignored.\n", sets[i].freq); continue; - } else if (error != 0 && bootverbose) { - device_printf(dev, "Can't check freq %u, " - "it may be invalid\n", - sets[i].freq); } table[j].freq = sets[i].freq; table[j].volts = sets[i].volts;