From: "Steven Rostedt (Red Hat)" <rost...@goodmis.org>

The branch tracer should not be checking the trace_seq_printf() return value
as that will soon be void. There's a new trace_handle_return() helper function
that will return TRACE_TYPE_PARTIAL_LINE if the trace_seq overflowed
and TRACE_TYPE_HANDLED otherwise.

Reviewed-by: Petr Mladek <pmla...@suse.cz>
Signed-off-by: Steven Rostedt <rost...@goodmis.org>
---
 kernel/trace/trace_branch.c | 15 +++++++--------
 1 file changed, 7 insertions(+), 8 deletions(-)

diff --git a/kernel/trace/trace_branch.c b/kernel/trace/trace_branch.c
index a3916f68a1c7..7d6e2afde669 100644
--- a/kernel/trace/trace_branch.c
+++ b/kernel/trace/trace_branch.c
@@ -151,14 +151,13 @@ static enum print_line_t trace_branch_print(struct 
trace_iterator *iter,
 
        trace_assign_type(field, iter->ent);
 
-       if (trace_seq_printf(&iter->seq, "[%s] %s:%s:%d\n",
-                            field->correct ? "  ok  " : " MISS ",
-                            field->func,
-                            field->file,
-                            field->line))
-               return TRACE_TYPE_PARTIAL_LINE;
-
-       return TRACE_TYPE_HANDLED;
+       trace_seq_printf(&iter->seq, "[%s] %s:%s:%d\n",
+                        field->correct ? "  ok  " : " MISS ",
+                        field->func,
+                        field->file,
+                        field->line);
+
+       return trace_handle_return(&iter->seq);
 }
 
 static void branch_print_header(struct seq_file *s)
-- 
2.1.1


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to