perf scripts python cs-etm: Restore first sample log in verbose mode
authorJames Clark <james.clark@linaro.org>
Tue, 23 Jul 2024 13:28:58 +0000 (14:28 +0100)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 31 Jul 2024 19:58:18 +0000 (16:58 -0300)
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: 2d98dbb4c9c5b09c ("perf scripts python arm-cs-trace-disasm.py: Do not ignore disam first sample")
Reviewed-by: Leo Yan <leo.yan@arm.com>
Signed-off-by: James 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.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/scripts/python/arm-cs-trace-disasm.py

index d973c2baed1c8559d2c769516a07542ecc993dc5..7aff02d84ffb3b763f44a4013285d7e9bba8e523 100755 (executable)
@@ -192,17 +192,16 @@ def process_event(param_dict):
        ip = sample["ip"]
        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
        # if this is the first tracing event for this CPU.
        if (cpu_data.get(str(cpu) + 'addr') == None):
                cpu_data[str(cpu) + 'addr'] = addr
                return
 
-
-       if (options.verbose == True):
-               print("Event type: %s" % name)
-               print_sample(sample)
-
        # If cannot find dso so cannot dump assembler, bail out
        if (dso == '[unknown]'):
                return