diff options
author | Steven Rostedt (Red Hat) <rostedt@goodmis.org> | 2014-11-17 13:12:22 -0500 |
---|---|---|
committer | Steven Rostedt <rostedt@goodmis.org> | 2014-11-19 22:01:14 -0500 |
commit | 74f06bb72347302a19aac087314388ebd0e4fee9 (patch) | |
tree | 214e56f4c4fc93f1954631c1f28bcdfa65de627e /kernel/trace/seq_buf.c | |
parent | eeab98154dc0b49afd398afdd71c464a8af5911f (diff) |
tracing: Clean up tracing_fill_pipe_page()
The function tracing_fill_pipe_page() logic is a little confusing with the
use of count saving the seq.len and reusing it.
Instead of subtracting a number that is calculated from the saved
value of the seq.len from seq.len, just save the seq.len at the start
and if we need to reset it, just assign it again.
When the seq_buf overflow is len == size + 1, the current logic will
break. Changing it to use a saved length for resetting back to the
original value is more robust and will work when we change the way
seq_buf sets the overflow.
Link: http://lkml.kernel.org/r/20141118161546.GJ23958@pathway.suse.cz
Reviewed-by: Petr Mladek <pmladek@suse.cz>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/seq_buf.c')
0 files changed, 0 insertions, 0 deletions