Re: [PATCH V7 01/25] perf header: Add AUX area tracing feature

[Jiri Olsa <jolsa@redhat.com>]

On Tue, Mar 31, 2015 at 02:38:30PM +0300, Adrian Hunter wrote:
> Add a feature to indicate that a perf.data file
> contains AUX area data.
> 
> Signed-off-by: Adrian Hunter <adrian.hunter@intel.com>

Acked-by: Jiri Olsa <jolsa@kernel.org>

> ---
>  tools/perf/util/header.c | 14 ++++++++++++++
>  tools/perf/util/header.h |  1 +
>  2 files changed, 15 insertions(+)
> 
> diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c
> index fb43215..c6739dd 100644
> --- a/tools/perf/util/header.c
> +++ b/tools/perf/util/header.c
> @@ -869,6 +869,13 @@ static int write_branch_stack(int fd __maybe_unused,
>  	return 0;
>  }
>  
> +static int write_auxtrace(int fd __maybe_unused,
> +			  struct perf_header *h __maybe_unused,
> +			  struct perf_evlist *evlist __maybe_unused)
> +{
> +	return 0;
> +}
> +
>  static void print_hostname(struct perf_header *ph, int fd __maybe_unused,
>  			   FILE *fp)
>  {
> @@ -1163,6 +1170,12 @@ static void print_branch_stack(struct perf_header *ph __maybe_unused,
>  	fprintf(fp, "# contains samples with branch stack\n");
>  }
>  
> +static void print_auxtrace(struct perf_header *ph __maybe_unused,
> +			   int fd __maybe_unused, FILE *fp)
> +{
> +	fprintf(fp, "# contains AUX area data (e.g. instruction trace)\n");
> +}
> +
>  static void print_pmu_mappings(struct perf_header *ph, int fd __maybe_unused,
>  			       FILE *fp)
>  {
> @@ -1873,6 +1886,7 @@ static const struct feature_ops feat_ops[HEADER_LAST_FEATURE] = {
>  	FEAT_OPA(HEADER_BRANCH_STACK,	branch_stack),
>  	FEAT_OPP(HEADER_PMU_MAPPINGS,	pmu_mappings),
>  	FEAT_OPP(HEADER_GROUP_DESC,	group_desc),
> +	FEAT_OPA(HEADER_AUXTRACE,	auxtrace),
>  };
>  
>  struct header_print_data {
> diff --git a/tools/perf/util/header.h b/tools/perf/util/header.h
> index 3bb90ac..d4d5796 100644
> --- a/tools/perf/util/header.h
> +++ b/tools/perf/util/header.h
> @@ -30,6 +30,7 @@ enum {
>  	HEADER_BRANCH_STACK,
>  	HEADER_PMU_MAPPINGS,
>  	HEADER_GROUP_DESC,
> +	HEADER_AUXTRACE,
>  	HEADER_LAST_FEATURE,
>  	HEADER_FEAT_BITS	= 256,
>  };
> -- 
> 1.9.1
>