| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <20190308182435.GF10690@kernel.org>
Date: Fri, 8 Mar 2019 15:24:35 -0300
From: Arnaldo Carvalho de Melo <arnaldo.melo@...il.com>
To: Jiri Olsa <jolsa@...nel.org>
Cc: lkml <linux-kernel@...r.kernel.org>,
Ingo Molnar <mingo@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Peter Zijlstra <a.p.zijlstra@...llo.nl>,
Adrian Hunter <adrian.hunter@...el.com>,
Andi Kleen <ak@...ux.intel.com>,
Stephane Eranian <eranian@...gle.com>,
Alexey Budankov <alexey.budankov@...ux.intel.com>
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@...nel.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
Powered by blists - more mailing lists