linux-kernel - Re: [PATCH 29/30] perf tools: Add support to configure builtin commands inclusion

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150108071527.GE7268@sejong>
Date:	Thu, 8 Jan 2015 16:15:27 +0900
From:	Namhyung Kim <namhyung@...nel.org>
To:	Jiri Olsa <jolsa@...nel.org>
Cc:	linux-kernel@...r.kernel.org,
	Alexis Berlemont <alexis.berlemont@...il.com>,
	Arnaldo Carvalho de Melo <acme@...hat.com>,
	Borislav Petkov <bp@...en8.de>,
	Corey Ashford <cjashfor@...ux.vnet.ibm.com>,
	David Ahern <dsahern@...il.com>,
	Frederic Weisbecker <fweisbec@...il.com>,
	Ingo Molnar <mingo@...nel.org>,
	Paul Mackerras <paulus@...ba.org>,
	Peter Zijlstra <peterz@...radead.org>,
	Stephane Eranian <eranian@...gle.com>
Subject: Re: [PATCH 29/30] perf tools: Add support to configure builtin
 commands inclusion

On Fri, Jan 02, 2015 at 04:36:11PM +0100, Jiri Olsa wrote:
> NOTE: hack
> 
> Adding support to configure builtin commands inclusion
> in the build, so we could later disable particular
> builtin command.
> 
> Signed-off-by: Jiri Olsa <jolsa@...nel.org>
> Cc: Alexis Berlemont <alexis.berlemont@...il.com>
> Cc: Arnaldo Carvalho de Melo <acme@...hat.com>
> Cc: Borislav Petkov <bp@...en8.de>
> Cc: Corey Ashford <cjashfor@...ux.vnet.ibm.com>
> Cc: David Ahern <dsahern@...il.com>
> Cc: Frederic Weisbecker <fweisbec@...il.com>
> Cc: Ingo Molnar <mingo@...nel.org>
> Cc: Namhyung Kim <namhyung@...nel.org>
> Cc: Paul Mackerras <paulus@...ba.org>
> Cc: Peter Zijlstra <peterz@...radead.org>
> Cc: Stephane Eranian <eranian@...gle.com>
> ---
>  tools/perf/Build                    | 53 ++++++++++++++++++++-----------------
>  tools/perf/Makefile.build           |  3 +++
>  tools/perf/Makefile.perf            |  7 +++--
>  tools/perf/builtin-cmds.h           | 29 ++++++++++++++++++++
>  tools/perf/builtin-help.c           |  1 +
>  tools/perf/config.default           | 23 ++++++++++++++++
>  tools/perf/perf.c                   |  1 +
>  tools/perf/util/generate-cmdlist.sh | 10 +++++++
>  8 files changed, 101 insertions(+), 26 deletions(-)
>  create mode 100644 tools/perf/builtin-cmds.h
>  create mode 100644 tools/perf/config.default
> 
> diff --git a/tools/perf/Build b/tools/perf/Build
> index c95d6378ad65..8e8f2c58f2b0 100644
> --- a/tools/perf/Build
> +++ b/tools/perf/Build
> @@ -1,29 +1,34 @@
> -perf-y += builtin-bench.o
> -perf-y += builtin-annotate.o
> -perf-y += builtin-diff.o
> -perf-y += builtin-evlist.o
> -perf-y += builtin-help.o
> -perf-y += builtin-sched.o
> -perf-y += builtin-buildid-list.o
> -perf-y += builtin-buildid-cache.o
> -perf-y += builtin-list.o
> -perf-y += builtin-record.o
> -perf-y += builtin-report.o
> -perf-y += builtin-stat.o
> -perf-y += builtin-timechart.o
> -perf-y += builtin-top.o
> -perf-y += builtin-script.o
> -perf-y += builtin-kmem.o
> -perf-y += builtin-lock.o
> -perf-y += builtin-kvm.o
> -perf-y += builtin-inject.o
> -perf-y += builtin-mem.o
> -
> -perf-$(CONFIG_AUDIT) += builtin-trace.o
> -perf-$(CONFIG_LIBELF) += builtin-probe.o
> +perf-$(CONFIG_BUILTIN_BENCH)         += builtin-bench.o
> +perf-$(CONFIG_BUILTIN_ANNOTATE)      += builtin-annotate.o
> +perf-$(CONFIG_BUILTIN_DIFF)          += builtin-diff.o
> +perf-$(CONFIG_BUILTIN_EVLIST)        += builtin-evlist.o
> +perf-$(CONFIG_BUILTIN_HELP)          += builtin-help.o
> +perf-$(CONFIG_BUILTIN_SCHED)         += builtin-sched.o
> +perf-$(CONFIG_BUILTIN_BUILDID_LIST)  += builtin-buildid-list.o
> +perf-$(CONFIG_BUILTIN_BUILDID_CACHE) += builtin-buildid-cache.o
> +perf-$(CONFIG_BUILTIN_LIST)          += builtin-list.o
> +perf-$(CONFIG_BUILTIN_RECORD)        += builtin-record.o
> +perf-$(CONFIG_BUILTIN_REPORT)        += builtin-report.o
> +perf-$(CONFIG_BUILTIN_STAT)          += builtin-stat.o
> +perf-$(CONFIG_BUILTIN_TIMECHART)     += builtin-timechart.o
> +perf-$(CONFIG_BUILTIN_TOP)           += builtin-top.o
> +perf-$(CONFIG_BUILTIN_SCRIPT)        += builtin-script.o
> +perf-$(CONFIG_BUILTIN_KMEM)          += builtin-kmem.o
> +perf-$(CONFIG_BUILTIN_LOCK)          += builtin-lock.o
> +perf-$(CONFIG_BUILTIN_KVM)           += builtin-kvm.o
> +perf-$(CONFIG_BUILTIN_INJECT)        += builtin-inject.o
> +perf-$(CONFIG_BUILTIN_MEM)           += builtin-mem.o
> +
> +ifeq ($(CONFIG_AUDIT),y)
> +perf-$(CONFIG_BUILTIN_TRACE)         += builtin-trace.o
> +endif
> +
> +ifeq ($(CONFIG_LIBELF),y)
> +perf-$(CONFIG_BUILTIN_PROBE)         += builtin-probe.o
> +endif
>  
>  perf-y += bench/

I guess it also needs to be changed like below?

   perf-$(CONFIG_BUILTIN_BENCH) += bench/

Thanks,
Namhyung


> -perf-y += tests/
> +perf-$(CONFIG_BUILTIN_TEST) += tests/
>  
>  perf-y += perf.o
--
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/