From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id DC96BFA3755 for ; Fri, 13 Sep 2024 13:41:43 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type:In-Reply-To:From:References:Cc:To:Subject:MIME-Version:Date: Message-ID:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=90rBWxsHwqIq/UEH5bkPRLfBSWO+RKXquYpk21qvzk4=; b=aOxEee/PGDSkqYixFzc+FYgxli S9lynrflbr3tp23SV2Iru2b54I/UpAfBk0BDEg9jOz+laJpviC1YWMKJYjQcX2FzvmyYaOARc0UlT 4rGEm+3ws0BAKj7uXz6LnVreYGoBj3Pn08/0zj6a65RRoT2v3nL2ka2JoGaGQ5moe0KSZLiq/uA/P zboIXxmYDMpTt9Jy/p7xoGeAxPW45NN0+4GTA/QK0Tu3BPO8C+GKMRKkuBUSQEBQbVNsEgSNvPtS6 3i84isLAwnvSnDe8pAxtAyvrVJ0qzPXSZ2PQ7FoIXISpa74tw6A6qgL4SykjOgD7UrvRqSSrnb32b /h9hILVw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sp6YH-0000000G40D-0qcT; Fri, 13 Sep 2024 13:41:33 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1sp6XC-0000000G3r3-37iP for linux-arm-kernel@lists.infradead.org; Fri, 13 Sep 2024 13:40:28 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7649213D5; Fri, 13 Sep 2024 06:40:55 -0700 (PDT) Received: from [10.2.76.71] (e132581.arm.com [10.2.76.71]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 2624D3F73B; Fri, 13 Sep 2024 06:40:23 -0700 (PDT) Message-ID: Date: Fri, 13 Sep 2024 14:40:21 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 3/7] perf scripting python: Add function to get a config value To: James Clark , linux-perf-users@vger.kernel.org, gankulkarni@os.amperecomputing.com, coresight@lists.linaro.org, scclevenger@os.amperecomputing.com Cc: Peter Zijlstra , Ingo Molnar , Arnaldo Carvalho de Melo , Namhyung Kim , Mark Rutland , Alexander Shishkin , Jiri Olsa , Ian Rogers , Adrian Hunter , "Liang, Kan" , Suzuki K Poulose , Mike Leach , John Garry , Will Deacon , Leo Yan , Ben Gainey , Ruidong Tian , Benjamin Gray , Mathieu Poirier , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20240912151143.1264483-1-james.clark@linaro.org> <20240912151143.1264483-4-james.clark@linaro.org> Content-Language: en-US From: Leo Yan In-Reply-To: <20240912151143.1264483-4-james.clark@linaro.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240913_064026_910596_CD4FF21C X-CRM114-Status: GOOD ( 22.49 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 9/12/24 16:11, James Clark wrote: > Warning: EXTERNAL SENDER, use caution when opening links or attachments. > > > This can be used to get config values like which objdump Perf uses for > disassembly. > > Signed-off-by: James Clark Reviewed-by: Leo Yan > --- > .../perf/Documentation/perf-script-python.txt | 2 +- > .../scripts/python/Perf-Trace-Util/Context.c | 11 ++++++++++ > tools/perf/util/config.c | 22 +++++++++++++++++++ > tools/perf/util/config.h | 1 + > 4 files changed, 35 insertions(+), 1 deletion(-) > > diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt > index 13e37e9385ee..27a1cac6fe76 100644 > --- a/tools/perf/Documentation/perf-script-python.txt > +++ b/tools/perf/Documentation/perf-script-python.txt > @@ -624,7 +624,7 @@ as perf_trace_context.perf_script_context . > perf_set_itrace_options(context, itrace_options) - set --itrace options if they have not been set already > perf_sample_srcline(context) - returns source_file_name, line_number > perf_sample_srccode(context) - returns source_file_name, line_number, source_line > - > + perf_config_get(config_name) - returns the value of the named config item, or None if unset > > Util.py Module > ~~~~~~~~~~~~~~ > diff --git a/tools/perf/scripts/python/Perf-Trace-Util/Context.c b/tools/perf/scripts/python/Perf-Trace-Util/Context.c > index 3954bd1587ce..01f54d6724a5 100644 > --- a/tools/perf/scripts/python/Perf-Trace-Util/Context.c > +++ b/tools/perf/scripts/python/Perf-Trace-Util/Context.c > @@ -12,6 +12,7 @@ > #define PY_SSIZE_T_CLEAN > > #include > +#include "../../../util/config.h" > #include "../../../util/trace-event.h" > #include "../../../util/event.h" > #include "../../../util/symbol.h" > @@ -182,6 +183,15 @@ static PyObject *perf_sample_srccode(PyObject *obj, PyObject *args) > return perf_sample_src(obj, args, true); > } > > +static PyObject *__perf_config_get(PyObject *obj, PyObject *args) > +{ > + const char *config_name; > + > + if (!PyArg_ParseTuple(args, "s", &config_name)) > + return NULL; > + return Py_BuildValue("s", perf_config_get(config_name)); > +} > + > static PyMethodDef ContextMethods[] = { > #ifdef HAVE_LIBTRACEEVENT > { "common_pc", perf_trace_context_common_pc, METH_VARARGS, > @@ -199,6 +209,7 @@ static PyMethodDef ContextMethods[] = { > METH_VARARGS, "Get source file name and line number."}, > { "perf_sample_srccode", perf_sample_srccode, > METH_VARARGS, "Get source file name, line number and line."}, > + { "perf_config_get", __perf_config_get, METH_VARARGS, "Get perf config entry"}, > { NULL, NULL, 0, NULL} > }; > > diff --git a/tools/perf/util/config.c b/tools/perf/util/config.c > index 7a650de0db83..68f9407ca74b 100644 > --- a/tools/perf/util/config.c > +++ b/tools/perf/util/config.c > @@ -912,6 +912,7 @@ void set_buildid_dir(const char *dir) > struct perf_config_scan_data { > const char *name; > const char *fmt; > + const char *value; > va_list args; > int ret; > }; > @@ -939,3 +940,24 @@ int perf_config_scan(const char *name, const char *fmt, ...) > > return d.ret; > } > + > +static int perf_config_get_cb(const char *var, const char *value, void *data) > +{ > + struct perf_config_scan_data *d = data; > + > + if (!strcmp(var, d->name)) > + d->value = value; > + > + return 0; > +} > + > +const char *perf_config_get(const char *name) > +{ > + struct perf_config_scan_data d = { > + .name = name, > + .value = NULL, > + }; > + > + perf_config(perf_config_get_cb, &d); > + return d.value; > +} > diff --git a/tools/perf/util/config.h b/tools/perf/util/config.h > index 2e5e808928a5..9971313d61c1 100644 > --- a/tools/perf/util/config.h > +++ b/tools/perf/util/config.h > @@ -30,6 +30,7 @@ typedef int (*config_fn_t)(const char *, const char *, void *); > int perf_default_config(const char *, const char *, void *); > int perf_config(config_fn_t fn, void *); > int perf_config_scan(const char *name, const char *fmt, ...) __scanf(2, 3); > +const char *perf_config_get(const char *name); > int perf_config_set(struct perf_config_set *set, > config_fn_t fn, void *data); > int perf_config_int(int *dest, const char *, const char *); > -- > 2.34.1 >