summaryrefslogtreecommitdiffstats
path: root/kernel/trace
diff options
context:
space:
mode:
authorAbhishek Sagar <sagar.abhishek@gmail.com>2008-06-03 05:03:41 +0200
committerIngo Molnar <mingo@elte.hu>2008-06-10 11:59:05 +0200
commit34078a5e44db3cbed2e0ed580c29a39d94e0cd97 (patch)
tree30bc17928302d6e38f352ab7fb11d1907906d058 /kernel/trace
parentftrace: add debugfs entry 'failures' (diff)
downloadlinux-34078a5e44db3cbed2e0ed580c29a39d94e0cd97.tar.xz
linux-34078a5e44db3cbed2e0ed580c29a39d94e0cd97.zip
ftrace: prevent freeing of all failed updates
Steven Rostedt wrote: > If we unload a module and reload it, will it ever get converted again? The intent was always to filter core kernel functions to prevent their freeing. Here's a fix which should allow re-recording of module call-sites. Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/trace')
-rw-r--r--kernel/trace/ftrace.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index ad568c742bfc..0118979e211f 100644
--- a/kernel/trace/ftrace.c
+++ b/kernel/trace/ftrace.c
@@ -430,7 +430,7 @@ static void ftrace_replace_code(int enable)
if (failed && (rec->flags & FTRACE_FL_CONVERTED)) {
rec->flags |= FTRACE_FL_FAILED;
if ((system_state == SYSTEM_BOOTING) ||
- !kernel_text_address(rec->ip)) {
+ !core_kernel_text(rec->ip)) {
ftrace_del_hash(rec);
ftrace_free_rec(rec);
}
@@ -651,10 +651,9 @@ static int __ftrace_update_code(void *ignore)
ftrace_update_cnt++;
} else {
if ((system_state == SYSTEM_BOOTING) ||
- !kernel_text_address(p->ip)) {
+ !core_kernel_text(p->ip)) {
ftrace_del_hash(p);
ftrace_free_rec(p);
-
}
}
}