[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <47AF5D58.8030303@web.de>
Date: Sun, 10 Feb 2008 21:23:52 +0100
From: Jan Kiszka <jan.kiszka@....de>
To: Sam Ravnborg <sam@...nborg.org>
CC: Ingo Molnar <mingo@...e.hu>, linux-kernel@...r.kernel.org,
Linus Torvalds <torvalds@...ux-foundation.org>,
Andrew Morton <akpm@....com.au>,
Thomas Gleixner <tglx@...utronix.de>,
Jason Wessel <jason.wessel@...driver.com>
Subject: Re: [0/6] kgdb light
Sam Ravnborg wrote:
> On Sun, Feb 10, 2008 at 02:25:20PM +0100, Jan Kiszka wrote:
>> Sam Ravnborg wrote:
>>> ...
>>> +extern int kgdb_may_fault;
>>> I searched but I could not find any places this
>>> variable were set to anuthing else than 0 neither where it was tested.
>> This is a leftover from the old jmp-on-fault logic that was missed by
>> this tree. I have a patch under test that kills this (and further 150
>> LOC) from the core.
>>
>>> diff --git a/lib/Kconfig.kgdb b/lib/Kconfig.kgdb
>>> new file mode 100644
>>> index 0000000..00263c0
>>> --- /dev/null
>>> +++ b/lib/Kconfig.kgdb
>>> @@ -0,0 +1,37 @@
>>> +
>>> +menuconfig KGDB
>>> + bool "KGDB: kernel debugging with remote gdb"
>>> + select FRAME_POINTER
>>> + depends on HAVE_ARCH_KGDB
>>> + depends on DEBUG_KERNEL && EXPERIMENTAL
>>> + help
>>> + If you say Y here, it will be possible to remotely debug the
>>> + kernel using gdb. Documentation of kernel debugger is available
>>> + at http://kgdb.sourceforge.net as well as in DocBook form
>>> + in Documentation/DocBook/. If unsure, say N.
>>> +
>>> +config HAVE_ARCH_KGDB_SHADOW_INFO
>>> + bool
>>> +
>>>
>>> Please add:
>>> config HAVE_ARCH_KGDB
>>> bool
>>>
>>> So we later in x86/Kconfig can do:
>>> config X86
>>> select HAVE_ARCH_KGDB
>>>
>>> And we can get rid of (from x86/Kconfig):
>>> +config HAVE_ARCH_KGDB
>>> + def_bool y
>>>
>>>
>>> Back to Kconfig.kgdb:
>>>
>>> +config KGDBOC
>>> + tristate "KGDB: use kgdb over the serial console"
>>> + depends on KGDB
>>>
>>> Can we have a more descriptive name here.
>>> For example:
>>> config KGDB_SERIAL_CONSOLE
>>> It is only used in one place so there is no
>>> specific need for such a magic short name.
>>>
>> Is the patch below OK? I also added an "if KGB" to unbreak kgdb's
>> kconfig menu again and included two minor cleanups I posted yesterday.
>
> Looks OK - but I think Ingo already addressed this.
Should be merged meanwhile.
At this chance: Is it necessary that prompt-less configs like
HAVE_ARCH_KGDB break the menu indention? I had to work around this via
"if KGDB" (or even "depends on"). This is easy to miss IMHO.
Jan
Download attachment "signature.asc" of type "application/pgp-signature" (251 bytes)
Powered by blists - more mailing lists