tracing: Clean up trace_seq.c
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>
Fri, 20 Jun 2014 21:38:01 +0000 (17:38 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Tue, 1 Jul 2014 11:13:36 +0000 (07:13 -0400)
commit36aabfff50b6a03bcfd2c3cfbd7b83eb0a9ce0c1
treea8e9df6eb61e93288596fcd3e60ba138214118a2
parent12306276fabcb746a14979e96f43a13c724dec49
tracing: Clean up trace_seq.c

For using trace_seq_*() functions in NMI context, I posted a patch to move
it to the lib/ directory. This caused Andrew Morton to take a look at the code.
He went through and gave a lot of comments about missing kernel doc,
inconsistent types for the save variable, mix match of EXPORT_SYMBOL_GPL()
and EXPORT_SYMBOL() as well as missing EXPORT_SYMBOL*()s. There were
a few comments about the way variables were being compared (int vs uint).

All these were good review comments and should be implemented regardless of
if trace_seq.c should be moved to lib/ or not.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
include/linux/trace_seq.h
kernel/trace/trace_seq.c