On Thu, Jan 15, 2004 at 11:57:35AM -0800, Kevin Paul Herbert wrote: > In the top level makefile, there is already: > > ifdef CONFIG_DEBUG_INFO > CFLAGS += -g > endif > > I don't see why you need to add it to arch/mips/Makefile. Your Kconfig > changes seem fine though. > That is right. Thanks for catching this. The original problem started when KGDB did not cause "-g" to be included. I simply looked at 2.4 and cooked the patch. The corrected patch is attached. Jun
diff -Nru link/arch/mips/Kconfig.orig link/arch/mips/Kconfig --- link/arch/mips/Kconfig.orig Thu Jan 15 10:55:57 2004 +++ link/arch/mips/Kconfig Thu Jan 15 11:12:23 2004 @@ -1233,23 +1233,6 @@ This allows applications to run more reliably even when the system is under load. -config DEBUG_INFO - bool "Compile the kernel with debug info" - depends on DEBUG_KERNEL - default y if KGDB - help - If you say Y here the resulting kernel image will include - debugging info resulting in a larger kernel image. - Say Y here only if you plan to use gdb to debug the kernel. - If you don't debug the kernel, you can say N. - -config SB1XXX_CORELIS - bool "Corelis Debugger" - depends on SIBYTE_SB1xxx_SOC && DEBUG_INFO - help - Select compile flags that produce code that can be processed by the - Corelis mksym utility and UDB Emulator. - config DEBUG_SPINLOCK bool "Spinlock debugging" depends on DEBUG_KERNEL @@ -1471,6 +1454,7 @@ config KGDB bool "Remote GDB kernel debugging" depends on DEBUG_KERNEL + select DEBUG_INFO help If you say Y here, it will be possible to remotely debug the MIPS kernel using gdb. This enlarges your kernel image disk size by @@ -1486,6 +1470,23 @@ would like kernel messages to be formatted into GDB $O packets so that GDB prints them as program output, say 'Y'. +config DEBUG_INFO + bool "Compile the kernel with debug info" + depends on DEBUG_KERNEL && !KGDB + default y if KGDB + help + If you say Y here the resulting kernel image will include + debugging info resulting in a larger kernel image. + Say Y here only if you plan to use gdb to debug the kernel. + If you don't debug the kernel, you can say N. + +config SB1XXX_CORELIS + bool "Corelis Debugger" + depends on SIBYTE_SB1xxx_SOC && DEBUG_INFO + help + Select compile flags that produce code that can be processed by the + Corelis mksym utility and UDB Emulator. + config RUNTIME_DEBUG bool "Enable run-time debugging" depends on DEBUG_KERNEL
