perf intel-pt: Tidy messages into called function intel_pt_synth_event()
authorAdrian Hunter <adrian.hunter@intel.com>
Fri, 26 May 2017 08:17:31 +0000 (11:17 +0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Fri, 30 Jun 2017 14:44:35 +0000 (11:44 -0300)
Tidy print messages into called function intel_pt_synth_event().

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Link: http://lkml.kernel.org/r/1495786658-18063-31-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/util/intel-pt.c

index a9486b57584f4ae2b330e5f020805bc7ceb3dc7e..81907f60e7dacb0de9c780a2f6cbbfe13615ca0a 100644 (file)
@@ -1910,16 +1910,25 @@ static int intel_pt_event_synth(struct perf_tool *tool,
                                                 NULL);
 }
 
-static int intel_pt_synth_event(struct perf_session *session,
+static int intel_pt_synth_event(struct perf_session *session, const char *name,
                                struct perf_event_attr *attr, u64 id)
 {
        struct intel_pt_synth intel_pt_synth;
+       int err;
+
+       pr_debug("Synthesizing '%s' event with id %" PRIu64 " sample type %#" PRIx64 "\n",
+                name, id, (u64)attr->sample_type);
 
        memset(&intel_pt_synth, 0, sizeof(struct intel_pt_synth));
        intel_pt_synth.session = session;
 
-       return perf_event__synthesize_attr(&intel_pt_synth.dummy_tool, attr, 1,
-                                          &id, intel_pt_event_synth);
+       err = perf_event__synthesize_attr(&intel_pt_synth.dummy_tool, attr, 1,
+                                         &id, intel_pt_event_synth);
+       if (err)
+               pr_err("%s: failed to synthesize '%s' event type\n",
+                      __func__, name);
+
+       return err;
 }
 
 static struct perf_evsel *intel_pt_evsel(struct intel_pt *pt,
@@ -1984,14 +1993,9 @@ static int intel_pt_synth_events(struct intel_pt *pt,
                        attr.sample_type |= PERF_SAMPLE_CALLCHAIN;
                if (pt->synth_opts.last_branch)
                        attr.sample_type |= PERF_SAMPLE_BRANCH_STACK;
-               pr_debug("Synthesizing 'instructions' event with id %" PRIu64 " sample type %#" PRIx64 "\n",
-                        id, (u64)attr.sample_type);
-               err = intel_pt_synth_event(session, &attr, id);
-               if (err) {
-                       pr_err("%s: failed to synthesize 'instructions' event type\n",
-                              __func__);
+               err = intel_pt_synth_event(session, "instructions", &attr, id);
+               if (err)
                        return err;
-               }
                pt->sample_instructions = true;
                pt->instructions_sample_type = attr.sample_type;
                pt->instructions_id = id;
@@ -2005,14 +2009,9 @@ static int intel_pt_synth_events(struct intel_pt *pt,
                        attr.sample_type |= PERF_SAMPLE_CALLCHAIN;
                if (pt->synth_opts.last_branch)
                        attr.sample_type |= PERF_SAMPLE_BRANCH_STACK;
-               pr_debug("Synthesizing 'transactions' event with id %" PRIu64 " sample type %#" PRIx64 "\n",
-                        id, (u64)attr.sample_type);
-               err = intel_pt_synth_event(session, &attr, id);
-               if (err) {
-                       pr_err("%s: failed to synthesize 'transactions' event type\n",
-                              __func__);
+               err = intel_pt_synth_event(session, "transactions", &attr, id);
+               if (err)
                        return err;
-               }
                pt->sample_transactions = true;
                pt->transactions_sample_type = attr.sample_type;
                pt->transactions_id = id;
@@ -2033,14 +2032,9 @@ static int intel_pt_synth_events(struct intel_pt *pt,
                attr.sample_type |= PERF_SAMPLE_ADDR;
                attr.sample_type &= ~(u64)PERF_SAMPLE_CALLCHAIN;
                attr.sample_type &= ~(u64)PERF_SAMPLE_BRANCH_STACK;
-               pr_debug("Synthesizing 'branches' event with id %" PRIu64 " sample type %#" PRIx64 "\n",
-                        id, (u64)attr.sample_type);
-               err = intel_pt_synth_event(session, &attr, id);
-               if (err) {
-                       pr_err("%s: failed to synthesize 'branches' event type\n",
-                              __func__);
+               err = intel_pt_synth_event(session, "branches", &attr, id);
+               if (err)
                        return err;
-               }
                pt->sample_branches = true;
                pt->branches_sample_type = attr.sample_type;
                pt->branches_id = id;