Commit cb1fab91 authored by Namhyung Kim's avatar Namhyung Kim Committed by Arnaldo Carvalho de Melo
Browse files

perf report: Left align dynamic entries in hierarchy 



The dynamic entries are right-aligned unlike other entries since it
usually has numeric value.  But for the hierarchy mode, left alignment
is more appropriate IMHO.  Also trim spaces on the left so that we can
easily identify the hierarchy.

Before:

  $ perf report --hierarchy -i perf.data.kmem -s gfp_flags,ptr,bytes_req --stdio -g none
  ...
  #
  #       Overhead                                        gfp_flags /                ptr /          bytes_req
  # ..............  .................................................................................................
  #
      91.67%                   GFP_ATOMIC|GFP_NOWARN|GFP_NOMEMALLOC
         37.50%        0xffff8803f7669400
            37.50%                       448
          8.33%        0xffff8803f766be00
             8.33%                        96
          4.17%        0xffff8800d156dc00
             4.17%                       704

After:

  #       Overhead  gfp_flags / ptr / bytes_req
  # ..............  ....................................
  #
      91.67%        GFP_ATOMIC|GFP_NOWARN|GFP_NOMEMALLOC
         37.50%        0xffff8803f7669400
            37.50%        448
          8.33%        0xffff8803f766be00
             8.33%        96
          4.17%        0xffff8800d156dc00
             4.17%        704

Signed-off-by: default avatarNamhyung Kim <namhyung@kernel.org>
Tested-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
Acked-by: default avatarJiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <andi@firstfloor.org>
Cc: David Ahern <dsahern@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>
Cc: Stephane Eranian <eranian@google.com>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1456512767-1164-3-git-send-email-namhyung@kernel.org


Signed-off-by: default avatarArnaldo Carvalho de Melo <acme@redhat.com>
parent d3a72fd8

tools/perf/ui/browsers/hists.c

+14 −2
Original line number Diff line number Diff line
@@ -1400,8 +1400,13 @@ static int hist_browser__show_hierarchy_entry(struct hist_browser *browser, 
		if (fmt->color) {

			width -= fmt->color(fmt, &hpp, entry);

		} else {

			int i = 0;



			width -= fmt->entry(fmt, &hpp, entry);

			ui_browser__printf(&browser->b, "%s", s);

			ui_browser__printf(&browser->b, "%s", ltrim(s));



			while (isspace(s[i++]))

				width++;

		}

	}
@@ -1576,6 +1581,8 @@ static int hists_browser__scnprintf_hierarchy_headers(struct hist_browser *brows 
		return ret;



	hists__for_each_format(hists, fmt) {

		char *start;



		if (!perf_hpp__is_sort_entry(fmt) && !perf_hpp__is_dynamic_entry(fmt))

			continue;

		if (perf_hpp__should_skip(fmt, hists))
@@ -1593,7 +1600,12 @@ static int hists_browser__scnprintf_hierarchy_headers(struct hist_browser *brows 
		dummy_hpp.buf[ret] = '\0';

		rtrim(dummy_hpp.buf);



		ret = strlen(dummy_hpp.buf);

		start = ltrim(dummy_hpp.buf);

		ret = strlen(start);



		if (start != dummy_hpp.buf)

			memmove(dummy_hpp.buf, start, ret + 1);



		if (advance_hpp_check(&dummy_hpp, ret))

			break;

	}

tools/perf/ui/stdio/hist.c

+19 −9
Original line number Diff line number Diff line
@@ -418,6 +418,7 @@ static int hist_entry__hierarchy_fprintf(struct hist_entry *he, 
	const char *sep = symbol_conf.field_sep;

	struct perf_hpp_fmt *fmt;

	char *buf = hpp->buf;

	size_t size = hpp->size;

	int ret, printed = 0;

	bool first = true;
@@ -457,6 +458,11 @@ static int hist_entry__hierarchy_fprintf(struct hist_entry *he, 
				(nr_sort_key - 1) * HIERARCHY_INDENT + 2, "");

	advance_hpp(hpp, ret);



	printed += fprintf(fp, "%s", buf);



	hpp->buf  = buf;

	hpp->size = size;



	/*

	 * No need to call hist_entry__snprintf_alignment() since this

	 * fmt is always the last column in the hierarchy mode.
@@ -467,7 +473,11 @@ static int hist_entry__hierarchy_fprintf(struct hist_entry *he, 
	else

		fmt->entry(fmt, hpp, he);



	printed += fprintf(fp, "%s\n", buf);

	/*

	 * dynamic entries are right-aligned but we want left-aligned

	 * in the hierarchy mode

	 */

	printed += fprintf(fp, "%s\n", ltrim(buf));



	if (symbol_conf.use_callchain && he->leaf) {

		u64 total = hists__total_period(hists);
@@ -525,6 +535,7 @@ static int print_hierarchy_header(struct hists *hists, struct perf_hpp *hpp, 
{

	bool first = true;

	int nr_sort;

	int depth;

	unsigned width = 0;

	unsigned header_width = 0;

	struct perf_hpp_fmt *fmt;
@@ -558,19 +569,16 @@ static int print_hierarchy_header(struct hists *hists, struct perf_hpp *hpp, 
		if (!first)

			header_width += fprintf(fp, " / ");

		else {

			header_width += fprintf(fp, "%s", sep ?: "  ");

			fprintf(fp, "%s", sep ?: "  ");

			first = false;

		}



		fmt->header(fmt, hpp, hists_to_evsel(hists));

		rtrim(hpp->buf);



		header_width += fprintf(fp, "%s", hpp->buf);

		header_width += fprintf(fp, "%s", ltrim(hpp->buf));

	}



	/* preserve max indent depth for combined sort headers */

	print_hierarchy_indent(sep, nr_sort, spaces, fp);



	fprintf(fp, "\n# ");



	/* preserve max indent depth for initial dots */
@@ -590,6 +598,7 @@ static int print_hierarchy_header(struct hists *hists, struct perf_hpp *hpp, 
		fprintf(fp, "%.*s", width, dots);

	}



	depth = 0;

	hists__for_each_format(hists, fmt) {

		if (!perf_hpp__is_sort_entry(fmt) && !perf_hpp__is_dynamic_entry(fmt))

			continue;
@@ -597,15 +606,16 @@ static int print_hierarchy_header(struct hists *hists, struct perf_hpp *hpp, 
			continue;



		width = fmt->width(fmt, hpp, hists_to_evsel(hists));

		width += depth * HIERARCHY_INDENT;



		if (width > header_width)

			header_width = width;



		depth++;

	}



	fprintf(fp, "%s%-.*s", sep ?: "  ", header_width, dots);



	/* preserve max indent depth for dots under sort headers */

	print_hierarchy_indent(sep, nr_sort, dots, fp);



	fprintf(fp, "\n#\n");



	return 2;

CRA Git | Maintained and supported by SUSTech CRA and CCSE