[coreboot-gerrit] Patch set updated for coreboot: cpu/x86/mtrr/mtrr: Add MTRR index and total MTRRs to error message

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Thu Oct 15 12:57:52 CEST 2015


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11905

-gerrit

commit 951c2d9b9be25337323f10636474c0c6618cacc0
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Thu Oct 15 12:41:53 2015 +0200

    cpu/x86/mtrr/mtrr: Add MTRR index and total MTRRs to error message
    
    Change-Id: I626a11c17c9d05c174c507d50684e498c8604cbc
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/cpu/x86/mtrr/mtrr.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c
index 072b887..2015462 100644
--- a/src/cpu/x86/mtrr/mtrr.c
+++ b/src/cpu/x86/mtrr/mtrr.c
@@ -429,8 +429,9 @@ static void prep_var_mtrr(struct var_mtrr_state *var_state,
 	if (var_state->mtrr_index >= bios_mtrrs)
 		printk(BIOS_WARNING, "Taking a reserved OS MTRR.\n");
 	if (var_state->mtrr_index >= total_mtrrs) {
-		printk(BIOS_ERR, "ERROR: Not enough MTRRs available!\n");
-		return;
+		printk(BIOS_ERR, "ERROR: Not enough MTRRs available! MTRR index"
+		       "is %d with %d MTTRs in total.\n",
+		       var_state->mtrr_index, total_mtrrs); return;
 	}
 
 	rbase = base;



More information about the coreboot-gerrit mailing list