lkml.org 
[lkml]   [2008]   [Oct]   [22]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    From
    Subject[PATCH 05/11] ftrace: only have ftrace_kill atomic
    Only have a way to completely disable ftrace in atomic sections.

    Signed-off-by: Steven Rostedt <srostedt@redhat.com>
    ---
    include/linux/ftrace.h | 3 +--
    kernel/trace/ftrace.c | 43 ++-----------------------------------------
    kernel/trace/trace.c | 2 +-
    3 files changed, 4 insertions(+), 44 deletions(-)

    Index: linux-compile.git/include/linux/ftrace.h
    ===================================================================
    --- linux-compile.git.orig/include/linux/ftrace.h 2008-10-22 13:17:26.000000000 -0400
    +++ linux-compile.git/include/linux/ftrace.h 2008-10-22 13:17:29.000000000 -0400
    @@ -40,7 +40,7 @@ extern void ftrace_stub(unsigned long a0
    # define register_ftrace_function(ops) do { } while (0)
    # define unregister_ftrace_function(ops) do { } while (0)
    # define clear_ftrace_function(ops) do { } while (0)
    -static inline void ftrace_kill_atomic(void) { }
    +static inline void ftrace_kill(void) { }
    #endif /* CONFIG_FTRACE */

    #ifdef CONFIG_DYNAMIC_FTRACE
    @@ -97,7 +97,6 @@ static inline void ftrace_release(void *

    /* totally disable ftrace - can not re-enable after this */
    void ftrace_kill(void);
    -void ftrace_kill_atomic(void);

    static inline void tracer_disable(void)
    {
    Index: linux-compile.git/kernel/trace/ftrace.c
    ===================================================================
    --- linux-compile.git.orig/kernel/trace/ftrace.c 2008-10-22 13:17:26.000000000 -0400
    +++ linux-compile.git/kernel/trace/ftrace.c 2008-10-22 13:17:29.000000000 -0400
    @@ -606,7 +606,6 @@ ftrace_code_disable(struct dyn_ftrace *r
    print_ip_ins(" expected: ", call);
    print_ip_ins(" replace: ", nop);
    printk(KERN_CONT "\n");
    - break;

    rec->flags |= FTRACE_FL_FAILED;
    return 0;
    @@ -1526,22 +1525,6 @@ int ftrace_force_update(void)
    return ret;
    }

    -static void ftrace_force_shutdown(void)
    -{
    - struct task_struct *task;
    - int command = FTRACE_DISABLE_CALLS | FTRACE_UPDATE_TRACE_FUNC;
    -
    - mutex_lock(&ftraced_lock);
    - task = ftraced_task;
    - ftraced_task = NULL;
    - ftraced_suspend = -1;
    - ftrace_run_update_code(command);
    - mutex_unlock(&ftraced_lock);
    -
    - if (task)
    - kthread_stop(task);
    -}
    -
    static __init int ftrace_init_debugfs(void)
    {
    struct dentry *d_tracer;
    @@ -1734,17 +1717,16 @@ core_initcall(ftrace_dynamic_init);
    # define ftrace_shutdown() do { } while (0)
    # define ftrace_startup_sysctl() do { } while (0)
    # define ftrace_shutdown_sysctl() do { } while (0)
    -# define ftrace_force_shutdown() do { } while (0)
    #endif /* CONFIG_DYNAMIC_FTRACE */

    /**
    - * ftrace_kill_atomic - kill ftrace from critical sections
    + * ftrace_kill - kill ftrace
    *
    * This function should be used by panic code. It stops ftrace
    * but in a not so nice way. If you need to simply kill ftrace
    * from a non-atomic section, use ftrace_kill.
    */
    -void ftrace_kill_atomic(void)
    +void ftrace_kill(void)
    {
    ftrace_disabled = 1;
    ftrace_enabled = 0;
    @@ -1755,27 +1737,6 @@ void ftrace_kill_atomic(void)
    }

    /**
    - * ftrace_kill - totally shutdown ftrace
    - *
    - * This is a safety measure. If something was detected that seems
    - * wrong, calling this function will keep ftrace from doing
    - * any more modifications, and updates.
    - * used when something went wrong.
    - */
    -void ftrace_kill(void)
    -{
    - mutex_lock(&ftrace_sysctl_lock);
    - ftrace_disabled = 1;
    - ftrace_enabled = 0;
    -
    - clear_ftrace_function();
    - mutex_unlock(&ftrace_sysctl_lock);
    -
    - /* Try to totally disable ftrace */
    - ftrace_force_shutdown();
    -}
    -
    -/**
    * register_ftrace_function - register a function for profiling
    * @ops - ops structure that holds the function for profiling.
    *
    Index: linux-compile.git/kernel/trace/trace.c
    ===================================================================
    --- linux-compile.git.orig/kernel/trace/trace.c 2008-10-22 13:17:26.000000000 -0400
    +++ linux-compile.git/kernel/trace/trace.c 2008-10-22 13:17:29.000000000 -0400
    @@ -3097,7 +3097,7 @@ void ftrace_dump(void)
    dump_ran = 1;

    /* No turning back! */
    - ftrace_kill_atomic();
    + ftrace_kill();

    for_each_tracing_cpu(cpu) {
    atomic_inc(&global_trace.data[cpu]->disabled);
    --


    \
     
     \ /
      Last update: 2008-10-22 20:57    [W:0.039 / U:70.084 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site