From: Arnaldo Carvalho de Melo <acme@kernel.org>
To: Mike Galbraith <umgwanakikbuti@gmail.com>
Cc: Namhyung Kim <namhyung@gmail.com>, LKML <linux-kernel@vger.kernel.org>
Subject: Re: perf top -g -U --sort=symbol --children == lalalalala?
Date: Thu, 11 Sep 2014 10:43:38 -0300 [thread overview]
Message-ID: <20140911134338.GE10158@kernel.org> (raw)
In-Reply-To: <1410439392.20184.17.camel@marge.simpson.net>
Em Thu, Sep 11, 2014 at 02:43:12PM +0200, Mike Galbraith escreveu:
> On Thu, 2014-09-11 at 17:09 +0900, Namhyung Kim wrote:
> > On Tue, Sep 9, 2014 at 10:37 PM, Arnaldo Carvalho de Melo
> > <acme@ghostprotocols.net> wrote:
> > > Em Tue, Sep 09, 2014 at 01:54:01PM +0200, Mike Galbraith escreveu:
> > >> Seems the now default on --children thingy doesn't like -U much.
> > >
> > > Namhyung, can you please take a look at this?
> >
> > So what is the problem here?
>
> Well, if you don't see anything wrong, I guess nothing at all.
:-)
I think that when we decide that it is so better to change defaults like
we did this time, we should be required to add a big fat warning (a
--tui popup, use the first lines on --stdio, etc) about why the default
was changed and allow quick, easy opt out, restoring previous behaviour
after the user, being warned, knows what to expect, tries it, and then
is in a better position to decide if keeping the new default is what is
desired.
- Arnaldo
> > >> Samples: 5K of event 'cycles', Event count (approx.): 2268660922
> > >> Children Self Symbol
> > >> + 46.42% 0.04% [k] system_call_fastpath
> I'll just turn it off until I figure out what cool stuff this is telling
> me. why that symbol becomes the number one hit, and why total% > 100.
> To me, it looks like top smoked it's breakfast, went to lala land ;-)
Yeah, its confusing, I'll let Namhyung explain it ;-)
:-)
- Arnaldo
next prev parent reply other threads:[~2014-09-11 13:43 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-09 11:54 perf top -g -U --sort=symbol --children == lalalalala? Mike Galbraith
2014-09-09 13:37 ` Arnaldo Carvalho de Melo
2014-09-11 8:09 ` Namhyung Kim
2014-09-11 12:43 ` Mike Galbraith
2014-09-11 13:43 ` Arnaldo Carvalho de Melo [this message]
2014-09-11 21:30 ` Arnaldo Carvalho de Melo
2014-09-12 2:30 ` Mike Galbraith
2014-09-12 7:24 ` Namhyung Kim
2014-09-12 8:41 ` Mike Galbraith
2014-09-12 12:10 ` Arnaldo Carvalho de Melo
2014-09-12 12:07 ` Arnaldo Carvalho de Melo
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=20140911134338.GE10158@kernel.org \
--to=acme@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=namhyung@gmail.com \
--cc=umgwanakikbuti@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