diff options
author | Ryan Zuklie <rzuklie@google.com> | 2022-05-20 17:53:01 +0000 |
---|---|---|
committer | Cherrypicker Worker <android-build-cherrypicker-worker@google.com> | 2022-06-03 17:56:54 +0000 |
commit | df142880e86418dad6b40fddb001ebd897919595 (patch) | |
tree | f3f3abd005afd96c07379261e18a98cf40f3ee60 /simpleperf/cmd_boot_record_test.cpp | |
parent | cd6b038e799ea13120e9676e0f1b23cdda4223db (diff) | |
download | extras-df142880e86418dad6b40fddb001ebd897919595.tar.gz |
populate attributes when re-writing perf dataandroid13-dev
JoinCallChains, PostUnwindRecords, and MergeMapRecords all flush the
current data to file and start a new data file using MoveRecordFile.
Normally, creating the file fills in the attributes from the event
selection set. However, DoRecording recently added CloseEventFiles
calls which clear the event_fds which are used to populate the attrs.
This change fixes the issue by copying the attrs from the old file.
Test: ran simpleperf on device
$ adb shell
$ cd /data/local/tmp
$ simpleperf record -a --exclude-perf --duration 2 -g \
-e power:sugov_next_freq -e power:sugov_util_update
$ simpleperf report
Before the fix, all samples would be attributed to the first event type.
With this fix, the samples are correctly attributed.
Bug: 231357972
Change-Id: Id077e3c6f6c4460df595fad04c355717985a914e
(cherry picked from commit 62fe04581f910f5a847ad0653908a3e57b6ed8a2)
Merged-In: Id077e3c6f6c4460df595fad04c355717985a914e
Diffstat (limited to 'simpleperf/cmd_boot_record_test.cpp')
0 files changed, 0 insertions, 0 deletions