Re: [PATCH V9 2/6] perf,tools: read msr pmu type from header.

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

Em Tue, Sep 08, 2015 at 03:32:45PM -0400, kan.liang@intel.com escreveu:
> From: Kan Liang <kan.liang@intel.com>
> 
> Get msr pmu type when processing pmu_mappings

This one looks ok, I am porting it to my current perf/core, that
includes that perf/env branch, will fix up things as we discuss it.

- Arnaldo
 
> Signed-off-by: Kan Liang <kan.liang@intel.com>
> Acked-by: Jiri Olsa <jolsa@kernel.org>
> ---
>  tools/perf/util/header.c | 3 +++
>  tools/perf/util/header.h | 1 +
>  2 files changed, 4 insertions(+)
> 
> diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c
> index 3535dcb..04f415d 100644
> --- a/tools/perf/util/header.c
> +++ b/tools/perf/util/header.c
> @@ -1879,6 +1879,9 @@ static int process_pmu_mappings(struct perf_file_section *section __maybe_unused
>  		/* include a NULL character at the end */
>  		strbuf_add(&sb, "", 1);
>  
> +		if (!strcmp(name, "msr"))
> +			ph->env.msr_pmu_type = type;
> +
>  		free(name);
>  		pmu_num--;
>  	}
> diff --git a/tools/perf/util/header.h b/tools/perf/util/header.h
> index dd9f6b0..d9034e8 100644
> --- a/tools/perf/util/header.h
> +++ b/tools/perf/util/header.h
> @@ -107,6 +107,7 @@ struct perf_env {
>  			u64	max_freq;
>  		} cpuattr;
>  	};
> +	unsigned int		msr_pmu_type;
>  };
>  
>  struct perf_header {
> -- 
> 1.8.3.1