From: Sami Tolvanen <samitolvanen@google.com> To: Masahiro Yamada <masahiroy@kernel.org>, Steven Rostedt <rostedt@goodmis.org>, Will Deacon <will@kernel.org> Cc: Josh Poimboeuf <jpoimboe@redhat.com>, Peter Zijlstra <peterz@infradead.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, "Paul E. McKenney" <paulmck@kernel.org>, Kees Cook <keescook@chromium.org>, Nick Desaulniers <ndesaulniers@google.com>, clang-built-linux@googlegroups.com, kernel-hardening@lists.openwall.com, linux-arch@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, Sami Tolvanen <samitolvanen@google.com> Subject: [PATCH v7 01/17] tracing: move function tracer options to Kconfig Date: Wed, 18 Nov 2020 14:07:15 -0800 [thread overview] Message-ID: <20201118220731.925424-2-samitolvanen@google.com> (raw) In-Reply-To: <20201118220731.925424-1-samitolvanen@google.com> Move function tracer options to Kconfig to make it easier to add new methods for generating __mcount_loc, and to make the options available also when building kernel modules. Note that FTRACE_MCOUNT_USE_* options are updated on rebuild and therefore, work even if the .config was generated in a different environment. Signed-off-by: Sami Tolvanen <samitolvanen@google.com> --- Makefile | 20 ++++++++------------ kernel/trace/Kconfig | 16 ++++++++++++++++ scripts/Makefile.build | 6 ++---- 3 files changed, 26 insertions(+), 16 deletions(-) diff --git a/Makefile b/Makefile index e2c3f65c4721..8c8feb4245a6 100644 --- a/Makefile +++ b/Makefile @@ -851,12 +851,8 @@ KBUILD_CFLAGS += $(DEBUG_CFLAGS) export DEBUG_CFLAGS ifdef CONFIG_FUNCTION_TRACER -ifdef CONFIG_FTRACE_MCOUNT_RECORD - # gcc 5 supports generating the mcount tables directly - ifeq ($(call cc-option-yn,-mrecord-mcount),y) - CC_FLAGS_FTRACE += -mrecord-mcount - export CC_USING_RECORD_MCOUNT := 1 - endif +ifdef CONFIG_FTRACE_MCOUNT_USE_CC + CC_FLAGS_FTRACE += -mrecord-mcount ifdef CONFIG_HAVE_NOP_MCOUNT ifeq ($(call cc-option-yn, -mnop-mcount),y) CC_FLAGS_FTRACE += -mnop-mcount @@ -864,6 +860,12 @@ ifdef CONFIG_FTRACE_MCOUNT_RECORD endif endif endif +ifdef CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT + ifdef CONFIG_HAVE_C_RECORDMCOUNT + BUILD_C_RECORDMCOUNT := y + export BUILD_C_RECORDMCOUNT + endif +endif ifdef CONFIG_HAVE_FENTRY ifeq ($(call cc-option-yn, -mfentry),y) CC_FLAGS_FTRACE += -mfentry @@ -873,12 +875,6 @@ endif export CC_FLAGS_FTRACE KBUILD_CFLAGS += $(CC_FLAGS_FTRACE) $(CC_FLAGS_USING) KBUILD_AFLAGS += $(CC_FLAGS_USING) -ifdef CONFIG_DYNAMIC_FTRACE - ifdef CONFIG_HAVE_C_RECORDMCOUNT - BUILD_C_RECORDMCOUNT := y - export BUILD_C_RECORDMCOUNT - endif -endif endif # We trigger additional mismatches with less inlining diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig index a4020c0b4508..927ad004888a 100644 --- a/kernel/trace/Kconfig +++ b/kernel/trace/Kconfig @@ -595,6 +595,22 @@ config FTRACE_MCOUNT_RECORD depends on DYNAMIC_FTRACE depends on HAVE_FTRACE_MCOUNT_RECORD +config FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + bool + depends on FTRACE_MCOUNT_RECORD + +config FTRACE_MCOUNT_USE_CC + def_bool y + depends on $(cc-option,-mrecord-mcount) + depends on !FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + depends on FTRACE_MCOUNT_RECORD + +config FTRACE_MCOUNT_USE_RECORDMCOUNT + def_bool y + depends on !FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + depends on !FTRACE_MCOUNT_USE_CC + depends on FTRACE_MCOUNT_RECORD + config TRACING_MAP bool depends on ARCH_HAVE_NMI_SAFE_CMPXCHG diff --git a/scripts/Makefile.build b/scripts/Makefile.build index ae647379b579..2175ddb1ee0c 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -178,8 +178,7 @@ cmd_modversions_c = \ fi endif -ifdef CONFIG_FTRACE_MCOUNT_RECORD -ifndef CC_USING_RECORD_MCOUNT +ifdef CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT # compiler will not generate __mcount_loc use recordmcount or recordmcount.pl ifdef BUILD_C_RECORDMCOUNT ifeq ("$(origin RECORDMCOUNT_WARN)", "command line") @@ -206,8 +205,7 @@ recordmcount_source := $(srctree)/scripts/recordmcount.pl endif # BUILD_C_RECORDMCOUNT cmd_record_mcount = $(if $(findstring $(strip $(CC_FLAGS_FTRACE)),$(_c_flags)), \ $(sub_cmd_record_mcount)) -endif # CC_USING_RECORD_MCOUNT -endif # CONFIG_FTRACE_MCOUNT_RECORD +endif # CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT ifdef CONFIG_STACK_VALIDATION ifneq ($(SKIP_STACK_VALIDATION),1) -- 2.29.2.299.gdc1121823c-goog
WARNING: multiple messages have this Message-ID (diff)
From: Sami Tolvanen <samitolvanen@google.com> To: Masahiro Yamada <masahiroy@kernel.org>, Steven Rostedt <rostedt@goodmis.org>, Will Deacon <will@kernel.org> Cc: linux-arch@vger.kernel.org, Kees Cook <keescook@chromium.org>, "Paul E. McKenney" <paulmck@kernel.org>, kernel-hardening@lists.openwall.com, Peter Zijlstra <peterz@infradead.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, linux-kbuild@vger.kernel.org, Nick Desaulniers <ndesaulniers@google.com>, linux-kernel@vger.kernel.org, linux-pci@vger.kernel.org, clang-built-linux@googlegroups.com, Sami Tolvanen <samitolvanen@google.com>, Josh Poimboeuf <jpoimboe@redhat.com>, linux-arm-kernel@lists.infradead.org Subject: [PATCH v7 01/17] tracing: move function tracer options to Kconfig Date: Wed, 18 Nov 2020 14:07:15 -0800 [thread overview] Message-ID: <20201118220731.925424-2-samitolvanen@google.com> (raw) In-Reply-To: <20201118220731.925424-1-samitolvanen@google.com> Move function tracer options to Kconfig to make it easier to add new methods for generating __mcount_loc, and to make the options available also when building kernel modules. Note that FTRACE_MCOUNT_USE_* options are updated on rebuild and therefore, work even if the .config was generated in a different environment. Signed-off-by: Sami Tolvanen <samitolvanen@google.com> --- Makefile | 20 ++++++++------------ kernel/trace/Kconfig | 16 ++++++++++++++++ scripts/Makefile.build | 6 ++---- 3 files changed, 26 insertions(+), 16 deletions(-) diff --git a/Makefile b/Makefile index e2c3f65c4721..8c8feb4245a6 100644 --- a/Makefile +++ b/Makefile @@ -851,12 +851,8 @@ KBUILD_CFLAGS += $(DEBUG_CFLAGS) export DEBUG_CFLAGS ifdef CONFIG_FUNCTION_TRACER -ifdef CONFIG_FTRACE_MCOUNT_RECORD - # gcc 5 supports generating the mcount tables directly - ifeq ($(call cc-option-yn,-mrecord-mcount),y) - CC_FLAGS_FTRACE += -mrecord-mcount - export CC_USING_RECORD_MCOUNT := 1 - endif +ifdef CONFIG_FTRACE_MCOUNT_USE_CC + CC_FLAGS_FTRACE += -mrecord-mcount ifdef CONFIG_HAVE_NOP_MCOUNT ifeq ($(call cc-option-yn, -mnop-mcount),y) CC_FLAGS_FTRACE += -mnop-mcount @@ -864,6 +860,12 @@ ifdef CONFIG_FTRACE_MCOUNT_RECORD endif endif endif +ifdef CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT + ifdef CONFIG_HAVE_C_RECORDMCOUNT + BUILD_C_RECORDMCOUNT := y + export BUILD_C_RECORDMCOUNT + endif +endif ifdef CONFIG_HAVE_FENTRY ifeq ($(call cc-option-yn, -mfentry),y) CC_FLAGS_FTRACE += -mfentry @@ -873,12 +875,6 @@ endif export CC_FLAGS_FTRACE KBUILD_CFLAGS += $(CC_FLAGS_FTRACE) $(CC_FLAGS_USING) KBUILD_AFLAGS += $(CC_FLAGS_USING) -ifdef CONFIG_DYNAMIC_FTRACE - ifdef CONFIG_HAVE_C_RECORDMCOUNT - BUILD_C_RECORDMCOUNT := y - export BUILD_C_RECORDMCOUNT - endif -endif endif # We trigger additional mismatches with less inlining diff --git a/kernel/trace/Kconfig b/kernel/trace/Kconfig index a4020c0b4508..927ad004888a 100644 --- a/kernel/trace/Kconfig +++ b/kernel/trace/Kconfig @@ -595,6 +595,22 @@ config FTRACE_MCOUNT_RECORD depends on DYNAMIC_FTRACE depends on HAVE_FTRACE_MCOUNT_RECORD +config FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + bool + depends on FTRACE_MCOUNT_RECORD + +config FTRACE_MCOUNT_USE_CC + def_bool y + depends on $(cc-option,-mrecord-mcount) + depends on !FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + depends on FTRACE_MCOUNT_RECORD + +config FTRACE_MCOUNT_USE_RECORDMCOUNT + def_bool y + depends on !FTRACE_MCOUNT_USE_PATCHABLE_FUNCTION_ENTRY + depends on !FTRACE_MCOUNT_USE_CC + depends on FTRACE_MCOUNT_RECORD + config TRACING_MAP bool depends on ARCH_HAVE_NMI_SAFE_CMPXCHG diff --git a/scripts/Makefile.build b/scripts/Makefile.build index ae647379b579..2175ddb1ee0c 100644 --- a/scripts/Makefile.build +++ b/scripts/Makefile.build @@ -178,8 +178,7 @@ cmd_modversions_c = \ fi endif -ifdef CONFIG_FTRACE_MCOUNT_RECORD -ifndef CC_USING_RECORD_MCOUNT +ifdef CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT # compiler will not generate __mcount_loc use recordmcount or recordmcount.pl ifdef BUILD_C_RECORDMCOUNT ifeq ("$(origin RECORDMCOUNT_WARN)", "command line") @@ -206,8 +205,7 @@ recordmcount_source := $(srctree)/scripts/recordmcount.pl endif # BUILD_C_RECORDMCOUNT cmd_record_mcount = $(if $(findstring $(strip $(CC_FLAGS_FTRACE)),$(_c_flags)), \ $(sub_cmd_record_mcount)) -endif # CC_USING_RECORD_MCOUNT -endif # CONFIG_FTRACE_MCOUNT_RECORD +endif # CONFIG_FTRACE_MCOUNT_USE_RECORDMCOUNT ifdef CONFIG_STACK_VALIDATION ifneq ($(SKIP_STACK_VALIDATION),1) -- 2.29.2.299.gdc1121823c-goog _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2020-11-18 22:07 UTC|newest] Thread overview: 134+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-11-18 22:07 [PATCH v7 00/17] Add support for Clang LTO Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen [this message] 2020-11-18 22:07 ` [PATCH v7 01/17] tracing: move function tracer options to Kconfig Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 02/17] kbuild: add support for Clang LTO Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 23:48 ` Nick Desaulniers 2020-11-18 23:48 ` Nick Desaulniers 2020-11-18 23:48 ` Nick Desaulniers 2020-11-20 16:23 ` Sami Tolvanen 2020-11-20 16:23 ` Sami Tolvanen 2020-11-20 16:23 ` Sami Tolvanen 2020-11-20 19:47 ` Kees Cook 2020-11-20 19:47 ` Kees Cook 2020-11-20 20:29 ` Nathan Chancellor 2020-11-20 20:29 ` Nathan Chancellor 2020-11-20 20:43 ` Kees Cook 2020-11-20 20:43 ` Kees Cook 2020-11-20 20:58 ` Sami Tolvanen 2020-11-20 20:58 ` Sami Tolvanen 2020-11-20 20:58 ` Sami Tolvanen 2020-11-20 23:59 ` Kees Cook 2020-11-20 23:59 ` Kees Cook 2020-11-21 1:46 ` Sami Tolvanen 2020-11-21 1:46 ` Sami Tolvanen 2020-11-21 1:46 ` Sami Tolvanen 2020-11-21 20:11 ` Kees Cook 2020-11-21 20:11 ` Kees Cook 2020-11-18 22:07 ` [PATCH v7 03/17] kbuild: lto: fix module versioning Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 04/17] kbuild: lto: limit inlining Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 05/17] kbuild: lto: merge module sections Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 06/17] kbuild: lto: remove duplicate dependencies from .mod files Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 07/17] init: lto: ensure initcall ordering Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 08/17] init: lto: fix PREL32 relocations Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 09/17] PCI: Fix PREL32 relocations for LTO Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 10/17] modpost: lto: strip .lto from module names Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 11/17] scripts/mod: disable LTO for empty.c Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 12/17] efi/libstub: disable LTO Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 13/17] drivers/misc/lkdtm: disable LTO for rodata.o Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 14/17] arm64: vdso: disable LTO Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-30 11:52 ` Will Deacon 2020-11-30 11:52 ` Will Deacon 2020-11-30 23:44 ` Sami Tolvanen 2020-11-30 23:44 ` Sami Tolvanen 2020-11-30 23:44 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 15/17] KVM: arm64: disable LTO for the nVHE directory Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-23 10:21 ` David Brazdil 2020-11-23 10:21 ` David Brazdil 2020-11-23 18:34 ` Sami Tolvanen 2020-11-23 18:34 ` Sami Tolvanen 2020-11-23 18:34 ` Sami Tolvanen 2020-11-18 22:07 ` [PATCH v7 16/17] arm64: disable recordmcount with DYNAMIC_FTRACE_WITH_REGS Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-30 11:59 ` Will Deacon 2020-11-30 11:59 ` Will Deacon 2020-11-18 22:07 ` [PATCH v7 17/17] arm64: allow LTO_CLANG and THINLTO to be selected Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-18 22:07 ` Sami Tolvanen 2020-11-30 12:00 ` Will Deacon 2020-11-30 12:00 ` Will Deacon 2020-11-18 23:42 ` [PATCH v7 00/17] Add support for Clang LTO Nick Desaulniers 2020-11-18 23:42 ` Nick Desaulniers 2020-11-18 23:42 ` Nick Desaulniers 2020-11-20 10:29 ` Ard Biesheuvel 2020-11-20 10:29 ` Ard Biesheuvel 2020-11-20 10:29 ` Ard Biesheuvel 2020-11-20 20:19 ` Nick Desaulniers 2020-11-20 20:19 ` Nick Desaulniers 2020-11-20 20:19 ` Nick Desaulniers 2020-11-20 23:30 ` Ard Biesheuvel 2020-11-20 23:30 ` Ard Biesheuvel 2020-11-20 23:30 ` Ard Biesheuvel 2020-11-20 23:53 ` Nick Desaulniers 2020-11-20 23:53 ` Nick Desaulniers 2020-11-20 23:53 ` Nick Desaulniers 2020-11-21 7:35 ` Ard Biesheuvel 2020-11-21 7:35 ` Ard Biesheuvel 2020-11-21 7:35 ` Ard Biesheuvel 2020-11-21 11:40 ` Marc Zyngier 2020-11-21 11:40 ` Marc Zyngier 2020-11-21 3:14 ` Nathan Chancellor 2020-11-21 3:14 ` Nathan Chancellor 2020-11-20 4:04 ` Josh Poimboeuf 2020-11-20 4:04 ` Josh Poimboeuf 2020-11-20 20:25 ` Sami Tolvanen 2020-11-20 20:25 ` Sami Tolvanen 2020-11-20 20:25 ` Sami Tolvanen 2020-11-30 12:01 ` Will Deacon 2020-11-30 12:01 ` Will Deacon 2020-12-01 17:31 ` Kees Cook 2020-12-01 17:31 ` Kees Cook 2020-12-01 19:51 ` Nick Desaulniers 2020-12-01 19:51 ` Nick Desaulniers 2020-12-01 19:51 ` Nick Desaulniers 2020-12-01 21:38 ` Sami Tolvanen 2020-12-01 21:38 ` Sami Tolvanen 2020-12-01 21:38 ` Sami Tolvanen 2020-12-02 2:42 ` Masahiro Yamada 2020-12-02 2:42 ` Masahiro Yamada 2020-12-02 5:46 ` Sami Tolvanen 2020-12-02 5:46 ` Sami Tolvanen 2020-12-02 5:46 ` Sami Tolvanen 2020-12-02 18:54 ` Kees Cook 2020-12-02 18:54 ` Kees Cook
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20201118220731.925424-2-samitolvanen@google.com \ --to=samitolvanen@google.com \ --cc=clang-built-linux@googlegroups.com \ --cc=gregkh@linuxfoundation.org \ --cc=jpoimboe@redhat.com \ --cc=keescook@chromium.org \ --cc=kernel-hardening@lists.openwall.com \ --cc=linux-arch@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kbuild@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-pci@vger.kernel.org \ --cc=masahiroy@kernel.org \ --cc=ndesaulniers@google.com \ --cc=paulmck@kernel.org \ --cc=peterz@infradead.org \ --cc=rostedt@goodmis.org \ --cc=will@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.