diff options
| author | Steven Rostedt <srostedt@redhat.com> | 2009-09-14 11:18:02 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2009-09-14 11:18:02 -0400 |
| commit | a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf (patch) | |
| tree | 08c56167c5a3152c10c006e0aa67fe19f6e66831 /include/trace/ftrace.h | |
| parent | ec827c7ece8901044e6b3f92aeea489be9e1bcf7 (diff) | |
| download | op-kernel-dev-a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf.zip op-kernel-dev-a48f494e1dbdcf4fb7c02100ae3208c4c1daecbf.tar.gz | |
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 fa8ce03..72a3b43 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 |
