ftrace: Remove unnecessary disabling of irqs
authorSteven Rostedt <srostedt@redhat.com>
Fri, 22 Apr 2011 02:41:35 +0000 (22:41 -0400)
committerSteven Rostedt <rostedt@goodmis.org>
Sat, 30 Apr 2011 02:52:55 +0000 (22:52 -0400)
The disabling of interrupts around ftrace_update_code() was used
to protect against the evil ftrace daemon from years past. But that
daemon has long been killed. It is safe to keep interrupts enabled
while updating the initial mcount into nops.

The ftrace_mutex is also held which keeps other users at bay.

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

index 4ff65599c9731f15c20cd4bd9125196f62edfe78..f199fb2e1d2c4380dcac0de6413ee37b6698a028 100644 (file)
@@ -2707,7 +2707,6 @@ static int ftrace_process_locs(struct module *mod,
 {
        unsigned long *p;
        unsigned long addr;
-       unsigned long flags;
 
        mutex_lock(&ftrace_lock);
        p = start;
@@ -2724,10 +2723,7 @@ static int ftrace_process_locs(struct module *mod,
                ftrace_record_ip(addr);
        }
 
-       /* disable interrupts to prevent kstop machine */
-       local_irq_save(flags);
        ftrace_update_code(mod);
-       local_irq_restore(flags);
        mutex_unlock(&ftrace_lock);
 
        return 0;