[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <225ff37c-0e50-48d5-bb4c-e8a63e72991a@arm.com>
Date: Wed, 24 Jul 2024 16:00:04 +0100
From: Leo Yan <leo.yan@....com>
To: James Clark <james.clark@...aro.org>, coresight@...ts.linaro.org,
gankulkarni@...amperecomputing.com, mike.leach@...aro.org,
suzuki.poulose@....com
Cc: Peter Zijlstra <peterz@...radead.org>, Ingo Molnar <mingo@...hat.com>,
Arnaldo Carvalho de Melo <acme@...nel.org>,
Namhyung Kim <namhyung@...nel.org>, Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...nel.org>, Ian Rogers <irogers@...gle.com>,
Adrian Hunter <adrian.hunter@...el.com>,
"Liang, Kan" <kan.liang@...ux.intel.com>,
Ruidong Tian <tianruidong@...ux.alibaba.com>,
Benjamin Gray <bgray@...ux.ibm.com>, linux-perf-users@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] perf scripts python: cs-etm: Update example to provide
vmlinux path to Perf
On 7/24/2024 3:33 PM, James Clark wrote:
>
> The example shows the vmlinux path being given to the script, but this
> only works when running on the target. If the script is run off the
> target, then confusingly the vmlinux argument also needs to be given to
> Perf as well.
>
> Without going into too much detail in the example about when it is or
> isn't required, just include it. It doesn't do any harm even when
> running on the target. Now the example command works in both places.
>
> Signed-off-by: James Clark <james.clark@...aro.org>
The change makes senses to me. Just check a bit, does it need to add the same
option for the command "only source line and symbols"?
Thanks,
Leo
> ---
> tools/perf/scripts/python/arm-cs-trace-disasm.py | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tools/perf/scripts/python/arm-cs-trace-disasm.py b/tools/perf/scripts/python/arm-cs-trace-disasm.py
> index 7aff02d84ffb..4aeb9b497f7a 100755
> --- a/tools/perf/scripts/python/arm-cs-trace-disasm.py
> +++ b/tools/perf/scripts/python/arm-cs-trace-disasm.py
> @@ -19,10 +19,10 @@ from perf_trace_context import perf_set_itrace_options, \
> # Below are some example commands for using this script.
> #
> # Output disassembly with objdump:
> -# perf script -s scripts/python/arm-cs-trace-disasm.py \
> +# perf script -k path/to/vmlinux -s scripts/python/arm-cs-trace-disasm.py \
> # -- -d objdump -k path/to/vmlinux
> # Output disassembly with llvm-objdump:
> -# perf script -s scripts/python/arm-cs-trace-disasm.py \
> +# perf script -k path/to/vmlinux -s scripts/python/arm-cs-trace-disasm.py \
> # -- -d llvm-objdump-11 -k path/to/vmlinux
> # Output only source line and symbols:
> # perf script -s scripts/python/arm-cs-trace-disasm.py
> --
> 2.34.1
>
Powered by blists - more mailing lists