diff options
| author | Ingo Molnar <mingo@kernel.org> | 2016-01-30 03:15:49 -0500 |
|---|---|---|
| committer | Ingo Molnar <mingo@kernel.org> | 2016-01-30 03:15:49 -0500 |
| commit | 28fb8a5b6e233fc384fb27f9f91f811b40ba9cf8 (patch) | |
| tree | 6239311b9d3bf0fc1b62e8abe27a460a3959b458 | |
| parent | 5fa7c8ec57f70a7b5c6fe269fa9c51b9e465989c (diff) | |
| parent | 3f416f22d1e21709a631189ba169f76fd267b374 (diff) | |
Merge tag 'perf-urgent-for-mingo' of git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux into perf/urgent
Pull perf/urgent fixes from Arnaldo Carvalho de Melo:
- Fix 'perf stat' stddev reporting due to mistakenly cleaning event
private stats (Jiri Olsa)
- Fix 'perf test CQM' endless loop detected by 'gcc6 -Wmisleading-indentation'
(Markus Trippelsdorf)
- Fix behaviour of Shift-Tab when nothing is focussed in the annotate TUI browser,
detected with gcc6 -Wmisleading-indentation (Markus Trippelsdorf)
- Fix mem data cacheline hists browser width setting for unresolved
addresses (Jiri Olsa)
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: Ingo Molnar <mingo@kernel.org>
| -rw-r--r-- | tools/perf/arch/x86/tests/intel-cqm.c | 2 | ||||
| -rw-r--r-- | tools/perf/ui/browsers/annotate.c | 4 | ||||
| -rw-r--r-- | tools/perf/util/hist.c | 2 | ||||
| -rw-r--r-- | tools/perf/util/stat.c | 1 |
4 files changed, 5 insertions, 4 deletions
diff --git a/tools/perf/arch/x86/tests/intel-cqm.c b/tools/perf/arch/x86/tests/intel-cqm.c index 3e89ba825f6b..7f064eb37158 100644 --- a/tools/perf/arch/x86/tests/intel-cqm.c +++ b/tools/perf/arch/x86/tests/intel-cqm.c | |||
| @@ -17,7 +17,7 @@ static pid_t spawn(void) | |||
| 17 | if (pid) | 17 | if (pid) |
| 18 | return pid; | 18 | return pid; |
| 19 | 19 | ||
| 20 | while(1); | 20 | while(1) |
| 21 | sleep(5); | 21 | sleep(5); |
| 22 | return 0; | 22 | return 0; |
| 23 | } | 23 | } |
diff --git a/tools/perf/ui/browsers/annotate.c b/tools/perf/ui/browsers/annotate.c index d4d7cc27252f..718bd46d47fa 100644 --- a/tools/perf/ui/browsers/annotate.c +++ b/tools/perf/ui/browsers/annotate.c | |||
| @@ -755,11 +755,11 @@ static int annotate_browser__run(struct annotate_browser *browser, | |||
| 755 | nd = browser->curr_hot; | 755 | nd = browser->curr_hot; |
| 756 | break; | 756 | break; |
| 757 | case K_UNTAB: | 757 | case K_UNTAB: |
| 758 | if (nd != NULL) | 758 | if (nd != NULL) { |
| 759 | nd = rb_next(nd); | 759 | nd = rb_next(nd); |
| 760 | if (nd == NULL) | 760 | if (nd == NULL) |
| 761 | nd = rb_first(&browser->entries); | 761 | nd = rb_first(&browser->entries); |
| 762 | else | 762 | } else |
| 763 | nd = browser->curr_hot; | 763 | nd = browser->curr_hot; |
| 764 | break; | 764 | break; |
| 765 | case K_F1: | 765 | case K_F1: |
diff --git a/tools/perf/util/hist.c b/tools/perf/util/hist.c index c226303e3da0..68a7612019dc 100644 --- a/tools/perf/util/hist.c +++ b/tools/perf/util/hist.c | |||
| @@ -131,6 +131,8 @@ void hists__calc_col_len(struct hists *hists, struct hist_entry *h) | |||
| 131 | symlen = unresolved_col_width + 4 + 2; | 131 | symlen = unresolved_col_width + 4 + 2; |
| 132 | hists__new_col_len(hists, HISTC_MEM_DADDR_SYMBOL, | 132 | hists__new_col_len(hists, HISTC_MEM_DADDR_SYMBOL, |
| 133 | symlen); | 133 | symlen); |
| 134 | hists__new_col_len(hists, HISTC_MEM_DCACHELINE, | ||
| 135 | symlen); | ||
| 134 | } | 136 | } |
| 135 | 137 | ||
| 136 | if (h->mem_info->iaddr.sym) { | 138 | if (h->mem_info->iaddr.sym) { |
diff --git a/tools/perf/util/stat.c b/tools/perf/util/stat.c index 2f901d15e063..2b58edccd56f 100644 --- a/tools/perf/util/stat.c +++ b/tools/perf/util/stat.c | |||
| @@ -310,7 +310,6 @@ int perf_stat_process_counter(struct perf_stat_config *config, | |||
| 310 | int i, ret; | 310 | int i, ret; |
| 311 | 311 | ||
| 312 | aggr->val = aggr->ena = aggr->run = 0; | 312 | aggr->val = aggr->ena = aggr->run = 0; |
| 313 | init_stats(ps->res_stats); | ||
| 314 | 313 | ||
| 315 | if (counter->per_pkg) | 314 | if (counter->per_pkg) |
| 316 | zero_per_pkg(counter); | 315 | zero_per_pkg(counter); |
