Werner Zeh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63541 )
Change subject: arch/x86/postcar_loader: Fix printed MTRR range computation ......................................................................
arch/x86/postcar_loader: Fix printed MTRR range computation
There is a log message in 'postcar_var_mtrr_set()' showing the used variable MTRRs which prints the start and the end address as well as the size of the MTRR region. Instead of the end address the next address after the end is computed which leads to a wrong log output.
This patch uses 'addr + size - 1' instead of 'addr + size' to compute the end address correctly.
Change-Id: I0ca292f9cf272564cb5ef1c4ea38f5c483605c94 Signed-off-by: Werner Zeh werner.zeh@siemens.com --- M src/arch/x86/postcar_loader.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/63541/1
diff --git a/src/arch/x86/postcar_loader.c b/src/arch/x86/postcar_loader.c index 2609fd6..0fe174b 100644 --- a/src/arch/x86/postcar_loader.c +++ b/src/arch/x86/postcar_loader.c @@ -60,7 +60,7 @@ struct postcar_frame *pcf = ctx->arg;
printk(BIOS_DEBUG, "MTRR Range: Start=%lx End=%lx (Size %zx)\n", - addr, addr + size, size); + addr, addr + size - 1, size);
stack_push(pcf, mask.hi); stack_push(pcf, mask.lo);