ftrace: Add context level recursion bit checking
authorSteven Rostedt <srostedt@redhat.com>
Fri, 2 Nov 2012 21:17:59 +0000 (17:17 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Wed, 23 Jan 2013 04:38:00 +0000 (23:38 -0500)
Currently for recursion checking in the function tracer, ftrace
tests a task_struct bit to determine if the function tracer had
recursed or not. If it has, then it will will return without going
further.

But this leads to races. If an interrupt came in after the bit
was set, the functions being traced would see that bit set and
think that the function tracer recursed on itself, and would return.

Instead add a bit for each context (normal, softirq, irq and nmi).

A check of which context the task is in is made before testing the
associated bit. Now if an interrupt preempts the function tracer
after the previous context has been set, the interrupt functions
can still be traced.

Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
kernel/trace/ftrace.c
kernel/trace/trace.h

index 1330969d8447708edc6214446e18bbe1b0496518..639b6ab1f04c5f0c155b791aa7c2d8d12dbb7233 100644 (file)
@@ -156,14 +156,27 @@ static void
 ftrace_global_list_func(unsigned long ip, unsigned long parent_ip,
                        struct ftrace_ops *op, struct pt_regs *regs)
 {
-       if (unlikely(trace_recursion_test(TRACE_GLOBAL_BIT)))
+       int bit;
+
+       if (in_interrupt()) {
+               if (in_nmi())
+                       bit = TRACE_GLOBAL_NMI_BIT;
+
+               else if (in_irq())
+                       bit = TRACE_GLOBAL_IRQ_BIT;
+               else
+                       bit = TRACE_GLOBAL_SIRQ_BIT;
+       } else
+               bit = TRACE_GLOBAL_BIT;
+
+       if (unlikely(trace_recursion_test(bit)))
                return;
 
-       trace_recursion_set(TRACE_GLOBAL_BIT);
+       trace_recursion_set(bit);
        do_for_each_ftrace_op(op, ftrace_global_list) {
                op->func(ip, parent_ip, op, regs);
        } while_for_each_ftrace_op(op);
-       trace_recursion_clear(TRACE_GLOBAL_BIT);
+       trace_recursion_clear(bit);
 }
 
 static void ftrace_pid_func(unsigned long ip, unsigned long parent_ip,
@@ -4132,14 +4145,27 @@ __ftrace_ops_list_func(unsigned long ip, unsigned long parent_ip,
                       struct ftrace_ops *ignored, struct pt_regs *regs)
 {
        struct ftrace_ops *op;
+       unsigned int bit;
 
        if (function_trace_stop)
                return;
 
-       if (unlikely(trace_recursion_test(TRACE_INTERNAL_BIT)))
-               return;
+       if (in_interrupt()) {
+               if (in_nmi())
+                       bit = TRACE_INTERNAL_NMI_BIT;
+
+               else if (in_irq())
+                       bit = TRACE_INTERNAL_IRQ_BIT;
+               else
+                       bit = TRACE_INTERNAL_SIRQ_BIT;
+       } else
+               bit = TRACE_INTERNAL_BIT;
+
+       if (unlikely(trace_recursion_test(bit)))
+                       return;
+
+       trace_recursion_set(bit);
 
-       trace_recursion_set(TRACE_INTERNAL_BIT);
        /*
         * Some of the ops may be dynamically allocated,
         * they must be freed after a synchronize_sched().
@@ -4150,7 +4176,7 @@ __ftrace_ops_list_func(unsigned long ip, unsigned long parent_ip,
                        op->func(ip, parent_ip, op, regs);
        } while_for_each_ftrace_op(op);
        preempt_enable_notrace();
-       trace_recursion_clear(TRACE_INTERNAL_BIT);
+       trace_recursion_clear(bit);
 }
 
 /*
index c75d7988902caa5b7ce38d3cdad888d9ade9f105..fe6ccff9cc7054f592161f3d75a4629ddeedd553 100644 (file)
@@ -299,8 +299,14 @@ struct tracer {
 
 /* for function tracing recursion */
 #define TRACE_INTERNAL_BIT             (1<<11)
-#define TRACE_GLOBAL_BIT               (1<<12)
-#define TRACE_CONTROL_BIT              (1<<13)
+#define TRACE_INTERNAL_NMI_BIT         (1<<12)
+#define TRACE_INTERNAL_IRQ_BIT         (1<<13)
+#define TRACE_INTERNAL_SIRQ_BIT                (1<<14)
+#define TRACE_GLOBAL_BIT               (1<<15)
+#define TRACE_GLOBAL_NMI_BIT           (1<<16)
+#define TRACE_GLOBAL_IRQ_BIT           (1<<17)
+#define TRACE_GLOBAL_SIRQ_BIT          (1<<18)
+#define TRACE_CONTROL_BIT              (1<<19)
 
 /*
  * Abuse of the trace_recursion.
@@ -309,7 +315,7 @@ struct tracer {
  * was called in irq context but we have irq tracing off. Since this
  * can only be modified by current, we can reuse trace_recursion.
  */
-#define TRACE_IRQ_BIT                  (1<<13)
+#define TRACE_IRQ_BIT                  (1<<20)
 
 #define trace_recursion_set(bit)       do { (current)->trace_recursion |= (bit); } while (0)
 #define trace_recursion_clear(bit)     do { (current)->trace_recursion &= ~(bit); } while (0)