• Steven Rostedt's avatar
    ftrace: use kretprobe trampoline name to test in output · b3aa5577
    Steven Rostedt authored
    Impact: ia64+tracing build fix
    
    When a function is kprobed, the return address is set to the
    kprobe_trampoline, or something similar. This caused the output
    of the trace to look confusing when the parent seemed to be this
    "kprobe_trampoline" function.
    
    To fix this, Abhishek Sagar added a test of the instruction pointer
    of the parent to see if it matched the kprobe_trampoline. If it
    did, the output would print a "[unknown/kretprobe'd]" instead.
    
    Unfortunately, not all archs do this the same way, and the trampoline
    function may not be exported, which causes failures in builds.
    
    This patch will compare the name instead of the pointer to see
    if it matches. This prevents us from depending on a function from
    being exported, and should work on all archs. The worst that can
    happen is that an arch might use a different name and then we
    go back to the confusing output. At least the arch will still build.
    Reported-by: default avatarAbhishek Sagar <sagar.abhishek@gmail.com>
    Signed-off-by: default avatarSteven Rostedt <srostedt@redhat.com>
    Signed-off-by: default avatarIngo Molnar <mingo@elte.hu>
    Tested-by: default avatarAbhishek Sagar <sagar.abhishek@gmail.com>
    Acked-by: default avatarAbhishek Sagar <sagar.abhishek@gmail.com>
    b3aa5577
trace.c 71.4 KB