[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:28:02 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 25670c8a59e53b253c899e60e28b517995fae172
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                    | 3 ++-
 src/mainboard/emulation/Kconfig | 1 +
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/Makefile.inc b/Makefile.inc
index 127f372..8c2705b 100644
--- a/Makefile.inc
+++ b/Makefile.inc
@@ -487,7 +487,8 @@ $(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) --version
+	$(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