Re: [GIT PULL] perf tools: Factorize common code

From: Frederic Weisbecker
Date: Wed Aug 12 2009 - 05:27:28 EST


On Wed, Aug 12, 2009 at 11:25:59AM +0200, Frederic Weisbecker wrote:
> Ingo,
>
> This set factorizes the common definition of structs, functions,
> variables inside perf tools to make it easier for perf trace to
> be created.
>
> I guess this patchset only factorizes 10 % of what could be done,
> the largest part is the intersection between perf report and perf
> annotate. It's a lot of work.
>
> I guess I will do the rest in separate patchsets while writing
> perf trace.
>
> Thanks,
> Frederic.
>
>
> The following changes since commit 9f8666971185b86615a074bcac67c90fdf8af8bc:
> Brice Goglin (1):
> perf report: Add raw displaying of per-thread counters
>
> are available in the git repository at:
>
> git://git.kernel.org/pub/scm/linux/kernel/git/frederic/random-tracing.git ..BRANCH.NOT.VERIFIED..


s/BRANCH.NOT.VERIFIED/"perfcounters/core"


>
> Frederic Weisbecker (3):
> perf tools: Factorize high level dso helpers
> perf tools: Factorize the event structure definitions in a single file
> perf tools: Factorize the map helpers
>
> tools/perf/Makefile | 2 +
> tools/perf/builtin-annotate.c | 209 +-------------------------------
> tools/perf/builtin-record.c | 20 +---
> tools/perf/builtin-report.c | 274 +----------------------------------------
> tools/perf/builtin-stat.c | 1 -
> tools/perf/builtin-top.c | 24 ----
> tools/perf/builtin.h | 1 +
> tools/perf/perf.h | 1 +
> tools/perf/util/callchain.h | 1 +
> tools/perf/util/event.h | 84 +++++++++++++
> tools/perf/util/map.c | 97 +++++++++++++++
> tools/perf/util/symbol.c | 97 +++++++++++++++
> tools/perf/util/symbol.h | 12 ++
> tools/perf/util/util.h | 1 +
> 14 files changed, 303 insertions(+), 521 deletions(-)
> create mode 100644 tools/perf/util/event.h
> create mode 100644 tools/perf/util/map.c

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/