From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934237AbcI0TIL (ORCPT ); Tue, 27 Sep 2016 15:08:11 -0400 Received: from mail.kernel.org ([198.145.29.136]:44222 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752367AbcI0TIH (ORCPT ); Tue, 27 Sep 2016 15:08:07 -0400 Subject: [PATCH] CodingStyle: Expand IS_ENABLED() documentation To: Jonathan Corbet From: Bjorn Helgaas Cc: linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org Date: Tue, 27 Sep 2016 14:08:02 -0500 Message-ID: <20160927190802.13636.10320.stgit@bhelgaas-glaptop2.roam.corp.google.com> User-Agent: StGit/0.16 MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org CodingStyle recommends IS_ENABLED(CONFIG_FOO) over #ifdef. Add an example of the #ifdef, since it's not completely obvious that in many cases the #ifdef needs to test both CONFIG_FOO and CONFIG_FOO_MODULE. Signed-off-by: Bjorn Helgaas --- Documentation/CodingStyle | 15 +++++++++++++-- include/linux/kconfig.h | 3 ++- 2 files changed, 15 insertions(+), 3 deletions(-) diff --git a/Documentation/CodingStyle b/Documentation/CodingStyle index a096836..b3e9743 100644 --- a/Documentation/CodingStyle +++ b/Documentation/CodingStyle @@ -911,8 +911,19 @@ The compiler will constant-fold the conditional away, and include or exclude the block of code just as with an #ifdef, so this will not add any runtime overhead. However, this approach still allows the C compiler to see the code inside the block, and check it for correctness (syntax, types, symbol -references, etc). Thus, you still have to use an #ifdef if the code inside the -block references symbols that will not exist if the condition is not met. +references, etc). + +Because the compiler processes the block, you have to use an #ifdef instead +of IS_ENABLED() when code inside the block references symbols that will not +exist if the condition is not met. Different CONFIG_FOO autoconf.h symbols +are generated for modular Kconfig options than for builtin ones, so you +need "#if defined(CONFIG_FOO) || defined(CONFIG_FOO_MODULE)" if FOO can be +a module: + + .config include/generated/autoconf.h + ------------ ---------------------------- + CONFIG_FOO=y #define CONFIG_FOO 1 + CONFIG_FOO=m #define CONFIG_FOO_MODULE 1 At the end of any non-trivial #if or #ifdef block (more than a few lines), place a comment after the #endif on the same line, noting the conditional diff --git a/include/linux/kconfig.h b/include/linux/kconfig.h index 15ec117..51a5f66 100644 --- a/include/linux/kconfig.h +++ b/include/linux/kconfig.h @@ -60,7 +60,8 @@ /* * IS_ENABLED(CONFIG_FOO) evaluates to 1 if CONFIG_FOO is set to 'y' or 'm', - * 0 otherwise. + * 0 otherwise. Note that CONFIG_FOO=y results in "#define CONFIG_FOO 1" + * in autoconf.h, while CONFIG_FOO=m results in "#define CONFIG_FOO_MODULE 1" */ #define IS_ENABLED(option) __or(IS_BUILTIN(option), IS_MODULE(option))