From b6974ab13deccb89d8fa749a37ad97fc951eddc8 Mon Sep 17 00:00:00 2001 From: Maximilian Brune Date: Thu, 15 Aug 2024 23:08:33 +0200 Subject: 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/83934 Tested-by: build bot (Jenkins) Reviewed-by: Martin L Roth --- src/arch/x86/Kconfig | 8 -------- src/arch/x86/null_breakpoint.c | 4 ++-- 2 files changed, 2 insertions(+), 10 deletions(-) (limited to 'src') diff --git a/src/arch/x86/Kconfig b/src/arch/x86/Kconfig index 2db526e34c..e99b3af7c2 100644 --- a/src/arch/x86/Kconfig +++ b/src/arch/x86/Kconfig @@ -368,14 +368,6 @@ config DEBUG_NULL_DEREF_BREAKPOINTS_IN_ALL_STAGES 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 43e3727448..44c7ae91c4 100644 --- a/src/arch/x86/null_breakpoint.c +++ b/src/arch/x86/null_breakpoint.c @@ -12,7 +12,7 @@ static struct breakpoint_handle null_fetch_bp; 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 @@ static int handle_deref_breakpoint(struct breakpoint_handle handle, struct eregs #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) -- cgit v1.2.3