Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-08 Thread Jiri Olsa
On Fri, Jun 05, 2020 at 06:23:17PM +0300, Alexey Budankov wrote: SNIP > >> Or even clearer: > >> > >> --ctl-fifo /tmp/my-perf --ctl-fifo-ack /tmp/my-perf-ack > > > > If people are OK with having so many options, then that is fine by me. > > the single option Adrian

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-08 Thread Alexey Budankov
On 05.06.2020 18:23, Alexey Budankov wrote: > > On 05.06.2020 17:47, Alexey Budankov wrote: >> >> On 05.06.2020 16:57, Jiri Olsa wrote: >>> On Fri, Jun 05, 2020 at 04:15:52PM +0300, Alexey Budankov wrote: On 05.06.2020 13:51, Jiri Olsa wrote: > On Tue, Jun 02, 2020 at 04:43:58PM

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-06 Thread Jiri Olsa
On Fri, Jun 05, 2020 at 05:47:28PM +0300, Alexey Budankov wrote: > > On 05.06.2020 16:57, Jiri Olsa wrote: > > On Fri, Jun 05, 2020 at 04:15:52PM +0300, Alexey Budankov wrote: > >> > >> On 05.06.2020 13:51, Jiri Olsa wrote: > >>> On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: >

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-05 Thread Alexey Budankov
On 05.06.2020 17:47, Alexey Budankov wrote: > > On 05.06.2020 16:57, Jiri Olsa wrote: >> On Fri, Jun 05, 2020 at 04:15:52PM +0300, Alexey Budankov wrote: >>> >>> On 05.06.2020 13:51, Jiri Olsa wrote: On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: > On 2/06/20 12:12 pm,

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-05 Thread Alexey Budankov
On 05.06.2020 16:57, Jiri Olsa wrote: > On Fri, Jun 05, 2020 at 04:15:52PM +0300, Alexey Budankov wrote: >> >> On 05.06.2020 13:51, Jiri Olsa wrote: >>> On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: On 2/06/20 12:12 pm, Alexey Budankov wrote: > > On 02.06.2020

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-05 Thread Jiri Olsa
On Fri, Jun 05, 2020 at 04:15:52PM +0300, Alexey Budankov wrote: > > On 05.06.2020 13:51, Jiri Olsa wrote: > > On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: > >> On 2/06/20 12:12 pm, Alexey Budankov wrote: > >>> > >>> On 02.06.2020 11:32, Alexey Budankov wrote: > > On

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-05 Thread Alexey Budankov
On 05.06.2020 13:51, Jiri Olsa wrote: > On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: >> On 2/06/20 12:12 pm, Alexey Budankov wrote: >>> >>> On 02.06.2020 11:32, Alexey Budankov wrote: On 02.06.2020 2:37, Andi Kleen wrote: >>> or a pathname, or including also the

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-05 Thread Jiri Olsa
On Tue, Jun 02, 2020 at 04:43:58PM +0300, Adrian Hunter wrote: > On 2/06/20 12:12 pm, Alexey Budankov wrote: > > > > On 02.06.2020 11:32, Alexey Budankov wrote: > >> > >> On 02.06.2020 2:37, Andi Kleen wrote: > > or a pathname, or including also the event default of "disabled". > >

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-02 Thread Adrian Hunter
On 2/06/20 12:12 pm, Alexey Budankov wrote: > > On 02.06.2020 11:32, Alexey Budankov wrote: >> >> On 02.06.2020 2:37, Andi Kleen wrote: > or a pathname, or including also the event default of "disabled". For my cases conversion of pathnames into open fds belongs to external

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-02 Thread Alexey Budankov
On 02.06.2020 11:32, Alexey Budankov wrote: > > On 02.06.2020 2:37, Andi Kleen wrote: or a pathname, or including also the event default of "disabled". >>> >>> For my cases conversion of pathnames into open fds belongs to external >>> controlling process e.g. like in the examples provided

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-02 Thread Alexey Budankov
On 02.06.2020 2:37, Andi Kleen wrote: >>> or a pathname, or including also the event default of "disabled". >> >> For my cases conversion of pathnames into open fds belongs to external >> controlling process e.g. like in the examples provided in the patch set. >> Not sure about "event default of

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-01 Thread Andi Kleen
> > or a pathname, or including also the event default of "disabled". > > For my cases conversion of pathnames into open fds belongs to external > controlling process e.g. like in the examples provided in the patch set. > Not sure about "event default of 'disabled'" It would be nicer for manual

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-01 Thread Alexey Budankov
Hi Adrian, On 01.06.2020 19:30, Adrian Hunter wrote: > On 1/06/20 7:05 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] >> options description. Document possible usage

Re: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-01 Thread Adrian Hunter
On 1/06/20 7:05 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] > options description. Document possible usage model introduced by > --ctl-fd[-ack] options by providing

[PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options

2020-06-01 Thread Alexey Budankov
Introduce --ctl-fd[-ack] options to pass open file descriptors numbers from command line. Extend perf-record.txt file with --ctl-fd[-ack] options description. Document possible usage model introduced by --ctl-fd[-ack] options by providing example bash shell script. Signed-off-by: Alexey