i386 could do something similiar but it currently sets up
update_cpuspeed callbacks early and one of these is called from piixpcib...

This will hopefully reduce the number of people claiming
their processor is in some way not working because of a printf
in the bus clock code.

Index: est.c
===================================================================
RCS file: /cvs/src/sys/arch/amd64/amd64/est.c,v
retrieving revision 1.27
diff -u -p -r1.27 est.c
--- est.c       19 Mar 2012 00:49:08 -0000      1.27
+++ est.c       19 Mar 2012 01:15:27 -0000
@@ -341,10 +341,14 @@ est_init(struct cpu_info *ci)
 #if NACPICPU > 0
        est_fqlist = est_acpi_init();
 #endif
-       if (ci->ci_family == 0xf) {
-               p4_get_bus_clock(ci);
-       } else if (ci->ci_family == 6) {
-               p3_get_bus_clock(ci);
+
+       /* bus_clock is only used if we can't get values from ACPI */
+       if (est_fqlist == NULL) {
+               if (ci->ci_family == 0xf) {
+                       p4_get_bus_clock(ci);
+               } else if (ci->ci_family == 6) {
+                       p3_get_bus_clock(ci);
+               }
        }
 
        /*

Reply via email to