On 1/06/20 11:20 pm, Alexey Budankov wrote: > > Introduce --ctl-fd[-ack] options to pass open file descriptors numbers > from command line. Extend perf-record.txt file with --ctl-fd[-ack]
perf-record.txt change seems to be missing. > options description. Document possible usage model introduced by > --ctl-fd[-ack] options by providing example bash shell script. > > Signed-off-by: Alexey Budankov <alexey.budan...@linux.intel.com> > --- > tools/perf/builtin-record.c | 10 ++++++++++ > tools/perf/util/record.h | 2 ++ > 2 files changed, 12 insertions(+) > > diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c > index 0394e068dde8..8494ce964738 100644 > --- a/tools/perf/builtin-record.c > +++ b/tools/perf/builtin-record.c > @@ -1749,6 +1749,9 @@ static int __cmd_record(struct record *rec, int argc, > const char **argv) > perf_evlist__start_workload(rec->evlist); > } > > + if (evlist__initialize_ctlfd(rec->evlist, opts->ctl_fd, > opts->ctl_fd_ack)) > + goto out_child; > + > if (opts->initial_delay) { > pr_info(EVLIST_DISABLED_MSG); > if (opts->initial_delay > 0) { > @@ -1895,6 +1898,7 @@ static int __cmd_record(struct record *rec, int argc, > const char **argv) > record__synthesize_workload(rec, true); > > out_child: > + evlist__finalize_ctlfd(rec->evlist); > record__mmap_read_all(rec, true); > record__aio_mmap_read_sync(rec); > > @@ -2380,6 +2384,8 @@ static struct record record = { > }, > .mmap_flush = MMAP_FLUSH_DEFAULT, > .nr_threads_synthesize = 1, > + .ctl_fd = -1, > + .ctl_fd_ack = -1, > }, > .tool = { > .sample = process_sample_event, > @@ -2581,6 +2587,10 @@ static struct option __record_options[] = { > "libpfm4 event selector. use 'perf list' to list available > events", > parse_libpfm_events_option), > #endif > + OPT_INTEGER(0, "ctl-fd", &record.opts.ctl_fd, > + "Listen on fd descriptor for command to control measurement > ('enable': enable events, 'disable': disable events)"), > + OPT_INTEGER(0, "ctl-fd-ack", &record.opts.ctl_fd_ack, > + "Send control command completion ('ack') to fd ack > descriptor"), > OPT_END() > }; > > diff --git a/tools/perf/util/record.h b/tools/perf/util/record.h > index da138dcb4d34..4cb72a478af1 100644 > --- a/tools/perf/util/record.h > +++ b/tools/perf/util/record.h > @@ -70,6 +70,8 @@ struct record_opts { > int mmap_flush; > unsigned int comp_level; > unsigned int nr_threads_synthesize; > + int ctl_fd; > + int ctl_fd_ack; > }; > > extern const char * const *record_usage; >