lkml.org 
[lkml]   [2020]   [Jun]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v5 13/13] perf record: introduce --ctl-fd[-ack] options
    From
    Date

    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 model introduced by
    >> --ctl-fd[-ack] options by providing example bash shell script.
    >
    > Hi
    >
    > I am interested in using this also for taking snapshots.

    Good to hear from you.

    >
    > Did you consider using a single option, or allowing either a file descriptor

    Single option use case is already possible like --ctl-fd <NUM_1>.
    Synchronization messages can be provided via --ctl-fd-ack <NUM_2>.

    > 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'"

    >
    > e.g. add "--control" and support all of:
    >
    > --control
    > --control 11
    > --control 11,15
    > --control 11,15,disabled
    > --control 11,,disabled
    > --control /tmp/my-perf.fifo
    > --control /tmp/my-perf.fifo,/tmp/my-perf-ack.fifo
    > --control /tmp/my-perf.fifo,/tmp/my-perf-ack.fifo,disabled
    > --control /tmp/my-perf.fifo,,disabled
    >
    > Regards
    > Adrian
    >

    Regards,
    Alexey

    \
     
     \ /
      Last update: 2020-06-01 19:13    [W:2.966 / U:0.040 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site