1. 14 Feb, 2015 1 commit
  2. 20 Nov, 2014 3 commits
  3. 19 Nov, 2014 2 commits
  4. 01 Jul, 2014 4 commits
    • Steven Rostedt (Red Hat)'s avatar
      tracing: Remove trace_seq_reserve() · 9096032f
      Steven Rostedt (Red Hat) authored
      trace_seq_reserve() has no users in the kernel, it just wastes space.
      Remove it.
      
      Cc: Eduard - Gabriel Munteanu <eduard.munteanu@linux360.ro>
      Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
      9096032f
    • Steven Rostedt (Red Hat)'s avatar
      tracing: Make trace_seq_putmem_hex() more robust · 6d2289f3
      Steven Rostedt (Red Hat) authored
      Currently trace_seq_putmem_hex() can only take as a parameter a pointer
      to something that is 8 bytes or less, otherwise it will overflow the
      buffer. This is protected by a macro that encompasses the call to
      trace_seq_putmem_hex() that has a BUILD_BUG_ON() for the variable before
      it is passed in. This is not very robust and if trace_seq_putmem_hex() ever
      gets used outside that macro it will cause issues.
      
      Instead of only being able to produce a hex output of memory that is for
      a single word, change it to be more robust and allow any size input.
      Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
      6d2289f3
    • Steven Rostedt (Red Hat)'s avatar
      tracing: Clean up trace_seq.c · 36aabfff
      Steven Rostedt (Red Hat) authored
      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: default avatarSteven Rostedt <rostedt@goodmis.org>
      36aabfff
    • Steven Rostedt (Red Hat)'s avatar
      tracing: Move the trace_seq_* functions into its own trace_seq.c file · 12306276
      Steven Rostedt (Red Hat) authored
      The trace_seq_*() functions are a nice utility that allows users to manipulate
      buffers with printf() like formats. It has its own trace_seq.h header in
      include/linux and should be in its own file. Being tied with trace_output.c
      is rather awkward.
      Signed-off-by: default avatarSteven Rostedt <rostedt@goodmis.org>
      12306276