Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754066AbaJWWbL (ORCPT ); Thu, 23 Oct 2014 18:31:11 -0400 Received: from mail-lb0-f172.google.com ([209.85.217.172]:39524 "EHLO mail-lb0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753575AbaJWWbG (ORCPT ); Thu, 23 Oct 2014 18:31:06 -0400 From: Alexis Berlemont To: linux-kernel@vger.kernel.org Cc: Jiri Olsa , dsahern@gmail.com, mingo@redhat.com, a.p.zijlstra@chello.nl, paulus@samba.org, acme@kernel.org, namhyung@kernel.org, mmarek@suse.cz, Arnaldo Carvalho de Melo , Ingo Molnar , Corey Ashford , Frederic Weisbecker , Borislav Petkov , linux-kbuild@vger.kernel.org, Stephane Eranian , Alexis Berlemont Subject: [PATCH v3 03/14] perf tools: Kbuild source related fixies Date: Fri, 24 Oct 2014 00:28:11 +0200 Message-Id: <1414103302-14906-4-git-send-email-alexis.berlemont@gmail.com> X-Mailer: git-send-email 2.1.1 In-Reply-To: <1414103302-14906-1-git-send-email-alexis.berlemont@gmail.com> References: <1414103302-14906-1-git-send-email-alexis.berlemont@gmail.com> In-Reply-To: <1414101372-14443-1-git-send-email-alexis.berlemont@gmail.com> References: <1414101372-14443-1-git-send-email-alexis.berlemont@gmail.com> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Jiri Olsa Fixing several sources config dependencies to allow separate config builds. This commit was originally created by Jiri Olsa in 2013. Minor changes were needed to get it working one year later: * Remove #ifdef directives related with GUI / TUI (because of code changes in ui/setup.c) * Fix a link issue if the option BUILTIN_TRACE is disabled. * Fix typo (CONFIG_BULTIN_ instead CONFIG_BUILTIN_) Signed-off-by: Jiri Olsa Cc: Arnaldo Carvalho de Melo Cc: Peter Zijlstra Cc: Ingo Molnar Cc: Paul Mackerras Cc: Corey Ashford Cc: Frederic Weisbecker Cc: Namhyung Kim Cc: Borislav Petkov Cc: Michal Marek Cc: linux-kbuild@vger.kernel.org Cc: Stephane Eranian Signed-off-by: Alexis Berlemont --- tools/perf/builtin-annotate.c | 6 +++--- tools/perf/builtin-lock.c | 5 ++++- tools/perf/builtin-report.c | 2 +- tools/perf/builtin-sched.c | 3 +++ tools/perf/builtin-top.c | 4 ++-- tools/perf/perf.c | 10 ++++++++-- tools/perf/ui/browsers/scripts.c | 4 ++++ tools/perf/ui/setup.c | 6 +++--- 8 files changed, 28 insertions(+), 12 deletions(-) diff --git a/tools/perf/builtin-annotate.c b/tools/perf/builtin-annotate.c index 19cb967..a46af8f 100644 --- a/tools/perf/builtin-annotate.c +++ b/tools/perf/builtin-annotate.c @@ -155,7 +155,7 @@ find_next: /* skip missing symbols */ nd = rb_next(nd); -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) } else if (use_browser == 1) { struct rb_node *next = NULL; key = hist_entry__tui_annotate(he, evsel, NULL); @@ -303,9 +303,9 @@ int cmd_annotate(int argc, const char **argv, const char *prefix __maybe_unused) OPT_BOOLEAN('D', "dump-raw-trace", &dump_trace, "dump raw trace in ASCII"), OPT_BOOLEAN(0, "gtk", &annotate.use_gtk, "Use the GTK interface"), -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) OPT_BOOLEAN(0, "tui", &annotate.use_tui, "Use the TUI interface"), -#endif /* CONFIG_TUI */ +#endif OPT_BOOLEAN(0, "stdio", &annotate.use_stdio, "Use the stdio interface"), OPT_STRING('k', "vmlinux", &symbol_conf.vmlinux_name, "file", "vmlinux pathname"), diff --git a/tools/perf/builtin-lock.c b/tools/perf/builtin-lock.c index e7ec715..05c0e1c 100644 --- a/tools/perf/builtin-lock.c +++ b/tools/perf/builtin-lock.c @@ -1,4 +1,5 @@ #include "builtin.h" +#include "builtin-cmds.h" #include "perf.h" #include "util/evlist.h" @@ -995,9 +996,11 @@ int cmd_lock(int argc, const char **argv, const char *prefix __maybe_unused) usage_with_options(report_usage, report_options); } rc = __cmd_report(false); +#ifdef CONFIG_BUILTIN_SCRIPT } else if (!strcmp(argv[0], "script")) { /* Aliased to 'perf script' */ - return cmd_script(argc, argv, prefix); + rc = cmd_script(argc, argv, prefix); +#endif } else if (!strcmp(argv[0], "info")) { if (argc) { argc = parse_options(argc, argv, diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c index 00d93ff..b639e58 100644 --- a/tools/perf/builtin-report.c +++ b/tools/perf/builtin-report.c @@ -407,7 +407,7 @@ static int report__browse_hists(struct report *rep) const char *help = "For a higher level overview, try: perf report --sort comm,dso"; switch (use_browser) { -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) case 1: ret = perf_evlist__tui_browse_hists(evlist, help, NULL, rep->min_percent, diff --git a/tools/perf/builtin-sched.c b/tools/perf/builtin-sched.c index 891c393..e38b021 100644 --- a/tools/perf/builtin-sched.c +++ b/tools/perf/builtin-sched.c @@ -1,4 +1,5 @@ #include "builtin.h" +#include "builtin-cmds.h" #include "perf.h" #include "util/util.h" @@ -1742,11 +1743,13 @@ int cmd_sched(int argc, const char **argv, const char *prefix __maybe_unused) if (!argc) usage_with_options(sched_usage, sched_options); +#ifdef CONFIG_BUILTIN_SCRIPT /* * Aliased to 'perf script' for now: */ if (!strcmp(argv[0], "script")) return cmd_script(argc, argv, prefix); +#endif if (!strncmp(argv[0], "rec", 3)) { return __cmd_record(argc, argv); diff --git a/tools/perf/builtin-top.c b/tools/perf/builtin-top.c index a2ff8b7..6f04452 100644 --- a/tools/perf/builtin-top.c +++ b/tools/perf/builtin-top.c @@ -535,7 +535,7 @@ static bool perf_top__handle_keypress(struct perf_top *top, int c) return ret; } -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) static void perf_top__sort_new_samples(void *arg) { struct perf_top *t = arg; @@ -981,7 +981,7 @@ static int __cmd_top(struct perf_top *top) ret = -1; -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) if (use_browser > 0) display_fn = display_thread_tui; #endif diff --git a/tools/perf/perf.c b/tools/perf/perf.c index ab363b5..976ded6 100644 --- a/tools/perf/perf.c +++ b/tools/perf/perf.c @@ -93,8 +93,10 @@ static struct cmd_struct commands[] = { #ifdef CONFIG_BUILTIN_KVM { "kvm", cmd_kvm, 0 }, #endif +#ifdef CONFIG_BUILTIN_TEST { "test", cmd_test, 0 }, -#if defined HAVE_LIBAUDIT_SUPPORT && defined CONFIG_BUILTIN_TRACE +#endif +#ifdef CONFIG_BUILTIN_TRACE { "trace", cmd_trace, 0 }, #endif #ifdef CONFIG_BUILTIN_INJECT @@ -539,7 +541,7 @@ int main(int argc, const char **argv) goto out; } if (!prefixcmp(cmd, "trace")) { -#ifdef HAVE_LIBAUDIT_SUPPORT +#ifdef CONFIG_BUILTIN_TRACE set_buildid_dir(); setup_path(); argv[0] = "trace"; @@ -563,13 +565,17 @@ int main(int argc, const char **argv) } else { /* The user didn't specify a command; give them help */ printf("\n usage: %s\n\n", perf_usage_string); +#ifdef CONFIG_BUILTIN_HELP list_common_cmds_help(); printf("\n %s\n\n", perf_more_info_string); +#endif goto out; } cmd = argv[0]; +#ifdef CONFIG_BUILTIN_TEST test_attr__init(); +#endif /* * We use PATH to find perf commands, but we prepend some higher diff --git a/tools/perf/ui/browsers/scripts.c b/tools/perf/ui/browsers/scripts.c index 402d2bd..128988d 100644 --- a/tools/perf/ui/browsers/scripts.c +++ b/tools/perf/ui/browsers/scripts.c @@ -59,7 +59,11 @@ static int list_scripts(char *script_name) paths[i] = names[i] + SCRIPT_NAMELEN; } +#ifdef CONFIG_BUILTIN_SCRIPT num = find_scripts(names, paths); +#else + num = 0; +#endif if (num > 0) { choice = ui__popup_menu(num, names); if (choice < num && choice >= 0) { diff --git a/tools/perf/ui/setup.c b/tools/perf/ui/setup.c index ec7fd36..efc2b21 100644 --- a/tools/perf/ui/setup.c +++ b/tools/perf/ui/setup.c @@ -77,7 +77,7 @@ void setup_browser(bool fallback_to_pager) PERF_GTK_DSO); sleep(1); /* fall through */ -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) case 1: use_browser = 1; if (ui__init() == 0) @@ -92,13 +92,13 @@ void setup_browser(bool fallback_to_pager) } } -void exit_browser(bool wait_for_ok) +void exit_browser(bool wait_for_ok __maybe_unused) { switch (use_browser) { case 2: exit_gtk_browser(wait_for_ok); break; -#ifdef CONFIG_TUI +#if defined(CONFIG_TUI) || !defined(CONFIG_KBUILD) case 1: ui__exit(wait_for_ok); break; -- 2.1.1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/