[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150102173612.GA5941@gmail.com>
Date: Fri, 2 Jan 2015 18:36:12 +0100
From: Ingo Molnar <mingo@...nel.org>
To: Jiri Olsa <jolsa@...nel.org>
Cc: linux-kernel@...r.kernel.org,
Adrian Hunter <adrian.hunter@...el.com>,
Alexis Berlemont <alexis.berlemont@...il.com>,
Andi Kleen <ak@...ux.intel.com>,
Anton Blanchard <anton@...ba.org>,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Borislav Petkov <bp@...en8.de>, Borislav Petkov <bp@...e.de>,
Cody P Schafer <cody@...ux.vnet.ibm.com>,
Corey Ashford <cjashfor@...ux.vnet.ibm.com>,
David Ahern <dsahern@...il.com>,
Florian Fainelli <florian@...nwrt.org>,
Frederic Weisbecker <fweisbec@...il.com>,
Jean Pihet <jean.pihet@...aro.org>,
Mark Salter <msalter@...hat.com>,
Namhyung Kim <namhyung@...nel.org>,
Paul Mackerras <paulus@...ba.org>,
Peter Zijlstra <peterz@...radead.org>,
Sam Ravnborg <sam@...nborg.org>,
Stephane Eranian <eranian@...gle.com>,
Sukadev Bhattiprolu <sukadev@...ux.vnet.ibm.com>,
Will Deacon <will.deacon@....com>
Subject: Re: [RFC/PATCHSET 00/30] perf tools: New build framework
* Jiri Olsa <jolsa@...nel.org> wrote:
> [...] The basic idea is the same as in kbuild system, with some
> minor changes to allow for multiple binaries build definitions.
> tools/perf/Build | 48 ++
> tools/perf/Build.include | 75 ++++
> tools/perf/Documentation/Build | 123 +++++
> tools/perf/Makefile.build | 122 +++++
> tools/perf/Makefile.perf | 566 +++---------------------
> 47 files changed, 827 insertions(+), 541 deletions(-)
Even the diffstat looks pretty good: only a modest line count
increase.
It would be useful to put this (and other build features - such
as features/parallelisation support) into tools/kbuild/ though,
so that other tooling can make easy use of it as well. For
example tools/kvm/ (an external project) started off with an
earlier version of the perf build system.
Sharing that would make quite a bit of sense, as tooling (and
most user-space project) build systems are usually have pretty
poor quality.
Thanks,
Ingo
--
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