Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932930AbaGQOHf (ORCPT ); Thu, 17 Jul 2014 10:07:35 -0400 Received: from mail.kernel.org ([198.145.19.201]:45647 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755415AbaGQOHe (ORCPT ); Thu, 17 Jul 2014 10:07:34 -0400 Date: Thu, 17 Jul 2014 11:07:27 -0300 From: Arnaldo Carvalho de Melo To: Jiri Olsa Cc: linux-kernel@vger.kernel.org, Corey Ashford , David Ahern , Frederic Weisbecker , Ingo Molnar , Namhyung Kim , Paul Mackerras , Peter Zijlstra Subject: Re: [PATCH 2/5] perf tools: Move pr_* debug macros into debug object Message-ID: <20140717140727.GC4312@kernel.org> References: <1405374411-29012-1-git-send-email-jolsa@kernel.org> <1405374411-29012-3-git-send-email-jolsa@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <1405374411-29012-3-git-send-email-jolsa@kernel.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Mon, Jul 14, 2014 at 11:46:48PM +0200, Jiri Olsa escreveu: > Moving pr_* debug macros to have it with in same object as > debug variables, becase we will change them to use verbose > variable in next patch. CC /home/acme/git/build/perf/ui/gtk/util.o util/scripting-engines/trace-event-python.c: In function ‘python_process_callchain’: util/scripting-engines/trace-event-python.c:299:3: error: implicit declaration of function ‘pr_err’ [-Werror=implicit-function-declaration] util/scripting-engines/trace-event-python.c:299:3: error: nested extern declaration of ‘pr_err’ [-Werror=nested-externs] cc1: all warnings being treated as errors make[1]: *** [/home/acme/git/build/perf/util/scripting-engines/trace-event-python.o] Error 1 make[1]: *** Waiting for unfinished jobs.... make: *** [install] Error 2 make: Leaving directory `/home/acme/git/linux/tools/perf' [acme@ssdandy linux]$ Nevermind, I'll fix it. > Cc: Arnaldo Carvalho de Melo > Cc: Corey Ashford > Cc: David Ahern > Cc: Frederic Weisbecker > Cc: Ingo Molnar > Cc: Namhyung Kim > Cc: Paul Mackerras > Cc: Peter Zijlstra > Signed-off-by: Jiri Olsa > --- > tools/perf/arch/x86/tests/dwarf-unwind.c | 1 + > tools/perf/arch/x86/util/unwind-libunwind.c | 1 + > tools/perf/builtin-evlist.c | 1 + > tools/perf/builtin-help.c | 1 + > tools/perf/builtin-timechart.c | 1 + > tools/perf/tests/dso-data.c | 1 + > tools/perf/tests/evsel-roundtrip-name.c | 1 + > tools/perf/tests/evsel-tp-sched.c | 1 + > tools/perf/tests/open-syscall-tp-fields.c | 1 + > tools/perf/tests/parse-events.c | 1 + > tools/perf/tests/parse-no-sample-id-all.c | 1 + > tools/perf/tests/sample-parsing.c | 1 + > tools/perf/tests/thread-mg-share.c | 1 + > tools/perf/util/data.c | 1 + > tools/perf/util/debug.h | 20 ++++++++++++++++++++ > tools/perf/util/include/linux/kernel.h | 21 --------------------- > tools/perf/util/pstack.c | 1 + > .../perf/util/scripting-engines/trace-event-perl.c | 1 + > tools/perf/util/trace-event-info.c | 1 + > tools/perf/util/trace-event-read.c | 1 + > tools/perf/util/unwind-libunwind.c | 1 + > tools/perf/util/util.c | 1 + > tools/perf/util/vdso.c | 1 + > 23 files changed, 41 insertions(+), 21 deletions(-) > > diff --git a/tools/perf/arch/x86/tests/dwarf-unwind.c b/tools/perf/arch/x86/tests/dwarf-unwind.c > index 9f89f899ccc7..d8bbf7ad1681 100644 > --- a/tools/perf/arch/x86/tests/dwarf-unwind.c > +++ b/tools/perf/arch/x86/tests/dwarf-unwind.c > @@ -3,6 +3,7 @@ > #include "thread.h" > #include "map.h" > #include "event.h" > +#include "debug.h" > #include "tests/tests.h" > > #define STACK_SIZE 8192 > diff --git a/tools/perf/arch/x86/util/unwind-libunwind.c b/tools/perf/arch/x86/util/unwind-libunwind.c > index 3261f68c6a7c..db25e93d989c 100644 > --- a/tools/perf/arch/x86/util/unwind-libunwind.c > +++ b/tools/perf/arch/x86/util/unwind-libunwind.c > @@ -3,6 +3,7 @@ > #include > #include "perf_regs.h" > #include "../../util/unwind.h" > +#include "../../util/debug.h" > > #ifdef HAVE_ARCH_X86_64_SUPPORT > int libunwind__arch_reg_id(int regnum) > diff --git a/tools/perf/builtin-evlist.c b/tools/perf/builtin-evlist.c > index c99e0de7e54a..66e12f55c052 100644 > --- a/tools/perf/builtin-evlist.c > +++ b/tools/perf/builtin-evlist.c > @@ -15,6 +15,7 @@ > #include "util/parse-options.h" > #include "util/session.h" > #include "util/data.h" > +#include "util/debug.h" > > static int __cmd_evlist(const char *file_name, struct perf_attr_details *details) > { > diff --git a/tools/perf/builtin-help.c b/tools/perf/builtin-help.c > index 178b88ae3d2f..0384d930480b 100644 > --- a/tools/perf/builtin-help.c > +++ b/tools/perf/builtin-help.c > @@ -11,6 +11,7 @@ > #include "util/parse-options.h" > #include "util/run-command.h" > #include "util/help.h" > +#include "util/debug.h" > > static struct man_viewer_list { > struct man_viewer_list *next; > diff --git a/tools/perf/builtin-timechart.c b/tools/perf/builtin-timechart.c > index 04c9c53becad..2f1a5220c090 100644 > --- a/tools/perf/builtin-timechart.c > +++ b/tools/perf/builtin-timechart.c > @@ -37,6 +37,7 @@ > #include "util/svghelper.h" > #include "util/tool.h" > #include "util/data.h" > +#include "util/debug.h" > > #define SUPPORT_OLD_POWER_EVENTS 1 > #define PWR_EVENT_EXIT -1 > diff --git a/tools/perf/tests/dso-data.c b/tools/perf/tests/dso-data.c > index 630808cd7cc2..caaf37f079b1 100644 > --- a/tools/perf/tests/dso-data.c > +++ b/tools/perf/tests/dso-data.c > @@ -10,6 +10,7 @@ > #include "machine.h" > #include "symbol.h" > #include "tests.h" > +#include "debug.h" > > static char *test_file(int size) > { > diff --git a/tools/perf/tests/evsel-roundtrip-name.c b/tools/perf/tests/evsel-roundtrip-name.c > index 465cdbc345cf..b8d8341b383e 100644 > --- a/tools/perf/tests/evsel-roundtrip-name.c > +++ b/tools/perf/tests/evsel-roundtrip-name.c > @@ -2,6 +2,7 @@ > #include "evsel.h" > #include "parse-events.h" > #include "tests.h" > +#include "debug.h" > > static int perf_evsel__roundtrip_cache_name_test(void) > { > diff --git a/tools/perf/tests/evsel-tp-sched.c b/tools/perf/tests/evsel-tp-sched.c > index 35d7fdb2328d..52162425c969 100644 > --- a/tools/perf/tests/evsel-tp-sched.c > +++ b/tools/perf/tests/evsel-tp-sched.c > @@ -1,6 +1,7 @@ > #include > #include "evsel.h" > #include "tests.h" > +#include "debug.h" > > static int perf_evsel__test_field(struct perf_evsel *evsel, const char *name, > int size, bool should_be_signed) > diff --git a/tools/perf/tests/open-syscall-tp-fields.c b/tools/perf/tests/open-syscall-tp-fields.c > index c505ef2af245..0785b64ffd6c 100644 > --- a/tools/perf/tests/open-syscall-tp-fields.c > +++ b/tools/perf/tests/open-syscall-tp-fields.c > @@ -3,6 +3,7 @@ > #include "evsel.h" > #include "thread_map.h" > #include "tests.h" > +#include "debug.h" > > int test__syscall_open_tp_fields(void) > { > diff --git a/tools/perf/tests/parse-events.c b/tools/perf/tests/parse-events.c > index deba66955f8c..5941927a4b7f 100644 > --- a/tools/perf/tests/parse-events.c > +++ b/tools/perf/tests/parse-events.c > @@ -5,6 +5,7 @@ > #include > #include > #include "tests.h" > +#include "debug.h" > #include > > #define PERF_TP_SAMPLE_TYPE (PERF_SAMPLE_RAW | PERF_SAMPLE_TIME | \ > diff --git a/tools/perf/tests/parse-no-sample-id-all.c b/tools/perf/tests/parse-no-sample-id-all.c > index 905019f9b740..2c63ea658541 100644 > --- a/tools/perf/tests/parse-no-sample-id-all.c > +++ b/tools/perf/tests/parse-no-sample-id-all.c > @@ -7,6 +7,7 @@ > #include "evlist.h" > #include "header.h" > #include "util.h" > +#include "debug.h" > > static int process_event(struct perf_evlist **pevlist, union perf_event *event) > { > diff --git a/tools/perf/tests/sample-parsing.c b/tools/perf/tests/sample-parsing.c > index 7ae8d17db3d9..ca292f9a4ae2 100644 > --- a/tools/perf/tests/sample-parsing.c > +++ b/tools/perf/tests/sample-parsing.c > @@ -4,6 +4,7 @@ > #include "util.h" > #include "event.h" > #include "evsel.h" > +#include "debug.h" > > #include "tests.h" > > diff --git a/tools/perf/tests/thread-mg-share.c b/tools/perf/tests/thread-mg-share.c > index 2b2e0dbe114f..b028499dd3cf 100644 > --- a/tools/perf/tests/thread-mg-share.c > +++ b/tools/perf/tests/thread-mg-share.c > @@ -2,6 +2,7 @@ > #include "machine.h" > #include "thread.h" > #include "map.h" > +#include "debug.h" > > int test__thread_mg_share(void) > { > diff --git a/tools/perf/util/data.c b/tools/perf/util/data.c > index 55de44ecebef..34cc8fe8852b 100644 > --- a/tools/perf/util/data.c > +++ b/tools/perf/util/data.c > @@ -7,6 +7,7 @@ > > #include "data.h" > #include "util.h" > +#include "debug.h" > > static bool check_pipe(struct perf_data_file *file) > { > diff --git a/tools/perf/util/debug.h b/tools/perf/util/debug.h > index 443694c36b03..8a8ceb3ccde9 100644 > --- a/tools/perf/util/debug.h > +++ b/tools/perf/util/debug.h > @@ -11,6 +11,24 @@ > extern int verbose; > extern bool quiet, dump_trace; > > +#ifndef pr_fmt > +#define pr_fmt(fmt) fmt > +#endif > + > +#define pr_err(fmt, ...) \ > + eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_warning(fmt, ...) \ > + eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_info(fmt, ...) \ > + eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_debug(fmt, ...) \ > + eprintf(1, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_debugN(n, fmt, ...) \ > + eprintf(n, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_debug2(fmt, ...) pr_debugN(2, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_debug3(fmt, ...) pr_debugN(3, pr_fmt(fmt), ##__VA_ARGS__) > +#define pr_debug4(fmt, ...) pr_debugN(4, pr_fmt(fmt), ##__VA_ARGS__) > + > int dump_printf(const char *fmt, ...) __attribute__((format(printf, 1, 2))); > void trace_event(union perf_event *event); > > @@ -19,4 +37,6 @@ int ui__warning(const char *format, ...) __attribute__((format(printf, 1, 2))); > > void pr_stat(const char *fmt, ...); > > +int eprintf(int level, const char *fmt, ...) __attribute__((format(printf, 2, 3))); > + > #endif /* __PERF_DEBUG_H */ > diff --git a/tools/perf/util/include/linux/kernel.h b/tools/perf/util/include/linux/kernel.h > index 9844c31b7c2b..09e8e7aea7c6 100644 > --- a/tools/perf/util/include/linux/kernel.h > +++ b/tools/perf/util/include/linux/kernel.h > @@ -94,27 +94,6 @@ static inline int scnprintf(char * buf, size_t size, const char * fmt, ...) > return (i >= ssize) ? (ssize - 1) : i; > } > > -int eprintf(int level, > - const char *fmt, ...) __attribute__((format(printf, 2, 3))); > - > -#ifndef pr_fmt > -#define pr_fmt(fmt) fmt > -#endif > - > -#define pr_err(fmt, ...) \ > - eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_warning(fmt, ...) \ > - eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_info(fmt, ...) \ > - eprintf(0, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_debug(fmt, ...) \ > - eprintf(1, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_debugN(n, fmt, ...) \ > - eprintf(n, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_debug2(fmt, ...) pr_debugN(2, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_debug3(fmt, ...) pr_debugN(3, pr_fmt(fmt), ##__VA_ARGS__) > -#define pr_debug4(fmt, ...) pr_debugN(4, pr_fmt(fmt), ##__VA_ARGS__) > - > /* > * This looks more complex than it should be. But we need to > * get the type for the ~ right in round_down (it needs to be > diff --git a/tools/perf/util/pstack.c b/tools/perf/util/pstack.c > index daa17aeb6c63..a126e6cc6e73 100644 > --- a/tools/perf/util/pstack.c > +++ b/tools/perf/util/pstack.c > @@ -6,6 +6,7 @@ > > #include "util.h" > #include "pstack.h" > +#include "debug.h" > #include > #include > > diff --git a/tools/perf/util/scripting-engines/trace-event-perl.c b/tools/perf/util/scripting-engines/trace-event-perl.c > index af7da565a750..b2dba9c0a3a1 100644 > --- a/tools/perf/util/scripting-engines/trace-event-perl.c > +++ b/tools/perf/util/scripting-engines/trace-event-perl.c > @@ -34,6 +34,7 @@ > #include "../event.h" > #include "../trace-event.h" > #include "../evsel.h" > +#include "../debug.h" > > void boot_Perf__Trace__Context(pTHX_ CV *cv); > void boot_DynaLoader(pTHX_ CV *cv); > diff --git a/tools/perf/util/trace-event-info.c b/tools/perf/util/trace-event-info.c > index 7e6fcfe8b438..a92d2437f3ee 100644 > --- a/tools/perf/util/trace-event-info.c > +++ b/tools/perf/util/trace-event-info.c > @@ -40,6 +40,7 @@ > #include "trace-event.h" > #include > #include "evsel.h" > +#include "debug.h" > > #define VERSION "0.5" > > diff --git a/tools/perf/util/trace-event-read.c b/tools/perf/util/trace-event-read.c > index e113e180c48f..ea3fd7fc6f5c 100644 > --- a/tools/perf/util/trace-event-read.c > +++ b/tools/perf/util/trace-event-read.c > @@ -36,6 +36,7 @@ > #include "../perf.h" > #include "util.h" > #include "trace-event.h" > +#include "debug.h" > > static int input_fd; > > diff --git a/tools/perf/util/unwind-libunwind.c b/tools/perf/util/unwind-libunwind.c > index 25578b98f5c5..92b56db52471 100644 > --- a/tools/perf/util/unwind-libunwind.c > +++ b/tools/perf/util/unwind-libunwind.c > @@ -30,6 +30,7 @@ > #include "unwind.h" > #include "symbol.h" > #include "util.h" > +#include "debug.h" > > extern int > UNW_OBJ(dwarf_search_unwind_table) (unw_addr_space_t as, > diff --git a/tools/perf/util/util.c b/tools/perf/util/util.c > index 95aefa78bb07..7d9a1e97ce8c 100644 > --- a/tools/perf/util/util.c > +++ b/tools/perf/util/util.c > @@ -1,5 +1,6 @@ > #include "../perf.h" > #include "util.h" > +#include "debug.h" > #include > #include > #ifdef HAVE_BACKTRACE_SUPPORT > diff --git a/tools/perf/util/vdso.c b/tools/perf/util/vdso.c > index 0ddb3b8a89ec..290582452da3 100644 > --- a/tools/perf/util/vdso.c > +++ b/tools/perf/util/vdso.c > @@ -12,6 +12,7 @@ > #include "util.h" > #include "symbol.h" > #include "linux/string.h" > +#include "debug.h" > > static bool vdso_found; > static char vdso_file[] = "/tmp/perf-vdso.so-XXXXXX"; > -- > 1.8.3.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/