[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20170628184324.zpfe6f54jbhzxq5g@naverao1-tp.localdomain>
Date: Thu, 29 Jun 2017 00:13:24 +0530
From: "Naveen N. Rao" <naveen.n.rao@...ux.vnet.ibm.com>
To: Masami Hiramatsu <mhiramat@...nel.org>
Cc: Steven Rostedt <rostedt@...dmis.org>,
Ananth N Mavinakayanahalli <ananth@...ux.vnet.ibm.com>,
Michael Ellerman <mpe@...erman.id.au>,
linux-kernel@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org
Subject: Re: [PATCH 2/2] selftests/ftrace: Update multiple kprobes test for
powerpc
On 2017/06/28 11:16PM, Masami Hiramatsu wrote:
> > > diff --git
> > > a/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_eventname.tc
> > > b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_eventname.tc
> > > new file mode 100644
> > > index 0000000..d259031
> > > --- /dev/null
> > > +++ b/tools/testing/selftests/ftrace/test.d/kprobe/kprobe_eventname.tc
> > > @@ -0,0 +1,28 @@
> > > +#!/bin/sh
> > > +# description: Kprobe event auto/manual naming
> > > +
> > > +disable_events
> > > +echo > kprobe_events
> > > +
> > > +:;: "Add an event on function without name" ;:
> > > +
> > > +FUNC=`grep -m 10 " [tT] [^.]*$" /proc/kallsyms | tail -n 1 | cut -f 3 -d " "`
> >
> > On powerpc, this always ends up using a blacklisted function. So, I
> > think we need a way to find a function that is not black listed.
>
> Hmm, if we increase the -m argument, like -m 100, is that still
> in a blacklisted function?
Yes, most of the initial symbols are exception vectors which are
blacklisted.
>
> > However, one of the issues is that debugfs does not show all the address
> > ranges that are blacklisted. I am coming up with a way to address that
> > and will post patches once I have it working.
>
> Would you find that is only on powerpc or generic issue?
I meant the address _ranges_ that are blacklisted such as the ones with
__kprobes annotation and __entry_text and so on.
>
> >
> > With those patches, we should be able to select symbols that are not
> > blacklisted.
>
> Or, maybe we can use generic function, like "schedule" or "vfs_read"
> etc.
Yes, I think this will be good for the generic test, but may not help
selecting a dot symbol on powerpc.
Thanks,
Naveen
Powered by blists - more mailing lists