diff options
author | Jiri Olsa <jolsa@redhat.com> | 2012-10-27 17:18:28 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2012-10-29 09:34:46 -0400 |
commit | ebb296c276ca19798c3116220eb8d0b2c0975ae0 (patch) | |
tree | f76d2264c75f6e7d8f3d6258578104e85dc97b56 | |
parent | 9175ce1f1244edd9f4d39605aa06ce5b0a50b8e0 (diff) |
perf tools: Move build_id__sprintf into build-id object
Moving build_id__sprintf function into build-id object.
Signed-off-by: Jiri Olsa <jolsa@redhat.com>
Reviewed-by: Namhyung Kim <namhyung@kernel.org>
Tested-by: Namhyung Kim <namhyung@kernel.org>
Cc: Corey Ashford <cjashfor@linux.vnet.ibm.com>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Paul Mackerras <paulus@samba.org>
Cc: Peter Zijlstra <a.p.zijlstra@chello.nl>
Link: http://lkml.kernel.org/r/1351372712-21104-2-git-send-email-jolsa@redhat.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
-rw-r--r-- | tools/perf/builtin-buildid-cache.c | 2 | ||||
-rw-r--r-- | tools/perf/util/build-id.c | 15 | ||||
-rw-r--r-- | tools/perf/util/build-id.h | 1 | ||||
-rw-r--r-- | tools/perf/util/header.c | 1 | ||||
-rw-r--r-- | tools/perf/util/map.c | 1 | ||||
-rw-r--r-- | tools/perf/util/symbol.c | 15 | ||||
-rw-r--r-- | tools/perf/util/symbol.h | 1 |
7 files changed, 19 insertions, 17 deletions
diff --git a/tools/perf/builtin-buildid-cache.c b/tools/perf/builtin-buildid-cache.c index d37e077f4b14..edb26ea78dd3 100644 --- a/tools/perf/builtin-buildid-cache.c +++ b/tools/perf/builtin-buildid-cache.c | |||
@@ -13,7 +13,7 @@ | |||
13 | #include "util/header.h" | 13 | #include "util/header.h" |
14 | #include "util/parse-options.h" | 14 | #include "util/parse-options.h" |
15 | #include "util/strlist.h" | 15 | #include "util/strlist.h" |
16 | #include "util/symbol.h" | 16 | #include "util/build-id.h" |
17 | 17 | ||
18 | static int build_id_cache__add_file(const char *filename, const char *debugdir) | 18 | static int build_id_cache__add_file(const char *filename, const char *debugdir) |
19 | { | 19 | { |
diff --git a/tools/perf/util/build-id.c b/tools/perf/util/build-id.c index 94ca117b8d6e..5295625c0c00 100644 --- a/tools/perf/util/build-id.c +++ b/tools/perf/util/build-id.c | |||
@@ -70,6 +70,21 @@ struct perf_tool build_id__mark_dso_hit_ops = { | |||
70 | .build_id = perf_event__process_build_id, | 70 | .build_id = perf_event__process_build_id, |
71 | }; | 71 | }; |
72 | 72 | ||
73 | int build_id__sprintf(const u8 *build_id, int len, char *bf) | ||
74 | { | ||
75 | char *bid = bf; | ||
76 | const u8 *raw = build_id; | ||
77 | int i; | ||
78 | |||
79 | for (i = 0; i < len; ++i) { | ||
80 | sprintf(bid, "%02x", *raw); | ||
81 | ++raw; | ||
82 | bid += 2; | ||
83 | } | ||
84 | |||
85 | return raw - build_id; | ||
86 | } | ||
87 | |||
73 | char *dso__build_id_filename(struct dso *self, char *bf, size_t size) | 88 | char *dso__build_id_filename(struct dso *self, char *bf, size_t size) |
74 | { | 89 | { |
75 | char build_id_hex[BUILD_ID_SIZE * 2 + 1]; | 90 | char build_id_hex[BUILD_ID_SIZE * 2 + 1]; |
diff --git a/tools/perf/util/build-id.h b/tools/perf/util/build-id.h index 45c500bd5b9f..f6d9ff9cab20 100644 --- a/tools/perf/util/build-id.h +++ b/tools/perf/util/build-id.h | |||
@@ -5,6 +5,7 @@ | |||
5 | 5 | ||
6 | extern struct perf_tool build_id__mark_dso_hit_ops; | 6 | extern struct perf_tool build_id__mark_dso_hit_ops; |
7 | 7 | ||
8 | int build_id__sprintf(const u8 *build_id, int len, char *bf); | ||
8 | char *dso__build_id_filename(struct dso *self, char *bf, size_t size); | 9 | char *dso__build_id_filename(struct dso *self, char *bf, size_t size); |
9 | 10 | ||
10 | int build_id__mark_dso_hit(struct perf_tool *tool, union perf_event *event, | 11 | int build_id__mark_dso_hit(struct perf_tool *tool, union perf_event *event, |
diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c index 7daad237dea5..514ed1bd41a2 100644 --- a/tools/perf/util/header.c +++ b/tools/perf/util/header.c | |||
@@ -23,6 +23,7 @@ | |||
23 | #include "pmu.h" | 23 | #include "pmu.h" |
24 | #include "vdso.h" | 24 | #include "vdso.h" |
25 | #include "strbuf.h" | 25 | #include "strbuf.h" |
26 | #include "build-id.h" | ||
26 | 27 | ||
27 | static bool no_buildid_cache = false; | 28 | static bool no_buildid_cache = false; |
28 | 29 | ||
diff --git a/tools/perf/util/map.c b/tools/perf/util/map.c index 6109fa4d14cd..9b40c444039c 100644 --- a/tools/perf/util/map.c +++ b/tools/perf/util/map.c | |||
@@ -10,6 +10,7 @@ | |||
10 | #include "thread.h" | 10 | #include "thread.h" |
11 | #include "strlist.h" | 11 | #include "strlist.h" |
12 | #include "vdso.h" | 12 | #include "vdso.h" |
13 | #include "build-id.h" | ||
13 | 14 | ||
14 | const char *map_type__name[MAP__NR_TYPES] = { | 15 | const char *map_type__name[MAP__NR_TYPES] = { |
15 | [MAP__FUNCTION] = "Functions", | 16 | [MAP__FUNCTION] = "Functions", |
diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c index e2e8c697cffe..e36e8c2755b6 100644 --- a/tools/perf/util/symbol.c +++ b/tools/perf/util/symbol.c | |||
@@ -504,21 +504,6 @@ void dso__sort_by_name(struct dso *dso, enum map_type type) | |||
504 | &dso->symbols[type]); | 504 | &dso->symbols[type]); |
505 | } | 505 | } |
506 | 506 | ||
507 | int build_id__sprintf(const u8 *build_id, int len, char *bf) | ||
508 | { | ||
509 | char *bid = bf; | ||
510 | const u8 *raw = build_id; | ||
511 | int i; | ||
512 | |||
513 | for (i = 0; i < len; ++i) { | ||
514 | sprintf(bid, "%02x", *raw); | ||
515 | ++raw; | ||
516 | bid += 2; | ||
517 | } | ||
518 | |||
519 | return raw - build_id; | ||
520 | } | ||
521 | |||
522 | size_t dso__fprintf_buildid(struct dso *dso, FILE *fp) | 507 | size_t dso__fprintf_buildid(struct dso *dso, FILE *fp) |
523 | { | 508 | { |
524 | char sbuild_id[BUILD_ID_SIZE * 2 + 1]; | 509 | char sbuild_id[BUILD_ID_SIZE * 2 + 1]; |
diff --git a/tools/perf/util/symbol.h b/tools/perf/util/symbol.h index 8b6ef7fac745..d70f6764524e 100644 --- a/tools/perf/util/symbol.h +++ b/tools/perf/util/symbol.h | |||
@@ -338,7 +338,6 @@ struct symbol *dso__find_symbol_by_name(struct dso *dso, enum map_type type, | |||
338 | int filename__read_build_id(const char *filename, void *bf, size_t size); | 338 | int filename__read_build_id(const char *filename, void *bf, size_t size); |
339 | int sysfs__read_build_id(const char *filename, void *bf, size_t size); | 339 | int sysfs__read_build_id(const char *filename, void *bf, size_t size); |
340 | bool __dsos__read_build_ids(struct list_head *head, bool with_hits); | 340 | bool __dsos__read_build_ids(struct list_head *head, bool with_hits); |
341 | int build_id__sprintf(const u8 *build_id, int len, char *bf); | ||
342 | int kallsyms__parse(const char *filename, void *arg, | 341 | int kallsyms__parse(const char *filename, void *arg, |
343 | int (*process_symbol)(void *arg, const char *name, | 342 | int (*process_symbol)(void *arg, const char *name, |
344 | char type, u64 start)); | 343 | char type, u64 start)); |