summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Gavin Corkery <gavincorkery@google.com> 2021-07-08 22:39:02 +0100
committer Gavin Corkery <gavincorkery@google.com> 2021-07-08 22:55:38 +0100
commitfac7e2b57ba6b9d461ed779a7fe6a620335d16c8 (patch)
treeb5aa8abc89364b52b0c9f9bfee35b5e39e23703e
parent2bfbdffc26b4da2670a74b8fef640fa3741ad229 (diff)
Increase meminfo dumpsys timeout
Increases the meminfo dumpsys timeout from 30s to 60s, since 30s is sometimes not enough. Test: Check that longer timeout is used in writeDump call Bug: 191615258 Change-Id: Ic270a81f3dde1329881023d533ffbff43a82127c
-rw-r--r--cmds/dumpstate/dumpstate.cpp11
1 files changed, 9 insertions, 2 deletions
diff --git a/cmds/dumpstate/dumpstate.cpp b/cmds/dumpstate/dumpstate.cpp
index c9a7c7b90e..8f466ca57e 100644
--- a/cmds/dumpstate/dumpstate.cpp
+++ b/cmds/dumpstate/dumpstate.cpp
@@ -1240,8 +1240,15 @@ static Dumpstate::RunStatus RunDumpsysTextByPriority(const std::string& title, i
if (status == OK) {
dumpsys.writeDumpHeader(STDOUT_FILENO, service, priority);
std::chrono::duration<double> elapsed_seconds;
- status = dumpsys.writeDump(STDOUT_FILENO, service, service_timeout,
- /* as_proto = */ false, elapsed_seconds, bytes_written);
+ if (priority == IServiceManager::DUMP_FLAG_PRIORITY_HIGH &&
+ service == String16("meminfo")) {
+ // Use a longer timeout for meminfo, since 30s is not always enough.
+ status = dumpsys.writeDump(STDOUT_FILENO, service, 60s,
+ /* as_proto = */ false, elapsed_seconds, bytes_written);
+ } else {
+ status = dumpsys.writeDump(STDOUT_FILENO, service, service_timeout,
+ /* as_proto = */ false, elapsed_seconds, bytes_written);
+ }
dumpsys.writeDumpFooter(STDOUT_FILENO, service, elapsed_seconds);
bool dump_complete = (status == OK);
dumpsys.stopDumpThread(dump_complete);