Messages in this thread | | | Date | Wed, 18 Dec 2013 14:47:15 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [RFC/PATCHSET 00/18] perf report: Add support to accumulate hist periods (v3) |
| |
Em Wed, Dec 18, 2013 at 11:37:49PM +0900, Namhyung Kim escreveu: > 2013-12-18 (수), 10:46 +0100, Ingo Molnar: > > * Namhyung Kim <namhyung@kernel.org> wrote:
> > > I changed the option as a separate --cumulate and added a new > > > "Total" column (and renamed the default "Overhead" column into > > > "Self"). The output will be sorted by total (cumulative) overhead > > > for now. The reason I changed to the --cumulate is that I still > > > think it's much different from other --callchain options and I > > > plan to add support for showing (remaining) callchains to > > > cumulative entries too. The --callchain option will take care of > > > it even with --cumulate option.
> > So I still think this is a fantastic feature, and while this variant > > of the user interface is an improvement over the previous version, we > > are not there yet ;-) > > Right. But I want to finish the core part of this patchset first. And > then I will work on the UI part since it'll be a quite large work by > itself, I guess.
I got the first few ones, that are not really related to this, to reduce the patchkit size, will go thru the others next.
> > So the natural way to get this would something like: > > > > perf report --total > > > > Or, if '--total' does not feel good, maybe we should change 'Total' to > > 'Siblings' or 'Children', and propagate that naming through the UI: > > Childrend looks better to me.
--children, ack
> > $ perf report --children --stdio > > > > # Self Children Command Shared Object Symbol > > # ........ ........ ....... ................. ....................... > > # > > 0.00% 88.29% abc libc-2.17.so [.] __libc_start_main > > 0.00% 88.29% abc abc [.] main > > 0.00% 88.29% abc abc [.] c > > 0.00% 88.29% abc abc [.] b > > 88.29% 88.29% abc abc [.] a > > 0.00% 11.61% abc ld-2.17.so [.] _dl_sysdep_start > > > > Which shows the sum of overhead of all child functions as well, not > > just ourselves. > > > > Anything but 'cumulative'. That word, beyond being a strange, hard to > > remember Latin word, also does not tell the user (or the developer) > > anything about _what_ is being accumulated. It could cover anything. > > > > I also think this option should be enabled by default - lets see how > > much people complain about that. > > > > Okay? > > Okay. I'll make it default in the next spin. > > Thanks, > Namhyung > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |