lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Tue, 23 Apr 2013 17:31:11 +0900 From: Namhyung Kim <namhyung@...nel.org> To: Arnaldo Carvalho de Melo <acme@...stprotocols.net> Cc: Peter Zijlstra <a.p.zijlstra@...llo.nl>, Paul Mackerras <paulus@...ba.org>, Ingo Molnar <mingo@...nel.org>, Namhyung Kim <namhyung.kim@....com>, LKML <linux-kernel@...r.kernel.org>, Steven Rostedt <rostedt@...dmis.org>, Frederic Weisbecker <fweisbec@...il.com>, Jiri Olsa <jolsa@...hat.com>, David Ahern <dsahern@...il.com>, Stephane Eranian <eranian@...gle.com> Subject: [PATCH 13/14] perf ftrace: Cleanup using ftrace_setup/teardown() From: Namhyung Kim <namhyung.kim@....com> The ftrace sub-commands share some common code so that factor it out to ftrace_setup() and ftrace_teardown() helpers. Cc: Steven Rostedt <rostedt@...dmis.org> Cc: Frederic Weisbecker <fweisbec@...il.com> Signed-off-by: Namhyung Kim <namhyung@...nel.org> --- tools/perf/builtin-ftrace.c | 168 +++++++++++++++++--------------------------- 1 file changed, 64 insertions(+), 104 deletions(-) diff --git a/tools/perf/builtin-ftrace.c b/tools/perf/builtin-ftrace.c index f4efa1475cf9..95f9a7fa3e04 100644 --- a/tools/perf/builtin-ftrace.c +++ b/tools/perf/builtin-ftrace.c @@ -1409,10 +1409,59 @@ out: return ret; } +static int ftrace_setup(struct perf_ftrace *ftrace, int argc, const char **argv) +{ + int ret; + char errbuf[512]; + + ret = perf_target__validate(&ftrace->target); + if (ret) { + perf_target__strerror(&ftrace->target, ret, errbuf, 512); + pr_err("%s\n", errbuf); + return -EINVAL; + } + + ftrace->evlist = perf_evlist__new(); + if (ftrace->evlist == NULL) + return -ENOMEM; + + ret = perf_evlist__create_maps(ftrace->evlist, &ftrace->target); + if (ret < 0) + goto out; + + if (ftrace->tracer == NULL) + ftrace->tracer = DEFAULT_TRACER; + + if (ftrace->dirname == NULL) + ftrace->dirname = DEFAULT_DIRNAME; + + if (argc) { + ret = perf_evlist__prepare_workload(ftrace->evlist, + &ftrace->target, + argv, false, true); + if (ret < 0) + goto out_maps; + } + return 0; + +out_maps: + perf_evlist__delete_maps(ftrace->evlist); +out: + perf_evlist__delete(ftrace->evlist); + + return ret; +} + +static void ftrace_teardown(struct perf_ftrace *ftrace) +{ + perf_evlist__delete_maps(ftrace->evlist); + perf_evlist__delete(ftrace->evlist); +} + static int __cmd_ftrace_live(struct perf_ftrace *ftrace, int argc, const char **argv) { - int ret = -1; + int ret; const char * const live_usage[] = { "perf ftrace live [<options>] [<command>]", "perf ftrace live [<options>] -- <command> [<options>]", @@ -1437,47 +1486,22 @@ __cmd_ftrace_live(struct perf_ftrace *ftrace, int argc, const char **argv) if (!argc && perf_target__none(&ftrace->target)) usage_with_options(live_usage, live_options); - ret = perf_target__validate(&ftrace->target); - if (ret) { - char errbuf[512]; - - perf_target__strerror(&ftrace->target, ret, errbuf, 512); - pr_err("%s\n", errbuf); - return -EINVAL; - } - - ftrace->evlist = perf_evlist__new(); - if (ftrace->evlist == NULL) - return -ENOMEM; - - ret = perf_evlist__create_maps(ftrace->evlist, &ftrace->target); + ret = ftrace_setup(ftrace, argc, argv); if (ret < 0) - goto out; - - if (ftrace->tracer == NULL) - ftrace->tracer = DEFAULT_TRACER; - - if (argc && perf_evlist__prepare_workload(ftrace->evlist, - &ftrace->target, - argv, false, true) < 0) - goto out_maps; + return ret; setup_pager(); ret = do_ftrace_live(ftrace); -out_maps: - perf_evlist__delete_maps(ftrace->evlist); -out: - perf_evlist__delete(ftrace->evlist); - + ftrace_teardown(ftrace); return ret; } static int __cmd_ftrace_record(struct perf_ftrace *ftrace, int argc, const char **argv) { - int ret = -1; + int ret; const char * const record_usage[] = { "perf ftrace record [<options>] [<command>]", "perf ftrace record [<options>] -- <command> [<options>]", @@ -1504,39 +1528,13 @@ __cmd_ftrace_record(struct perf_ftrace *ftrace, int argc, const char **argv) if (!argc && perf_target__none(&ftrace->target)) usage_with_options(record_usage, record_options); - ret = perf_target__validate(&ftrace->target); - if (ret) { - char errbuf[512]; - - perf_target__strerror(&ftrace->target, ret, errbuf, 512); - pr_err("%s\n", errbuf); - return -EINVAL; - } - - ftrace->evlist = perf_evlist__new(); - if (ftrace->evlist == NULL) - return -ENOMEM; - - ret = perf_evlist__create_maps(ftrace->evlist, &ftrace->target); + ret = ftrace_setup(ftrace, argc, argv); if (ret < 0) - goto out; - - if (ftrace->tracer == NULL) - ftrace->tracer = DEFAULT_TRACER; - - if (ftrace->dirname == NULL) - ftrace->dirname = DEFAULT_DIRNAME; - - if (argc && perf_evlist__prepare_workload(ftrace->evlist, - &ftrace->target, - argv, false, true) < 0) - goto out_maps; + return ret; ret = do_ftrace_record(ftrace); -out_maps: perf_evlist__delete_maps(ftrace->evlist); -out: perf_evlist__delete(ftrace->evlist); return ret; @@ -1545,7 +1543,7 @@ out: static int __cmd_ftrace_show(struct perf_ftrace *ftrace, int argc, const char **argv) { - int ret = -1; + int ret; const char * const show_usage[] = { "perf ftrace show [<options>]", NULL @@ -1565,41 +1563,22 @@ __cmd_ftrace_show(struct perf_ftrace *ftrace, int argc, const char **argv) if (argc) usage_with_options(show_usage, show_options); - ret = perf_target__validate(&ftrace->target); - if (ret) { - char errbuf[512]; - - perf_target__strerror(&ftrace->target, ret, errbuf, 512); - pr_err("%s\n", errbuf); - return -EINVAL; - } - - ftrace->evlist = perf_evlist__new(); - if (ftrace->evlist == NULL) - return -ENOMEM; - - ret = perf_evlist__create_maps(ftrace->evlist, &ftrace->target); + ret = ftrace_setup(ftrace, argc, argv); if (ret < 0) - goto out; - - if (ftrace->dirname == NULL) - ftrace->dirname = DEFAULT_DIRNAME; + return ret; setup_pager(); ret = do_ftrace_show(ftrace); - perf_evlist__delete_maps(ftrace->evlist); -out: - perf_evlist__delete(ftrace->evlist); - + ftrace_teardown(ftrace); return ret; } static int __cmd_ftrace_report(struct perf_ftrace *ftrace, int argc, const char **argv) { - int ret = -1; + int ret; const char * const report_usage[] = { "perf ftrace report [<options>]", NULL @@ -1625,25 +1604,9 @@ __cmd_ftrace_report(struct perf_ftrace *ftrace, int argc, const char **argv) if (argc) usage_with_options(report_usage, report_options); - ret = perf_target__validate(&ftrace->target); - if (ret) { - char errbuf[512]; - - perf_target__strerror(&ftrace->target, ret, errbuf, 512); - pr_err("%s\n", errbuf); - return -EINVAL; - } - - ftrace->evlist = perf_evlist__new(); - if (ftrace->evlist == NULL) - return -ENOMEM; - - ret = perf_evlist__create_maps(ftrace->evlist, &ftrace->target); + ret = ftrace_setup(ftrace, argc, argv); if (ret < 0) - goto out; - - if (ftrace->dirname == NULL) - ftrace->dirname = DEFAULT_DIRNAME; + return ret; perf_hpp__column_enable(PERF_HPP__OVERHEAD); perf_hpp__init(); @@ -1657,10 +1620,7 @@ __cmd_ftrace_report(struct perf_ftrace *ftrace, int argc, const char **argv) ret = do_ftrace_report(ftrace); - perf_evlist__delete_maps(ftrace->evlist); -out: - perf_evlist__delete(ftrace->evlist); - + ftrace_teardown(ftrace); return ret; } -- 1.7.11.7 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists