diff options
author | Felix Held <felix-coreboot@felixheld.de> | 2022-12-14 23:07:52 +0100 |
---|---|---|
committer | Martin L Roth <gaumless@gmail.com> | 2022-12-15 03:10:08 +0000 |
commit | 447f5777aae39600af421b230d6ab22a18b96e84 (patch) | |
tree | 21ae595e3ba38af3ba1458c2ff1f43cfa5a71dfe /src/cpu/x86/mtrr/debug.c | |
parent | 2cf2bd81974722114b74e1880425b9921f38a7f9 (diff) |
cpu/x86/mtrr: use lapicid instead of cpu_index calls
The cpu_index function can't be used before mpinit, so use lapicid calls
instead. This fixes the regression introduced by commit 4c3749884d71
("cpu/x86/mtrr: Print cpu index number when set up MTRRs for BSP/APs")
and also reverts also commit b3261661c703 ("cpu/x86/mtrr/mtrr: fix
printk format strings"), since lapicid returns an unsigned int while
cpu_index returns an unsigned long.
TEST=Mandolin boots again and doesn't fail when it first tries to print
the MTRR configuration
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Change-Id: I0d226704051ab171891775a618ce7897b74fde16
Reviewed-on: https://review.coreboot.org/c/coreboot/+/70797
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Martin Roth <martin.roth@amd.corp-partner.google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Fred Reitberger <reitbergerfred@gmail.com>
Reviewed-by: Matt DeVillier <matt.devillier@amd.corp-partner.google.com>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>
Diffstat (limited to 'src/cpu/x86/mtrr/debug.c')
0 files changed, 0 insertions, 0 deletions