From: Ingo Molnar <mingo@elte.hu>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: David Sharp <dhsharp@google.com>,
Vaibhav Nagarnaik <vnagarnaik@google.com>,
Michael Rubin <mrubin@google.com>,
Linus Torvalds <torvalds@linux-foundation.org>,
Arjan van de Ven <arjan@linux.intel.com>,
linux-kernel <linux-kernel@vger.kernel.org>,
Frederic Weisbecker <fweisbec@gmail.com>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Thomas Gleixner <tglx@linutronix.de>,
Christoph Hellwig <hch@infradead.org>,
Arnd Bergmann <arnd@arndb.de>
Subject: Re: Fix powerTOP regression with 2.6.39-rc5
Date: Tue, 10 May 2011 09:54:57 +0200 [thread overview]
Message-ID: <20110510075457.GJ11595@elte.hu> (raw)
In-Reply-To: <1304996847.2969.151.camel@frodo>
* Steven Rostedt <rostedt@goodmis.org> wrote:
> > I think its rather obvious how the unification should be done: check
> > tip:tmp.perf/trace for the 'trace' command that does tracing.
>
> I'll tell you what. I've been talking with other developers and one thing we
> came up with that we all seem to agree with is that ftrace is designed to
> trace the entire system, and it does it very well. Perf is designed to trace
> individual tasks, and it does it very well (trace is an example of this. It's
> focus is on tasks not the system). Ftrace can also trace individual tasks and
> perf can also trace the entire system, but they both do those poorly.
Not sure where you picked that up but it's 100% nonsense and you could not be
more wrong.
The reason why you see most instrumentation users use per task tracing and
profiling is very simple: they *can* do it and local views are what most
developer are interested in!
Otherwise perf has been designed to do system-wide (global) tracing pretty much
from day one on. In fact one of the first applications of perf: kerneltop, the
tool that evolved into 'perf top' has a system-wide view and never had any
other default but system-wide tracing+profiling ...
'perf top' is what many kernel developers use and it's very popular because the
kernel itself is 'system-wide' so obviously kernel developers want to have (and
need to have) a system-wide view.
ftrace uses system-wide tracing because that's pretty much the only model it
has. That is one of its many design mistakes, not a feature.
But the world is a lot more than just kernel focused workflows and perf
supports various other popular views:
- per task
- per task hierarchy (tree spanning fork()/exec()/clone() trees of tasks)
- per cgroup
- system-wide
And you want to keep ftrace a forked identity on the weird notion that somehow
perf can not do system-wide event collection and that somehow fundamentally
instrumentation can not serve these goals of event grouping?
Steve, your opinion is, sadly, very narrow.
Thanks,
Ingo
next prev parent reply other threads:[~2011-05-10 7:55 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-06 20:08 Fix powerTOP regression with 2.6.39-rc5 Arjan van de Ven
2011-05-06 20:20 ` Steven Rostedt
2011-05-06 20:51 ` Linus Torvalds
2011-05-06 21:10 ` Steven Rostedt
2011-05-06 21:24 ` Linus Torvalds
2011-05-06 21:14 ` Steven Rostedt
2011-05-06 21:28 ` Linus Torvalds
2011-05-06 21:29 ` Arjan van de Ven
2011-05-06 21:57 ` Steven Rostedt
2011-05-07 6:58 ` Ingo Molnar
2011-05-07 10:45 ` Steven Rostedt
2011-05-07 14:44 ` Ingo Molnar
2011-05-07 17:20 ` Steven Rostedt
2011-05-07 17:59 ` Arjan van de Ven
2011-05-08 21:08 ` Frederic Weisbecker
2011-05-08 21:56 ` Arjan van de Ven
2011-05-07 19:00 ` Ingo Molnar
2011-05-10 3:07 ` Steven Rostedt
2011-05-10 4:44 ` Dave Chinner
2011-05-10 5:39 ` Steven Rostedt
2011-05-10 7:36 ` Dave Chinner
2011-05-10 7:54 ` Ingo Molnar [this message]
2011-05-10 8:09 ` Ingo Molnar
2011-05-10 8:32 ` Arjan van de Ven
2011-05-10 8:44 ` Ingo Molnar
2011-05-10 9:14 ` Pekka Enberg
2011-05-10 8:41 ` Ingo Molnar
2011-05-10 13:06 ` Steven Rostedt
2011-05-11 21:51 ` Ingo Molnar
2011-05-11 22:36 ` Steven Rostedt
2011-05-17 7:15 ` Michael Rubin
2011-05-17 11:19 ` Steven Rostedt
2011-05-17 13:24 ` David Ahern
2011-05-17 13:27 ` Steven Rostedt
2011-05-17 13:30 ` Ingo Molnar
2011-05-10 8:47 ` Ingo Molnar
2011-05-10 10:33 ` Steven Rostedt
2011-05-10 19:13 ` David Sharp
2011-05-09 23:37 ` David Sharp
2011-05-10 7:39 ` Ingo Molnar
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=20110510075457.GJ11595@elte.hu \
--to=mingo@elte.hu \
--cc=a.p.zijlstra@chello.nl \
--cc=arjan@linux.intel.com \
--cc=arnd@arndb.de \
--cc=dhsharp@google.com \
--cc=fweisbec@gmail.com \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mrubin@google.com \
--cc=rostedt@goodmis.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=vnagarnaik@google.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 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.