ftrace: Cleanup ftrace_dyn_arch_init()
authorWeizhao Ouyang <o451686892@gmail.com>
Thu, 9 Sep 2021 09:02:16 +0000 (17:02 +0800)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Fri, 8 Oct 2021 23:41:39 +0000 (19:41 -0400)
Most of ARCHs use empty ftrace_dyn_arch_init(), introduce a weak common
ftrace_dyn_arch_init() to cleanup them.

Link: https://lkml.kernel.org/r/20210909090216.1955240-1-o451686892@gmail.com
Acked-by: Heiko Carstens <hca@linux.ibm.com> (s390)
Acked-by: Helge Deller <deller@gmx.de> (parisc)
Signed-off-by: Weizhao Ouyang <o451686892@gmail.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
13 files changed:
arch/arm/kernel/ftrace.c
arch/arm64/kernel/ftrace.c
arch/csky/kernel/ftrace.c
arch/ia64/kernel/ftrace.c
arch/microblaze/kernel/ftrace.c
arch/nds32/kernel/ftrace.c
arch/parisc/kernel/ftrace.c
arch/riscv/kernel/ftrace.c
arch/s390/kernel/ftrace.c
arch/sh/kernel/ftrace.c
arch/sparc/kernel/ftrace.c
arch/x86/kernel/ftrace.c
kernel/trace/ftrace.c

index 3c83b5d..a006585 100644 (file)
@@ -193,11 +193,6 @@ int ftrace_make_nop(struct module *mod,
 
        return ret;
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif /* CONFIG_DYNAMIC_FTRACE */
 
 #ifdef CONFIG_FUNCTION_GRAPH_TRACER
index 7f467bd..fc62dfe 100644 (file)
@@ -236,11 +236,6 @@ void arch_ftrace_update_code(int command)
        command |= FTRACE_MAY_SLEEP;
        ftrace_modify_all_code(command);
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif /* CONFIG_DYNAMIC_FTRACE */
 
 #ifdef CONFIG_FUNCTION_GRAPH_TRACER
index b4a7ec1..50bfcf1 100644 (file)
@@ -133,11 +133,6 @@ int ftrace_update_ftrace_func(ftrace_func_t func)
                                (unsigned long)func, true, true);
        return ret;
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif /* CONFIG_DYNAMIC_FTRACE */
 
 #ifdef CONFIG_DYNAMIC_FTRACE_WITH_REGS
index b2ab2d5..d6360fd 100644 (file)
@@ -194,9 +194,3 @@ int ftrace_update_ftrace_func(ftrace_func_t func)
        flush_icache_range(addr, addr + 16);
        return 0;
 }
-
-/* run from kstop_machine */
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
index 224eea4..188749d 100644 (file)
@@ -163,11 +163,6 @@ int ftrace_make_call(struct dyn_ftrace *rec, unsigned long addr)
        return ret;
 }
 
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
-
 int ftrace_update_ftrace_func(ftrace_func_t func)
 {
        unsigned long ip = (unsigned long)(&ftrace_call);
index 0e23e3a..f0ef484 100644 (file)
@@ -84,11 +84,6 @@ void _ftrace_caller(unsigned long parent_ip)
        /* restore all state needed by the compiler epilogue */
 }
 
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
-
 static unsigned long gen_sethi_insn(unsigned long addr)
 {
        unsigned long opcode = 0x46000000;
index 0a1e75a..01581f7 100644 (file)
@@ -94,11 +94,6 @@ int ftrace_disable_ftrace_graph_caller(void)
 #endif
 
 #ifdef CONFIG_DYNAMIC_FTRACE
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 int ftrace_update_ftrace_func(ftrace_func_t func)
 {
        return 0;
index 7f1e520..4716f4c 100644 (file)
@@ -154,11 +154,6 @@ int ftrace_update_ftrace_func(ftrace_func_t func)
 
        return ret;
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif
 
 #ifdef CONFIG_DYNAMIC_FTRACE_WITH_REGS
index 1d94ffd..5165bf3 100644 (file)
@@ -262,11 +262,6 @@ int ftrace_update_ftrace_func(ftrace_func_t func)
        return 0;
 }
 
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
-
 void arch_ftrace_update_code(int command)
 {
        if (ftrace_shared_hotpatch_trampoline(NULL))
index 295c433..930001b 100644 (file)
@@ -252,11 +252,6 @@ int ftrace_make_call(struct dyn_ftrace *rec, unsigned long addr)
 
        return ftrace_modify_code(rec->ip, old, new);
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif /* CONFIG_DYNAMIC_FTRACE */
 
 #ifdef CONFIG_FUNCTION_GRAPH_TRACER
index 684b84c..eaead3d 100644 (file)
@@ -82,11 +82,6 @@ int ftrace_update_ftrace_func(ftrace_func_t func)
        new = ftrace_call_replace(ip, (unsigned long)func);
        return ftrace_modify_code(ip, old, new);
 }
-
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
 #endif
 
 #ifdef CONFIG_FUNCTION_GRAPH_TRACER
index 1b3ce3b..23d221a 100644 (file)
@@ -252,11 +252,6 @@ void arch_ftrace_update_code(int command)
        ftrace_modify_all_code(command);
 }
 
-int __init ftrace_dyn_arch_init(void)
-{
-       return 0;
-}
-
 /* Currently only x86_64 supports dynamic trampolines */
 #ifdef CONFIG_X86_64
 
index 0a0dbc2..2c3e976 100644 (file)
@@ -6847,6 +6847,11 @@ void __init ftrace_free_init_mem(void)
        ftrace_free_mem(NULL, start, end);
 }
 
+int __init __weak ftrace_dyn_arch_init(void)
+{
+       return 0;
+}
+
 void __init ftrace_init(void)
 {
        extern unsigned long __start_mcount_loc[];