diff options
author | Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> | 2022-01-29 00:43:13 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2022-01-29 00:43:13 +0000 |
commit | 2cbc0596712eddd5c269133855a15b4d2b4c59b3 (patch) | |
tree | 5db64db0a0090ea9aeae83a600cd202ccf977b85 | |
parent | c54d12318f0e511868f61681dc9f1a6dec46c2df (diff) | |
parent | a331e9c67c6530f5c667855f7a784338eb03b141 (diff) | |
download | extras-2cbc0596712eddd5c269133855a15b4d2b4c59b3.tar.gz |
Merge "Merge "simpleperf: use 8 byte aligned stack size when resizing stack data." into android11-tests-dev am: 0eb3232339" into android12-tests-dev
-rw-r--r-- | simpleperf/RecordReadThread.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/simpleperf/RecordReadThread.cpp b/simpleperf/RecordReadThread.cpp index 16af9292..a8229303 100644 --- a/simpleperf/RecordReadThread.cpp +++ b/simpleperf/RecordReadThread.cpp @@ -540,7 +540,7 @@ void RecordReadThread::PushRecordToRecordBuffer(KernelRecordReader* kernel_recor // space in each sample to store stack data. However, a thread may use less stack than 64K. // So not all the 64K stack data in a sample is valid, and we only need to keep valid stack // data, whose size is dyn_stack_size. - uint64_t new_stack_size = std::min<uint64_t>(dyn_stack_size, stack_size_limit); + uint64_t new_stack_size = Align(std::min<uint64_t>(dyn_stack_size, stack_size_limit), 8); if (stack_size > new_stack_size) { // Remove part of the stack data. perf_event_header new_header = header; |