Re: [PATCH V7 4/8] perf,tools: add backpointer for perf_env to evlist

[Arnaldo Carvalho de Melo <acme@kernel.org>]

Em Fri, Aug 28, 2015 at 05:48:05AM -0400, Kan Liang escreveu:
> From: Kan Liang <kan.liang@intel.com>
> 
> Add backpointer to evlist, so we can easily access env when processing
> something where we have a evsel or evlist.

This is already in my perf/core branch.
 
> Suggested-by: Arnaldo Carvalho de Melo <acme@redhat.com>
> Signed-off-by: Kan Liang <kan.liang@intel.com>
> ---
>  tools/perf/util/evlist.h | 1 +
>  tools/perf/util/header.c | 1 +
>  2 files changed, 2 insertions(+)
> 
> diff --git a/tools/perf/util/evlist.h b/tools/perf/util/evlist.h
> index 436e358..b39a619 100644
> --- a/tools/perf/util/evlist.h
> +++ b/tools/perf/util/evlist.h
> @@ -56,6 +56,7 @@ struct perf_evlist {
>  	struct cpu_map	  *cpus;
>  	struct perf_evsel *selected;
>  	struct events_stats stats;
> +	struct perf_env	*env;
>  };
>  
>  struct perf_evsel_str_handler {
> diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c
> index 8df0582..c3c79cb 100644
> --- a/tools/perf/util/header.c
> +++ b/tools/perf/util/header.c
> @@ -2576,6 +2576,7 @@ int perf_session__read_header(struct perf_session *session)
>  	if (session->evlist == NULL)
>  		return -ENOMEM;
>  
> +	session->evlist->env = &header->env;
>  	if (perf_data_file__is_pipe(file))
>  		return perf_header__read_pipe(session);
>  
> -- 
> 1.8.3.1