lkml.org 
[lkml]   [2014]   [May]   [13]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 17/34] perf kbuild: remove legacy demangle-related build variables
Date
Remove NO_DEMANGLE (replaced by CONFIG_DEMANGLE)
Remove HAVE_LIBBFD_SUPPORT (replaced by CONFIG_LIBBFD)
Remove HAVE_CPLUS_DEMANGLE_SUPPORT (replaced by CONFIG_LIBIBERTY_ONLY)
---
tools/perf/Kconfig | 28 ++++++++++++++++++++++------
tools/perf/config/Makefile | 45 ++++++++++++++++++++++++---------------------
tools/perf/util/srcline.c | 8 +++++---
tools/perf/util/symbol.h | 25 +++++++++++++++++--------
4 files changed, 68 insertions(+), 38 deletions(-)

diff --git a/tools/perf/Kconfig b/tools/perf/Kconfig
index 76d0763..ffc1d81 100644
--- a/tools/perf/Kconfig
+++ b/tools/perf/Kconfig
@@ -190,6 +190,28 @@ config TIMERFD
---help---
Timer via file descriptor

+config DEMANGLE
+ bool "Demangle symbols"
+ default n
+ ---help---
+ demangle
+
+choice DEMANGLE_DEPS
+ prompt "Demangle dependency(ies)"
+ depends on DEMANGLE
+ default LIBBFD
+
+config LIBBFD
+ bool "BFD + libiberty libraries"
+ ---help---
+ Binary File Descriptor and libiberty libraries
+
+config LIBIBERTY_ONLY
+ bool "Libiberty only"
+ ---help---
+ Libiberty
+endchoice
+
choice
prompt "Elf library"
default LIBELF
@@ -223,12 +245,6 @@ config NUMA
---help---
numa

-config DEMANGLE
- bool "Demangle symbols"
- default n
- ---help---
- demangle
-
config BIONIC
bool "Bionic support"
default n
diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
index 33843b7..ff7f01c 100644
--- a/tools/perf/config/Makefile
+++ b/tools/perf/config/Makefile
@@ -534,33 +534,36 @@ ifeq ($(feature-libbfd), 1)
endif
endif

-ifdef NO_DEMANGLE
- CFLAGS += -DNO_DEMANGLE
-else
- ifdef HAVE_CPLUS_DEMANGLE_SUPPORT
- EXTLIBS += -liberty
- CFLAGS += -DHAVE_CPLUS_DEMANGLE_SUPPORT
- else
- ifneq ($(feature-libbfd), 1)
- ifneq ($(feature-liberty), 1)
- ifneq ($(feature-liberty-z), 1)
- # we dont have neither HAVE_CPLUS_DEMANGLE_SUPPORT
- # or any of 'bfd iberty z' trinity
- ifeq ($(feature-cplus-demangle), 1)
- EXTLIBS += -liberty
- CFLAGS += -DHAVE_CPLUS_DEMANGLE_SUPPORT
- else
- msg := $(warning No bfd.h/libbfd found, install binutils-dev[el]/zlib-static to gain symbol demangling)
- CFLAGS += -DNO_DEMANGLE
- endif
+ifdef CONFIG_LIBBFD
+ ifneq ($(feature-libbfd), 1)
+ # TODO: there might be an issue here: feature-liberty and
+ # feature-liberty-z cannot be set; they can be set only if
+ # feature-libbfd is set
+ ifneq ($(feature-liberty), 1)
+ ifneq ($(feature-liberty-z), 1)
+ # we dont have neither HAVE_CPLUS_DEMANGLE_SUPPORT
+ # or any of 'bfd iberty z' trinity
+ ifeq ($(feature-cplus-demangle), 1)
+ EXTLIBS += -liberty
+ $(shell $(KCONFIG_SCRIPT) -d CONFIG_LIBBFD)
+ $(shell $(KCONFIG_SCRIPT) -e CONFIG_LIBIBERTY_ONLY)
+ else
+ msg := $(warning No bfd.h/libbfd found, install binutils-dev[el]/zlib-static to gain symbol demangling)
+ $(shell $(KCONFIG_SCRIPT) -d CONFIG_DEMANGLE)
endif
endif
endif
endif
endif

-ifneq ($(filter -lbfd,$(EXTLIBS)),)
- CFLAGS += -DHAVE_LIBBFD_SUPPORT
+ifdef CONFIG_LIBIBERTY_ONLY
+ $(call feature_check,cplus-demangle)
+ ifeq ($(feature-cplus-demangle), 1)
+ EXTLIBS += -liberty
+ else
+ msg := $(warning No libiberty found, install binutils-dev[el]/zlib-static to gain symbol demangling)
+ $(shell $(KCONFIG_SCRIPT) -d CONFIG_DEMANGLE)
+ endif
endif

ifndef NO_ON_EXIT
diff --git a/tools/perf/util/srcline.c b/tools/perf/util/srcline.c
index f3e4bc5..38df8f4 100644
--- a/tools/perf/util/srcline.c
+++ b/tools/perf/util/srcline.c
@@ -4,11 +4,13 @@

#include <linux/kernel.h>

+#include "generated/autoconf.h"
+
#include "util/dso.h"
#include "util/util.h"
#include "util/debug.h"

-#ifdef HAVE_LIBBFD_SUPPORT
+#ifdef CONFIG_LIBBFD

/*
* Implement addr2line using libbfd.
@@ -189,7 +191,7 @@ void dso__free_a2l(struct dso *dso)
dso->a2l = NULL;
}

-#else /* HAVE_LIBBFD_SUPPORT */
+#else /* !CONFIG_LIBBFD */

static int addr2line(const char *dso_name, unsigned long addr,
char **file, unsigned int *line_nr,
@@ -242,7 +244,7 @@ void dso__free_a2l(struct dso *dso __maybe_unused)
{
}

-#endif /* HAVE_LIBBFD_SUPPORT */
+#endif /* CONFIG_LIBBFD */

/*
* Number of addr2line failures (without success) before disabling it for that
diff --git a/tools/perf/util/symbol.h b/tools/perf/util/symbol.h
index 501e4e7..d8e886e 100644
--- a/tools/perf/util/symbol.h
+++ b/tools/perf/util/symbol.h
@@ -1,6 +1,8 @@
#ifndef __PERF_SYMBOL
#define __PERF_SYMBOL 1

+#include "generated/autoconf.h"
+
#include <linux/types.h>
#include <stdbool.h>
#include <stdint.h>
@@ -21,26 +23,33 @@

#include "dso.h"

-#ifdef HAVE_CPLUS_DEMANGLE_SUPPORT
+#ifdef CONFIG_DEMANGLE
+
+#ifdef CONFIG_LIBBFD
+
+#define PACKAGE 'perf'
+#include <bfd.h>
+
+#elif defined(CONFIG_LIBIBERTY_ONLY)
extern char *cplus_demangle(const char *, int);

static inline char *bfd_demangle(void __maybe_unused *v, const char *c, int i)
{
return cplus_demangle(c, i);
}
-#else
-#ifdef NO_DEMANGLE
+
+#else /* Unknown dependency */
+#error "Inconsistent demangling configuration"
+#endif
+
+#else /* !CONFIG_DEMANGLE */
static inline char *bfd_demangle(void __maybe_unused *v,
const char __maybe_unused *c,
int __maybe_unused i)
{
return NULL;
}
-#else
-#define PACKAGE 'perf'
-#include <bfd.h>
-#endif
-#endif
+#endif /* CONFIG_DEMANGLE */

/*
* libelf 0.8.x and earlier do not support ELF_C_READ_MMAP;
--
1.9.2


\
 
 \ /
  Last update: 2014-05-17 15:01    [W:0.511 / U:0.504 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site