3 files changed, 32 insertions, 20 deletions

diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c
index 2a40b8e1def7..097b3ed77fdd 100644
--- a/tools/perf/util/evlist.c
+++ b/tools/perf/util/evlist.c
@@ -239,31 +239,13 @@ void perf_event_attr__set_max_precise_ip(struct perf_event_attr *attr)
 
 int perf_evlist__add_default(struct perf_evlist *evlist)
 {
-        struct perf_event_attr attr = {
-                .type = PERF_TYPE_HARDWARE,
-                .config = PERF_COUNT_HW_CPU_CYCLES,
-        };
-        struct perf_evsel *evsel;
-
-        event_attr_init(&attr);
+        struct perf_evsel *evsel = perf_evsel__new_cycles();
 
-        perf_event_attr__set_max_precise_ip(&attr);
-
-        evsel = perf_evsel__new(&attr);
         if (evsel == NULL)
-                goto error;
-
-        /* use asprintf() because free(evsel) assumes name is allocated */
-        if (asprintf(&evsel->name, "cycles%.*s",
-                     attr.precise_ip ? attr.precise_ip + 1 : 0, ":ppp") < 0)
-                goto error_free;
+                return -ENOMEM;
 
         perf_evlist__add(evlist, evsel);
         return 0;
-error_free:
-        perf_evsel__delete(evsel);
-error:
-        return -ENOMEM;
 }
 
 int perf_evlist__add_dummy(struct perf_evlist *evlist)
diff --git a/tools/perf/util/evsel.c b/tools/perf/util/evsel.c
index 8c54df61fe64..4c2b31dc4e4d 100644
--- a/tools/perf/util/evsel.c
+++ b/tools/perf/util/evsel.c
@@ -253,6 +253,34 @@ struct perf_evsel *perf_evsel__new_idx(struct perf_event_attr *attr, int idx)
         return evsel;
 }
 
+struct perf_evsel *perf_evsel__new_cycles(void)
+{
+        struct perf_event_attr attr = {
+                .type   = PERF_TYPE_HARDWARE,
+                .config = PERF_COUNT_HW_CPU_CYCLES,
+        };
+        struct perf_evsel *evsel;
+
+        event_attr_init(&attr);
+
+        perf_event_attr__set_max_precise_ip(&attr);
+
+        evsel = perf_evsel__new(&attr);
+        if (evsel == NULL)
+                goto out;
+
+        /* use asprintf() because free(evsel) assumes name is allocated */
+        if (asprintf(&evsel->name, "cycles%.*s",
+                     attr.precise_ip ? attr.precise_ip + 1 : 0, ":ppp") < 0)
+                goto error_free;
+out:
+        return evsel;
+error_free:
+        perf_evsel__delete(evsel);
+        evsel = NULL;
+        goto out;
+}
+
 /*
  * Returns pointer with encoded error via <linux/err.h> interface.
  */
diff --git a/tools/perf/util/evsel.h b/tools/perf/util/evsel.h
index 8a4a6c9f1480..4d44129e050b 100644
--- a/tools/perf/util/evsel.h
+++ b/tools/perf/util/evsel.h
@@ -175,6 +175,8 @@ static inline struct perf_evsel *perf_evsel__newtp(const char *sys, const char *
         return perf_evsel__newtp_idx(sys, name, 0);
 }
 
+struct perf_evsel *perf_evsel__new_cycles(void);
+
 struct event_format *event_format__new(const char *sys, const char *name);
 
 void perf_evsel__init(struct perf_evsel *evsel,