On Mon, Feb 11, 2019 at 11:22:38PM +0300, Alexey Budankov wrote:
SNIP
> +static int process_compressed(struct feat_fd *ff,
> + void *data __maybe_unused)
> +{
> + u64 compression_info;
> +
> + if (do_read_u64(ff, &compression_info))
> + return -1;
> +
> + ff->ph->env.comp_type = (compression_info >> 32) & 0xffffffffULL;
> + ff->ph->env.comp_level = compression_info & 0xffffffffULL;
> +
> + if (do_read_u64(ff, &compression_info))
> + return -1;
> +
> + ff->ph->env.comp_ratio = (compression_info >> 32) & 0xffffffffULL;
> + ff->ph->env.comp_mmap_len = compression_info & 0xffffffffULL;
> +
> + return 0;
> +}
> +
> struct feature_ops {
> int (*write)(struct feat_fd *ff, struct perf_evlist *evlist);
> void (*print)(struct feat_fd *ff, FILE *fp);
> @@ -2651,7 +2693,8 @@ static const struct feature_ops
> feat_ops[HEADER_LAST_FEATURE] = {
> FEAT_OPN(CACHE, cache, true),
> FEAT_OPR(SAMPLE_TIME, sample_time, false),
> FEAT_OPR(MEM_TOPOLOGY, mem_topology, true),
> - FEAT_OPR(CLOCKID, clockid, false)
> + FEAT_OPR(CLOCKID, clockid, false),
> + FEAT_OPR(COMPRESSED, compressed, false)
please separate the COMPRESSED feature ddition into single patch
thanks,
jirka
