Skip to content

Commit

Permalink
perf hists: Use struct perf_hpp_fmt::idx in perf_hpp__reset_width
Browse files Browse the repository at this point in the history
We are going to add dynamic hpp format fields, so we need to make the
'len' change for the format itself, not in the perf_hpp__format
template.

Signed-off-by: Jiri Olsa <jolsa@kernel.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1453109064-1026-5-git-send-email-jolsa@kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
  • Loading branch information
Jiri Olsa authored and Arnaldo Carvalho de Melo committed Feb 3, 2016
1 parent b21a763 commit 2e8b79e
Showing 1 changed file with 2 additions and 10 deletions.
12 changes: 2 additions & 10 deletions tools/perf/ui/hist.c
Original file line number Diff line number Diff line change
Expand Up @@ -629,20 +629,12 @@ unsigned int hists__sort_list_width(struct hists *hists)

void perf_hpp__reset_width(struct perf_hpp_fmt *fmt, struct hists *hists)
{
int idx;

if (perf_hpp__is_sort_entry(fmt))
return perf_hpp__reset_sort_width(fmt, hists);

for (idx = 0; idx < PERF_HPP__MAX_INDEX; idx++) {
if (fmt == &perf_hpp__format[idx])
break;
}

if (idx == PERF_HPP__MAX_INDEX)
return;
BUG_ON(fmt->idx >= PERF_HPP__MAX_INDEX);

switch (idx) {
switch (fmt->idx) {
case PERF_HPP__OVERHEAD:
case PERF_HPP__OVERHEAD_SYS:
case PERF_HPP__OVERHEAD_US:
Expand Down

0 comments on commit 2e8b79e

Please sign in to comment.