ftrace: Simplify ftrace hash lookup code in clear_func_from_hash()
authorChangbin Du <changbin.du@gmail.com>
Tue, 10 Sep 2019 14:33:36 +0000 (22:33 +0800)
committerSteven Rostedt (VMware) <rostedt@goodmis.org>
Tue, 17 Sep 2019 15:21:20 +0000 (11:21 -0400)
Function ftrace_lookup_ip() will check empty hash table. So we don't
need extra check outside.

Link: http://lkml.kernel.org/r/20190910143336.13472-1-changbin.du@gmail.com
Signed-off-by: Changbin Du <changbin.du@gmail.com>
Signed-off-by: Steven Rostedt (VMware) <rostedt@goodmis.org>
kernel/trace/ftrace.c

index f9821a3..c4cc048 100644 (file)
@@ -6036,11 +6036,7 @@ clear_func_from_hash(struct ftrace_init_func *func, struct ftrace_hash *hash)
 {
        struct ftrace_func_entry *entry;
 
-       if (ftrace_hash_empty(hash))
-               return;
-
-       entry = __ftrace_lookup_ip(hash, func->ip);
-
+       entry = ftrace_lookup_ip(hash, func->ip);
        /*
         * Do not allow this rec to match again.
         * Yeah, it may waste some memory, but will be removed