From: Ingo Molnar <mingo@elte.hu>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Martin Bligh <mbligh@google.com>,
Christoph Hellwig <hch@infradead.org>,
Peter Zijlstra <a.p.zijlstra@chello.nl>,
Al Viro <viro@zeniv.linux.org.uk>,
"David S. Miller" <davem@davemloft.net>,
Stephane Eranian <eranian@googlemail.com>,
linux-kernel@vger.kernel.org, Paul Mackerras <paulus@samba.org>,
Andrew Morton <akpm@linux-foundation.org>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [GIT PULL] Performance Counters for Linux
Date: Thu, 11 Jun 2009 23:14:08 +0200 [thread overview]
Message-ID: <20090611211408.GA6415@elte.hu> (raw)
In-Reply-To: <1244753357.27363.82.camel@violet>
* Marcel Holtmann <marcel@holtmann.org> wrote:
> Hi Ingo,
>
> > > What the "keep it in the kernel sources" approach hopefully allows is
> > >
> > > - taking advantage of new features in a timely manner.
> > >
> > > NOT with some ABI breakage, but simply things like supporting a
> > > new CPU architecture or new counters. The thing that oprofile
> > > failed at so badly in my experience.
> > >
> > > - Make it easier for developers, and _avoiding_ the horrible
> > > situation where you have two different groups that don't talk
> > > well to each other because one is a group of user-space
> > > weenies, and the other is a group of manly kernel people, and
> > > there is no common ground.
> >
> > Yes, very much agreed.
> >
> > Btw., here are a couple of other arguments why i find it useful to
> > have the tools/perf/ in the kernel repo:
> >
> > 1) Super-fast and synchronized release cycles
> >
> > The kernel is one of the fastest moving packages in Linux - most
> > user-space packages have (much!) longer release cycles than 3
> > months.
>
> that might be true for some projects, but for others this is
> wrong. You are just making an assumption out of thin air.
>
> > A tight release schedule forces a certain amount of release
> > discipline on tooling as well - so i'm glad that the two will be
> > coupled. It's so easy for a promising tool to degrade into
> > tinkerware with odd release cycles with time - if it's part of
> > the kernel then at least the release cycles wont be odd but at
> > precise 3 months.
>
> And you can't do that within a perf.git tree on kernel.org
> because?
We actually tried the tools as separate code, and for the first
three months of the project we only got three contributions - while
the kernel code was essentially finished. (Pekka reported a similar
experience in this thread, with another tool that has close kernel
ties.)
Once we moved it into the same repo as the kernel code (three months
ago), the patches started flowing in - at an amazing rate. We now
have a dozen contributors, most of them kernel developers, and we
have over a hundred good changes to the tools - in just another 3
months.
The key difference was the location of the tools. It is very
convenient and productive to have a shared repository for a project
that frequently involves both kernel and tool changes.
So my point is: this model clearly works in practice and all the
current tools/perf/ contributors like this kind of coding
environment.
Most of your arguments seem to center around the notion that it
could all be done in a separate repo too and that such a repo could
be run as well as the Linux kernel. If you think you could do it
even better in a separate repo you are certainly free to try it.
Ingo
next prev parent reply other threads:[~2009-06-11 21:14 UTC|newest]
Thread overview: 64+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-11 16:03 [GIT PULL] Performance Counters for Linux Ingo Molnar
2009-06-11 16:17 ` Christoph Hellwig
2009-06-11 16:26 ` Linus Torvalds
2009-06-11 16:34 ` Marcel Holtmann
2009-06-11 16:38 ` Linus Torvalds
2009-06-11 16:46 ` Christoph Hellwig
2009-06-11 16:54 ` Linus Torvalds
2009-06-11 16:47 ` Marcel Holtmann
2009-06-11 18:04 ` David Newall
2009-06-11 16:52 ` Al Viro
2009-06-11 16:56 ` Peter Zijlstra
2009-06-11 17:00 ` Christoph Hellwig
2009-06-11 17:05 ` Ray Lee
2009-06-11 17:08 ` Marcel Holtmann
2009-06-11 17:12 ` Al Viro
2009-06-11 17:22 ` Ray Lee
2009-06-11 17:06 ` Linus Torvalds
2009-06-11 17:59 ` Pekka Enberg
2009-06-11 18:10 ` David Newall
2009-06-11 18:21 ` Linus Torvalds
2009-06-11 18:38 ` David Newall
2009-06-11 18:44 ` Linus Torvalds
2009-06-11 19:07 ` David Newall
2009-06-11 19:23 ` Linus Torvalds
2009-06-11 19:29 ` Linus Torvalds
2009-06-11 19:35 ` David Newall
2009-06-11 19:49 ` Linus Torvalds
2009-06-12 1:43 ` Robert Richter
2009-06-12 3:21 ` David Newall
2009-06-11 19:37 ` David Newall
2009-06-11 18:51 ` Christoph Hellwig
2009-06-11 19:05 ` Marcel Holtmann
2009-06-11 18:24 ` Martin Bligh
2009-06-11 18:34 ` Linus Torvalds
2009-06-11 20:23 ` Ingo Molnar
2009-06-11 20:49 ` Marcel Holtmann
2009-06-11 21:08 ` Sam Ravnborg
2009-06-11 21:17 ` Marcel Holtmann
2009-06-11 21:26 ` Sam Ravnborg
2009-06-11 22:18 ` Jiri Slaby
2009-06-11 22:27 ` Linus Torvalds
2009-06-11 22:38 ` Alan Cox
2009-06-11 22:49 ` Linus Torvalds
2009-06-12 7:35 ` Alan Cox
2009-06-11 23:19 ` Al Viro
2009-06-11 23:25 ` Linus Torvalds
2009-06-12 0:26 ` Al Viro
2009-06-12 2:58 ` Linus Torvalds
2009-06-12 4:05 ` Al Viro
2009-06-11 21:59 ` Steven Rostedt
2009-06-12 10:19 ` Jörn Engel
2009-06-11 21:14 ` Ingo Molnar [this message]
2009-06-28 1:19 ` Felipe Contreras
2009-06-11 19:58 ` Andrew Morton
2009-06-11 20:09 ` Linus Torvalds
2009-06-12 4:07 ` Kyle McMartin
2009-06-11 16:58 ` Linus Torvalds
2009-06-11 18:50 ` Sam Ravnborg
2009-06-15 13:41 ` Giacomo A. Catenazzi
2009-06-15 15:18 ` Sam Ravnborg
2009-06-12 9:56 ` stephane eranian
2009-06-12 10:28 ` Ingo Molnar
2009-06-18 21:58 ` stephane eranian
2009-06-22 13:10 ` Performance analysis under Linux (was: Re: [GIT PULL] Performance Counters for Linux) Ingo Molnar
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090611211408.GA6415@elte.hu \
--to=mingo@elte.hu \
--cc=a.p.zijlstra@chello.nl \
--cc=akpm@linux-foundation.org \
--cc=davem@davemloft.net \
--cc=eranian@googlemail.com \
--cc=hch@infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcel@holtmann.org \
--cc=mbligh@google.com \
--cc=paulus@samba.org \
--cc=tglx@linutronix.de \
--cc=torvalds@linux-foundation.org \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox