perf tests time-to-tsc: No need to disable an event before deleting it
authorArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 22 Jun 2016 13:10:52 +0000 (10:10 -0300)
committerArnaldo Carvalho de Melo <acme@redhat.com>
Wed, 22 Jun 2016 13:10:52 +0000 (10:10 -0300)
Because at the destructor we will call close() and that will do the
disable. And we destructors can accept NULL, just like free(), so no
need to check it.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Cc: Jiri Olsa <jolsa@redhat.com>
Link: http://lkml.kernel.org/n/tip-i98mcyfkkjh5qp62dle27ac1@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
tools/perf/arch/x86/tests/perf-time-to-tsc.c

index d4aa567a29c4685ece1fb142577322ba9e57c890..5c76cc83186a24ae0df5f3986a922dc75e4ff959 100644 (file)
@@ -154,10 +154,6 @@ next_event:
        err = 0;
 
 out_err:
-       if (evlist) {
-               perf_evlist__disable(evlist);
-               perf_evlist__delete(evlist);
-       }
-
+       perf_evlist__delete(evlist);
        return err;
 }