From: Frederic Weisbecker <fweisbec@gmail.com>
To: Ingo Molnar <mingo@elte.hu>
Cc: LKML <linux-kernel@vger.kernel.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Mike Galbraith <efault@gmx.de>, Paul Mackerras <paulus@samba.org>,
Anton Blanchard <anton@samba.org>,
Jens Axboe <jens.axboe@oracle.com>,
Arnaldo Carvalho de Melo <acme@redhat.com>
Subject: Re: [PATCH 5/5] perf report: Support callchains with relative overhead rate
Date: Sun, 5 Jul 2009 15:23:35 +0200 [thread overview]
Message-ID: <20090705132333.GB4912@nowhere> (raw)
In-Reply-To: <20090705085949.GA23780@elte.hu>
On Sun, Jul 05, 2009 at 10:59:49AM +0200, Ingo Molnar wrote:
>
> btw., i get some buggy looking output with:
>
> $ perf record -f -g ~/hackbench 10
>
> $ perf report -c
>
>
> |--5.11%-- unix_stream_sendmsg
> | |
> | |--100.00%-- __sock_sendmsg
> | | sock_aio_write
> | | do_sync_write
> | | vfs_write
> | | sys_write
> | | sysenter_dispatch
> | | 0xf7f72430
> | | 0xffebbca000000014
> | |
> | --11.11%-- sock_aio_write
> | do_sync_write
> | vfs_write
> | sys_write
> | sysenter_dispatch
> | 0xf7f72430
> | 0xffebbca000000014
>
> Those percentages dont sum up to 100% :-)
Argh. I can reproduce it, will have a look.
> Another detail: i think we should signal when we crop the output due
> to the filter, via a line of:
>
> | [...]
>
> or so.
Ok.
> Plus, when doing 'perf report' on a call-chain recording, shouldnt
> we auto-detect this fact and default to fractal output
> automatically, instead of flat mode?
>
> User can still force flat mode via 'perf report -c flat'.
Yeah but the user won't be able to ignore the callchain.
May be I may add a -c none in this case?
next prev parent reply other threads:[~2009-07-05 13:23 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-07-05 5:39 [PATCH 1/5] perf report: Warn on callchain output request from non-callchain file Frederic Weisbecker
2009-07-05 5:39 ` [PATCH 2/5] perf report: Use a modifiable string for default callchain options Frederic Weisbecker
2009-07-05 9:52 ` [tip:perfcounters/urgent] " tip-bot for Frederic Weisbecker
2009-07-05 5:39 ` [PATCH 3/5] perf report: Change default callchain parameters Frederic Weisbecker
2009-07-05 9:52 ` [tip:perfcounters/urgent] " tip-bot for Frederic Weisbecker
2009-07-05 5:39 ` [PATCH 4/5] perf tools: callchains: Manage the cumul hits on the fly Frederic Weisbecker
2009-07-05 9:52 ` [tip:perfcounters/urgent] perf_counter " tip-bot for Frederic Weisbecker
2009-07-05 5:39 ` [PATCH 5/5] perf report: Support callchains with relative overhead rate Frederic Weisbecker
2009-07-05 8:34 ` Ingo Molnar
2009-07-05 8:59 ` Ingo Molnar
2009-07-05 13:23 ` Frederic Weisbecker [this message]
2009-07-05 13:19 ` Frederic Weisbecker
2009-07-05 9:52 ` [tip:perfcounters/urgent] perf report: Add "Fractal" mode output - support " tip-bot for Frederic Weisbecker
2009-07-05 9:51 ` [tip:perfcounters/urgent] perf report: Warn on callchain output request from non-callchain file tip-bot for Frederic 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=20090705132333.GB4912@nowhere \
--to=fweisbec@gmail.com \
--cc=a.p.zijlstra@chello.nl \
--cc=acme@redhat.com \
--cc=anton@samba.org \
--cc=efault@gmx.de \
--cc=jens.axboe@oracle.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mingo@elte.hu \
--cc=paulus@samba.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.