Commit 249ca1a8 authored by Taeung Song's avatar Taeung Song Committed by Arnaldo Carvalho de Melo

perf kmem: Fill in the missing session freeing after an error occurs

When an error occurs an error value is just returned without freeing the
session. So allocating and freeing session have to be matched as a pair
even if an error occurs.
Signed-off-by: default avatarTaeung Song <treeze.taeung@gmail.com>
Acked-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1435652124-22414-3-git-send-email-treeze.taeung@gmail.comSigned-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent 9fedfb0c
...@@ -1916,7 +1916,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused) ...@@ -1916,7 +1916,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused)
if (!perf_evlist__find_tracepoint_by_name(session->evlist, if (!perf_evlist__find_tracepoint_by_name(session->evlist,
"kmem:kmalloc")) { "kmem:kmalloc")) {
pr_err(errmsg, "slab", "slab"); pr_err(errmsg, "slab", "slab");
return -1; goto out_delete;
} }
} }
...@@ -1927,7 +1927,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused) ...@@ -1927,7 +1927,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused)
"kmem:mm_page_alloc"); "kmem:mm_page_alloc");
if (evsel == NULL) { if (evsel == NULL) {
pr_err(errmsg, "page", "page"); pr_err(errmsg, "page", "page");
return -1; goto out_delete;
} }
kmem_page_size = pevent_get_page_size(evsel->tp_format->pevent); kmem_page_size = pevent_get_page_size(evsel->tp_format->pevent);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment