From patchwork Mon Jul 17 20:39:47 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Steven Rostedt X-Patchwork-Id: 121596 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:c923:0:b0:3e4:2afc:c1 with SMTP id j3csp1339156vqt; Mon, 17 Jul 2023 13:55:02 -0700 (PDT) X-Google-Smtp-Source: APBJJlHGojK7D4Vj72kbUYRYHUBHX0wgKXPNPnn8qX9qevSlW4oLC4PJb2tCFzu0VNA50WV2IHlB X-Received: by 2002:a05:6a00:2d17:b0:668:7090:bf18 with SMTP id fa23-20020a056a002d1700b006687090bf18mr12008728pfb.34.1689627302536; Mon, 17 Jul 2023 13:55:02 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1689627302; cv=none; d=google.com; s=arc-20160816; b=m2UV3hpoylgV833R1EYatIZGAdlaDhfIqHlfwBP4JSuEYmSy0eGozacB9A62f3KSqI zGENl8xPDhNnXfOEALE6C9yhM7bHl01NPGphg069quOlRpqlE865cVjec4zgpTeVT80b NXMv5X4VFfiDuGiq+glgLQGepFOmjPi3cO6sbUahwrbCoLDNTrXiHR9OGSkBncmvx4Ys xXU0c8BITpqtpSaUqo4JTPV8tQtkhAYTJ71bTcxS/SaV2JGGvBBcuhNLT4zhsUP4tAT7 TiKBTQEL5EEkrIp/j7Q1Y6IeZlbQLEYYtH5nBiUreYxcxSW1ARHha4MH1zet/AWrU5qp b8EQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :message-id:subject:cc:to:from:date; bh=jdss6l+/Q7fmB/ltRx/TWx6a+GVD9Y3AVyoXyvZWjvw=; fh=rUHGDHCpkay18/bKjY/Oi8Lr8Udus8p97wPZ0FMwRkI=; b=i88fAj9SIAv5Y008jXN2zEgWzDbm2OMdZj/nXz0f/hlSeNZ+3AglmRqSGAPp142fsa HU7E35l8sjPdKPZ7es1Twd35zBi8N3vHHUOyRLy+cv6eTTlW6Gbp5SXVuX9NAVeC9OLd gqsyN17cc32F1DUVDHjWTvtd1sGdbg2Rkqas6n9G5TkTusIAGUqdVojz0E8K1dwrtF5C 3X9dkY6+60hHY/2pYjaQ2U1NrTSR080v7gaqZZVG6Rn+L1cx7kV5FpUyhBCj7smofuqg 4nPk81NCM9FkHC+sfcNFb73TvqoSiOd5hm8NlNHv/VgwViMmIIsZ4yfQT2kEjHcYOxLd 0UEQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id j3-20020a056a00234300b00666878f8de7si347731pfj.397.2023.07.17.13.54.49; Mon, 17 Jul 2023 13:55:02 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230451AbjGQUjx (ORCPT + 99 others); Mon, 17 Jul 2023 16:39:53 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:38552 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229634AbjGQUjv (ORCPT ); Mon, 17 Jul 2023 16:39:51 -0400 Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 81302103; Mon, 17 Jul 2023 13:39:50 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 204666125F; Mon, 17 Jul 2023 20:39:50 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1518EC433C7; Mon, 17 Jul 2023 20:39:48 +0000 (UTC) Date: Mon, 17 Jul 2023 16:39:47 -0400 From: Steven Rostedt To: Linux Trace Kernel , LKML Cc: Masami Hiramatsu , Mark Rutland Subject: [PATCH] tracing/probes: Differentiate BTF support from start of function Message-ID: <20230717163947.4969316a@gandalf.local.home> X-Mailer: Claws Mail 3.19.1 (GTK+ 2.24.33; x86_64-pc-linux-gnu) MIME-Version: 1.0 X-Spam-Status: No, score=-1.7 required=5.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,RCVD_IN_DNSWL_BLOCKED,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=no autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1771702638488832240 X-GMAIL-MSGID: 1771702638488832240 From: "Steven Rostedt (Google)" Having a typo in the function name to trace one of the arguments using BTF, gave me an error message that BTF was not supported by this kernel: ~# echo 'p:lock __raw_spin_lock lock=lock' > /sys/kernel/tracing/kprobe_events -bash: echo: write error: Invalid argument ~# cat /sys/kernel/tracing/error_log [ 34.382149] trace_kprobe: error: BTF is not available or not supported Command: p:lock __raw_spin_lock lock=lock ^ I added two underscores instead of one underscore for the function _raw_spin_lock(). But I spent too much time trying to figure out why BTF was not supported, because in reality it was! The problem was that the function wasn't found (this could happen if a function offset is used as well). Instead, add a new error message that gives a better description of the problem. ~# echo 'p:lock __raw_spin_lock lock=lock' > /sys/kernel/tracing/kprobe_events -bash: echo: write error: Invalid argument ~# cat /sys/kernel/tracing/error_log [ 45.922742] trace_kprobe: error: BTF arg can be used only on function entry Command: p:lock __raw_spin_lock lock=lock ^ Fixes: b576e09701c7d ("tracing/probes: Support function parameters if BTF is available") Signed-off-by: Steven Rostedt (Google) Acked-by: Masami Hiramatsu (Google) --- kernel/trace/trace_probe.c | 2 +- kernel/trace/trace_probe.h | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/kernel/trace/trace_probe.c b/kernel/trace/trace_probe.c index cf504efd0694..328a51b1d068 100644 --- a/kernel/trace/trace_probe.c +++ b/kernel/trace/trace_probe.c @@ -1023,7 +1023,7 @@ parse_probe_arg(char *arg, const struct fetch_type *type, default: if (isalpha(arg[0]) || arg[0] == '_') { /* BTF variable */ if (!tparg_is_function_entry(ctx->flags)) { - trace_probe_log_err(ctx->offset, NOSUP_BTFARG); + trace_probe_log_err(ctx->offset, NOFENTRY_BTFARG); return -EINVAL; } ret = parse_btf_arg(arg, pcode, end, ctx); diff --git a/kernel/trace/trace_probe.h b/kernel/trace/trace_probe.h index 604d6fb9c5ff..554fe2995ca1 100644 --- a/kernel/trace/trace_probe.h +++ b/kernel/trace/trace_probe.h @@ -493,6 +493,7 @@ extern int traceprobe_define_arg_fields(struct trace_event_call *event_call, C(BAD_ATTACH_ARG, "Attached event does not have this field"),\ C(NO_EP_FILTER, "No filter rule after 'if'"), \ C(NOSUP_BTFARG, "BTF is not available or not supported"), \ + C(NOFENTRY_BTFARG, "BTF arg can be used only on function entry"), \ C(NO_BTFARG, "This variable is not found at this probe point"),\ C(NO_BTF_ENTRY, "No BTF entry for this probe point"), \ C(BAD_VAR_ARGS, "$arg* must be an independent parameter without name etc."),\