linux-kernel - Re: [PATCH 3/3] perf tools: Add evlist/evlist-verbose control commands

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20201215162701.GB698181@krava>
Date:   Tue, 15 Dec 2020 17:27:01 +0100
From:   Jiri Olsa <jolsa@...hat.com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>
Cc:     Jiri Olsa <jolsa@...nel.org>, lkml <linux-kernel@...r.kernel.org>,
        Peter Zijlstra <a.p.zijlstra@...llo.nl>,
        Ingo Molnar <mingo@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Namhyung Kim <namhyung@...nel.org>,
        Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
        Michael Petlan <mpetlan@...hat.com>,
        Ian Rogers <irogers@...gle.com>,
        Stephane Eranian <eranian@...gle.com>,
        Alexei Budankov <abudankov@...wei.com>
Subject: Re: [PATCH 3/3] perf tools: Add evlist/evlist-verbose control
 commands

On Tue, Dec 15, 2020 at 01:09:33PM -0300, Arnaldo Carvalho de Melo wrote:

SNIP

> > instead of:
>  
> >   echo 'enable-cycles' > control
>  
> > I'd like to avoid any elaborate parsing logic..
>  
> > how about that?
> 
> Use space to separate command from its arguments, keep the same
> experience as:
> 
> [acme@...co ~]$ perf evlist -h
> 
>  Usage: perf evlist [<options>]
> 
>     -f, --force           don't complain, do it
>     -F, --freq            Show the sample frequency
>     -g, --group           Show event group information
>     -i, --input <file>    Input file name
>     -v, --verbose         Show all event attr details
>         --trace-fields    Show tracepoint fields
> 
> [acme@...co ~]$
> 
> and:
> 
> echo "evlist arguments"
> 
> ?
> 
> Fits the bill :-)

ok

> 
> The experience users have on existing commands, the same arguments, etc.
> 
> Even -h can have its uses, i.e. has this daemon support for some option
> or is it an old version?

Alexei just suggested to add 'stop' control command and use it
instead of kill signal, which is good idea

but other than that daemon just opens the control file and
shows its path

jirka