summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFrederic Weisbecker <fweisbec@gmail.com>2008-12-16 22:08:58 +0100
committerIngo Molnar <mingo@elte.hu>2008-12-17 00:26:35 +0100
commit2c2d7329d8afa9efa3ec24e19a53e7be9d14f242 (patch)
tree12dd840eef0475e81e0dc8f616229a030d7caae4
parent9dfc3bc7d21864d47797d64b8d531d4dbbc0b618 (diff)
downloadop-kernel-dev-2c2d7329d8afa9efa3ec24e19a53e7be9d14f242.zip
op-kernel-dev-2c2d7329d8afa9efa3ec24e19a53e7be9d14f242.tar.gz
tracing/ftrace: use preempt_enable_no_resched_notrace in ring_buffer_time_stamp()
Impact: prevent a trace recursion After some tests with function graph tracer under x86-32, I saw some recursions caused by ring_buffer_time_stamp() that calls preempt_enable_no_notrace() which calls preempt_schedule() which is traced itself. This patch re-enables preemption without rescheduling. Signed-off-by: Frederic Weisbecker <fweisbec@gmail.com> Signed-off-by: Ingo Molnar <mingo@elte.hu>
-rw-r--r--kernel/trace/ring_buffer.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/trace/ring_buffer.c b/kernel/trace/ring_buffer.c
index 7f69cfea..eab81f9 100644
--- a/kernel/trace/ring_buffer.c
+++ b/kernel/trace/ring_buffer.c
@@ -107,7 +107,7 @@ u64 ring_buffer_time_stamp(int cpu)
preempt_disable_notrace();
/* shift to debug/test normalization and TIME_EXTENTS */
time = sched_clock() << DEBUG_SHIFT;
- preempt_enable_notrace();
+ preempt_enable_no_resched_notrace();
return time;
}
OpenPOWER on IntegriCloud