[coreboot-gerrit] Patch set updated for coreboot: [DON'T MERGE] Investigating CL #16987

Jonathan Neuschäfer (j.neuschaefer@gmx.net) gerrit at coreboot.org
Wed Oct 19 21:36:19 CEST 2016


Jonathan Neuschäfer (j.neuschaefer at gmx.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/17060

-gerrit

commit 1cd2d1ea10b5f350fbb692523e7dab90586d1c23
Author: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
Date:   Wed Oct 19 02:18:54 2016 +0200

    [DON'T MERGE] Investigating CL #16987
    
    https://review.coreboot.org/#/c/16987/ results in a crash in AMDFWTOOL
    for some tools, on jenkins. With this patch, I try to figure out what's
    going on.
    
    Change-Id: Ic36b8be2b0ee458311dcc8e9139df2b4d113b4ba
    Signed-off-by: Jonathan Neuschäfer <j.neuschaefer at gmx.net>
---
 Makefile.inc                    | 2 +-
 src/mainboard/emulation/Kconfig | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/Makefile.inc b/Makefile.inc
index 127f372..1eb4b02 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -487,7 +487,7 @@ $(IFDFAKE): $(top)/util/ifdfake/ifdfake.c
 AMDFWTOOL:=$(objutil)/amdfwtool/amdfwtool
 $(AMDFWTOOL): $(top)/util/amdfwtool/amdfwtool.c
 	@printf "    HOSTCC     $(subst $(obj)/,,$(@))\n"
-	$(HOSTCC) $(HOSTCFLAGS) -DCONFIG_ROM_SIZE=$(CONFIG_ROM_SIZE) -o $@ $<
+	$(HOSTCC) $(HOSTCFLAGS) -DCONFIG_ROM_SIZE=$(CONFIG_ROM_SIZE) -fsanitize=address -o $@ $<
 
 CBOOTIMAGE:=$(objutil)/cbootimage/cbootimage
 
diff --git a/src/mainboard/emulation/Kconfig b/src/mainboard/emulation/Kconfig
index 1dc89ca..759b1de 100644
--- a/src/mainboard/emulation/Kconfig
+++ b/src/mainboard/emulation/Kconfig
@@ -2,6 +2,7 @@ if VENDOR_EMULATION
 
 choice
 	prompt "Mainboard model"
+	default BOARD_EMULATION_QEMU_X86_I440FX
 
 source "src/mainboard/emulation/*/Kconfig.name"
 



More information about the coreboot-gerrit mailing list