tree 697e55f1240115ef857aa6d791f5bd1f91fefe1f
parent 5ef5c10be17e6bde3a448f859963a67d8cd9e7e8
author Venkatesh Srinivas <venkateshs@google.com> 1560834880 +0000
committer CQ bot account: commit-bot@chromium.org <commit-bot@chromium.org> 1560834880 +0000

[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