linux-kernel - [PATCH v5 0/2] perf script: Add script per-event-dump support

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-Id: <1508921599-10832-1-git-send-email-yuzhoujian@didichuxing.com>
Date:   Wed, 25 Oct 2017 16:53:17 +0800
From:   yuzhoujian <ufo19890607@...il.com>
To:     peterz@...radead.org, mingo@...hat.com,
        alexander.shishkin@...ux.intel.com, jolsa@...nel.org,
        dsahern@...il.com, namhyung@...nel.org, milian.wolff@...b.com,
        arnaldo.melo@...il.com, yuzhoujian@...ichuxing.com,
        adrian.hunter@...el.com, wangnan0@...wei.com
Cc:     linux-perf-users@...r.kernel.org, linux-kernel@...r.kernel.org,
        acme@...hat.com
Subject: [PATCH v5 0/2] perf script: Add script per-event-dump support

Introduce a new option to dump trace output to files named by the
monitored events and update perf-script documentation accordingly.

Shown below is output of perf script command with the newly introduced
option.

         $perf record -e cycles -e cs -ag -- sleep 1
         $perf script --per-event-dump
         [ perf script: Wrote 0.051 MB perf.data-script-dump-cycles.txt (76 samples) ]
         [ perf script: Wrote 0.012 MB perf.data-script-dump-cs.txt (69 samples) ]
         $ls
         perf.data-script-dump-cycles.txt perf.data-script-dump-cs.txt

Without per-event-dump support, drawing flamegraphs for different events
would be a very troublesome thing. You can monitor only one event at a time
if you want to get flamegraphs for different events. Using this option, you
can get the trace output files named by the monitored events, and could draw
flamegraphs according to the event's name.

yuzhoujian (4):
Patch 1: Add new elements for per-event-dump option
Patch 2: Add the fp_selection_helper to set the fp for print functions

Changes since v4:
- Patch 1: None
- Patch 2: The fp_selection_helper will be invoked by process_event not process_sample_event 
	   Replace some printf with fprintf in process_event

Changes since v3:
- Patch 1: Remove three elements for perf_evsel struct and create the perf_script_evsel
 struct to save them.
- Patch 2: None
- Patch 3: None
- Patch 4: Free the evsel->priv by zfree().

Changes since v2:
- Patch 1: Add the last_evsel_name for per_tool struct and three elements for perf_evsel struct.
- Patch 2: None
- Patch 3: None
- Patch 4: Remove the file_name and per_event_dump_file variables.
           Add the fp_selection_helper function to select the fp and open the dump file
 for all print functions.
           Close the dump file for all the evsels and calculate the dump file's size at
 the end of the perf script.
           Solve the segmentation fault generated by perf script --per-event-dump --show-mmap-events

Changes since v1:
- Patch 1: Remove the set of script.tool.per_event_dump variable.
- Patch 2: Add the __maybe_unused attribute for the fp argument in the second patch.
- Patch 3: remove the fp_selection_helper function for setting the fp argument.
- Patch 2: split the original second patch(Makes all those related functions receive
 the FILE pointer) to two patches.
- Patch 4: modify the file name of per-event-dump to
 <ORIGINAL PERF DATA FILE NAME>-script-dump-<EVENT NAME>.txt

 tools/perf/builtin-script.c | 110 ++++++++++++++++++++++++++++++++++++--------
 tools/perf/util/evsel.h     |  11 +++++
 tools/perf/util/session.c   |  19 ++++++++
 tools/perf/util/tool.h      |   2 +
 4 files changed, 122 insertions(+), 20 deletions(-)

-- 
1.8.3.1