[coreboot-gerrit] Patch set updated for coreboot: cpu/x86/lapic: Add stack overrun detection

Timothy Pearson (tpearson@raptorengineeringinc.com) gerrit at coreboot.org
Sat Oct 31 02:49:06 CET 2015


Timothy Pearson (tpearson at raptorengineeringinc.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11963

-gerrit

commit 22808eac2251275037c8a6e8ded2db295d6a9c12
Author: Timothy Pearson <tpearson at raptorengineeringinc.com>
Date:   Sat Sep 5 19:31:20 2015 -0500

    cpu/x86/lapic: Add stack overrun detection
    
    Change-Id: I03e43f38e0d2e51141208ebb169ad8deba77ab78
    Signed-off-by: Timothy Pearson <tpearson at raptorengineeringinc.com>
---
 src/cpu/x86/lapic/lapic_cpu_init.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/cpu/x86/lapic/lapic_cpu_init.c b/src/cpu/x86/lapic/lapic_cpu_init.c
index 7fedd00..faa1f1f 100644
--- a/src/cpu/x86/lapic/lapic_cpu_init.c
+++ b/src/cpu/x86/lapic/lapic_cpu_init.c
@@ -5,6 +5,7 @@
  * Copyright (C) 2001 Ronald G. Minnich
  * Copyright (C) 2005 Yinghai Lu
  * Copyright (C) 2008 coresystems GmbH
+ * Copyright (C) 2015 Timothy Pearson <tpearson at raptorengineeringinc.com>, Raptor Engineering
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -494,6 +495,7 @@ static void wait_other_cpus_stop(struct bus *cpu_bus)
 		}
 	}
 	printk(BIOS_DEBUG, "All AP CPUs stopped (%ld loops)\n", loopcount);
+	checkstack(_estack, 0);
 	for(i = 1; i <= last_cpu_index; i++)
 		checkstack((void *)stacks[i] + CONFIG_STACK_SIZE, i);
 }



More information about the coreboot-gerrit mailing list