Age | Commit message (Collapse) | Author |
|
into main
|
|
After change in aosp/3086621, addrs are changed when base_addr != 0.
But they are still used to search in binary.range_count_map. This
causes segfault error.
Fix it by putting count values in the vector.
Bug: 322399357
Test: run simpleperf_unit_test
Test: run simpleperf manually
Change-Id: I6f02f32288b870a4f758e1c8e28fec4b6e567467
|
|
|
|
When converting ETM data for the kernel, the check
"CHECK_GE(vaddr, first_load_segment_addr)" may fail.
It happens when the ETM data asks the decoder to decode
.word data at the end of functions as instructions.
Not sure why this can happen (maybe because of garbage
ETM data caused by overflow), but we don't need to
abort in this situation. Instead, ignore the invalid
data.
Bug: 322399357
Test: run simpleperf_unit_test
Change-Id: Id42a9d84b610bba9003d899efc8fd6ceb39542f9
|
|
|
|
It makes us easier to locate kernel prebuilts in the build server.
Bug: 323054107
Test: run profcollectd
Change-Id: I3f42ca89e76335b16397b57fdc616d4096c78fea
|
|
|
|
If the Display interface is implemented the compiler will automatically
derive an implementation of ToString.
Test: m rust
Bug: 333887339
Change-Id: Ib07011b5004bdc42b60eff7405528b72e77b9496
|
|
|
|
options.PullUintValue() returns true even when --etm-flush-interval
isn't used. In this case, interval is still set to etm_flush_interval_,
which makes it 0.
Bug: 336375740
Test: run simpleperf manually
Change-Id: I95fafcf24b9ae821d8b19782813c7b2f312bf317
|
|
|
|
Add --etm-flush-interval to adjust the interval between ETM data
flushes. So we can use smaller interval when needed.
Bug: 336375740
Test: run simpleperf_unit_test
Change-Id: I1e6805446fa08ee85e9b4033ff827709c34ae530
|
|
* changes:
simpleperf: Delay processing ETM data until the buffer is half full
simpleperf: Rotate the last cpu disabling ETM events
simpleperf: Flush ETM data at the end of recording
simpleperf: Enable ETM events after create aux buffer
|
|
When using --decode-etm, the main thread is busy processing ETM data
and can miss flushing ETM data. To fix this, we can delay processing
ETM data after recording.
This patch triggers ETM data processing when the buffer is at least
half full. In the default configuration, the buffer is large enough
to hold ETM data for several seconds. So effectively, this delays
processing ETM data after recording.
Bug: 336375740
Test: run simpleperf_unit_test
Change-Id: Ib892aeea402773b1cab785f2c14886bb9f96d851
|
|
When using ETR, ETM data is flushed to the aux buffer of the last
cpu disabling ETM events. To avoid overflowing the aux buffer for
one cpu, rotate the last cpu disabling ETM events.
Bug: 336375740
Test: run simpleperf manually
Change-Id: I1b5f2b85935513a3485d84fe0b850b8223ad03b4
|
|
So we can get the last ETM data generated during recording.
Bug: 336375740
Test: run simpleperf manually
Change-Id: I449517fb959e4f8310cd1fc582e2182552d0353d
|
|
Otherwise, ETM events can't be enabled successfully.
Bug: 336375740
Test: run simpleperf manually
Change-Id: I6583aebced57d739f98769cc39dab862569ec8ad
|
|
|
|
When files have spaces in them, we need to use double quotes. Otherwise
some of the bash commands like `basename` and `mktemp` don't use the
full filename which results in incorrect behavior.
Bug: 337895632
Change-Id: Ia50946051bd8ef6cff460810d3559f98d55731c6
Signed-off-by: Will McVicker <willmcvicker@google.com>
|
|
|
|
|
|
Tripped me up in b/337956762#comment7, where I thought
this binary had no native libs, but it was really 32-bit
only.
Bug: 337956762
Test: manual
Change-Id: I225757e2fd5618aa058ef1a528850827feb309f1
|
|
|
|
Summary: After switching to a time-based approach for report generation, we want to
ensure that we don't accidentally include repeated trace files. Thus, after
creating a zip file for the report, we delete the originals.
Test: Ran adb shell device_config put profcollect_native_boot enabled true
and rebooted the device to enable profcollectd.
Ranprofcollectctl once to collect trace files, profcollectctl process to
process them, and then profcollectctl report to generate the report.
Verified that the processed trace files were removed after profcollectctl report
was run & that the config.json file remained.
Tags:
Change-Id: Ic6ab934f52c7af277817ce992faebd96fcde223f
Signed-off-by: Abhishek Gadewar <abhishekgadewar@meta.com>
|
|
If an APK is passed in, then check the zipalignment as well to see if
it's 16K aligned. This will help cover both requirements -- zipalignment
and ELF alignment.
Also, add a print message after checking ELF alignment to indicate that
it was successful. This helps indicate success when there are no shared
libraries in the APK.
Bug: 337895632
Change-Id: I2578dba77e13570aa5b880f950058cfcd939857f
|
|
|
|
Update the document to align with recent changes in
AutoFDO on GitHub and the method for adding profiles
to toolchain/pgo-profiles.
Bug: 337338041
Test: none
Change-Id: Ic03b512eadf9b31a77ee564c0167451d44ce814f
|
|
|
|
This script can be used to check the ELF alignment of shared libraries
in the provided path or in the provided *.apk file.
Bug: 337895632
Change-Id: I3b40383eab076809a73e2ffcf920e6bd8b638cff
|
|
|
|
Currently we do not print info about super partition in lpdump.
Let's add it so we can get to know super's size and used size.
Available space can be inferred from these two.
Bug: 335891825
Test: lpdump --json
Test: atest CtsEdiHostTestCases
Change-Id: Ic3dea6346492c1f94c6432a96c55bfc6164d1b89
Signed-off-by: Sandeep Dhavale <dhavale@google.com>
|
|
* changes:
Merge changes from topic "am-4212e85279cd4af49081bfd97be33764" into android14-tests-dev am: 6df0a78dd6 -s ours
[automerger skipped] Merge "Merge "Merge "simpleperf: Fix flaky test check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336" into android13-tests-dev am: 57b29e5ba9 -s ours am: 04d7e66ade -s ours
|
|
android14-tests-dev am: 6df0a78dd6 -s ours
am skip reason: Merged-In I846cd2bbf1159abf05e9d754c2e96c12912c7d72 with SHA-1 31ec381052 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: I0d5ac89b02a494ffc61043c67db5094f2d1d15ca
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336" into android13-tests-dev am: 57b29e5ba9 -s ours am: 04d7e66ade -s ours
am skip reason: Merged-In I846cd2bbf1159abf05e9d754c2e96c12912c7d72 with SHA-1 31ec381052 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: Ieefaaeceb1f3686c899492f2531aa4ac82260eef
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
* changes:
Merge changes from topic "am-58ca7d8a2c394672818c16262cc7b67b" into android14-tests-dev am: 8961961b84 -s ours
[automerger skipped] Merge "Merge "Merge "simpleperf: Fix record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703" into android13-tests-dev am: bc87a068b5 -s ours am: 39f3b7a553 -s ours
|
|
android14-tests-dev am: 8961961b84 -s ours
am skip reason: Merged-In I401b091e27e7b3ae86f003a270b53be4060c922f with SHA-1 b347fdbef6 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049052
Change-Id: Ifbe2c5d65e94cc97d7e455dfa2e67eac1253ddc4
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703" into android13-tests-dev am: bc87a068b5 -s ours am: 39f3b7a553 -s ours
am skip reason: Merged-In I401b091e27e7b3ae86f003a270b53be4060c922f with SHA-1 b347fdbef6 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049052
Change-Id: Icef8a8907710c2c45ed78df89ee02cc6398c7898
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
android14-tests-dev
* changes:
[automerger skipped] Merge "Merge "Merge "simpleperf: Fix flaky test check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336" into android13-tests-dev am: 57b29e5ba9 -s ours
[automerger skipped] Merge "Merge "simpleperf: Fix flaky test check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336 am: 5cec5340a8 -s ours
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336" into android13-tests-dev am: 57b29e5ba9 -s ours
am skip reason: Merged-In I846cd2bbf1159abf05e9d754c2e96c12912c7d72 with SHA-1 31ec381052 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: Ifc0f8273a783c81844e26593a7435c7f0a5d9b9e
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336 am: 5cec5340a8 -s ours
am skip reason: Merged-In I846cd2bbf1159abf05e9d754c2e96c12912c7d72 with SHA-1 31ec381052 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: Ie838d9cb80a7d6bc45e6b47d92be3b71f1b23b26
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
android14-tests-dev
* changes:
[automerger skipped] Merge "Merge "Merge "simpleperf: Fix record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703" into android13-tests-dev am: bc87a068b5 -s ours
[automerger skipped] Merge "Merge "simpleperf: Fix record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703 am: 0498708589 -s ours
|
|
record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703" into android13-tests-dev am: bc87a068b5 -s ours
am skip reason: Merged-In I401b091e27e7b3ae86f003a270b53be4060c922f with SHA-1 b347fdbef6 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049052
Change-Id: Ie0ed4de209fb78e6eb44039e1451b1d7fca238b1
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
record_cmd#check_trampoline test" into android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703 am: 0498708589 -s ours
am skip reason: Merged-In I401b091e27e7b3ae86f003a270b53be4060c922f with SHA-1 b347fdbef6 is already in history
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049052
Change-Id: I7dff312bee50058b2a8aa42f61164de18b28c360
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336" into android13-tests-dev
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev am: 6b5a8b2336
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: Ia476005ba9cc0e7a529f965696c7ee49ae8dff58
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703" into android13-tests-dev
|
|
android12-tests-dev am: 927e0ca800" into android12L-tests-dev am: d9fb8dd703
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049052
Change-Id: Ieaab6a03bcc4c5da5cbcd85365dda2427c3fd50f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
check_trampoline_after_art_jni_methods" into android12-tests-dev am: e4cc9c7725" into android12L-tests-dev
|
|
into android12-tests-dev am: e4cc9c7725
Original change: https://android-review.googlesource.com/c/platform/system/extras/+/3049053
Change-Id: I33ed7fa5e08bc1cf07e8478c8ecfe82fd11e9ee5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
|
|
android12-tests-dev am: 927e0ca800" into android12L-tests-dev
|