ring-buffer: Put back the length if crossed page with add_timestamp
authorSteven Rostedt (Red Hat) <rostedt@goodmis.org>
Mon, 23 Nov 2015 22:35:24 +0000 (17:35 -0500)
committerSteven Rostedt <rostedt@goodmis.org>
Tue, 24 Nov 2015 14:27:25 +0000 (09:27 -0500)
Commit fcc742eaad7c "ring-buffer: Add event descriptor to simplify passing
data" added a descriptor that holds various data instead of passing around
several variables through parameters. The problem was that one of the
parameters was modified in a function and the code was designed not to have
an effect on that modified  parameter. Now that the parameter is a
descriptor and any modifications to it are non-volatile, the size of the
data could be unnecessarily expanded.

Remove the extra space added if a timestamp was added and the event went
across the page.

Cc: stable@vger.kernel.org # 4.3+
Fixes: fcc742eaad7c "ring-buffer: Add event descriptor to simplify passing data"
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
kernel/trace/ring_buffer.c

index 4dd6d5bc4e11904cfed838c2db2e044aea8ab81f..9c6045a27ba356252546971fb26dbae30149c89c 100644 (file)
@@ -2797,8 +2797,11 @@ rb_reserve_next_event(struct ring_buffer *buffer,
 
        event = __rb_reserve_next(cpu_buffer, &info);
 
-       if (unlikely(PTR_ERR(event) == -EAGAIN))
+       if (unlikely(PTR_ERR(event) == -EAGAIN)) {
+               if (info.add_timestamp)
+                       info.length -= RB_LEN_TIME_EXTEND;
                goto again;
+       }
 
        if (!event)
                goto out_fail;