Message ID | 51cba452b38ae55049bd15b0aeac6060cc1105f2.1561634177.git.naveen.n.rao@linux.vnet.ibm.com (mailing list archive) |
---|---|
State | Changes Requested |
Headers | show |
Series | powerpc/ftrace: Patch out -mprofile-kernel instructions | expand |
Context | Check | Description |
---|---|---|
snowpatch_ozlabs/apply_patch | success | Successfully applied on branch next (c7d64b560ce80d8c44f082eee8352f0778a73195) |
snowpatch_ozlabs/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 22 lines checked |
Le 27/06/2019 à 13:23, Naveen N. Rao a écrit : > Since ftrace_replace_code() is a __weak function and can be overridden, > we need to expose the flags that can be set. So, move the flags enum to > the header file. > > Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org> > Signed-off-by: Naveen N. Rao <naveen.n.rao@linux.vnet.ibm.com> This series does apply anymore. We have a link to it in https://github.com/linuxppc/issues/issues/386 I'll flag it "change requested" Christophe
Christophe Leroy wrote: > > > Le 27/06/2019 à 13:23, Naveen N. Rao a écrit : >> Since ftrace_replace_code() is a __weak function and can be overridden, >> we need to expose the flags that can be set. So, move the flags enum to >> the header file. >> >> Reviewed-by: Steven Rostedt (VMware) <rostedt@goodmis.org> >> Signed-off-by: Naveen N. Rao <naveen.n.rao@linux.vnet.ibm.com> > > This series does apply anymore. > > We have a link to it in https://github.com/linuxppc/issues/issues/386 > > I'll flag it "change requested" There are a couple of changes being worked on as a prerequisite for this series. See: http://lkml.kernel.org/r/cover.1645096227.git.naveen.n.rao@linux.vnet.ibm.com - Naveen
diff --git a/include/linux/ftrace.h b/include/linux/ftrace.h index 25e2995d4a4c..e97789c95c4e 100644 --- a/include/linux/ftrace.h +++ b/include/linux/ftrace.h @@ -162,6 +162,11 @@ enum { FTRACE_OPS_FL_TRACE_ARRAY = 1 << 15, }; +enum { + FTRACE_MODIFY_ENABLE_FL = (1 << 0), + FTRACE_MODIFY_MAY_SLEEP_FL = (1 << 1), +}; + #ifdef CONFIG_DYNAMIC_FTRACE /* The hash used to know what functions callbacks trace */ struct ftrace_ops_hash { diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index 38277af44f5c..5710a6b3edc1 100644 --- a/kernel/trace/ftrace.c +++ b/kernel/trace/ftrace.c @@ -75,11 +75,6 @@ #define INIT_OPS_HASH(opsname) #endif -enum { - FTRACE_MODIFY_ENABLE_FL = (1 << 0), - FTRACE_MODIFY_MAY_SLEEP_FL = (1 << 1), -}; - struct ftrace_ops ftrace_list_end __read_mostly = { .func = ftrace_stub, .flags = FTRACE_OPS_FL_RECURSION_SAFE | FTRACE_OPS_FL_STUB,