From patchwork Tue Aug 27 10:36:21 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Masahiro Yamada X-Patchwork-Id: 172271 Delivered-To: patch@linaro.org Received: by 2002:a92:d204:0:0:0:0:0 with SMTP id y4csp5594484ily; Tue, 27 Aug 2019 03:37:17 -0700 (PDT) X-Google-Smtp-Source: APXvYqyXHxJj/hg9y9jj4h3RWmjWyummDaLgHaHaW1x4SW7xdyD1dZBpWZApBvqg0z922KE9p9Ci X-Received: by 2002:a65:6081:: with SMTP id t1mr20875423pgu.9.1566902237125; Tue, 27 Aug 2019 03:37:17 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1566902237; cv=none; d=google.com; s=arc-20160816; b=Pccm1ePrFHLE7AMeTSu/jehoqsrP2D1SE57oGKBCeJy4yTzg88C6aNC/VnblfWh1l/ +YiiRTQXtDVx5lra9e3en4wDdPdxPlN+bIyMFCUAmvOKzOaRGz9U8W0jb3mEjKXLadco ortmgrYyFFMEiPEJ0qbt5PV0BR08VeUutR/2HuYBcJsRuebg8QjyLvvZg2m4+5tYLjNz uFyTQHPxAK8lhDOW3E6MYBWafWRyn/eKLQP15cUoPQe+m8cTxLdRohvwXERSNsSwpXK7 yKPL141wOTbwgCvsD8B4948ktk08uqG45xVBcrsDU3eVMOXREqwkK134BiuUYkSzmR29 M37A== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:message-id:date:subject:cc:to:from :dkim-signature:dkim-filter; bh=FO349XjUNELPoDFkfp0JNDKza99Kv6R8kSRcxXIPYYs=; b=TNVoJkXP77Kjx2XiTNTYhLGT4cRCOJ56swko7MZeoX3LnC0UqMkVCVCyFVxZZrNuup 9BJgM8TtbfOYWa8qyYiSryn9uGnDNc64GkGrNeWbvzI/NiwZDq6bMuc28ox37o2a34Os MOsHFrVlmhvKzk8xS7U5La172vmSTxsd1xejNK8DjFO7cry5urp5T300wGbJupmsN8JM 73i3K4wn4ikOwQ3x7f47tfmzUkym/b66IIsGqAADIIFOqd/Rwt3n2bI2NjV7s+TPyK0z iX9New06UVtcv1mWgS68PCIUe1mJTh7uXwVTuSv8fT9MX0k0Bg2TqVfiWkhx/Hexdz35 UXcw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=u6qzukAh; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id p35si11396553pgb.484.2019.08.27.03.37.16; Tue, 27 Aug 2019 03:37:17 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@nifty.com header.s=dec2015msa header.b=u6qzukAh; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728978AbfH0KhP (ORCPT + 28 others); Tue, 27 Aug 2019 06:37:15 -0400 Received: from conuserg-10.nifty.com ([210.131.2.77]:47835 "EHLO conuserg-10.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725912AbfH0KhP (ORCPT ); Tue, 27 Aug 2019 06:37:15 -0400 Received: from localhost.localdomain (p14092-ipngnfx01kyoto.kyoto.ocn.ne.jp [153.142.97.92]) (authenticated) by conuserg-10.nifty.com with ESMTP id x7RAaNlw022326; Tue, 27 Aug 2019 19:36:24 +0900 DKIM-Filter: OpenDKIM Filter v2.10.3 conuserg-10.nifty.com x7RAaNlw022326 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nifty.com; s=dec2015msa; t=1566902184; bh=FO349XjUNELPoDFkfp0JNDKza99Kv6R8kSRcxXIPYYs=; h=From:To:Cc:Subject:Date:From; b=u6qzukAh6+Ep3piO54BqJloftQavacwDCNtvFPE00NCyS5GFfiGnhFnRILzRBFLiD ky04ajNvtoi6wyxHf5SxE6MCU0zRfcQk6KXGJf13q7BRwtJDCzDzMmE8OtZvkboj4R Nc7CUxDKHYS73cPsJR6rWU0Lk0DugAcmCpbb4k/L3oue4yH1pR0uRDpe4+S1itAsoN edX1NTcASyv/8O3Hbdc7WjBUDBr4hvhV7v5ltW51Ei7l7iUaMes6SPkrJ7aVKFDLi4 aHRYfWJV1NCr5llJxugtclgQu8WSs9aA8W4p0ysUgKY9BWa4w5ifBA9bloOTCHEjcj VKMQcwNr9YUVw== X-Nifty-SrcIP: [153.142.97.92] From: Masahiro Yamada To: linux-kbuild@vger.kernel.org Cc: Nick Desaulniers , Nathan Chancellor , Arnd Bergmann , Masahiro Yamada , Michal Marek , clang-built-linux@googlegroups.com, linux-kernel@vger.kernel.org Subject: [PATCH v2] kbuild: enable unused-function warnings for W= build with Clang Date: Tue, 27 Aug 2019 19:36:21 +0900 Message-Id: <20190827103621.1073-1-yamada.masahiro@socionext.com> X-Mailer: git-send-email 2.17.1 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org GCC and Clang have different policy for -Wunused-function; GCC never reports unused-function warnings for 'static inline' functions whereas Clang reports them if they are defined in source files instead of included headers although it has been suppressed since commit abb2ea7dfd82 ("compiler, clang: suppress warning for unused static inline functions"). We often miss to remove unused functions where 'static inline' is used in .c files since there is no tool to detect them. Unused code remains until somebody notices. For example, commit 075ddd75680f ("regulator: core: remove unused rdev_get_supply()"). Let's remove __maybe_unused from the inline macro to allow Clang to start finding unused static inline functions. As always, it is not a good idea to sprinkle warnings for the normal build, so I added -Wno-unsued-function for no W= build. Per the documentation [1], -Wno-unused-function will also disable -Wunneeded-internal-declaration, which can help find bugs like commit 8289c4b6f2e5 ("platform/x86: mlx-platform: Properly use mlxplat_mlxcpld_msn201x_items"). (pointed out by Nathan Chancellor) I added -Wunneeded-internal-declaration to address it. If you contribute to code clean-up, please run "make CC=clang W=1" and check -Wunused-function warnings. You will find lots of unused functions. Some of them are false-positives because the call-sites are disabled by #ifdef. I do not like to abuse the inline keyword for suppressing unused-function warnings because it is intended to be a hint for the compiler's optimization. I prefer __maybe_unused or #ifdef around the definition. [1]: https://clang.llvm.org/docs/DiagnosticsReference.html#wunused-function Signed-off-by: Masahiro Yamada Reviewed-by: Kees Cook --- Changes in v2: - Add -Wunneeded-internal-declaration (per Nathan Chancellor) include/linux/compiler_types.h | 10 ++-------- scripts/Makefile.extrawarn | 4 ++++ 2 files changed, 6 insertions(+), 8 deletions(-) -- 2.17.1 Reviewed-by: Nathan Chancellor Reviewed-by: Nick Desaulniers Tested-by: Nick Desaulniers Acked-by: Nick Desaulniers diff --git a/include/linux/compiler_types.h b/include/linux/compiler_types.h index 599c27b56c29..14de8d0162fb 100644 --- a/include/linux/compiler_types.h +++ b/include/linux/compiler_types.h @@ -130,10 +130,6 @@ struct ftrace_likely_data { /* * Force always-inline if the user requests it so via the .config. - * GCC does not warn about unused static inline functions for - * -Wunused-function. This turns out to avoid the need for complex #ifdef - * directives. Suppress the warning in clang as well by using "unused" - * function attribute, which is redundant but not harmful for gcc. * Prefer gnu_inline, so that extern inline functions do not emit an * externally visible function. This makes extern inline behave as per gnu89 * semantics rather than c99. This prevents multiple symbol definition errors @@ -143,11 +139,9 @@ struct ftrace_likely_data { * (which would break users of __always_inline). */ #if !defined(CONFIG_OPTIMIZE_INLINING) -#define inline inline __attribute__((__always_inline__)) __gnu_inline \ - __maybe_unused notrace +#define inline inline __attribute__((__always_inline__)) __gnu_inline notrace #else -#define inline inline __gnu_inline \ - __maybe_unused notrace +#define inline inline __gnu_inline notrace #endif #define __inline__ inline diff --git a/scripts/Makefile.extrawarn b/scripts/Makefile.extrawarn index a74ce2e3c33e..69589f4bac48 100644 --- a/scripts/Makefile.extrawarn +++ b/scripts/Makefile.extrawarn @@ -70,5 +70,9 @@ KBUILD_CFLAGS += -Wno-initializer-overrides KBUILD_CFLAGS += -Wno-format KBUILD_CFLAGS += -Wno-sign-compare KBUILD_CFLAGS += -Wno-format-zero-length +KBUILD_CFLAGS += -Wno-unused-function +# -Wno-unused-function would also disable -Wunneeded-internal-declaration, +# but we want to keep it enabled. +KBUILD_CFLAGS += -Wunneeded-internal-declaration endif endif