From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 27 Jul 2018 09:48:54 +0200 From: Christoph Hellwig Subject: Re: include architecture Kconfig files from top-level Kconfig v3 Message-ID: <20180727074854.GA21840@lst.de> References: <20180724175646.3621-1-hch@lst.de> <20180726172143.GA9836@lst.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Masahiro Yamada Cc: Christoph Hellwig , Randy Dunlap , Richard Weinberger , Ley Foon Tan , Michal Simek , Linux Kbuild mailing list , linux-um@lists.infradead.org, linux-arch@lists.infradead.org, Linux Kernel Mailing List List-ID: On Fri, Jul 27, 2018 at 10:32:19AM +0900, Masahiro Yamada wrote: > This will just add a new unmet dependency warning. > CONFIG_PREEMPT_COUNT will be still selected. True. I guess we simply need to prohibit CONFIG_DEBUG_ATOMIC_SLEEP explicitly if PREEMPT_COUNT isn't supported. E.g something like this: diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 26d3ff7e3cf4..373ce9fecd7e 100644 --- a/lib/Kconfig.debug +++ b/lib/Kconfig.debug @@ -1195,6 +1195,7 @@ config DEBUG_ATOMIC_SLEEP bool "Sleep inside atomic section checking" select PREEMPT_COUNT depends on DEBUG_KERNEL + depends on !ARCH_NO_PREEMPT help If you say Y here, various routines which may sleep will become very noisy if they are called inside atomic sections: when a spinlock is