public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Frederic Weisbecker <fweisbec@gmail.com>
To: Jiri Olsa <jolsa@redhat.com>
Cc: rostedt@goodmis.org, mingo@redhat.com, a.p.zijlstra@chello.nl,
	linux-kernel@vger.kernel.org, Tom Zanussi <tzanussi@gmail.com>,
	Arnaldo Carvalho de Melo <acme@redhat.com>,
	Li Zefan <lizf@cn.fujitsu.com>
Subject: Re: [RFC,PATCH 0/3] trace,perf: enabling ftrace/function tracepoint
Date: Thu, 21 Apr 2011 17:27:28 +0200	[thread overview]
Message-ID: <20110421152724.GB1772@nowhere> (raw)
In-Reply-To: <1303382458-11072-1-git-send-email-jolsa@redhat.com>

On Thu, Apr 21, 2011 at 12:40:55PM +0200, Jiri Olsa wrote:
> hi,
> 
> this is just RFC patch with patche series showing the
> direction I'm taking.. _RFC_ ;)
> 
> I was discussing with Frederic the possibility to have function trace
> available for perf processing, and the possibility to have it used
> as starting/stopping events, and probably more.
> 
> The 1st 2 patches adding the registration function for the
> ftrace/function tracepoint to be usable by both trace and perf
> via tracepoint interface.
> 
> The 3rd patch is the biggest hack and is trying to add filtering
> support.  I'm currently looking on the filtering code to come up
> with some better idea of hooking this type of filtering in.
> 
> attached patches:
> 1/3 - add support for enabling ftrace/function tracepoint event
> 2/3 - add support for registering ftrace/function tracepoint event via perf
> 3/3 - add filter support for ftrace/function tracepoint event
> 
> I could read perf counts from ftrace/function tracepoint using
> attached program.
> 
> Any thoughts/ideas about this direction or projecting some other
> would be just great :)
> 
> 
> thanks a lot,
> jirka

So the direction is kinda good but the thing needs to be unglobalized
(if that word ever exists). Filters need to apply individually to
each users of the function tracer, that's the trickiest part.

We also need to catch up with set_ftrace_filter and set_ftrace_notrace
using filters and triggers, so that we can get rid of the global files
in the end and have only local effects (per perf_event/ftrace).

But that's a good start.

Thanks.

  parent reply	other threads:[~2011-04-21 15:27 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-04-21 10:40 [RFC,PATCH 0/3] trace,perf: enabling ftrace/function tracepoint Jiri Olsa
2011-04-21 10:40 ` [RFC,PATCH 1/3] trace: add support for enabling ftrace/function tracepoint event Jiri Olsa
2011-04-21 10:40 ` [RFC,PATCH 2/3] trace,perf: add support for registering ftrace/function tracepoint event via perf Jiri Olsa
2011-04-21 10:40 ` [RFC,PATCH 3/3] trace,perf: add filter support for ftrace/function tracepoint event Jiri Olsa
2011-04-21 15:20   ` Frederic Weisbecker
2011-04-21 15:27 ` Frederic Weisbecker [this message]
2011-04-21 15:34   ` [RFC,PATCH 0/3] trace,perf: enabling ftrace/function tracepoint Steven Rostedt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20110421152724.GB1772@nowhere \
    --to=fweisbec@gmail.com \
    --cc=a.p.zijlstra@chello.nl \
    --cc=acme@redhat.com \
    --cc=jolsa@redhat.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lizf@cn.fujitsu.com \
    --cc=mingo@redhat.com \
    --cc=rostedt@goodmis.org \
    --cc=tzanussi@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox