Commit ae8e4f40 authored by James Clark's avatar James Clark Committed by Arnaldo Carvalho de Melo

perf scripts python cs-etm: Restore first sample log in verbose mode

The linked commit moved the early return on the first sample to before
the verbose log, so move the log earlier too. Now the first sample is
also logged and not skipped.

Fixes: 2d98dbb4 ("perf scripts python arm-cs-trace-disasm.py: Do not ignore disam first sample")
Reviewed-by: default avatarLeo Yan <leo.yan@arm.com>
Signed-off-by: default avatarJames Clark <james.clark@linaro.org>
Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Alexander Shishkin <alexander.shishkin@linux.intel.com>
Cc: Benjamin Gray <bgray@linux.ibm.com>
Cc: coresight@lists.linaro.org
Cc: gankulkarni@os.amperecomputing.com
Cc: Ian Rogers <irogers@google.com>
Cc: Ingo Molnar <mingo@redhat.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Mark Rutland <mark.rutland@arm.com>
Cc: Mike Leach <mike.leach@linaro.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Ruidong Tian <tianruidong@linux.alibaba.com>
Cc: Suzuki Poulouse <suzuki.poulose@arm.com>
Link: https://lore.kernel.org/r/20240723132858.12747-1-james.clark@linaro.orgSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 41947446
...@@ -192,17 +192,16 @@ def process_event(param_dict): ...@@ -192,17 +192,16 @@ def process_event(param_dict):
ip = sample["ip"] ip = sample["ip"]
addr = sample["addr"] addr = sample["addr"]
if (options.verbose == True):
print("Event type: %s" % name)
print_sample(sample)
# Initialize CPU data if it's empty, and directly return back # Initialize CPU data if it's empty, and directly return back
# if this is the first tracing event for this CPU. # if this is the first tracing event for this CPU.
if (cpu_data.get(str(cpu) + 'addr') == None): if (cpu_data.get(str(cpu) + 'addr') == None):
cpu_data[str(cpu) + 'addr'] = addr cpu_data[str(cpu) + 'addr'] = addr
return return
if (options.verbose == True):
print("Event type: %s" % name)
print_sample(sample)
# If cannot find dso so cannot dump assembler, bail out # If cannot find dso so cannot dump assembler, bail out
if (dso == '[unknown]'): if (dso == '[unknown]'):
return return
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment