linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCHSET 0/7] Cleanup hist printing code (v3)
@ 2012-08-20  4:52 Namhyung Kim
  2012-08-20  4:52 ` [PATCH 1/7] perf hists: Separate out hist print functions Namhyung Kim
                   ` (6 more replies)
  0 siblings, 7 replies; 15+ messages in thread
From: Namhyung Kim @ 2012-08-20  4:52 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, LKML

Hello,

This is a cleanup and refactoring patchset for the hist printing code
by adding hist_period_print functions and hpp_context.  I believe it
makes the code easy to maintain and to add new functionalities like
upcoming group viewing and callchain accumulation.

Any comments are welcome, thanks.
Namhyung

v2 -> v3:
 * Rebase to current acme/perf/core
 * Move fprintf code to ui/stdio/hist.c (Arnaldo)
 * Add ack from Pekka


Namhyung Kim (7):
  perf hists: Separate out hist print functions
  perf hists: Refactor some functions
  perf hists: Introduce hist_period_print functions
  perf hists: Handle field separator properly
  perf hists: Use hpp_functions->width to calculate the column widths
  perf ui/browser: Use hist_period_print functions
  perf gtk/browser: Use hist_period_print functions

 tools/perf/Makefile            |   5 +-
 tools/perf/builtin-diff.c      |   1 +
 tools/perf/ui/browsers/hists.c |  96 ++++--
 tools/perf/ui/gtk/browser.c    | 101 +++++-
 tools/perf/ui/gtk/gtk.h        |   1 +
 tools/perf/ui/gtk/setup.c      |   1 +
 tools/perf/ui/hist.c           | 391 +++++++++++++++++++++++
 tools/perf/ui/setup.c          |   8 +-
 tools/perf/ui/stdio/hist.c     | 498 +++++++++++++++++++++++++++++
 tools/perf/ui/tui/setup.c      |   4 +
 tools/perf/util/hist.c         | 710 ++---------------------------------------
 tools/perf/util/hist.h         |  43 ++-
 12 files changed, 1121 insertions(+), 738 deletions(-)
 create mode 100644 tools/perf/ui/hist.c
 create mode 100644 tools/perf/ui/stdio/hist.c

-- 
1.7.11.2


^ permalink raw reply	[flat|nested] 15+ messages in thread
* [PATCH 0/7] perf hists: Cleanup hist printing code (v2)
@ 2012-08-06  8:57 Namhyung Kim
  2012-08-06  8:57 ` [PATCH 1/7] perf hists: Separate out hist print functions Namhyung Kim
  0 siblings, 1 reply; 15+ messages in thread
From: Namhyung Kim @ 2012-08-06  8:57 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, LKML,
	Stephane Eranian, Pekka Enberg

Hi,

This is a cleanup and refactoring patchset for the hist printing code
by adding hist_period_print functions and hpp_context. I believe it
makes the code easy to maintain and to add new functionalities like
upcoming group viewing and callchain accumulation.

Any comments are welcome, thanks.
Namhyung


Namhyung Kim (7):
  perf hists: Separate out hist print functions
  perf hists: Refactor some functions
  perf hists: Introduce hist_period_print functions
  perf hists: Handle field separator properly
  perf hists: Use hpp_functions->width to calculate the column widths
  perf ui/browser: Use hist_period_print functions
  perf gtk/browser: Use hist_period_print functions

 tools/perf/Makefile            |   4 +-
 tools/perf/builtin-diff.c      |   1 +
 tools/perf/ui/browsers/hists.c |  96 +++--
 tools/perf/ui/gtk/browser.c    | 101 ++++-
 tools/perf/ui/gtk/gtk.h        |   2 +
 tools/perf/ui/gtk/setup.c      |   1 +
 tools/perf/ui/hist.c           | 885 +++++++++++++++++++++++++++++++++++++++++
 tools/perf/ui/setup.c          |   8 +-
 tools/perf/ui/tui/setup.c      |   4 +
 tools/perf/util/hist.c         | 710 +--------------------------------
 tools/perf/util/hist.h         |  41 +-
 11 files changed, 1115 insertions(+), 738 deletions(-)
 create mode 100644 tools/perf/ui/hist.c

-- 
1.7.11.2


^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2012-08-21 16:35 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-20  4:52 [PATCHSET 0/7] Cleanup hist printing code (v3) Namhyung Kim
2012-08-20  4:52 ` [PATCH 1/7] perf hists: Separate out hist print functions Namhyung Kim
2012-08-21 16:34   ` [tip:perf/core] " tip-bot for Namhyung Kim
2012-08-20  4:52 ` [PATCH 2/7] perf hists: Refactor some functions Namhyung Kim
2012-08-21 16:34   ` [tip:perf/core] perf hists: Rename and move " tip-bot for Namhyung Kim
2012-08-20  4:52 ` [PATCH 3/7] perf hists: Introduce hist_period_print functions Namhyung Kim
2012-08-20 13:08   ` Arnaldo Carvalho de Melo
2012-08-21  8:05     ` Namhyung Kim
2012-08-20  4:52 ` [PATCH 4/7] perf hists: Handle field separator properly Namhyung Kim
2012-08-20  4:52 ` [PATCH 5/7] perf hists: Use hpp_functions->width to calculate the column widths Namhyung Kim
2012-08-20  4:52 ` [PATCH 6/7] perf ui/browser: Use hist_period_print functions Namhyung Kim
2012-08-20  4:52 ` [PATCH 7/7] perf gtk/browser: " Namhyung Kim
  -- strict thread matches above, loose matches on Subject: below --
2012-08-06  8:57 [PATCH 0/7] perf hists: Cleanup hist printing code (v2) Namhyung Kim
2012-08-06  8:57 ` [PATCH 1/7] perf hists: Separate out hist print functions Namhyung Kim
2012-08-09 19:18   ` Arnaldo Carvalho de Melo
2012-08-10  3:08     ` Namhyung Kim

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).