perf script: Fix unnecessary machine_resolve()
authorAdrian Hunter <adrian.hunter@intel.com>
Wed, 11 Aug 2021 10:10:32 +0000 (13:10 +0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 11 Aug 2021 12:33:59 +0000 (09:33 -0300)
machine_resolve() may have already been called. Test for that to avoid
calling it again unnecessarily.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Link: https //lore.kernel.org/r/20210811101036.17986-3-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/builtin-script.c

index e2e165b..f469354 100644 (file)
@@ -2212,7 +2212,7 @@ static int process_sample_event(struct perf_tool *tool,
        if (filter_cpu(sample))
                goto out_put;
 
-       if (machine__resolve(machine, &al, sample) < 0) {
+       if (!al.thread && machine__resolve(machine, &al, sample) < 0) {
                pr_err("problem processing %d event, skipping it.\n",
                       event->header.type);
                ret = -1;