Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.37 as permitted sender) client-ip=23.128.96.37;
Message-ID: <203f8ab5-7728-40fe-bd31-d48440fb0726@efficios.com>
Date:   Fri, 8 Dec 2023 09:34:33 -0500
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] init/Kconfig: move more items into the EXPERT menu
Content-Language: en-US
To:     Randy Dunlap <rdunlap@infradead.org>, linux-kernel@vger.kernel.org
Cc:     Andrew Morton <akpm@linux-foundation.org>,
        Peter Zijlstra <peterz@infradead.org>,
        "Paul E. McKenney" <paulmck@kernel.org>,
        Boqun Feng <boqun.feng@gmail.com>
References: <20231208045819.2922-1-rdunlap@infradead.org>
From:   Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
In-Reply-To: <20231208045819.2922-1-rdunlap@infradead.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 2023-12-07 23:58, Randy Dunlap wrote:
> KCMP, RSEQ, CACHESTAT_SYSCALL, and PC104 depend on EXPERT but not shown
> in the EXPERT menu. Move some lines around so that they are displayed
> in the EXPERT menu.
> 
> Drop one useless comment.
> 
> Change "enabled" to "enable" for DEBUG_RSEQ.

Thanks!

Reviewed-by: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>

> 
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Andrew Morton <akpm@linux-foundation.org>
> Cc: Mathieu Desnoyers <mathieu.desnoyers@efficios.com>
> Cc: Peter Zijlstra <peterz@infradead.org>
> Cc: "Paul E. McKenney" <paulmck@kernel.org>
> Cc: Boqun Feng <boqun.feng@gmail.com>
> ---
>   init/Kconfig |  102 ++++++++++++++++++++++++-------------------------
>   1 file changed, 50 insertions(+), 52 deletions(-)
> 
> diff -- a/init/Kconfig b/init/Kconfig
> --- a/init/Kconfig
> +++ b/init/Kconfig
> @@ -1688,6 +1688,56 @@ config MEMBARRIER
>   
>   	  If unsure, say Y.
>   
> +config KCMP
> +	bool "Enable kcmp() system call" if EXPERT
> +	help
> +	  Enable the kernel resource comparison system call. It provides
> +	  user-space with the ability to compare two processes to see if they
> +	  share a common resource, such as a file descriptor or even virtual
> +	  memory space.
> +
> +	  If unsure, say N.
> +
> +config RSEQ
> +	bool "Enable rseq() system call" if EXPERT
> +	default y
> +	depends on HAVE_RSEQ
> +	select MEMBARRIER
> +	help
> +	  Enable the restartable sequences system call. It provides a
> +	  user-space cache for the current CPU number value, which
> +	  speeds up getting the current CPU number from user-space,
> +	  as well as an ABI to speed up user-space operations on
> +	  per-CPU data.
> +
> +	  If unsure, say Y.
> +
> +config DEBUG_RSEQ
> +	default n
> +	bool "Enable debugging of rseq() system call" if EXPERT
> +	depends on RSEQ && DEBUG_KERNEL
> +	help
> +	  Enable extra debugging checks for the rseq system call.
> +
> +	  If unsure, say N.
> +
> +config CACHESTAT_SYSCALL
> +	bool "Enable cachestat() system call" if EXPERT
> +	default y
> +	help
> +	  Enable the cachestat system call, which queries the page cache
> +	  statistics of a file (number of cached pages, dirty pages,
> +	  pages marked for writeback, (recently) evicted pages).
> +
> +	  If unsure say Y here.
> +
> +config PC104
> +	bool "PC/104 support" if EXPERT
> +	help
> +	  Expose PC/104 form factor device drivers and options available for
> +	  selection and configuration. Enable this option if your target
> +	  machine has a PC/104 bus.
> +
>   config KALLSYMS
>   	bool "Load all symbols for debugging/ksymoops" if EXPERT
>   	default y
> @@ -1752,57 +1802,12 @@ config KALLSYMS_BASE_RELATIVE
>   
>   # end of the "standard kernel features (expert users)" menu
>   
> -# syscall, maps, verifier
> -
>   config ARCH_HAS_MEMBARRIER_CALLBACKS
>   	bool
>   
>   config ARCH_HAS_MEMBARRIER_SYNC_CORE
>   	bool
>   
> -config KCMP
> -	bool "Enable kcmp() system call" if EXPERT
> -	help
> -	  Enable the kernel resource comparison system call. It provides
> -	  user-space with the ability to compare two processes to see if they
> -	  share a common resource, such as a file descriptor or even virtual
> -	  memory space.
> -
> -	  If unsure, say N.
> -
> -config RSEQ
> -	bool "Enable rseq() system call" if EXPERT
> -	default y
> -	depends on HAVE_RSEQ
> -	select MEMBARRIER
> -	help
> -	  Enable the restartable sequences system call. It provides a
> -	  user-space cache for the current CPU number value, which
> -	  speeds up getting the current CPU number from user-space,
> -	  as well as an ABI to speed up user-space operations on
> -	  per-CPU data.
> -
> -	  If unsure, say Y.
> -
> -config CACHESTAT_SYSCALL
> -	bool "Enable cachestat() system call" if EXPERT
> -	default y
> -	help
> -	  Enable the cachestat system call, which queries the page cache
> -	  statistics of a file (number of cached pages, dirty pages,
> -	  pages marked for writeback, (recently) evicted pages).
> -
> -	  If unsure say Y here.
> -
> -config DEBUG_RSEQ
> -	default n
> -	bool "Enabled debugging of rseq() system call" if EXPERT
> -	depends on RSEQ && DEBUG_KERNEL
> -	help
> -	  Enable extra debugging checks for the rseq system call.
> -
> -	  If unsure, say N.
> -
>   config HAVE_PERF_EVENTS
>   	bool
>   	help
> @@ -1817,13 +1822,6 @@ config PERF_USE_VMALLOC
>   	help
>   	  See tools/perf/design.txt for details
>   
> -config PC104
> -	bool "PC/104 support" if EXPERT
> -	help
> -	  Expose PC/104 form factor device drivers and options available for
> -	  selection and configuration. Enable this option if your target
> -	  machine has a PC/104 bus.
> -
>   menu "Kernel Performance Events And Counters"
>   
>   config PERF_EVENTS

-- 
Mathieu Desnoyers
EfficiOS Inc.
https://www.efficios.com