Add tracepoints to gs201 dumpstateBoard
This change adds tracepoints for the gs201 implementation of dumpstateBoard. This is intended to help identify long-running sections. Test: Build+flash, check that traces of 'adb bugreportz' show the expected trace sections. Bug: 258228821 Change-Id: Ia8f64f63609533eaaec1e78a93c1f67947d82466
This commit is contained in:
parent
b4ee8b12eb
commit
2381ca7f30
1 changed files with 6 additions and 0 deletions
|
@ -15,6 +15,7 @@
|
||||||
*/
|
*/
|
||||||
|
|
||||||
#define LOG_TAG "dumpstate_device"
|
#define LOG_TAG "dumpstate_device"
|
||||||
|
#define ATRACE_TAG ATRACE_TAG_ALWAYS
|
||||||
|
|
||||||
#include <inttypes.h>
|
#include <inttypes.h>
|
||||||
|
|
||||||
|
@ -22,6 +23,7 @@
|
||||||
#include <android-base/stringprintf.h>
|
#include <android-base/stringprintf.h>
|
||||||
#include <android-base/properties.h>
|
#include <android-base/properties.h>
|
||||||
#include <android-base/unique_fd.h>
|
#include <android-base/unique_fd.h>
|
||||||
|
#include <cutils/trace.h>
|
||||||
#include <log/log.h>
|
#include <log/log.h>
|
||||||
#include <sys/stat.h>
|
#include <sys/stat.h>
|
||||||
|
|
||||||
|
@ -187,6 +189,7 @@ void dumpModemEFS(std::string destDir) {
|
||||||
}
|
}
|
||||||
|
|
||||||
timepoint_t startSection(int fd, const std::string §ionName) {
|
timepoint_t startSection(int fd, const std::string §ionName) {
|
||||||
|
ATRACE_BEGIN(sectionName.c_str());
|
||||||
::android::base::WriteStringToFd(
|
::android::base::WriteStringToFd(
|
||||||
"\n"
|
"\n"
|
||||||
"------ Section start: " + sectionName + " ------\n"
|
"------ Section start: " + sectionName + " ------\n"
|
||||||
|
@ -195,6 +198,7 @@ timepoint_t startSection(int fd, const std::string §ionName) {
|
||||||
}
|
}
|
||||||
|
|
||||||
void endSection(int fd, const std::string §ionName, timepoint_t startTime) {
|
void endSection(int fd, const std::string §ionName, timepoint_t startTime) {
|
||||||
|
ATRACE_END();
|
||||||
auto endTime = std::chrono::steady_clock::now();
|
auto endTime = std::chrono::steady_clock::now();
|
||||||
auto elapsedMsec = std::chrono::duration_cast<std::chrono::milliseconds>
|
auto elapsedMsec = std::chrono::duration_cast<std::chrono::milliseconds>
|
||||||
(endTime - startTime).count();
|
(endTime - startTime).count();
|
||||||
|
@ -1149,6 +1153,7 @@ void Dumpstate::dumpLogSection(int fd, int fd_bin)
|
||||||
ndk::ScopedAStatus Dumpstate::dumpstateBoard(const std::vector<::ndk::ScopedFileDescriptor>& in_fds,
|
ndk::ScopedAStatus Dumpstate::dumpstateBoard(const std::vector<::ndk::ScopedFileDescriptor>& in_fds,
|
||||||
IDumpstateDevice::DumpstateMode in_mode,
|
IDumpstateDevice::DumpstateMode in_mode,
|
||||||
int64_t in_timeoutMillis) {
|
int64_t in_timeoutMillis) {
|
||||||
|
ATRACE_BEGIN("dumpstateBoard");
|
||||||
// Unused arguments.
|
// Unused arguments.
|
||||||
(void) in_timeoutMillis;
|
(void) in_timeoutMillis;
|
||||||
|
|
||||||
|
@ -1185,6 +1190,7 @@ ndk::ScopedAStatus Dumpstate::dumpstateBoard(const std::vector<::ndk::ScopedFile
|
||||||
|
|
||||||
dumpTextSection(fd, kAllSections);
|
dumpTextSection(fd, kAllSections);
|
||||||
|
|
||||||
|
ATRACE_END();
|
||||||
return ndk::ScopedAStatus::ok();
|
return ndk::ScopedAStatus::ok();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue