diff options
author | Wang Nan <[email protected]> | 2016-05-23 07:13:40 +0000 |
---|---|---|
committer | Arnaldo Carvalho de Melo <[email protected]> | 2016-05-23 18:22:47 -0300 |
commit | 09fa4f401296f555afb6f2f4282717644d94722e (patch) | |
tree | 1221a34bf153698a5a97a9f8fc40d7bbc9909249 /tools/perf/util/trace-event-scripting.c | |
parent | 2d11c65071d489e20b3a811167507939dd8c2eac (diff) |
perf record: Rename variable to make code clear
record__mmap_read() writes data from ring buffer into perf.data. 'head'
is maintained by the kernel, points to the last written record.
'old' is maintained by perf, points to the record read in previous
round. record__mmap_read() saves data from 'old' to 'head' to
perf.data.
The names of these variables are not very intutive. In addition,
when dealing with backward writing ring buffer, the md->prev pointer
should point to 'head' instead of the last byte it got.
Add 'start' and 'end' pointer to make code clear and set md->prev to
'head' instead of the moved 'old' pointer. This patch doesn't change
behavior since:
buf = &data[old & md->mask];
size = head - old;
old += size; <--- Here, old == head
Signed-off-by: Wang Nan <[email protected]>
Cc: Jiri Olsa <[email protected]>
Cc: Masami Hiramatsu <[email protected]>
Cc: Namhyung Kim <[email protected]>
Cc: Zefan Li <[email protected]>
Cc: [email protected]
Link: http://lkml.kernel.org/r/[email protected]
Signed-off-by: He Kuang <[email protected]>
Signed-off-by: Arnaldo Carvalho de Melo <[email protected]>
Diffstat (limited to 'tools/perf/util/trace-event-scripting.c')
0 files changed, 0 insertions, 0 deletions