diff options
author | Abhishek Sagar <sagar.abhishek@gmail.com> | 2008-06-03 08:33:41 +0530 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-06-10 11:59:05 +0200 |
commit | 34078a5e44db3cbed2e0ed580c29a39d94e0cd97 (patch) | |
tree | 30bc17928302d6e38f352ab7fb11d1907906d058 /kernel/trace | |
parent | eb9a7bf09172f409c10ec9560adeea95bb4045f5 (diff) | |
download | op-kernel-dev-34078a5e44db3cbed2e0ed580c29a39d94e0cd97.zip op-kernel-dev-34078a5e44db3cbed2e0ed580c29a39d94e0cd97.tar.gz |
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.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c index ad568c7..0118979 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); - } } } |