Messages in this thread | | | Date | Sun, 5 Jun 2011 10:06:39 -0700 | From | Josh Triplett <> | Subject | Re: [PATCH] Unhide DEBUG_BUGVERBOSE when EXPERT=y, even if DEBUG_KERNEL=n |
| |
On Sun, Jun 05, 2011 at 11:34:45AM +0200, Ingo Molnar wrote: > > * Josh Triplett <josh@joshtriplett.org> wrote: > > > config DEBUG_BUGVERBOSE > > - bool "Verbose BUG() reporting (adds 70K)" if DEBUG_KERNEL && EXPERT > > + bool "Verbose BUG() reporting (adds 70K)" if DEBUG_KERNEL || EXPERT > > Well, DEBUG_KERNEL really means two things: > > - make more debugging options available > - allow the *disabling* of existing (default-enabled) debug options
As well as one more: a quick "git grep DEBUG_KERNEL" turns up a few uses in actual kernel source code, to control debugging features. Those should likely use separately selectable debug options, but they currently don't.
~/src/linux-2.6$ find * -not -name 'Kconfig*' -not -name '*defconfig' | xargs grep -n DEBUG_KERNEL arch/parisc/mm/init.c:653:#ifdef CONFIG_DEBUG_KERNEL /* double-sanity-check paranoia */ arch/powerpc/kernel/sysfs.c:229:#ifdef CONFIG_DEBUG_KERNEL arch/powerpc/kernel/sysfs.c:258:#endif /* CONFIG_DEBUG_KERNEL */ arch/powerpc/kernel/sysfs.c:299:#ifdef CONFIG_DEBUG_KERNEL arch/powerpc/kernel/sysfs.c:328:#endif /* CONFIG_DEBUG_KERNEL */ arch/blackfin/include/asm/entry.h:53:/* As a debugging aid - we save IPEND when DEBUG_KERNEL is on, arch/blackfin/include/asm/entry.h:56:# ifndef CONFIG_DEBUG_KERNEL arch/blackfin/include/asm/entry.h:65:# else /* CONFIG_DEBUG_KERNEL */ arch/blackfin/include/asm/entry.h:77:# endif /* CONFIG_DEBUG_KERNEL */ arch/blackfin/include/asm/context.S:208:#ifdef CONFIG_DEBUG_KERNEL drivers/usb/musb/musb_core.c:557: * REVISIT: do delays from lots of DEBUG_KERNEL checks
> So i think the right solution would be to select DEBUG_KERNEL if > EXPERT is enabled - this would simplify things and would allow the > removal of a lot of EXPERT conditions from the debug options. > > Ok?
I could live with that, as long as CONFIG_DEBUG_KERNEL never directly enables any debugging code like it does above, and just acts like CONFIG_EXPERT in hiding a pile of unnecessary options. DEBUG_KERNEL should probably also have some text saying it doesn't actually enable any kernel debugging on its own, once that becomes true.
- Josh Triplett
| |