Maximilian Brune has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/83934?usp=email )
Change subject: arch/x86: Remove CONFIG_DEBUG_NULL_DEREF_HALT ......................................................................
arch/x86: Remove CONFIG_DEBUG_NULL_DEREF_HALT
For more than 2 years the option has been unconfigurable. Since no one seems to have fixed that, the options seems to be not needed by anyone. So instead of making it configurable now, we can just as well remove it.
Change-Id: I4055d497c7c23e148d2a09f216c7b910a9b3ea9b Signed-off-by: Maximilian Brune maximilian.brune@9elements.com --- M src/arch/x86/Kconfig M src/arch/x86/null_breakpoint.c 2 files changed, 2 insertions(+), 10 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/34/83934/1
diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig index 16d8a70..530f871 100644 --- a/src/arch/x86/Kconfig +++ b/src/arch/x86/Kconfig @@ -368,14 +368,6 @@ default y depends on DEBUG_NULL_DEREF_BREAKPOINTS && DEBUG_HW_BREAKPOINTS_IN_ALL_STAGES
-config DEBUG_NULL_DEREF_HALT - bool - default n - depends on DEBUG_NULL_DEREF_BREAKPOINTS - help - When enabled null dereferences and instruction fetches will halt execution. - Otherwise an error will be printed. - # Some EC need an "EC firmware pointer" (a data structure hinting the address # of its firmware blobs) being put at a fixed position. Its space # (__section__(".ecfw_ptr")) should be reserved if it lies in the range of a diff --git a/src/arch/x86/null_breakpoint.c b/src/arch/x86/null_breakpoint.c index 43e3727..44c7ae9 100644 --- a/src/arch/x86/null_breakpoint.c +++ b/src/arch/x86/null_breakpoint.c @@ -12,7 +12,7 @@ static int handle_fetch_breakpoint(struct breakpoint_handle handle, struct eregs *regs) { printk(BIOS_ERR, "Instruction fetch from address zero\n"); - return CONFIG(DEBUG_NULL_DEREF_HALT); + return 0; }
static int handle_deref_breakpoint(struct breakpoint_handle handle, struct eregs *regs) @@ -22,7 +22,7 @@ #else printk(BIOS_ERR, "Null dereference at eip: 0x%x\n", regs->eip); #endif - return CONFIG(DEBUG_NULL_DEREF_HALT); + return 0; }
static void create_deref_breakpoint(void)