Re: [PATCH] Compiling v2.0-2.4.17-i386-1 with CONFIG_KDB=n

[Keith Owens <kaos@xxxxxxx>]

On 29 Dec 2001 23:04:48 +0200, 
Tommi Virtanen <tv@xxxxxxxxxx> wrote:
>
>       This patch is needed for compilation of v2.0-2.4.17-i386-1
>       with CONFIG_KDB=n -- otherwise kdb() and friends are undefined.
>
>--- linux-2.4.17.orig/arch/i386/kernel/traps.c Sat Dec 29 21:27:08 2001
>+++ linux-2.4.17/arch/i386/kernel/traps.c      Sat Dec 29 21:27:34 2001
>@@ -30,9 +30,7 @@
> #include <asm/processor.h>
> #endif
> 
>-#ifdef        CONFIG_KDB
> #include <linux/kdb.h>
>-#endif        /* CONFIG_KDB */
> 
> #include <asm/system.h>
> #include <asm/uaccess.h>

Other way around.  Calls to kdb from non-kdb code need to be wrapped in
#ifdef CONFIG_KDB, I missed one.

--- 2.4.17-kdb.orig/arch/i386/kernel/traps.c    Sun Dec 30 12:31:30 2001
+++ 2.4.17-kdb/arch/i386/kernel/traps.c Sun Dec 30 12:32:49 2001
@@ -581,7 +581,9 @@
                return;
        }
 #endif
+#ifdef CONFIG_KDB
        (void)kdb(KDB_REASON_NMI, reason, regs);
+#endif /* CONFIG_KDB */
        printk("Uhhuh. NMI received for unknown reason %02x.\n", reason);
        printk("Dazed and confused, but trying to continue\n");
        printk("Do you have a strange power saving mode enabled?\n");