]> Git Repo - linux.git/commitdiff
tracing: Fix trace_pipe behavior for instance traces
authorTahsin Erdogan <[email protected]>
Sun, 17 Sep 2017 10:23:48 +0000 (03:23 -0700)
committerSteven Rostedt (VMware) <[email protected]>
Tue, 19 Sep 2017 22:33:42 +0000 (18:33 -0400)
When reading data from trace_pipe, tracing_wait_pipe() performs a
check to see if tracing has been turned off after some data was read.
Currently, this check always looks at global trace state, but it
should be checking the trace instance where trace_pipe is located at.

Because of this bug, cat instances/i1/trace_pipe in the following
script will immediately exit instead of waiting for data:

cd /sys/kernel/debug/tracing
echo 0 > tracing_on
mkdir -p instances/i1
echo 1 > instances/i1/tracing_on
echo 1 > instances/i1/events/sched/sched_process_exec/enable
cat instances/i1/trace_pipe

Link: http://lkml.kernel.org/r/[email protected]
Cc: [email protected]
Fixes: 10246fa35d4f ("tracing: give easy way to clear trace buffer")
Signed-off-by: Tahsin Erdogan <[email protected]>
Signed-off-by: Steven Rostedt (VMware) <[email protected]>
kernel/trace/trace.c

index d3ca35f388037208b4e972c3708f4498ada1b816..752e5daf0896fc529876f8801a95e23118713338 100644 (file)
@@ -5680,7 +5680,7 @@ static int tracing_wait_pipe(struct file *filp)
                 *
                 * iter->pos will be 0 if we haven't read anything.
                 */
-               if (!tracing_is_on() && iter->pos)
+               if (!tracer_tracing_is_on(iter->tr) && iter->pos)
                        break;
 
                mutex_unlock(&iter->mutex);
This page took 0.070812 seconds and 4 git commands to generate.