2019-09-09 08:16:48

by Changbin Du

[permalink] [raw]
Subject: [PATCH 0/8] kconfig/hacking: make 'kernel hacking' menu better structured

This series is a trivial improvment for the layout of 'kernel hacking'
configuration menu. Now we have many items in it which makes takes
a little time to look up them since they are not well structured yet.

Early discussion is here:
https://lkml.org/lkml/2019/9/1/39

Changbin Du (8):
kconfig/hacking: Group sysrq/kgdb/ubsan into 'Generic Kernel Debugging
Instruments'
kconfig/hacking: Create submenu for arch special debugging options
kconfig/hacking: Group kernel data structures debugging together
kconfig/hacking: Move kernel testing and coverage options to same
submenu
kconfig/hacking: Move Oops into 'Lockups and Hangs'
kconfig/hacking: Move SCHED_STACK_END_CHECK after DEBUG_STACK_USAGE
kconfig/hacking: Create a submenu for scheduler debugging options
kconfig/hacking: Move DEBUG_BUGVERBOSE to 'printk and dmesg options'

lib/Kconfig.debug | 627 ++++++++++++++++++++++++----------------------
1 file changed, 324 insertions(+), 303 deletions(-)

--
2.20.1


2019-09-09 18:16:17

by Randy Dunlap

[permalink] [raw]
Subject: Re: [PATCH 0/8] kconfig/hacking: make 'kernel hacking' menu better structured

On 9/7/19 6:27 PM, Changbin Du wrote:
> This series is a trivial improvment for the layout of 'kernel hacking'
> configuration menu. Now we have many items in it which makes takes
> a little time to look up them since they are not well structured yet.
>
> Early discussion is here:
> https://lkml.org/lkml/2019/9/1/39
>
> Changbin Du (8):
> kconfig/hacking: Group sysrq/kgdb/ubsan into 'Generic Kernel Debugging
> Instruments'
> kconfig/hacking: Create submenu for arch special debugging options
> kconfig/hacking: Group kernel data structures debugging together
> kconfig/hacking: Move kernel testing and coverage options to same
> submenu
> kconfig/hacking: Move Oops into 'Lockups and Hangs'
> kconfig/hacking: Move SCHED_STACK_END_CHECK after DEBUG_STACK_USAGE
> kconfig/hacking: Create a submenu for scheduler debugging options
> kconfig/hacking: Move DEBUG_BUGVERBOSE to 'printk and dmesg options'
>
> lib/Kconfig.debug | 627 ++++++++++++++++++++++++----------------------
> 1 file changed, 324 insertions(+), 303 deletions(-)

Hi,

Series applies to v5.3-rc7. Has some problems applying to linux-next.

Under 'Compile-time checks and compiler options', "Debug Filesystem" does not belong here.
Maybe move it under 'Generic Kernel Debugging Instruments'.

I mostly like it. I might have put 'Debug notifier call chains' under
'Debug kernel data structures', but that's not a big deal.

--
~Randy

2019-09-10 18:21:45

by Changbin Du

[permalink] [raw]
Subject: Re: [PATCH 0/8] kconfig/hacking: make 'kernel hacking' menu better structured

On Sun, Sep 08, 2019 at 05:44:54PM -0700, Randy Dunlap wrote:
> On 9/7/19 6:27 PM, Changbin Du wrote:
> > This series is a trivial improvment for the layout of 'kernel hacking'
> > configuration menu. Now we have many items in it which makes takes
> > a little time to look up them since they are not well structured yet.
> >
> > Early discussion is here:
> > https://lkml.org/lkml/2019/9/1/39
> >
> > Changbin Du (8):
> > kconfig/hacking: Group sysrq/kgdb/ubsan into 'Generic Kernel Debugging
> > Instruments'
> > kconfig/hacking: Create submenu for arch special debugging options
> > kconfig/hacking: Group kernel data structures debugging together
> > kconfig/hacking: Move kernel testing and coverage options to same
> > submenu
> > kconfig/hacking: Move Oops into 'Lockups and Hangs'
> > kconfig/hacking: Move SCHED_STACK_END_CHECK after DEBUG_STACK_USAGE
> > kconfig/hacking: Create a submenu for scheduler debugging options
> > kconfig/hacking: Move DEBUG_BUGVERBOSE to 'printk and dmesg options'
> >
> > lib/Kconfig.debug | 627 ++++++++++++++++++++++++----------------------
> > 1 file changed, 324 insertions(+), 303 deletions(-)
>
> Hi,
>
> Series applies to v5.3-rc7. Has some problems applying to linux-next.
>
Let me rebase to linux-next.

> Under 'Compile-time checks and compiler options', "Debug Filesystem" does not belong here.
> Maybe move it under 'Generic Kernel Debugging Instruments'.
>
yes, let's move it to 'Generic Kernel Debugging Instruments'.

> I mostly like it. I might have put 'Debug notifier call chains' under
> 'Debug kernel data structures', but that's not a big deal.
>
no problem.

> --
> ~Randy

--
Cheers,
Changbin Du