Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1175549 > unrolled thread

[GIT PULL 0/8] perf/core fixes

Started byArnaldo Carvalho de Melo <acme@kernel.org>
First post2015-07-01 23:30 +0200
Last post2015-07-02 11:30 +0200
Articles 10 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [GIT PULL 0/8] perf/core fixes Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    [PATCH 7/8] perf tools: Create config.detected into OUTPUT directory Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    [PATCH 8/8] perf tools: Allow to specify custom linker command Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    [PATCH 5/8] perf kvm: Fill in the missing session freeing after an error occurs Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    [PATCH 4/8] perf report: Fill in the missing session freeing after an error occurs Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    [PATCH 3/8] perf kmem: Fill in the missing session freeing after an error occurs Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-07-01 23:30 +0200
    Re: [GIT PULL 0/8] perf/core fixes Ingo Molnar <mingo@kernel.org> - 2015-07-02 11:00 +0200
    Re: [GIT PULL 0/8] perf/core fixes Ingo Molnar <mingo@kernel.org> - 2015-07-02 11:00 +0200
      Re: [GIT PULL 0/8] perf/core fixes Jiri Olsa <jolsa@redhat.com> - 2015-07-02 11:20 +0200
        Re: [GIT PULL 0/8] perf/core fixes Ingo Molnar <mingo@kernel.org> - 2015-07-02 11:30 +0200

#1175549 — [GIT PULL 0/8] perf/core fixes

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[GIT PULL 0/8] perf/core fixes
Message-ID<pHxJU-23b-13@gated-at.bofh.it>
Hi Ingo,

	Please consider pulling,

- Arnaldo

The following changes since commit 93472aff802fd7b61f2209335207e9bd793012f7:

  perf/x86: Fix 'active_events' imbalance (2015-06-30 13:08:46 +0200)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo

for you to fetch changes up to 5ef7bbb09f7b91ef06524c72e1ab1fc48e0d6682:

  perf tools: Allow to specify custom linker command (2015-07-01 17:53:50 -0300)

----------------------------------------------------------------
perf/core fixes:

Build fixes:

- Create config.detected into OUTPUT directory, fixing parallel
  builds sharing the same source directory (Aaro Kiskinen)

- Allow to specify custom linker command, fixing some MIPS64
  builds. (Aaro Kiskinen)

Infrastructure:

- Add missing break for PERF_RECORD_ITRACE_START, which caused those events
  samples to be parsed as well as PERF_RECORD_LOST_SAMPLES. ITRACE_START only
  appears when Intel PT or BTS are present, so  (Jiri Olsa)

- Call the perf_session destructor when bailing out in the inject, kmem, report,
  kvm and mem tools (Taeung Song)

Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

----------------------------------------------------------------
Aaro Koskinen (2):
      perf tools: Create config.detected into OUTPUT directory
      perf tools: Allow to specify custom linker command

Jiri Olsa (1):
      perf tools: Add missing break for PERF_RECORD_ITRACE_START

Taeung Song (5):
      perf inject: Fill in the missing session freeing after an error occurs
      perf kmem: Fill in the missing session freeing after an error occurs
      perf report: Fill in the missing session freeing after an error occurs
      perf kvm: Fill in the missing session freeing after an error occurs
      perf mem: Fill in the missing session freeing after an error occurs

 tools/build/Makefile.build  |  2 +-
 tools/perf/Makefile.perf    |  4 ++--
 tools/perf/builtin-inject.c |  7 ++++---
 tools/perf/builtin-kmem.c   |  4 ++--
 tools/perf/builtin-kvm.c    | 14 ++++++++++----
 tools/perf/builtin-mem.c    | 16 ++++++----------
 tools/perf/builtin-report.c |  6 ++++--
 tools/perf/config/Makefile  |  6 +++---
 tools/perf/util/machine.c   |  3 +--
 9 files changed, 33 insertions(+), 29 deletions(-)
--
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/

[toc] | [next] | [standalone]

#1175550 — [PATCH 7/8] perf tools: Create config.detected into OUTPUT directory

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[PATCH 7/8] perf tools: Create config.detected into OUTPUT directory
Message-ID<pHxJV-23b-41@gated-at.bofh.it>
In reply to#1175549
From: Aaro Koskinen <aaro.koskinen@nokia.com>

Create config.detected into OUTPUT directory instead of source
directory.

This fixes parallel builds that share the same source directory.

Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1435751683-18500-1-git-send-email-aaro.koskinen@nokia.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/build/Makefile.build | 2 +-
 tools/perf/Makefile.perf   | 2 +-
 tools/perf/config/Makefile | 6 +++---
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/tools/build/Makefile.build b/tools/build/Makefile.build
index a51244a8022f..faca2bf6a430 100644
--- a/tools/build/Makefile.build
+++ b/tools/build/Makefile.build
@@ -25,7 +25,7 @@ build-dir := $(srctree)/tools/build
 include $(build-dir)/Build.include
 
 # do not force detected configuration
--include .config-detected
+-include $(OUTPUT).config-detected
 
 # Init all relevant variables used in build files so
 # 1) they have correct type
diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
index 1af0cfeb7a57..0e0938afbbb1 100644
--- a/tools/perf/Makefile.perf
+++ b/tools/perf/Makefile.perf
@@ -545,7 +545,7 @@ config-clean:
 clean: $(LIBTRACEEVENT)-clean $(LIBAPI)-clean config-clean
 	$(call QUIET_CLEAN, core-objs)  $(RM) $(LIB_FILE) $(OUTPUT)perf-archive $(OUTPUT)perf-with-kcore $(LANG_BINDINGS)
 	$(Q)find . -name '*.o' -delete -o -name '\.*.cmd' -delete -o -name '\.*.d' -delete
-	$(Q)$(RM) .config-detected
+	$(Q)$(RM) $(OUTPUT).config-detected
 	$(call QUIET_CLEAN, core-progs) $(RM) $(ALL_PROGRAMS) perf perf-read-vdso32 perf-read-vdsox32
 	$(call QUIET_CLEAN, core-gen)   $(RM)  *.spec *.pyc *.pyo */*.pyc */*.pyo $(OUTPUT)common-cmds.h TAGS tags cscope* $(OUTPUT)PERF-VERSION-FILE $(OUTPUT)FEATURE-DUMP $(OUTPUT)util/*-bison* $(OUTPUT)util/*-flex*
 	$(QUIET_SUBDIR0)Documentation $(QUIET_SUBDIR1) clean
diff --git a/tools/perf/config/Makefile b/tools/perf/config/Makefile
index 317001c94660..094ddaee104c 100644
--- a/tools/perf/config/Makefile
+++ b/tools/perf/config/Makefile
@@ -11,9 +11,9 @@ ifneq ($(obj-perf),)
 obj-perf := $(abspath $(obj-perf))/
 endif
 
-$(shell echo -n > .config-detected)
-detected     = $(shell echo "$(1)=y"       >> .config-detected)
-detected_var = $(shell echo "$(1)=$($(1))" >> .config-detected)
+$(shell echo -n > $(OUTPUT).config-detected)
+detected     = $(shell echo "$(1)=y"       >> $(OUTPUT).config-detected)
+detected_var = $(shell echo "$(1)=$($(1))" >> $(OUTPUT).config-detected)
 
 CFLAGS := $(EXTRA_CFLAGS) $(EXTRA_WARNINGS)
 
-- 
2.1.0

--
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/

[toc] | [prev] | [next] | [standalone]

#1175551 — [PATCH 8/8] perf tools: Allow to specify custom linker command

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[PATCH 8/8] perf tools: Allow to specify custom linker command
Message-ID<pHxJV-23b-45@gated-at.bofh.it>
In reply to#1175549
From: Aaro Koskinen <aaro.koskinen@nokia.com>

Allow to specify custom linker command. This fixes MIPS64 builds for
64-bit userspace as it will allow to pass a linker using the correct
linker flags for 64-bit ABI (by default GNU binutils ld will assume
N32).

Signed-off-by: Aaro Koskinen <aaro.koskinen@nokia.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1435751683-18500-2-git-send-email-aaro.koskinen@nokia.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/Makefile.perf | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
index 0e0938afbbb1..7a4b549214e3 100644
--- a/tools/perf/Makefile.perf
+++ b/tools/perf/Makefile.perf
@@ -110,7 +110,7 @@ $(OUTPUT)PERF-VERSION-FILE: ../../.git/HEAD
 	$(Q)touch $(OUTPUT)PERF-VERSION-FILE
 
 CC = $(CROSS_COMPILE)gcc
-LD = $(CROSS_COMPILE)ld
+LD ?= $(CROSS_COMPILE)ld
 AR = $(CROSS_COMPILE)ar
 PKG_CONFIG = $(CROSS_COMPILE)pkg-config
 
-- 
2.1.0

--
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/

[toc] | [prev] | [next] | [standalone]

#1175552 — [PATCH 5/8] perf kvm: Fill in the missing session freeing after an error occurs

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[PATCH 5/8] perf kvm: Fill in the missing session freeing after an error occurs
Message-ID<pHxJV-23b-51@gated-at.bofh.it>
In reply to#1175549
From: Taeung Song <treeze.taeung@gmail.com>

When an error occurs an error value is just returned without freeing the
session. So allocating and freeing session have to be matched as a pair
even if an error occurs.

Signed-off-by: Taeung Song <treeze.taeung@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1435677525-28055-1-git-send-email-treeze.taeung@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/builtin-kvm.c | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/tools/perf/builtin-kvm.c b/tools/perf/builtin-kvm.c
index 74878cd75078..fc1cffb1b7a2 100644
--- a/tools/perf/builtin-kvm.c
+++ b/tools/perf/builtin-kvm.c
@@ -1061,8 +1061,10 @@ static int read_events(struct perf_kvm_stat *kvm)
 
 	symbol__init(&kvm->session->header.env);
 
-	if (!perf_session__has_traces(kvm->session, "kvm record"))
-		return -EINVAL;
+	if (!perf_session__has_traces(kvm->session, "kvm record")) {
+		ret = -EINVAL;
+		goto out_delete;
+	}
 
 	/*
 	 * Do not use 'isa' recorded in kvm_exit tracepoint since it is not
@@ -1070,9 +1072,13 @@ static int read_events(struct perf_kvm_stat *kvm)
 	 */
 	ret = cpu_isa_config(kvm);
 	if (ret < 0)
-		return ret;
+		goto out_delete;
 
-	return perf_session__process_events(kvm->session);
+	ret = perf_session__process_events(kvm->session);
+
+out_delete:
+	perf_session__delete(kvm->session);
+	return ret;
 }
 
 static int parse_target_str(struct perf_kvm_stat *kvm)
-- 
2.1.0

--
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/

[toc] | [prev] | [next] | [standalone]

#1175553 — [PATCH 4/8] perf report: Fill in the missing session freeing after an error occurs

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[PATCH 4/8] perf report: Fill in the missing session freeing after an error occurs
Message-ID<pHxJV-23b-53@gated-at.bofh.it>
In reply to#1175549
From: Taeung Song <treeze.taeung@gmail.com>

When an error occurs an error value is just returned without freeing the
session. So allocating and freeing session have to be matched as a pair
even if an error occurs.

Signed-off-by: Taeung Song <treeze.taeung@gmail.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1435652124-22414-6-git-send-email-treeze.taeung@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/builtin-report.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/tools/perf/builtin-report.c b/tools/perf/builtin-report.c
index 348bed4a2abf..95a47719aec3 100644
--- a/tools/perf/builtin-report.c
+++ b/tools/perf/builtin-report.c
@@ -839,8 +839,10 @@ repeat:
 	if (report.header || report.header_only) {
 		perf_session__fprintf_info(session, stdout,
 					   report.show_full_info);
-		if (report.header_only)
-			return 0;
+		if (report.header_only) {
+			ret = 0;
+			goto error;
+		}
 	} else if (use_browser == 0) {
 		fputs("# To display the perf.data header info, please use --header/--header-only options.\n#\n",
 		      stdout);
-- 
2.1.0

--
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/

[toc] | [prev] | [next] | [standalone]

#1175554 — [PATCH 3/8] perf kmem: Fill in the missing session freeing after an error occurs

FromArnaldo Carvalho de Melo <acme@kernel.org>
Date2015-07-01 23:30 +0200
Subject[PATCH 3/8] perf kmem: Fill in the missing session freeing after an error occurs
Message-ID<pHxJV-23b-59@gated-at.bofh.it>
In reply to#1175549
From: Taeung Song <treeze.taeung@gmail.com>

When an error occurs an error value is just returned without freeing the
session. So allocating and freeing session have to be matched as a pair
even if an error occurs.

Signed-off-by: Taeung Song <treeze.taeung@gmail.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: http://lkml.kernel.org/r/1435652124-22414-3-git-send-email-treeze.taeung@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/builtin-kmem.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/tools/perf/builtin-kmem.c b/tools/perf/builtin-kmem.c
index 950f296dfcf7..23b1faaaa4cc 100644
--- a/tools/perf/builtin-kmem.c
+++ b/tools/perf/builtin-kmem.c
@@ -1916,7 +1916,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused)
 		if (!perf_evlist__find_tracepoint_by_name(session->evlist,
 							  "kmem:kmalloc")) {
 			pr_err(errmsg, "slab", "slab");
-			return -1;
+			goto out_delete;
 		}
 	}
 
@@ -1927,7 +1927,7 @@ int cmd_kmem(int argc, const char **argv, const char *prefix __maybe_unused)
 							     "kmem:mm_page_alloc");
 		if (evsel == NULL) {
 			pr_err(errmsg, "page", "page");
-			return -1;
+			goto out_delete;
 		}
 
 		kmem_page_size = pevent_get_page_size(evsel->tp_format->pevent);
-- 
2.1.0

--
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/

[toc] | [prev] | [next] | [standalone]

#1175837

FromIngo Molnar <mingo@kernel.org>
Date2015-07-02 11:00 +0200
Message-ID<pHIvE-qX-1@gated-at.bofh.it>
In reply to#1175549
* Arnaldo Carvalho de Melo <acme@kernel.org> wrote:

> Hi Ingo,
> 
> 	Please consider pulling,
> 
> - Arnaldo
> 
> The following changes since commit 93472aff802fd7b61f2209335207e9bd793012f7:
> 
>   perf/x86: Fix 'active_events' imbalance (2015-06-30 13:08:46 +0200)
> 
> are available in the git repository at:
> 
>   git://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git tags/perf-core-for-mingo
> 
> for you to fetch changes up to 5ef7bbb09f7b91ef06524c72e1ab1fc48e0d6682:
> 
>   perf tools: Allow to specify custom linker command (2015-07-01 17:53:50 -0300)
> 
> ----------------------------------------------------------------
> perf/core fixes:
> 
> Build fixes:
> 
> - Create config.detected into OUTPUT directory, fixing parallel
>   builds sharing the same source directory (Aaro Kiskinen)
> 
> - Allow to specify custom linker command, fixing some MIPS64
>   builds. (Aaro Kiskinen)
> 
> Infrastructure:
> 
> - Add missing break for PERF_RECORD_ITRACE_START, which caused those events
>   samples to be parsed as well as PERF_RECORD_LOST_SAMPLES. ITRACE_START only
>   appears when Intel PT or BTS are present, so  (Jiri Olsa)
> 
> - Call the perf_session destructor when bailing out in the inject, kmem, report,
>   kvm and mem tools (Taeung Song)
> 
> Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> 
> ----------------------------------------------------------------
> Aaro Koskinen (2):
>       perf tools: Create config.detected into OUTPUT directory
>       perf tools: Allow to specify custom linker command
> 
> Jiri Olsa (1):
>       perf tools: Add missing break for PERF_RECORD_ITRACE_START
> 
> Taeung Song (5):
>       perf inject: Fill in the missing session freeing after an error occurs
>       perf kmem: Fill in the missing session freeing after an error occurs
>       perf report: Fill in the missing session freeing after an error occurs
>       perf kvm: Fill in the missing session freeing after an error occurs
>       perf mem: Fill in the missing session freeing after an error occurs
> 
>  tools/build/Makefile.build  |  2 +-
>  tools/perf/Makefile.perf    |  4 ++--
>  tools/perf/builtin-inject.c |  7 ++++---
>  tools/perf/builtin-kmem.c   |  4 ++--
>  tools/perf/builtin-kvm.c    | 14 ++++++++++----
>  tools/perf/builtin-mem.c    | 16 ++++++----------
>  tools/perf/builtin-report.c |  6 ++++--
>  tools/perf/config/Makefile  |  6 +++---
>  tools/perf/util/machine.c   |  3 +--
>  9 files changed, 33 insertions(+), 29 deletions(-)

Pulled, thanks a lot Arnaldo!

	Ingo
--
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/

[toc] | [prev] | [next] | [standalone]

#1175844

FromIngo Molnar <mingo@kernel.org>
Date2015-07-02 11:00 +0200
Message-ID<pHIvF-qX-25@gated-at.bofh.it>
In reply to#1175549
Hm, had to unpull it, due to:

make[3]: *** No rule to make target 'util/intel-pt.h', needed by 'util/auxtrace.o'.  Stop.
/home/mingo/tip/tools/build/Makefile.build:107: recipe for target 'util' failed
make[2]: *** [util] Error 2
Makefile.perf:373: recipe for target 'libperf-in.o' failed
make[1]: *** [libperf-in.o] Error 2
Makefile:87: recipe for target 'install' failed
make: *** [install] Error 2

Thanks,

	Ingo
--
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/

[toc] | [prev] | [next] | [standalone]

#1175848

FromJiri Olsa <jolsa@redhat.com>
Date2015-07-02 11:20 +0200
Message-ID<pHIP0-Na-3@gated-at.bofh.it>
In reply to#1175844
On Thu, Jul 02, 2015 at 10:56:07AM +0200, Ingo Molnar wrote:
> 
> Hm, had to unpull it, due to:
> 
> make[3]: *** No rule to make target 'util/intel-pt.h', needed by 'util/auxtrace.o'.  Stop.
> /home/mingo/tip/tools/build/Makefile.build:107: recipe for target 'util' failed

Any chance this is leftover from previous intel pt pull/revert?

And 'util/intel-pt.h' dependency stayed in the util/.auxtrace.o.cmd file?
Removing the util/.auxtrace.o.cmd file should fix that.

I'm looking at this issue currently.. need somehow to check the
dependency validity, need to check how kbuild deal with this ;-)

jirka


> make[2]: *** [util] Error 2
> Makefile.perf:373: recipe for target 'libperf-in.o' failed
> make[1]: *** [libperf-in.o] Error 2
> Makefile:87: recipe for target 'install' failed
> make: *** [install] Error 2
> 
> Thanks,
> 
> 	Ingo
--
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/

[toc] | [prev] | [next] | [standalone]

#1175866

FromIngo Molnar <mingo@kernel.org>
Date2015-07-02 11:30 +0200
Message-ID<pHIYH-QJ-33@gated-at.bofh.it>
In reply to#1175848
* Jiri Olsa <jolsa@redhat.com> wrote:

> On Thu, Jul 02, 2015 at 10:56:07AM +0200, Ingo Molnar wrote:
> > 
> > Hm, had to unpull it, due to:
> > 
> > make[3]: *** No rule to make target 'util/intel-pt.h', needed by 'util/auxtrace.o'.  Stop.
> > /home/mingo/tip/tools/build/Makefile.build:107: recipe for target 'util' failed
> 
> Any chance this is leftover from previous intel pt pull/revert?
> 
> And 'util/intel-pt.h' dependency stayed in the util/.auxtrace.o.cmd file?
> Removing the util/.auxtrace.o.cmd file should fix that.
> 
> I'm looking at this issue currently.. need somehow to check the
> dependency validity, need to check how kbuild deal with this ;-)

That's exactly what happened - it now builds fine so I restored the pull, thanks 
Jiri!

	Ingo
--
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/

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web