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


Groups > linux.kernel > #1399758

[PATCH v3 1/7] perf tools: Set vdso name to vdso[64,32] depending on platform

From He Kuang <hekuang@huawei.com>
Newsgroups linux.kernel
Subject [PATCH v3 1/7] perf tools: Set vdso name to vdso[64,32] depending on platform
Date 2016-05-12 10:50 +0200
Message-ID <rxUtI-1Wg-3@gated-at.bofh.it> (permalink)
References <rxUtI-1Wg-5@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


This is a preparation for cross-platform vdso lookup.

There is a naming confusion about vdso name, vdso buildid generated by
a 32-bit machine stores it with the name 'vdso', but when processing
buildid on a 64-bit machine with the same 'perf.data', perf will
search for vdso named as 'vdso32' and get failed.

This patch uses different names when storing the buildid, i.e. vdso64
for 64-bit machine and vdso32 for 32-bit machine, and eliminates this
naming confusion.

Signed-off-by: He Kuang <hekuang@huawei.com>
---
 tools/perf/util/vdso.h | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/tools/perf/util/vdso.h b/tools/perf/util/vdso.h
index cdc4fab..45e9ef4 100644
--- a/tools/perf/util/vdso.h
+++ b/tools/perf/util/vdso.h
@@ -4,10 +4,15 @@
 #include <linux/types.h>
 #include <string.h>
 #include <stdbool.h>
+#include "util.h"
 
 #define VDSO__MAP_NAME "[vdso]"
 
-#define DSO__NAME_VDSO    "[vdso]"
+#if BITS_PER_LONG == 64
+#define DSO__NAME_VDSO    "[vdso64]"
+#else
+#define DSO__NAME_VDSO    "[vdso32]"
+#endif
 #define DSO__NAME_VDSO32  "[vdso32]"
 #define DSO__NAME_VDSOX32 "[vdsox32]"
 
-- 
1.8.5.2

Back to linux.kernel | Previous | NextNext in thread | Find similar | Unroll thread


Thread

[PATCH v3 1/7] perf tools: Set vdso name to vdso[64,32] depending on platform He Kuang <hekuang@huawei.com> - 2016-05-12 10:50 +0200
  Re: [PATCH v3 1/7] perf tools: Set vdso name to vdso[64,32] depending  on platform Adrian Hunter <adrian.hunter@intel.com> - 2016-05-12 12:20 +0200
    [PATCH v3 1/7 UPDATE] perf tools: Find vdso with the consider of cross-platform He Kuang <hekuang@huawei.com> - 2016-05-13 11:00 +0200
      Re: [PATCH v3 1/7 UPDATE] perf tools: Find vdso with the consider of  cross-platform Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-16 15:40 +0200
        Re: [PATCH v3 1/7 UPDATE] perf tools: Find vdso with the consider of  cross-platform Adrian Hunter <adrian.hunter@intel.com> - 2016-05-17 09:40 +0200
      Re: [PATCH v3 1/7 UPDATE] perf tools: Find vdso with the consider of  cross-platform Adrian Hunter <adrian.hunter@intel.com> - 2016-05-17 09:40 +0200
        [PATCH v3 1/7 UPDATE2] perf tools: Find vdso with the consider of cross-platform He Kuang <hekuang@huawei.com> - 2016-05-17 11:10 +0200
          Re: [PATCH v3 1/7 UPDATE2] perf tools: Find vdso with the consider of  cross-platform Adrian Hunter <adrian.hunter@intel.com> - 2016-05-17 11:30 +0200
            Re: [PATCH v3 1/7 UPDATE2] perf tools: Find vdso with the consider  of cross-platform Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-05-17 14:50 +0200
        Re: [PATCH v3 1/7 UPDATE] perf tools: Find vdso with the consider of  cross-platform Hekuang <hekuang@huawei.com> - 2016-05-17 11:10 +0200
    Re: [PATCH v3 1/7] perf tools: Set vdso name to vdso[64,32] depending  on platform Hekuang <hekuang@huawei.com> - 2016-05-13 11:00 +0200

csiph-web