All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: "Frédéric Weisbecker" <fweisbec@gmail.com>
Cc: Steven Rostedt <rostedt@goodmis.org>,
	Linux Kernel <linux-kernel@vger.kernel.org>
Subject: Re: Human readable output for function return tracer
Date: Tue, 25 Nov 2008 16:36:53 +0100	[thread overview]
Message-ID: <20081125153653.GA21001@elte.hu> (raw)
In-Reply-To: <c62985530811250718q3aa5f2ednd795b7c927abb8e4@mail.gmail.com>


* Frédéric Weisbecker <fweisbec@gmail.com> wrote:

> I made a patch last evening and I almost finished implementing the
> desired output.
>
> But while I went through, I was adding some more keywords with other 
> "return-function-tracing" semantics, plus the risk of confusion 
> resulting from the entry-on-function-for-return-tracing keywords....
> 
> I planned to change all the keywords according to a new name for 
> this tracer just after this patch. But I can't seriously submit such 
> a mess, even with a second patch that fixes the keywords....
> 
> I would like to change the name of the tracer and its keywords 
> before sending the output changes.

> Do you agree with "full function tracer" (since we hook now on the 
> two sides)?

"full function tracer" sounds a bit funny and quirky. How about 
"function call tracer"? Versus the "function tracer" or "function 
entry tracer" which is the lighter variant - both in name and in 
overhead. So we'd have:

 # cat /debug/tracing/available_tracers 
 mmiotrace wakeup irqsoff function function-call sysprof sched_switch initcall nop

note how intuitive it is: "function-call" is 'more' than just the 
plain function-tracer. It also expresses its main property: it traces 
the full call, entry and exit and return code as well.

Hm?

	Ingo

  reply	other threads:[~2008-11-25 15:37 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-24 14:39 Human readable output for function return tracer Frédéric Weisbecker
2008-11-24 16:27 ` Ingo Molnar
2008-11-24 17:16   ` Frédéric Weisbecker
2008-11-25 15:18   ` Frédéric Weisbecker
2008-11-25 15:36     ` Ingo Molnar [this message]
2008-11-25 15:40       ` Ingo Molnar
2008-11-25 15:49         ` Frédéric Weisbecker
2008-11-25 15:56         ` Ingo Molnar
2008-11-25 16:01           ` Frédéric Weisbecker
2008-11-24 17:21 ` Steven Rostedt
2008-11-24 17:34   ` Frédéric Weisbecker
2008-11-24 18:15 ` Arnaldo Carvalho de Melo
2008-11-24 19:10   ` Frédéric Weisbecker
2008-11-24 19:39     ` Arnaldo Carvalho de Melo
2008-11-24 20:00       ` Frédéric Weisbecker
2008-11-24 20:13       ` Frédéric Weisbecker

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=20081125153653.GA21001@elte.hu \
    --to=mingo@elte.hu \
    --cc=fweisbec@gmail.com \
    --cc=linux-kernel@vger.kernel.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.