aboutsummaryrefslogtreecommitdiffstats
path: root/tools/perf/scripts/python/export-to-postgresql.py
diff options
context:
space:
mode:
authorNamhyung Kim <namhyung@kernel.org>2016-01-28 00:40:53 +0900
committerArnaldo Carvalho de Melo <acme@redhat.com>2016-02-01 17:41:32 -0300
commit7ed5d6e28a0a1a54f554b0ab9c38a6061e7cac9e (patch)
tree4857e8eea3617ee54f2eb10ce3d4204709ad2981 /tools/perf/scripts/python/export-to-postgresql.py
parentperf callchain: Pass parent_samples to __callchain__fprintf_graph() (diff)
downloadlinux-7ed5d6e28a0a1a54f554b0ab9c38a6061e7cac9e.tar.gz
linux-7ed5d6e28a0a1a54f554b0ab9c38a6061e7cac9e.zip
perf report: Fix percent display in callchains on --stdio
When there's only a single callchain, perf doesn't print its percentage in front of the symbols. This is because it assumes that the percentage is same as parents. But if a percent limit is applied, it's possible that there are actually a couple of child nodes but only one of them is shown. In this case it should display the percent to prevent misunderstanding of its percentage is same as the parent's. For example, let's see the following callchain. $ perf report -s comm --percent-limit 0.01 --stdio ... 9.95% swapper | |--7.57%--intel_idle | cpuidle_enter_state | cpuidle_enter | call_cpuidle | cpu_startup_entry | | | |--4.89%--start_secondary | | | --2.68%--rest_init | start_kernel | x86_64_start_reservations | x86_64_start_kernel | |--0.15%--__schedule | | | |--0.13%--schedule | | schedule_preempt_disable | | cpu_startup_entry | | | | | |--0.09%--start_secondary | | | | | --0.04%--rest_init | | start_kernel | | x86_64_start_reservations | | x86_64_start_kernel | | | --0.01%--schedule_preempt_disabled | cpu_startup_entry ... Current code omits the percent if 'intel_idle' becomes the only node when percent limit is set to 0.5%, its percent is not 9.95% but users will assume it incorrectly. Before: $ perf report --percent-limit 0.5 --stdio ... 9.95% swapper | ---intel_idle cpuidle_enter_state cpuidle_enter call_cpuidle cpu_startup_entry | |--4.89%--start_secondary | --2.68%--rest_init start_kernel x86_64_start_reservations x86_64_start_kernel After: $ perf report --percent-limit 0.5 --stdio ... 9.95% swapper | --7.57%--intel_idle cpuidle_enter_state cpuidle_enter call_cpuidle cpu_startup_entry | |--4.89%--start_secondary | --2.68%--rest_init start_kernel x86_64_start_reservations x86_64_start_kernel Signed-off-by: Namhyung Kim <namhyung@kernel.org> Tested-by: Arnaldo Carvalho de Melo <acme@redhat.com> Cc: Andi Kleen <andi@firstfloor.org> Cc: David Ahern <dsahern@gmail.com> Cc: Frederic Weisbecker <fweisbec@gmail.com> Cc: Jiri Olsa <jolsa@kernel.org> Cc: Peter Zijlstra <peterz@infradead.org> Cc: Wang Nan <wangnan0@huawei.com> Link: http://lkml.kernel.org/r/1453909257-26015-7-git-send-email-namhyung@kernel.org Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions
ject_dir>/pack directoryBrandon Casey1-10/+12 2008-09-22t4018-diff-funcname: test syntax of builtin xfuncname patternsBrandon Casey1-0/+11 2008-09-22Do not perform cross-directory renames when creating packsPetr Baudis4-5/+5 2008-09-22Use dashless git commands in setgitperms.perlTodd Zullinger1-2/+2 2008-09-22git-remote: do not use user input in a printf format stringJohannes Sixt1-10/+8 2008-09-22make "git remote" report multiple URLsMichael J Gruber1-8/+15 2008-09-20diff hunk pattern: fix misconverted "\{" tex macro introducersJunio C Hamano1-2/+2 2008-09-19diff: use extended regexp to find hunk headersJunio C Hamano1-14/+17 2008-09-19Start draft release notes for 1.6.0.3Junio C Hamano2-1/+30 2008-09-19git-repack uses --no-repack-object, not --no-repack-delta.Mikael Magnusson2-2/+2 2008-09-19Typo "bogos" in format-patch error message.Mikael Magnusson1-1/+1 2008-09-19builtin-clone: fix typoFabrizio Chiarello1-1/+1 2008-09-19Bust the ghost of long-defunct diffcore-pathspec.Yann Dirson2-34/+22 2008-09-19completion: git commit should list --interactiveEric Raible1-1/+1 2008-09-18diff.*.xfuncname which uses "extended" regex's for hunk header selectionBrandon Casey3-3/+8 2008-09-18diff.c: associate a flag with each pattern and use it for compiling regexBrandon Casey3-12/+15 2008-09-18diff.c: return pattern entry pointer rather than just the hunk header patternBrandon Casey1-27/+28 2008-09-18sha1_file: link() returns -1 on failure, not errnoThomas Rast1-1/+3 2008-09-18Make git archive respect core.autocrlf when creating zip format archivesCharles Bailey2-1/+3