Re: [PATCH v3] Kconfig.debug: Make DEBUG_INFO selectable from a choice

From: Masahiro Yamada
Date: Tue Jan 25 2022 - 03:28:58 EST


On Tue, Jan 25, 2022 at 4:51 PM Kees Cook <keescook@xxxxxxxxxxxx> wrote:
>
> Currently it's not possible to enable DEBUG_INFO for an all*config build,
> since it is marked as "depends on !COMPILE_TEST". This generally makes
> sense because a debug build of an all*config target ends up taking much
> longer and the output is much larger. Having this be "default off" makes
> sense. However, there are cases where enabling DEBUG_INFO for such builds
> is useful for doing treewide A/B comparisons of build options, etc.
>
> Make DEBUG_INFO selectable from any of the DWARF version choice options,
> with DEBUG_INFO_NONE being the default for COMPILE_TEST. The mutually
> exclusive relationship between DWARF5 and BTF must be inverted, but the
> result remains the same. Additionally moves DEBUG_KERNEL and DEBUG_MISC
> up to the top of the menu because they were enabling features _above_
> it, making it weird to navigate menuconfig.
>
> Suggested-by: Arnd Bergmann <arnd@xxxxxxxx>
> Cc: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>
> Cc: Masahiro Yamada <masahiroy@xxxxxxxxxx>


Reviewed-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>


> Cc: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> Cc: Tetsuo Handa <penguin-kernel@xxxxxxxxxxxxxxxxxxx>
> Reviewed-by: Arnd Bergmann <arnd@xxxxxxxx>
> Reviewed-by: Nathan Chancellor <nathan@xxxxxxxxxx>
> Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> Tested-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
> Signed-off-by: Kees Cook <keescook@xxxxxxxxxxxx>
> ---
> v1: https://lore.kernel.org/lkml/20211210000910.3597609-1-keescook@xxxxxxxxxxxx
> v2: https://lore.kernel.org/lkml/20220121001204.4023842-1-keescook@xxxxxxxxxxxx
> v3: - rebase to v5.17-rc1
> ---


--
Best Regards
Masahiro Yamada