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

Groups > linux.kernel > #1432198

Re: [PATCH v11 01/10] perf record: Prepare mmap multiple evlists

Show all headers | View raw

Em Mon, Jun 27, 2016 at 10:24:03AM +0000, Wang Nan escreveu:
> Following commits introduce multiple evlists to record. This patch
> extracts perf_evlist__mmap_ex() processing to a new function, creates
> record__mmap() and record__mmap_evlist() to wrap perf_evlist__mmap_ex()
> and its error processing. They will be improvemented to create mmap
> for all evlists.

Thanks for following my suggestion, as you can see this patch has value
irrespective of the other patches in this series and can be applied
straight away.

- Arnaldo
 
> Signed-off-by: Wang Nan <wangnan0@huawei.com>
> Cc: Arnaldo Carvalho de Melo <acme@redhat.com>
> Cc: Jiri Olsa <jolsa@kernel.org>
> Cc: Masami Hiramatsu <mhiramat@kernel.org>
> Cc: Namhyung Kim <namhyung@kernel.org>
> Cc: Zefan Li <lizefan@huawei.com>
> Cc: He Kuang <hekuang@huawei.com>
> Cc: Nilay Vaish <nilayvaish@gmail.com>
> ---
>  tools/perf/builtin-record.c | 56 +++++++++++++++++++++++++++++----------------
>  1 file changed, 36 insertions(+), 20 deletions(-)
> 
> diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c
> index 81411b1..7eb8d7d 100644
> --- a/tools/perf/builtin-record.c
> +++ b/tools/perf/builtin-record.c
> @@ -342,6 +342,40 @@ int auxtrace_record__snapshot_start(struct auxtrace_record *itr __maybe_unused)
>  
>  #endif
>  
> +static int record__mmap_evlist(struct record *rec,
> +			       struct perf_evlist *evlist)
> +{
> +	struct record_opts *opts = &rec->opts;
> +	char msg[512];
> +
> +	if (perf_evlist__mmap_ex(evlist, opts->mmap_pages, false,
> +				 opts->auxtrace_mmap_pages,
> +				 opts->auxtrace_snapshot_mode) < 0) {
> +		if (errno == EPERM) {
> +			pr_err("Permission error mapping pages.\n"
> +			       "Consider increasing "
> +			       "/proc/sys/kernel/perf_event_mlock_kb,\n"
> +			       "or try again with a smaller value of -m/--mmap_pages.\n"
> +			       "(current value: %u,%u)\n",
> +			       opts->mmap_pages, opts->auxtrace_mmap_pages);
> +			return -errno;
> +		} else {
> +			pr_err("failed to mmap with %d (%s)\n", errno,
> +				strerror_r(errno, msg, sizeof(msg)));
> +			if (errno)
> +				return -errno;
> +			else
> +				return -EINVAL;
> +		}
> +	}
> +	return 0;
> +}
> +
> +static int record__mmap(struct record *rec)
> +{
> +	return record__mmap_evlist(rec, rec->evlist);
> +}
> +
>  static int record__open(struct record *rec)
>  {
>  	char msg[512];
> @@ -378,27 +412,9 @@ try_again:
>  		goto out;
>  	}
>  
> -	if (perf_evlist__mmap_ex(evlist, opts->mmap_pages, false,
> -				 opts->auxtrace_mmap_pages,
> -				 opts->auxtrace_snapshot_mode) < 0) {
> -		if (errno == EPERM) {
> -			pr_err("Permission error mapping pages.\n"
> -			       "Consider increasing "
> -			       "/proc/sys/kernel/perf_event_mlock_kb,\n"
> -			       "or try again with a smaller value of -m/--mmap_pages.\n"
> -			       "(current value: %u,%u)\n",
> -			       opts->mmap_pages, opts->auxtrace_mmap_pages);
> -			rc = -errno;
> -		} else {
> -			pr_err("failed to mmap with %d (%s)\n", errno,
> -				strerror_r(errno, msg, sizeof(msg)));
> -			if (errno)
> -				rc = -errno;
> -			else
> -				rc = -EINVAL;
> -		}
> +	rc = record__mmap(rec);
> +	if (rc)
>  		goto out;
> -	}
>  
>  	session->evlist = evlist;
>  	perf_session__set_id_hdr_size(session);
> -- 
> 1.8.3.4

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

Thread

[PATCH v11 00/10] perf tools: Support overwritable ring buffer Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 05/10] perf tests: Add testcase for auxiliary evlist Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 06/10] perf record: Introduce rec->overwrite_evlist for overwritable events Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 08/10] perf tools: Enable overwrite settings Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 09/10] perf tools: Don't warn about out of order event if write_backward is used Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 04/10] perf evlist: Introduce aux evlist Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
  [PATCH v11 01/10] perf record: Prepare mmap multiple evlists Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
    Re: [PATCH v11 01/10] perf record: Prepare mmap multiple evlists Arnaldo Carvalho de Melo <acme@kernel.org> - 2016-06-27 19:50 +0200
    [tip:perf/core] perf record: Move mmap setup block to separate  function tip-bot for Wang Nan <tipbot@zytor.com> - 2016-06-29 11:50 +0200
  [PATCH v11 03/10] perf record: Prepare picking perf_event_mmap_page from multiple evlists Wang Nan <wangnan0@huawei.com> - 2016-06-27 12:30 +0200
    [tip:perf/core] perf record: Prepare picking perf_event_mmap_page  from multiple evlists tip-bot for Wang Nan <tipbot@zytor.com> - 2016-06-29 11:50 +0200

csiph-web