Messages in this thread | | | From | Arnaldo Carvalho de Melo <> | Date | Fri, 8 Mar 2019 15:24:35 -0300 | Subject | Re: [PATCH 05/11] perf header: Add DIR_FORMAT feature to describe directory data |
| |
Em Fri, Mar 08, 2019 at 02:47:39PM +0100, Jiri Olsa escreveu: > The data files layout is described by HEADER_DIR_FORMAT > feature. Currently it holds only version number (1): > > uint64_t version; > > The current version holds only version value (1) means > that data files: > - follow the 'data.*' name format > - contain raw events data in standard perf format as > read from kernel (and need to be sorted)
and need to be sorted when multiple files are processed, right?
- Arnaldo
> Future versions are expected to describe different data > files layout according to special needs. > > Link: http://lkml.kernel.org/n/tip-6l53s3oabwr1r4n6wke21d8g@git.kernel.org > Signed-off-by: Jiri Olsa <jolsa@kernel.org> > --- > tools/perf/builtin-record.c | 2 ++ > tools/perf/util/data.c | 10 +++++++-- > tools/perf/util/data.h | 1 + > tools/perf/util/header.c | 44 ++++++++++++++++++++++++++++++++++++- > tools/perf/util/header.h | 5 +++++ > 5 files changed, 59 insertions(+), 3 deletions(-) > > diff --git a/tools/perf/builtin-record.c b/tools/perf/builtin-record.c > index e983c8d71a79..a468d882e74f 100644 > --- a/tools/perf/builtin-record.c > +++ b/tools/perf/builtin-record.c > @@ -837,6 +837,8 @@ static void record__init_features(struct record *rec) > if (!(rec->opts.use_clockid && rec->opts.clockid_res_ns)) > perf_header__clear_feat(&session->header, HEADER_CLOCKID); > > + perf_header__clear_feat(&session->header, HEADER_DIR_FORMAT); > + > perf_header__clear_feat(&session->header, HEADER_STAT); > } > > diff --git a/tools/perf/util/data.c b/tools/perf/util/data.c > index 498c531e2cb9..41a9f478f0f0 100644 > --- a/tools/perf/util/data.c > +++ b/tools/perf/util/data.c > @@ -14,6 +14,7 @@ > #include "data.h" > #include "util.h" > #include "debug.h" > +#include "header.h" > > static void close_dir(struct perf_data_file *files, int nr) > { > @@ -41,8 +42,9 @@ int perf_data__create_dir(struct perf_data *data, int nr) > if (!files) > return -ENOMEM; > > - data->dir.files = files; > - data->dir.nr = nr; > + data->dir.version = PERF_DIR_VERSION; > + data->dir.files = files; > + data->dir.nr = nr; > > for (i = 0; i < nr; i++) { > struct perf_data_file *file = &files[i]; > @@ -75,6 +77,10 @@ int perf_data__open_dir(struct perf_data *data) > if (WARN_ON(!data->is_dir)) > return -EINVAL; > > + /* The version is provided by DIR_FORMAT feature. */ > + if (WARN_ON(data->dir.version != PERF_DIR_VERSION)) > + return -1; > + > dir = opendir(data->path); > if (!dir) > return -EINVAL; > diff --git a/tools/perf/util/data.h b/tools/perf/util/data.h > index d342469bdfda..6aef8746469f 100644 > --- a/tools/perf/util/data.h > +++ b/tools/perf/util/data.h > @@ -24,6 +24,7 @@ struct perf_data { > enum perf_data_mode mode; > > struct { > + u64 version; > struct perf_data_file *files; > int nr; > } dir; > diff --git a/tools/perf/util/header.c b/tools/perf/util/header.c > index 01b324c275b9..b0683bf4d9f3 100644 > --- a/tools/perf/util/header.c > +++ b/tools/perf/util/header.c > @@ -861,6 +861,21 @@ static int write_clockid(struct feat_fd *ff, > sizeof(ff->ph->env.clockid_res_ns)); > } > > +static int write_dir_format(struct feat_fd *ff, > + struct perf_evlist *evlist __maybe_unused) > +{ > + struct perf_session *session; > + struct perf_data *data; > + > + session = container_of(ff->ph, struct perf_session, header); > + data = session->data; > + > + if (WARN_ON(!perf_data__is_dir(data))) > + return -1; > + > + return do_write(ff, &data->dir.version, sizeof(data->dir.version)); > +} > + > static int cpu_cache_level__sort(const void *a, const void *b) > { > struct cpu_cache_level *cache_a = (struct cpu_cache_level *)a; > @@ -1341,6 +1356,17 @@ static void print_clockid(struct feat_fd *ff, FILE *fp) > ff->ph->env.clockid_res_ns * 1000); > } > > +static void print_dir_format(struct feat_fd *ff, FILE *fp) > +{ > + struct perf_session *session; > + struct perf_data *data; > + > + session = container_of(ff->ph, struct perf_session, header); > + data = session->data; > + > + fprintf(fp, "# directory data version : %"PRIu64"\n", data->dir.version); > +} > + > static void free_event_desc(struct perf_evsel *events) > { > struct perf_evsel *evsel; > @@ -2373,6 +2399,21 @@ static int process_clockid(struct feat_fd *ff, > return 0; > } > > +static int process_dir_format(struct feat_fd *ff, > + void *_data __maybe_unused) > +{ > + struct perf_session *session; > + struct perf_data *data; > + > + session = container_of(ff->ph, struct perf_session, header); > + data = session->data; > + > + if (WARN_ON(!perf_data__is_dir(data))) > + return -1; > + > + return do_read_u64(ff, &data->dir.version); > +} > + > struct feature_ops { > int (*write)(struct feat_fd *ff, struct perf_evlist *evlist); > void (*print)(struct feat_fd *ff, FILE *fp); > @@ -2432,7 +2473,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_OPN(DIR_FORMAT, dir_format, false) > }; > > struct header_print_data { > diff --git a/tools/perf/util/header.h b/tools/perf/util/header.h > index 0d553ddca0a3..6a231340238d 100644 > --- a/tools/perf/util/header.h > +++ b/tools/perf/util/header.h > @@ -39,6 +39,7 @@ enum { > HEADER_SAMPLE_TIME, > HEADER_MEM_TOPOLOGY, > HEADER_CLOCKID, > + HEADER_DIR_FORMAT, > HEADER_LAST_FEATURE, > HEADER_FEAT_BITS = 256, > }; > @@ -48,6 +49,10 @@ enum perf_header_version { > PERF_HEADER_VERSION_2, > }; > > +enum perf_dir_version { > + PERF_DIR_VERSION = 1, > +}; > + > struct perf_file_section { > u64 offset; > u64 size; > -- > 2.17.2
--
- Arnaldo
| |