lkml.org 
[lkml]   [2019]   [Feb]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Subject[PATCHv2 00/17] perf record: Add support to store data in directory
Date
hi,
this patchset adds the --dir option to record command (and all
the other record command that overload cmd_record) that allows
the data to be stored in directory with multiple data files.

It's next step for multiple threads implementation in record.
It's now possible to make directory data via --dir option, like:

$ perf record --dir perf bench sched messaging
$ ls -l perf.data
total 344
-rw-------. 1 jolsa jolsa 43864 Jan 20 22:26 data.0
-rw-------. 1 jolsa jolsa 30464 Jan 20 22:26 data.1
-rw-------. 1 jolsa jolsa 53816 Jan 20 22:26 data.2
-rw-------. 1 jolsa jolsa 30368 Jan 20 22:26 data.3
-rw-------. 1 jolsa jolsa 40088 Jan 20 22:26 data.4
-rw-------. 1 jolsa jolsa 42592 Jan 20 22:26 data.5
-rw-------. 1 jolsa jolsa 56136 Jan 20 22:26 data.6
-rw-------. 1 jolsa jolsa 25992 Jan 20 22:26 data.7
-rw-------. 1 jolsa jolsa 8832 Jan 20 22:26 header

There's a data file created for every cpu and it's storing
data for those cpu maps. The report command will read it
transparently, sort it and display as single file data.

There's new DIR_FORMAT feature to describe directory data
layout/format. In future we can describe different data files
layout according to special needs.

It's possible to transform directory data into standard
perf.data file via simple inject command:

$ perf inject -o perf.data.file -i perf.data

The old perf fails over the directory data with following message:
$ perf report
incompatible file format (rerun with -v to learn more)

I'm now testing the record threads support, so I'd like to
have some agreement on the directory data support before.

v2 changes:
- rm_rf changes are already accepted with requested changes
- updated doc/man plus adding perf.data-directory-format.txt
to describe directory format/layout
- the --switch-output options now works over directory data
- data rollback is not part of this patchset, updated my TODO though ;-)
- added --output-dir option to combine -o and --dir
- added DIR_FORMAT feature to describe directory data
- disabling directory output for aio for now

It's also available in here:
git://git.kernel.org/pub/scm/linux/kernel/git/jolsa/perf.git
perf/dir

thanks,
jirka


---
Jiri Olsa (17):
perf data: Move size to struct perf_data_file
perf data: Add global path holder
perf data: Make check_backup work over directories
perf data: Fail check_backup in case of error
perf data: Add perf_data__(create_dir|close_dir) functions
perf data: Add perf_data__open_dir_data function
perf data: Add directory support
perf data: Don't store auxtrace index for directory data file
perf data: Add perf_data__update_dir function
perf data: Make perf_data__size to work over directory
perf header: Add DIR_FORMAT feature to describe directory data
perf session: Add process callback to reader object
perf session: Add __perf_session__process_dir_events function
perf session: Add path to reader object
perf record: Add --dir option to store data in directory
perf record: Add --output-dir option to store data in directory
perf record: Describe perf.data directory format

tools/lib/subcmd/parse-options.h | 4 +++
tools/perf/Documentation/perf-record.txt | 6 ++++
tools/perf/Documentation/perf.data-directory-format.txt | 54 ++++++++++++++++++++++++++++++++
tools/perf/builtin-annotate.c | 4 +--
tools/perf/builtin-buildid-cache.c | 4 +--
tools/perf/builtin-buildid-list.c | 8 ++---
tools/perf/builtin-c2c.c | 4 +--
tools/perf/builtin-diff.c | 12 +++----
tools/perf/builtin-evlist.c | 4 +--
tools/perf/builtin-inject.c | 10 +++---
tools/perf/builtin-kmem.c | 2 +-
tools/perf/builtin-kvm.c | 8 ++---
tools/perf/builtin-lock.c | 8 ++---
tools/perf/builtin-mem.c | 8 ++---
tools/perf/builtin-record.c | 104 ++++++++++++++++++++++++++++++++++++++++++++++++++++++-------
tools/perf/builtin-report.c | 6 ++--
tools/perf/builtin-sched.c | 16 ++++------
tools/perf/builtin-script.c | 12 +++----
tools/perf/builtin-stat.c | 6 ++--
tools/perf/builtin-timechart.c | 8 ++---
tools/perf/builtin-trace.c | 8 ++---
tools/perf/util/data-convert-bt.c | 2 +-
tools/perf/util/data.c | 254 +++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----------
tools/perf/util/data.h | 26 ++++++++++++----
tools/perf/util/header.c | 44 +++++++++++++++++++++++++-
tools/perf/util/header.h | 5 +++
tools/perf/util/mmap.h | 23 +++++++-------
tools/perf/util/session.c | 125 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-----
28 files changed, 643 insertions(+), 132 deletions(-)
create mode 100644 tools/perf/Documentation/perf.data-directory-format.txt

\
 
 \ /
  Last update: 2019-02-21 10:43    [W:0.154 / U:2.520 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site