perf mmap: Store mmap scope in struct perf_mmap()

There is too much boilerplate in the perf_mmap__read*() interfaces.

The 'start' and 'end' variables should be stored in struct perf_mmap at
initialization. They will be used later.

The old 'startp' and 'endp' pointers are used by perf_mmap__read_event()
now.  They cannot be removed. So the old 'startp/endp' and new
'md->start/md->end' will exist simultaneously now.  The old one will be
removed later.

Signed-off-by: Kan Liang <kan.liang@linux.intel.com>
Suggested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Acked-by: Jiri Olsa <jolsa@kernel.org>
Cc: Andi Kleen <ak@linux.intel.com>
Cc: Namhyung Kim <namhyung@kernel.org>
Cc: Wang Nan <wangnan0@huawei.com>
Link: http://lkml.kernel.org/r/1520350567-80082-2-git-send-email-kan.liang@linux.intel.com
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>

This commit is contained in:

Kan Liang

2018-03-06 10:36:01 -05:00

committed by

Arnaldo Carvalho de Melo

parent 2c5f6d876b

commit 4fda3459e3

2 changed files with 10 additions and 4 deletions

									
										tools/perf/util/mmap.h
									
		+2
		
												View File
												
				@@ -20,6 +20,8 @@ struct perf_mmap {

					int		 fd;

					refcount_t	 refcnt;

					u64		 prev;

					u64		 start;

					u64		 end;

					bool		 overwrite;

					struct auxtrace_mmap auxtrace_mmap;

					char		 event_copy[PERF_SAMPLE_MAX_SIZE] __aligned(8);