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>

diff --git a/kernel/trace/ftrace.c b/kernel/trace/ftrace.c
index ad568c742bfc7e5c6e6a6c89644c33d5b8900d66..0118979e211f34eaa2ad97e4904ef0807530b7bb 100644 (file)
--- 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);
-
                                }
                        }
                }