[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20090812110027.GC6081@nowhere>
Date: Wed, 12 Aug 2009 13:00:28 +0200
From: Frederic Weisbecker <fweisbec@...il.com>
To: Ingo Molnar <mingo@...e.hu>
Cc: LKML <linux-kernel@...r.kernel.org>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Peter Zijlstra <peterz@...radead.org>,
Mike Galbraith <efault@....de>,
Brice Goglin <Brice.Goglin@...ia.fr>
Subject: Re: [GIT PULL v2] perf tools: Factorize common code
On Wed, Aug 12, 2009 at 12:54:33PM +0200, Ingo Molnar wrote:
>
> * Frederic Weisbecker <fweisbec@...il.com> wrote:
>
> > On Wed, Aug 12, 2009 at 12:08:51PM +0200, Ingo Molnar wrote:
> > > yeah. I've zapped those commits from perfcounters/core.
> > >
> > > Ingo
> >
> > Ok, sorry about that.
> > I've rebuilt a clean tree. It should be fine now.
>
> btw., you can always check whether there's any missing
> git-add, via:
>
> git ls-files --others
Ah nice tip.
> And the safest way to commit is to use 'git commit -a' -
> that will auto-include new files. (but the tree has to be
> clean for that)
Well, I always have junks in my directories. .rej/.orig files,
or more especially scripts and tests in the case of the tools/perf
directory. So it's hard to check everything this way.
Note that most of the time I use git-commit -a, but it doesn't include
untracked files AFAICS.
> >
> > Thanks.
> >
> > 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 \
> > 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/debug.c | 22 ++++
> > tools/perf/util/debug.h | 5 +
> > 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 +
> > 16 files changed, 330 insertions(+), 521 deletions(-)
> > create mode 100644 tools/perf/util/debug.c
> > create mode 100644 tools/perf/util/debug.h
> > create mode 100644 tools/perf/util/event.h
> > create mode 100644 tools/perf/util/map.c
>
> Re-pulled it - it works fine here too - thanks!
>
> Ingo
Cool, thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists