Re: [PATCH 5/5] kconfig: move the "Executable file formats" menu to fs/Kconfig.binfmt

[Randy Dunlap]

On 07/02/18 07:47, Christoph Hellwig wrote:
> No need to have this in the top-level Kconfig.
> 
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>

Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx>


> ---
>  Kconfig           | 2 --
>  fs/Kconfig.binfmt | 5 +++++
>  2 files changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/Kconfig b/Kconfig
> index 5f580c81d85d..f9969589a6b4 100644
> --- a/Kconfig
> +++ b/Kconfig
> @@ -15,9 +15,7 @@ source "init/Kconfig"
>  
>  source "kernel/Kconfig.freezer"
>  
> -menu "Executable file formats"
>  source "fs/Kconfig.binfmt"
> -endmenu
>  
>  source "mm/Kconfig"
>  
> diff --git a/fs/Kconfig.binfmt b/fs/Kconfig.binfmt
> index 56df483de619..b795f8da81f3 100644
> --- a/fs/Kconfig.binfmt
> +++ b/fs/Kconfig.binfmt
> @@ -1,3 +1,6 @@
> +
> +menu "Executable file formats"
> +
>  config BINFMT_ELF
>  	bool "Kernel support for ELF binaries"
>  	depends on MMU
> @@ -187,3 +190,5 @@ config COREDUMP
>  	  This option enables support for performing core dumps. You almost
>  	  certainly want to say Y here. Not necessary on systems that never
>  	  need debugging or only ever run flawless code.
> +
> +endmenu
> 


-- 
~Randy