diff options
author | David Ahern <dsahern@gmail.com> | 2015-04-07 11:20:37 -0400 |
---|---|---|
committer | Arnaldo Carvalho de Melo <acme@redhat.com> | 2015-04-10 10:29:20 -0400 |
commit | 7b8283b56d9fb36106ff1c459dfd399a20bd374d (patch) | |
tree | f0d3281f2d9b5ff8beb6209e5e226778a5d803b6 /tools | |
parent | 8cb0aa4c2db395b143cd5165586dc17677960002 (diff) |
perf evlist: Fix type for references to data_head/tail
The data_head and data_tail fields are defined as __u64 in
linux/perf_event.h, but perf userspace uses int and unsigned int.
Convert all references to u64 for consistency.
Signed-off-by: David Ahern <dsahern@gmail.com>
Acked-by: Peter Zijlstra <peterz@infradead.org>
Cc: Frederic Weisbecker <fweisbec@gmail.com>
Cc: Jiri Olsa <jolsa@redhat.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Stephane Eranian <eranian@google.com>
Link: http://lkml.kernel.org/r/1428420037-26599-1-git-send-email-dsahern@gmail.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Diffstat (limited to 'tools')
-rw-r--r-- | tools/perf/builtin-record.c | 4 | ||||
-rw-r--r-- | tools/perf/util/evlist.c | 6 | ||||
-rw-r--r-- | tools/perf/util/evlist.h | 9 |
3 files changed, 9 insertions, 10 deletions
diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c index ac610488d2e1..c3efdfb630b5 100644 --- a/tools/perf/builtin-record.c +++ b/tools/perf/builtin-record.c | |||
@@ -70,8 +70,8 @@ static int process_synthesized_event(struct perf_tool *tool, | |||
70 | static int record__mmap_read(struct record *rec, int idx) | 70 | static int record__mmap_read(struct record *rec, int idx) |
71 | { | 71 | { |
72 | struct perf_mmap *md = &rec->evlist->mmap[idx]; | 72 | struct perf_mmap *md = &rec->evlist->mmap[idx]; |
73 | unsigned int head = perf_mmap__read_head(md); | 73 | u64 head = perf_mmap__read_head(md); |
74 | unsigned int old = md->prev; | 74 | u64 old = md->prev; |
75 | unsigned char *data = md->base + page_size; | 75 | unsigned char *data = md->base + page_size; |
76 | unsigned long size; | 76 | unsigned long size; |
77 | void *buf; | 77 | void *buf; |
diff --git a/tools/perf/util/evlist.c b/tools/perf/util/evlist.c index 76ef7ee62640..080be93eea96 100644 --- a/tools/perf/util/evlist.c +++ b/tools/perf/util/evlist.c | |||
@@ -634,8 +634,8 @@ static struct perf_evsel *perf_evlist__event2evsel(struct perf_evlist *evlist, | |||
634 | union perf_event *perf_evlist__mmap_read(struct perf_evlist *evlist, int idx) | 634 | union perf_event *perf_evlist__mmap_read(struct perf_evlist *evlist, int idx) |
635 | { | 635 | { |
636 | struct perf_mmap *md = &evlist->mmap[idx]; | 636 | struct perf_mmap *md = &evlist->mmap[idx]; |
637 | unsigned int head = perf_mmap__read_head(md); | 637 | u64 head = perf_mmap__read_head(md); |
638 | unsigned int old = md->prev; | 638 | u64 old = md->prev; |
639 | unsigned char *data = md->base + page_size; | 639 | unsigned char *data = md->base + page_size; |
640 | union perf_event *event = NULL; | 640 | union perf_event *event = NULL; |
641 | 641 | ||
@@ -716,7 +716,7 @@ void perf_evlist__mmap_consume(struct perf_evlist *evlist, int idx) | |||
716 | struct perf_mmap *md = &evlist->mmap[idx]; | 716 | struct perf_mmap *md = &evlist->mmap[idx]; |
717 | 717 | ||
718 | if (!evlist->overwrite) { | 718 | if (!evlist->overwrite) { |
719 | unsigned int old = md->prev; | 719 | u64 old = md->prev; |
720 | 720 | ||
721 | perf_mmap__write_tail(md, old); | 721 | perf_mmap__write_tail(md, old); |
722 | } | 722 | } |
diff --git a/tools/perf/util/evlist.h b/tools/perf/util/evlist.h index fb19c47b8aac..b5cce95d644e 100644 --- a/tools/perf/util/evlist.h +++ b/tools/perf/util/evlist.h | |||
@@ -27,7 +27,7 @@ struct perf_mmap { | |||
27 | void *base; | 27 | void *base; |
28 | int mask; | 28 | int mask; |
29 | int refcnt; | 29 | int refcnt; |
30 | unsigned int prev; | 30 | u64 prev; |
31 | char event_copy[PERF_SAMPLE_MAX_SIZE] __attribute__((aligned(8))); | 31 | char event_copy[PERF_SAMPLE_MAX_SIZE] __attribute__((aligned(8))); |
32 | }; | 32 | }; |
33 | 33 | ||
@@ -189,16 +189,15 @@ size_t perf_evlist__fprintf(struct perf_evlist *evlist, FILE *fp); | |||
189 | int perf_evlist__strerror_open(struct perf_evlist *evlist, int err, char *buf, size_t size); | 189 | int perf_evlist__strerror_open(struct perf_evlist *evlist, int err, char *buf, size_t size); |
190 | int perf_evlist__strerror_mmap(struct perf_evlist *evlist, int err, char *buf, size_t size); | 190 | int perf_evlist__strerror_mmap(struct perf_evlist *evlist, int err, char *buf, size_t size); |
191 | 191 | ||
192 | static inline unsigned int perf_mmap__read_head(struct perf_mmap *mm) | 192 | static inline u64 perf_mmap__read_head(struct perf_mmap *mm) |
193 | { | 193 | { |
194 | struct perf_event_mmap_page *pc = mm->base; | 194 | struct perf_event_mmap_page *pc = mm->base; |
195 | int head = ACCESS_ONCE(pc->data_head); | 195 | u64 head = ACCESS_ONCE(pc->data_head); |
196 | rmb(); | 196 | rmb(); |
197 | return head; | 197 | return head; |
198 | } | 198 | } |
199 | 199 | ||
200 | static inline void perf_mmap__write_tail(struct perf_mmap *md, | 200 | static inline void perf_mmap__write_tail(struct perf_mmap *md, u64 tail) |
201 | unsigned long tail) | ||
202 | { | 201 | { |
203 | struct perf_event_mmap_page *pc = md->base; | 202 | struct perf_event_mmap_page *pc = md->base; |
204 | 203 | ||