From: Frederic Weisbecker <fweisbec@gmail.com>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: linux-kernel@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
Andrew Morton <akpm@linux-foundation.org>,
Peter Zijlstra <peterz@infradead.org>,
Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [PATCH 00/15] [git pull] for tip/tracing/ftrace
Date: Tue, 17 Feb 2009 15:24:15 +0100 [thread overview]
Message-ID: <20090217142414.GA5888@nowhere> (raw)
In-Reply-To: <20090217051227.957864159@goodmis.org>
On Tue, Feb 17, 2009 at 12:12:27AM -0500, Steven Rostedt wrote:
>
> Ingo,
>
> This is a patch series that offers a variety of features.
>
> The first is to add:
>
> #### all functions enabled ####
>
> to set_ftrace_filter when all functions will be traced when function
> tracing is on.
>
> The next is to add a :command: interface, and the module command:
>
> echo '*:mod:ext4' > set_ftrace_filter
>
> Will select all functions in the ext4 module.
>
> echo '!*write*:mod:ext4' > set_ftrace_filter
>
> will remove all functions with the characters 'write' and are
> in ext4.
>
> Next we extend the :command: interface as well as add a way to
> hook tracers to specific functions. Instead of tracing all functions
> the same, this patch series adds a hash to be able to trace different
> functions differently.
>
> I implement a 'traceon' and 'traceoff' to have a function start or
> stop tracing when it is hit. It also includes a counter for the
> number of times to implement the trace on/off.
>
> echo '*_lock:traceon:4' > set_ftrace_filter
> echo '*_unlock:traceoff:5' > set_ftrace_filter
That's great!
I guess it could be useful to be able to pass it as a kernel parameter.
Something like that ftrace=function_tracer ftrace_filter='*_lock:traceon:4'
> This will have all functions that end with _lock turn on tracing
> and all that end with _unlock disable it, 4 times each.
>
> The following patches are in:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-2.6-trace.git
>
> branch: tip/tracing/ftrace
>
>
> Steven Rostedt (15):
> ftrace: state that all functions are enabled in set_ftrace_filter
> ftrace: add do_for_each_ftrace_rec and while_for_each_ftrace_rec
> ftrace: rename ftrace_match to ftrace_match_records
> ftrace: break up ftrace_match_records into smaller components
> ftrace: add module command function filter selection
> ftrace: enable filtering only when a function is filtered on
> ftrace: add command interface for function selection
> ftrace: convert ftrace_lock from a spinlock to mutex
> ftrace: consolidate mutexes
> ftrace: trace different functions with a different tracer
> ring-buffer: add tracing_is_on to test if ring buffer is enabled
> ftrace: add traceon traceoff commands to enable/disable the buffers
> ftrace: show selected functions in set_ftrace_filter
> ftrace: add pretty print to selected fuction traces
> ftrace: add pretty print function for traceon and traceoff hooks
>
> ----
> include/linux/ftrace.h | 40 ++
> include/linux/ring_buffer.h | 2 +
> kernel/trace/ftrace.c | 929 +++++++++++++++++++++++++++++++---------
> kernel/trace/ring_buffer.c | 9 +
> kernel/trace/trace_functions.c | 163 +++++++
> 5 files changed, 946 insertions(+), 197 deletions(-)
> --
prev parent reply other threads:[~2009-02-17 14:24 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-17 5:12 [PATCH 00/15] [git pull] for tip/tracing/ftrace Steven Rostedt
2009-02-17 5:12 ` [PATCH 01/15] ftrace: state that all functions are enabled in set_ftrace_filter Steven Rostedt
2009-02-17 5:12 ` [PATCH 02/15] ftrace: add do_for_each_ftrace_rec and while_for_each_ftrace_rec Steven Rostedt
2009-02-17 5:12 ` [PATCH 03/15] ftrace: rename ftrace_match to ftrace_match_records Steven Rostedt
2009-02-17 5:12 ` [PATCH 04/15] ftrace: break up ftrace_match_records into smaller components Steven Rostedt
2009-02-17 10:34 ` Ingo Molnar
2009-02-17 5:12 ` [PATCH 05/15] ftrace: add module command function filter selection Steven Rostedt
2009-02-17 5:12 ` [PATCH 06/15] ftrace: enable filtering only when a function is filtered on Steven Rostedt
2009-02-17 5:12 ` [PATCH 07/15] ftrace: add command interface for function selection Steven Rostedt
2009-02-17 5:12 ` [PATCH 08/15] ftrace: convert ftrace_lock from a spinlock to mutex Steven Rostedt
2009-02-17 5:12 ` [PATCH 09/15] ftrace: consolidate mutexes Steven Rostedt
2009-02-17 5:12 ` [PATCH 10/15] ftrace: trace different functions with a different tracer Steven Rostedt
2009-02-17 18:45 ` Paul E. McKenney
2009-02-17 18:53 ` Steven Rostedt
2009-02-17 5:12 ` [PATCH 11/15] ring-buffer: add tracing_is_on to test if ring buffer is enabled Steven Rostedt
2009-02-17 5:12 ` [PATCH 12/15] ftrace: add traceon traceoff commands to enable/disable the buffers Steven Rostedt
2009-02-17 10:37 ` Ingo Molnar
2009-02-17 12:45 ` Steven Rostedt
2009-02-17 12:47 ` Ingo Molnar
2009-02-17 5:12 ` [PATCH 13/15] ftrace: show selected functions in set_ftrace_filter Steven Rostedt
2009-02-17 5:12 ` [PATCH 14/15] ftrace: add pretty print to selected fuction traces Steven Rostedt
2009-02-17 5:12 ` [PATCH 15/15] ftrace: add pretty print function for traceon and traceoff hooks Steven Rostedt
2009-02-17 10:42 ` [PATCH 00/15] [git pull] for tip/tracing/ftrace Ingo Molnar
2009-02-17 12:49 ` Steven Rostedt
2009-02-17 14:37 ` Ingo Molnar
2009-02-18 0:41 ` Sam Ravnborg
2009-02-18 0:51 ` Ingo Molnar
2009-02-17 17:09 ` Steven Rostedt
2009-02-17 23:46 ` Ingo Molnar
2009-02-17 14:24 ` Frederic Weisbecker [this message]
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=20090217142414.GA5888@nowhere \
--to=fweisbec@gmail.com \
--cc=acme@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=peterz@infradead.org \
--cc=rostedt@goodmis.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.