Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933122Ab0KPWUU (ORCPT ); Tue, 16 Nov 2010 17:20:20 -0500 Received: from hqemgate03.nvidia.com ([216.228.121.140]:1950 "EHLO hqemgate03.nvidia.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932822Ab0KPWUS (ORCPT ); Tue, 16 Nov 2010 17:20:18 -0500 X-PGP-Universal: processed; by hqnvupgp05.nvidia.com on Tue, 16 Nov 2010 14:16:27 -0800 From: Robert Morell To: Peter Zijlstra , Paul Mackerras , Ingo Molnar , Arnaldo Carvalho de Melo , Masami Hiramatsu Cc: linux-kernel@vger.kernel.org, Robert Morell Subject: [PATCH RESEND] perf: Remove hardcoded include paths for elfutils Date: Tue, 16 Nov 2010 14:16:33 -0800 Message-Id: <1289945793-31441-1-git-send-email-rmorell@nvidia.com> X-Mailer: git-send-email 1.7.2.2 X-NVConfidentiality: public Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3007 Lines: 83 This change removes the use of hardcoded absolute "/usr/include/elfutils" paths from the perf build. The problem with hardcoded paths is that it prevents them from being overridden by $prefix or by -I in CFLAGS (e.g., for cross-compiling purposes). Instead, just include the "elfutils/" subdirectory as a relative path when files are needed from that directory. Tested by building perf: - Cross-compiled for ARM on x86_64 - Built natively on x86_64 - Built on x86_64 with /usr/include/elfutils moved to another location and manually included in CFLAGS Signed-off-by: Robert Morell --- tools/perf/Makefile | 4 ++-- tools/perf/feature-tests.mak | 4 ++-- tools/perf/util/probe-finder.h | 6 +++--- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/tools/perf/Makefile b/tools/perf/Makefile index d1db0f6..74b684d 100644 --- a/tools/perf/Makefile +++ b/tools/perf/Makefile @@ -507,7 +507,7 @@ PERFLIBS = $(LIB_FILE) -include config.mak ifndef NO_DWARF -FLAGS_DWARF=$(ALL_CFLAGS) -I/usr/include/elfutils -ldw -lelf $(ALL_LDFLAGS) $(EXTLIBS) +FLAGS_DWARF=$(ALL_CFLAGS) -ldw -lelf $(ALL_LDFLAGS) $(EXTLIBS) ifneq ($(call try-cc,$(SOURCE_DWARF),$(FLAGS_DWARF)),y) msg := $(warning No libdw.h found or old libdw.h found or elfutils is older than 0.138, disables dwarf support. Please install new elfutils-devel/libdw-dev); NO_DWARF := 1 @@ -554,7 +554,7 @@ ifndef NO_DWARF ifeq ($(origin PERF_HAVE_DWARF_REGS), undefined) msg := $(warning DWARF register mappings have not been defined for architecture $(ARCH), DWARF support disabled); else - BASIC_CFLAGS += -I/usr/include/elfutils -DDWARF_SUPPORT + BASIC_CFLAGS += -DDWARF_SUPPORT EXTLIBS += -lelf -ldw LIB_OBJS += $(OUTPUT)util/probe-finder.o endif # PERF_HAVE_DWARF_REGS diff --git a/tools/perf/feature-tests.mak b/tools/perf/feature-tests.mak index b253db6..b041ca6 100644 --- a/tools/perf/feature-tests.mak +++ b/tools/perf/feature-tests.mak @@ -9,8 +9,8 @@ endef ifndef NO_DWARF define SOURCE_DWARF #include -#include -#include +#include +#include #ifndef _ELFUTILS_PREREQ #error #endif diff --git a/tools/perf/util/probe-finder.h b/tools/perf/util/probe-finder.h index bba69d4..beaefc3 100644 --- a/tools/perf/util/probe-finder.h +++ b/tools/perf/util/probe-finder.h @@ -34,9 +34,9 @@ extern int find_available_vars_at(int fd, struct perf_probe_event *pev, bool externs); #include -#include -#include -#include +#include +#include +#include struct probe_finder { struct perf_probe_event *pev; /* Target probe event */ -- 1.7.2.2 -- 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/