diff options
author | Steven Rostedt <srostedt@redhat.com> | 2009-09-14 17:18:02 +0200 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2009-09-14 17:18:02 +0200 |
commit | a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf (patch) | |
tree | 08c56167c5a3152c10c006e0aa67fe19f6e66831 /include/trace/ftrace.h | |
parent | tracing: add static to generated TRACE_EVENT functions (diff) | |
download | linux-a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf.tar.xz linux-a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf.zip |
tracing: have TRACE_EVENT macro use __flags to not shadow parameter
The generated functions of TRACE_EVENT uses "flags" in one of the
sub macros which shadows a parameter in the outside macro.
Simple fix is to make the submacro use __flags instead.
Discovered by sparse.
Reported-by: Jaswinder Singh Rajput <jaswinder@kernel.org>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'include/trace/ftrace.h')
-rw-r--r-- | include/trace/ftrace.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/trace/ftrace.h b/include/trace/ftrace.h index fa8ce03f836c..72a3b437b829 100644 --- a/include/trace/ftrace.h +++ b/include/trace/ftrace.h @@ -239,9 +239,9 @@ ftrace_format_##call(struct ftrace_event_call *unused, \ #undef __print_flags #define __print_flags(flag, delim, flag_array...) \ ({ \ - static const struct trace_print_flags flags[] = \ + static const struct trace_print_flags __flags[] = \ { flag_array, { -1, NULL }}; \ - ftrace_print_flags_seq(p, delim, flag, flags); \ + ftrace_print_flags_seq(p, delim, flag, __flags); \ }) #undef __print_symbolic |