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

Groups > linux.kernel > #1269222 > unrolled thread

[GIT PULL 0/5] perf/urgent fixes

Back to article view | Back to linux.kernel

Contents

  [GIT PULL 0/5] perf/urgent fixes Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-11-13 22:00 +0100
    [PATCH 2/5] perf buildid-list: Requires ordered events Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-11-13 22:00 +0100

#1269222 — [GIT PULL 0/5] perf/urgent fixes

Hi Ingo,

	Please consider pulling,

- Arnaldo

The following changes since commit 2059fc7a5a9e667797b8ec503bfb4685afee48d8:

  perf symbols: Allow forcing reading of non-root owned files by root (2015-11-12 18:58:18 -0300)

are available in the git repository at:

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

for you to fetch changes up to 092b1f0b5f9f797812da0de927c3aa26acbe8762:

  perf probe: Clear probe_trace_event when add_probe_trace_event() fails (2015-11-13 12:28:09 -0300)

----------------------------------------------------------------
perf/urgent fixes:

- Do not change the key of an object in a rbtree, this time it was
  the one for DSOs lookup by its long_name, and the noticed symptom was
  with 'perf buildid-list --with-hits' (Adrian Hunter)

- 'perf inject' is a pipe, events it doesn't touch should be passed
  on, PERF_RECORD_LOST wasn't, fix it (Adrian Hunter)

- Make 'perf buildid-list' request event ordering, as it needs to
  first get the mmap events to be able to mark wich DSOs had hits
  (Adrian Hunter)

- Fix memory leaks on failure in 'perf probe' (Masami Hiramatsu, Wang Nan)

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

----------------------------------------------------------------
Adrian Hunter (3):
      perf symbols: Fix dso lookup by long name and missing buildids
      perf buildid-list: Requires ordered events
      perf inject: Also re-pipe lost_samples event

Masami Hiramatsu (1):
      perf probe: Fix memory leaking on failure by clearing all probe_trace_events

Wang Nan (1):
      perf probe: Clear probe_trace_event when add_probe_trace_event() fails

 tools/perf/builtin-inject.c    |  1 +
 tools/perf/util/build-id.c     |  1 +
 tools/perf/util/dso.c          | 17 +++++++++++++++++
 tools/perf/util/dso.h          |  1 +
 tools/perf/util/machine.c      |  1 +
 tools/perf/util/probe-finder.c | 24 +++++++++++++++++-------
 6 files changed, 38 insertions(+), 7 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]

#1269223 — [PATCH 2/5] perf buildid-list: Requires ordered events

From: Adrian Hunter <adrian.hunter@intel.com>

'perf buildid-list' processes events to determine hits (i.e. with-hits
option).  That may not work if events are not sorted in order. i.e. MMAP
events must be processed before the samples that depend on them so that
sample processing can 'hit' the DSO to which the MMAP refers.

Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Link: http://lkml.kernel.org/r/1447408112-1920-3-git-send-email-adrian.hunter@intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
 tools/perf/util/build-id.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c
index d909459fb54c..217b5a60e2ab 100644
--- a/tools/perf/util/build-id.c
+++ b/tools/perf/util/build-id.c
@@ -76,6 +76,7 @@ struct perf_tool build_id__mark_dso_hit_ops = {
 	.exit	= perf_event__exit_del_thread,
 	.attr		 = perf_event__process_attr,
 	.build_id	 = perf_event__process_build_id,
+	.ordered_events	 = true,
 };
 
 int build_id__sprintf(const u8 *build_id, int len, char *bf)
-- 
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] | [standalone]

Back to top | Article view | linux.kernel

csiph-web