[kernel][x86] Remove printf in early microarch selection code
Early in Fuchsia boot, features.cpp caches and partly decodes CPUID
information; it uses Intel/AMD Family/Model/Stepping to select a
'microarch' config table, with information about C1E promotion,
timer information, and shutdown information.
If an unknown CPU family was found, Fuchsia would attempt to printf;
unfortunately printf() that early in boot, at least on the NUC6, locks
up. The NUC6CAYH has an Intel 'Goldmont' CPU, which is not matched by
the existing Silvermont entry; it is the only system I had access to
that triggered this bug.
Tested:
* Added debug tracing in lk_main(); once this printf() is removed, we
see lk_main() proceed to last init level.
ZX-4201 Fuchsia should run on Intel NUC6CAYH
Change-Id: Ia206a68a044007a645aaf9b29cf39f70706e1d72
diff --git a/zircon/kernel/arch/x86/feature.cpp b/zircon/kernel/arch/x86/feature.cpp
index 24bd84a..878f5c9 100644
--- a/zircon/kernel/arch/x86/feature.cpp
+++ b/zircon/kernel/arch/x86/feature.cpp
@@ -737,8 +737,6 @@
case X86_MICROARCH_AMD_ZEN:
return &zen_config;
case X86_MICROARCH_UNKNOWN: {
- printf("WARNING: Could not identify microarch.\n");
- printf("Please file a bug with your boot log and description of hardware.\n");
switch (x86_vendor) {
case X86_VENDOR_INTEL:
return &intel_default_config;