]> Pileus Git - ~andy/linux/commit
tracing: Fix sparse warning with is_signed_type() macro
authorSteven Rostedt <srostedt@redhat.com>
Fri, 14 Dec 2012 18:15:08 +0000 (13:15 -0500)
committerSteven Rostedt <rostedt@goodmis.org>
Mon, 21 Jan 2013 18:22:28 +0000 (13:22 -0500)
commit418c59e49ddc77fcb7054f2c8d52c9d47403b43e
treea6ad61da008f95849076912a3485237d14ec3e46
parentc1bf08ac26e92122faab9f6c32ea8aba94612dae
tracing: Fix sparse warning with is_signed_type() macro

Sparse complains when is_signed_type() is used on a pointer.
This macro is needed for the format output used for ftrace
and perf, to know if a binary field is a signed type or not.
The is_signed_type() macro is used against all fields that are
recorded by events to automate the operation.

The problem sparse has is with the current way is_signed_type()
works:

  ((type)-1 < 0)

If "type" is a poiner, than sparse does not like it being compared
to an integer (zero). The simple fix is to just give zero the
same type. The runtime result stays the same.

Reported-by: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
include/linux/ftrace_event.h