Date: Tue, 5 May 2015 12:39:52 -0300
From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Alexei Starovoitov <ast@plumgrid.com>
Cc: Ingo Molnar <mingo@kernel.org>, Peter Zijlstra <peterz@infradead.org>,
        Wang Nan <wangnan0@huawei.com>, davem@davemloft.net,
        masami.hiramatsu.pt@hitachi.com, jolsa@kernel.org, lizefan@kernel.org,
        linux-kernel@vger.kernel.org, pi3orama@163.com, hekuang@huawei.com,
        bgregg@netflix.com
Subject: Re: [RFC PATCH 00/22] perf tools: introduce 'perf bpf' command to
 load eBPF programs.
Message-ID: <20150505153952.GV10475@kernel.org>
References: <1430391165-30267-1-git-send-email-wangnan0@huawei.com>
 <554302F0.3070101@plumgrid.com>
 <20150501110659.GE5029@twins.programming.kicks-ass.net>
 <20150501114943.GA24094@gmail.com>
 <5543B037.9070504@plumgrid.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <5543B037.9070504@plumgrid.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1404
Lines: 31

Em Fri, May 01, 2015 at 09:56:23AM -0700, Alexei Starovoitov escreveu:
> Anyway, back to my original question about long term home.
> where to land 'perf/bpf' branch ?

I don't care, but for me to merge it, please go on addressing the
comments made in this thread (perf bpf command --args, etc) and at some
point provide a small patchset that implements the most basic stuff,
like, say, a "hello, world" style proggie, together with the
tools/perf/Documentation/perf-bpf.txt file, detailed instructions on how
to use the feature, i.e. what dependencies are needed, what kernel
options should be enabled, etc.

Nice warning/error messages for when the user doesn't have those options
enabled or doesn't have appropriate permissions, etc.

I.e. just by following what is in each changeset comment log I should be
able to test patch after patch.

After we get one such, say, 10-long patchkit with a very basic feature
of eBPF exposed via 'perf bpf', we can go to the next, and so on.

Try to use 'perf trace usleep 1', 'perf trace -a usleep 1' as non-root,
for instance, to see examples on how to inform the user about what is
needed to use the tool.

- Arnaldo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/