lkml.org 
[lkml]   [2019]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
Subject[tip: perf/core] perf map: Move comparision of map's dso_id to a separate function
The following commit has been merged into the perf/core branch of tip:

Commit-ID: 7b59a82493b49b715224bfe3b35fae52e48e5fa1
Gitweb: https://git.kernel.org/tip/7b59a82493b49b715224bfe3b35fae52e48e5fa1
Author: Arnaldo Carvalho de Melo <acme@redhat.com>
AuthorDate: Tue, 19 Nov 2019 16:30:56 -03:00
Committer: Arnaldo Carvalho de Melo <acme@redhat.com>
CommitterDate: Tue, 19 Nov 2019 16:30:56 -03:00

perf map: Move comparision of map's dso_id to a separate function

We'll use it when doing DSO lookups using dso_ids.

Cc: Adrian Hunter <adrian.hunter@intel.com>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Jiri Olsa <jolsa@kernel.org>
Cc: Namhyung Kim <namhyung@kernel.org>
Link: https://lkml.kernel.org/n/tip-u2nr1oq03o0i29w2ay9jx03s@git.kernel.org
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
---
tools/perf/util/dsos.c | 25 +++++++++++++++++++++++++
tools/perf/util/map.h | 2 ++
tools/perf/util/sort.c | 16 ++++------------
3 files changed, 31 insertions(+), 12 deletions(-)

diff --git a/tools/perf/util/dsos.c b/tools/perf/util/dsos.c
index 3ea80d2..ecf8d73 100644
--- a/tools/perf/util/dsos.c
+++ b/tools/perf/util/dsos.c
@@ -2,6 +2,7 @@
#include "debug.h"
#include "dsos.h"
#include "dso.h"
+#include "map.h"
#include "vdso.h"
#include "namespaces.h"
#include <libgen.h>
@@ -9,6 +10,30 @@
#include <string.h>
#include <symbol.h> // filename__read_build_id

+int dso_id__cmp(struct dso_id *a, struct dso_id *b)
+{
+ /*
+ * The second is always dso->id, so zeroes if not set, assume passing
+ * NULL for a means a zeroed id
+ */
+ if (a == NULL)
+ return 0;
+
+ if (a->maj > b->maj) return -1;
+ if (a->maj < b->maj) return 1;
+
+ if (a->min > b->min) return -1;
+ if (a->min < b->min) return 1;
+
+ if (a->ino > b->ino) return -1;
+ if (a->ino < b->ino) return 1;
+
+ if (a->ino_generation > b->ino_generation) return -1;
+ if (a->ino_generation < b->ino_generation) return 1;
+
+ return 0;
+}
+
bool __dsos__read_build_ids(struct list_head *head, bool with_hits)
{
bool have_build_id = false;
diff --git a/tools/perf/util/map.h b/tools/perf/util/map.h
index f962eb9..e1e573a 100644
--- a/tools/perf/util/map.h
+++ b/tools/perf/util/map.h
@@ -28,6 +28,8 @@ struct dso_id {
u64 ino_generation;
};

+int dso_id__cmp(struct dso_id *a, struct dso_id *b);
+
struct map {
union {
struct rb_node rb_node;
diff --git a/tools/perf/util/sort.c b/tools/perf/util/sort.c
index bc58943..f148100 100644
--- a/tools/perf/util/sort.c
+++ b/tools/perf/util/sort.c
@@ -1194,6 +1194,7 @@ sort__dcacheline_cmp(struct hist_entry *left, struct hist_entry *right)
{
u64 l, r;
struct map *l_map, *r_map;
+ int rc;

if (!left->mem_info) return -1;
if (!right->mem_info) return 1;
@@ -1212,18 +1213,9 @@ sort__dcacheline_cmp(struct hist_entry *left, struct hist_entry *right)
if (!l_map) return -1;
if (!r_map) return 1;

- if (l_map->dso_id.maj > r_map->dso_id.maj) return -1;
- if (l_map->dso_id.maj < r_map->dso_id.maj) return 1;
-
- if (l_map->dso_id.min > r_map->dso_id.min) return -1;
- if (l_map->dso_id.min < r_map->dso_id.min) return 1;
-
- if (l_map->dso_id.ino > r_map->dso_id.ino) return -1;
- if (l_map->dso_id.ino < r_map->dso_id.ino) return 1;
-
- if (l_map->dso_id.ino_generation > r_map->dso_id.ino_generation) return -1;
- if (l_map->dso_id.ino_generation < r_map->dso_id.ino_generation) return 1;
-
+ rc = dso_id__cmp(&l_map->dso_id, &r_map->dso_id);
+ if (rc)
+ return rc;
/*
* Addresses with no major/minor numbers are assumed to be
* anonymous in userspace. Sort those on pid then address.
\
 
 \ /
  Last update: 2019-11-23 09:16    [W:0.047 / U:0.224 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site