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

From	Arnaldo Carvalho de Melo <acme@kernel.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH V9 2/6] perf,tools: read msr pmu type from header.
Date	2015-09-10 16:00 +0200
Message-ID	<q7aym-3dh-27@gated-at.bofh.it> (permalink)
References	<q6DCq-6DS-9@gated-at.bofh.it> <q6DCq-6DS-13@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH V9 0/6] Freq/CPU%/CORE_BUSY% support kan.liang@intel.com - 2015-09-09 04:50 +0200
  [PATCH V9 4/6] perf,tools: Dump per-sample freq/CPU%/CORE_BUSY% in report -D kan.liang@intel.com - 2015-09-09 04:50 +0200
    Re: [PATCH V9 4/6] perf,tools: Dump per-sample freq/CPU%/CORE_BUSY%  in report -D Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-09-10 16:30 +0200
  [PATCH V9 2/6] perf,tools: read msr pmu type from header. kan.liang@intel.com - 2015-09-09 04:50 +0200
    Re: [PATCH V9 2/6] perf,tools: read msr pmu type from header. Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-09-10 16:00 +0200
    Re: [PATCH V9 2/6] perf,tools: read msr pmu type from header. Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-09-14 23:20 +0200
  [PATCH V9 6/6] perf,tools: Show freq/CPU%/CORE_BUSY% in perf report by --freq-perf kan.liang@intel.com - 2015-09-09 04:50 +0200
    Re: [PATCH V9 6/6] perf,tools: Show freq/CPU%/CORE_BUSY% in perf  report by --freq-perf Jiri Olsa <jolsa@redhat.com> - 2015-09-09 16:40 +0200
  [PATCH V9 5/6] perf,tools: caculate and save freq/CPU%/CORE_BUSY% in he_stat kan.liang@intel.com - 2015-09-09 04:50 +0200
  [PATCH V9 3/6] perf, record: introduce --freq-perf option kan.liang@intel.com - 2015-09-09 04:50 +0200
    Re: [PATCH V9 3/6] perf, record: introduce --freq-perf option Jiri Olsa <jolsa@redhat.com> - 2015-09-09 16:40 +0200
    Re: [PATCH V9 3/6] perf, record: introduce --freq-perf option Arnaldo Carvalho de Melo <acme@kernel.org> - 2015-09-14 23:20 +0200
  Re: [PATCH V9 0/6] Freq/CPU%/CORE_BUSY% support Jiri Olsa <jolsa@redhat.com> - 2015-09-09 16:40 +0200

csiph-web