[coreboot-gerrit] Change in coreboot[master]: x86/mtrr: Print address ranges inclusive to be more consistent

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/63541 )

Change subject: x86/mtrr: Print address ranges inclusive to be more consistent
......................................................................

x86/mtrr: Print address ranges inclusive to be more consistent

The printed address ranges in the tree (resource allocator and even
some MTRR code) usually shows the range inclusive (meaning from start
address to the real end address of the range). Though there is still
some code in the MTRR context which prints the ranges with an exclusive
end. This patch aligns the printing of ranges in the MTRR code to be
consistent among the tree so that the shown end addresses are now
inclusive.

Change-Id: I0ca292f9cf272564cb5ef1c4ea38f5c483605c94
Signed-off-by: Werner Zeh 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/63541
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Subrata Banik 
Reviewed-by: Eric Lai 
Reviewed-by: Christian Walter 
---
M src/arch/x86/postcar_loader.c
M src/cpu/x86/mtrr/mtrr.c
2 files changed, 4 insertions(+), 4 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, approved
  Christian Walter: Looks good to me, approved
  Subrata Banik: Looks good to me, approved
  Eric Lai: Looks good to me, approved

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);
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c
index 288f06d..d8903b8 100644
--- a/src/cpu/x86/mtrr/mtrr.c
+++ b/src/cpu/x86/mtrr/mtrr.c
@@ -154,7 +154,7 @@
    memranges_each_entry(r, addr_space)
    	printk(BIOS_DEBUG,
    	       "0x%016llx - 0x%016llx size 0x%08llx type %ld\n",
-		       range_entry_base(r), range_entry_end(r),
+		       range_entry_base(r), range_entry_end(r) - 1,
    	       range_entry_size(r), range_entry_tag(r));
 }

@@ -272,7 +272,7 @@
    		type = range_entry_tag(r);
    		printk(MTRR_VERBOSE_LEVEL,
    		       "MTRR addr 0x%x-0x%x set to %d type @ %d\n",
-			       begin, begin + desc->step, type, type_index);
+			       begin, begin + desc->step - 1, type, type_index);
    		if (type == MTRR_TYPE_WRBACK)
    			type |= MTRR_FIXED_WRBACK_BITS;
    		fixed_mtrr_types[type_index] = type;
@@ -905,7 +905,7 @@

if (commit_var_mtrrs(&sol) < 0)
    	printk(BIOS_WARNING, "Unable to insert temporary MTRR range: 0x%016llx - 0x%016llx size 0x%08llx type %d\n",
-			(long long)begin, (long long)begin + size,
+			(long long)begin, (long long)begin + size - 1,
    		(long long)size, type);
    else
    	need_restore_mtrr();