diff options
| author | Carsten Emde <[email protected]> | 2009-12-06 14:02:44 +0100 | 
|---|---|---|
| committer | Steven Rostedt <[email protected]> | 2009-12-09 14:20:08 -0500 | 
| commit | f2942487ffb0c0a80b2312f667ea30dd55a24bb0 (patch) | |
| tree | 8e6e4c8ae8230f2648142c863e8482b7656e6614 /kernel/trace/trace_output.c | |
| parent | be1eca39319689aed7d3aedb9c3bece9469fe10f (diff) | |
tracing: Remove comparing of NULL to va_list in trace_array_vprintk()
Olof Johansson stated the following:
  Comparing a va_list with NULL is bogus. It's supposed to be treated like
  an opaque type and only be manipulated with va_* accessors.
Olof noticed that this code broke the ARM builds:
    kernel/trace/trace.c: In function 'trace_array_vprintk':
    kernel/trace/trace.c:1364: error: invalid operands to binary == (have 'va_list' and 'void *')
    kernel/trace/trace.c: In function 'tracing_mark_write':
    kernel/trace/trace.c:3349: error: incompatible type for argument 3 of 'trace_vprintk'
This patch partly reverts c13d2f7c3231e873f30db92b96c8caa48f100f33 and
re-installs the original mark_printk() mechanism.
Reported-by: Olof Johansson <[email protected]>
Signed-off-by: Carsten Emde <[email protected]>
LKML-Reference: <[email protected]>
Signed-off-by: Steven Rostedt <[email protected]>
Diffstat (limited to 'kernel/trace/trace_output.c')
0 files changed, 0 insertions, 0 deletions