diff options
author | Tej Singh <singhtejinder@google.com> | 2018-02-06 15:57:38 -0800 |
---|---|---|
committer | Mark Salyzyn <salyzyn@google.com> | 2018-06-04 13:31:05 -0700 |
commit | 993f65619e534d29a1e63cdc3d76a5bdec97225b (patch) | |
tree | f4722454450fb1ac0039c4851d7c67494fd54f44 /libutils/ProcessCallStack.cpp | |
parent | dc17e9efa043dc74c177c50834937a84457aaf3f (diff) |
Fix performance degradation from BootSequence atom
(partial cherry pick from commit fe3e762b6de94ab43b3019d38cdc2abfad3a786c)
Adding the boot sequence reported atom in ag/3518079 caused the duration
of bootstat to increase, as seen in b/72864061. I isolated the cause
down to calling BootReasonStrToReason. However, this function also gets
called in ReportBootReason, so I created another function that does the
parsing and sets the system boot reason property, and made
RecordBootReason and statsd logging get that property.
Bug: 72864061
Test: rebooted phone, verified boot events were received in adb shell
logcat -b stats and verified adb shell bootstat -p printed correct
values. Ran timing tests as well on walleye with 20 boots: before this
change, the average was ~150-160ms. After, it was ~80ms.
Change-Id: I92dbc9880328835647be7d9d50c7861b42f27bdb
Merged-In: I92dbc9880328835647be7d9d50c7861b42f27bdb
Diffstat (limited to 'libutils/ProcessCallStack.cpp')
0 files changed, 0 insertions, 0 deletions