This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
Re: [RFC] perf-cache command interface design
- From: Arnaldo Carvalho de Melo <acme at redhat dot com>
- To: Namhyung Kim <namhyung at kernel dot org>
- Cc: Masami Hiramatsu <masami dot hiramatsu dot pt at hitachi dot com>, Hemant Kumar <hemant at linux dot vnet dot ibm dot com>, linux-kernel at vger dot kernel dot org, srikar at linux dot vnet dot ibm dot com, peterz at infradead dot org, oleg at redhat dot com, hegdevasant at linux dot vnet dot ibm dot com, mingo at redhat dot com, systemtap at sourceware dot org, aravinda at linux dot vnet dot ibm dot com, penberg at iki dot fi, brendan dot d dot gregg at gmail dot com, "yrl dot pp-manager dot tt at hitachi dot com" <yrl dot pp-manager dot tt at hitachi dot com>
- Date: Mon, 17 Nov 2014 16:58:57 -0200
- Subject: Re: [RFC] perf-cache command interface design
- Authentication-results: sourceware.org; auth=none
- References: <874muew2hk dot fsf at sejong dot aot dot lge dot com> <5459E865 dot 6050207 at hitachi dot com> <545B1DDE dot 9000202 at linux dot vnet dot ibm dot com> <545C80F4 dot 4020905 at hitachi dot com> <54609A8C dot 4050308 at hitachi dot com> <20141110122321 dot GC4468 at redhat dot com> <5461B276 dot 50004 at hitachi dot com> <20141111131030 dot GG4468 at redhat dot com> <54637C05 dot 5090807 at hitachi dot com> <87oas6ttf8 dot fsf at sejong dot aot dot lge dot com>
Em Mon, Nov 17, 2014 at 12:08:59PM +0900, Namhyung Kim escreveu:
> On Thu, 13 Nov 2014 00:25:57 +0900, Masami Hiramatsu wrote:
> > (2014/11/11 22:10), Arnaldo Carvalho de Melo wrote:
> >> What I meant was, what is wrong with replacing:
> >> perf cache --probe <SPEC> # for the current kernel
> >> With:
> >> perf cache --add <PROBE-SPEC> # for the current kernel
> >> And have it figure out that what is being added is a probe and do the
> >> right thing?
> > As I've said previously, PROBE-SPEC can be same as FILES (imagine that a binary
> > file which has same name function in the kernel.)
> > Moreover, PROBE-SPEC requires the target binary(or kernel module) except for
> > kernel probes. In that case, anyway we need -x or -m options with file-path
> > for --add, that is very strange.
> > e.g.
> > For me,
> > perf cache --add ./binary --probe '*'
> > looks more natural than
> > perf cache --add '*' -exec ./binary
> > since in other cases(sdt/elf), we'll just do
> > perf cache --add ./binary
> I prefer this too. But I'd like make the 'add' part a subcommand rather
> than option like we do in perf kmem/kvm/list/lock/mem/sched ... And it
> can handle multiple files at once. What about this?
> perf cache add [--elf|--sdt|--probe <spec>] <binary> [<binary>...]
In the end we can have it both ways, i.e. if the user does just:
perf cache add something
or:
perf cache add --elf something
And 'something' is an ELF file, then in the first case (no --elf
specified) it will figure it out (checking the magic number, etc) and do
the right thing.
In the second case since we're being more verbose and think we know what
'something' is (an ELF file) the tool can check if it indeed is an ELF
file and if not, bail out.
- Arnaldo