From: Masami Hiramatsu (Google) <[email protected]>

Remove unneeded warnings for handled errors from __arm_kprobe_ftrace()
because all caller handled the error correctly.

Reported-by: Zw Tang <[email protected]>
Closes: 
https://lore.kernel.org/all/caphj_v+j6ydb_wx2nhxu6kh466dt_nydsas-1i_y8s7tqy-...@mail.gmail.com/
Fixes: 9c89bb8e3272 ("kprobes: treewide: Cleanup the error messages for 
kprobes")
Cc: [email protected]
Signed-off-by: Masami Hiramatsu (Google) <[email protected]>
---
 kernel/kprobes.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kernel/kprobes.c b/kernel/kprobes.c
index ab25b4aa9095..2c2b7bd24dd4 100644
--- a/kernel/kprobes.c
+++ b/kernel/kprobes.c
@@ -1144,12 +1144,12 @@ static int __arm_kprobe_ftrace(struct kprobe *p, struct 
ftrace_ops *ops,
        lockdep_assert_held(&kprobe_mutex);
 
        ret = ftrace_set_filter_ip(ops, (unsigned long)p->addr, 0, 0);
-       if (WARN_ONCE(ret < 0, "Failed to arm kprobe-ftrace at %pS (error 
%d)\n", p->addr, ret))
+       if (ret < 0)
                return ret;
 
        if (*cnt == 0) {
                ret = register_ftrace_function(ops);
-               if (WARN(ret < 0, "Failed to register kprobe-ftrace (error 
%d)\n", ret)) {
+               if (ret < 0) {
                        /*
                         * At this point, sinec ops is not registered, we 
should be sefe from
                         * registering empty filter.


Reply via email to