Re: [RFC PATCH 00/22] perf tools: introduce 'perf bpf' command to load eBPF programs.

[Ingo Molnar <mingo@kernel.org>]

* Wang Nan <wangnan0@huawei.com> wrote:

> This series of patches is an approach to integrate eBPF with perf.

Very promising!

> After applying these patches, users are allowed to use following
> command to load eBPF program compiled by LLVM into kernel:
> 
>  $ perf bpf sample_bpf.o

Please keep space for a subcommand space as most other perf 
subcommands do, i.e. make it something like:

    perf bpf add sample_bpf.o

or:

    perf bpf run sample_bpf.o

or:

    perf bpf load sample_bpf.o

So that future subcommands can be added:

    perf bpf list
    perf bpf del <...>
    perf bpf enable <...>
    perf bpf disable <...>
    perf bpf help

and 'perf bpf' should probably display the help page by default, so if 
curious perf users stumble into the new subcommand, they get a basic 
idea about what it's all about.

I.e. you should think about the high level subcommand space right now, 
and pick proper names - because this is going to determine the future 
usability and the success of the tool to a large degree.

Thanks,

	Ingo