diff options
-rw-r--r-- | tools/perf/builtin-record.c | 2 | ||||
-rw-r--r-- | tools/perf/builtin-trace.c | 2 | ||||
-rw-r--r-- | tools/perf/tests/perf-record.c | 2 | ||||
-rw-r--r-- | tools/perf/util/evlist.c | 5 | ||||
-rw-r--r-- | tools/perf/util/evlist.h | 3 |
5 files changed, 6 insertions, 8 deletions
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index a80301797e89..2a43c4423f6a 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c | |||
@@ -475,7 +475,7 @@ static int __cmd_record(struct perf_record *rec, int argc, const char **argv) | |||
475 | 475 | ||
476 | if (forks) { | 476 | if (forks) { |
477 | err = perf_evlist__prepare_workload(evsel_list, &opts->target, | 477 | err = perf_evlist__prepare_workload(evsel_list, &opts->target, |
478 | opts, argv); | 478 | argv, opts->pipe_output); |
479 | if (err < 0) { | 479 | if (err < 0) { |
480 | pr_err("Couldn't run the workload!\n"); | 480 | pr_err("Couldn't run the workload!\n"); |
481 | goto out_delete_session; | 481 | goto out_delete_session; |
diff --git a/tools/perf/builtin-trace.c b/tools/perf/builtin-trace.c index 1de3971437c9..3d9944c3d851 100644 --- a/tools/perf/builtin-trace.c +++ b/tools/perf/builtin-trace.c | |||
@@ -462,7 +462,7 @@ static int trace__run(struct trace *trace, int argc, const char **argv) | |||
462 | 462 | ||
463 | if (forks) { | 463 | if (forks) { |
464 | err = perf_evlist__prepare_workload(evlist, &trace->opts.target, | 464 | err = perf_evlist__prepare_workload(evlist, &trace->opts.target, |
465 | &trace->opts, argv); | 465 | argv, false); |
466 | if (err < 0) { | 466 | if (err < 0) { |
467 | printf("Couldn't run the workload!\n"); | 467 | printf("Couldn't run the workload!\n"); |
468 | goto out_delete_evlist; | 468 | goto out_delete_evlist; |
diff --git a/tools/perf/tests/perf-record.c b/tools/perf/tests/perf-record.c index adf6b4a21a60..a1c41b7d3c07 100644 --- a/tools/perf/tests/perf-record.c +++ b/tools/perf/tests/perf-record.c | |||
@@ -93,7 +93,7 @@ int test__PERF_RECORD(void) | |||
93 | * so that we have time to open the evlist (calling sys_perf_event_open | 93 | * so that we have time to open the evlist (calling sys_perf_event_open |
94 | * on all the fds) and then mmap them. | 94 | * on all the fds) and then mmap them. |
95 | */ | 95 | */ |
96 | err = perf_evlist__prepare_workload(evlist, &opts.target, &opts, argv); | 96 | err = perf_evlist__prepare_workload(evlist, &opts.target, argv, false); |
97 | if (err < 0) { | 97 | if (err < 0) { |
98 | pr_debug("Couldn't run the workload!\n"); | 98 | pr_debug("Couldn't run the workload!\n"); |
99 | goto out_delete_maps; | 99 | goto out_delete_maps; |
diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index 291884c804e9..9a337f091b22 100644 --- a/tools/perf/util/evlist.c +++ b/tools/perf/util/evlist.c | |||
@@ -746,8 +746,7 @@ out_err: | |||
746 | 746 | ||
747 | int perf_evlist__prepare_workload(struct perf_evlist *evlist, | 747 | int perf_evlist__prepare_workload(struct perf_evlist *evlist, |
748 | struct perf_target *target, | 748 | struct perf_target *target, |
749 | struct perf_record_opts *opts, | 749 | const char *argv[], bool pipe_output) |
750 | const char *argv[]) | ||
751 | { | 750 | { |
752 | int child_ready_pipe[2], go_pipe[2]; | 751 | int child_ready_pipe[2], go_pipe[2]; |
753 | char bf; | 752 | char bf; |
@@ -769,7 +768,7 @@ int perf_evlist__prepare_workload(struct perf_evlist *evlist, | |||
769 | } | 768 | } |
770 | 769 | ||
771 | if (!evlist->workload.pid) { | 770 | if (!evlist->workload.pid) { |
772 | if (opts->pipe_output) | 771 | if (pipe_output) |
773 | dup2(2, 1); | 772 | dup2(2, 1); |
774 | 773 | ||
775 | close(child_ready_pipe[0]); | 774 | close(child_ready_pipe[0]); |
diff --git a/tools/perf/util/evlist.h b/tools/perf/util/evlist.h index e089906cb4de..276a5acc56e6 100644 --- a/tools/perf/util/evlist.h +++ b/tools/perf/util/evlist.h | |||
@@ -87,8 +87,7 @@ void perf_evlist__config(struct perf_evlist *evlist, | |||
87 | 87 | ||
88 | int perf_evlist__prepare_workload(struct perf_evlist *evlist, | 88 | int perf_evlist__prepare_workload(struct perf_evlist *evlist, |
89 | struct perf_target *target, | 89 | struct perf_target *target, |
90 | struct perf_record_opts *opts, | 90 | const char *argv[], bool pipe_output); |
91 | const char *argv[]); | ||
92 | int perf_evlist__start_workload(struct perf_evlist *evlist); | 91 | int perf_evlist__start_workload(struct perf_evlist *evlist); |
93 | 92 | ||
94 | int perf_evlist__mmap(struct perf_evlist *evlist, unsigned int pages, | 93 | int perf_evlist__mmap(struct perf_evlist *evlist, unsigned int pages, |