commit 6996827fa06e9c83324da4da485251a78b081da2
author Jayachandran C <jayachandranc@google.com> Mon Jul 08 09:46:05 2019 -0700
committer Jayachandran Chinnakkannu <jayachandranc@google.com> Fri Aug 16 02:00:24 2019 +0000
tree 3458beca78de1db3d88a90318c4b12c0c0c4d6b7
parent e75ab2a74185a6c3d7008fd96a048118b3eb5956

Dump HALs and enable persistent logs for radio bugreports

HALs dumps are needed to debug WFC releated issues
Persistent logs capture is needed to capture the history of logs that
captures the scenario of telephony going to bad state.

Bug: 138261934
Test: Triggered telephony dumpsys using connectivity monitor app and verified

Change-Id: I158e327d5c5a13c64e48d5711f82903d4a50b8f0
Merged-In: I158e327d5c5a13c64e48d5711f82903d4a50b8f0

cmds/dumpstate/dumpstate.cpp

diff --git a/cmds/dumpstate/dumpstate.cpp b/cmds/dumpstate/dumpstate.cpp
index 957dfa1..3c4e33e 100644
--- a/cmds/dumpstate/dumpstate.cpp
+++ b/cmds/dumpstate/dumpstate.cpp
@@ -1509,6 +1509,8 @@
 static void DumpstateRadioCommon() {
     DumpIpTablesAsRoot();
 
+    ds.AddDir(LOGPERSIST_DATA_DIR, false);
+
     if (!DropRootUser()) {
         return;
     }
@@ -1519,6 +1521,7 @@
     DoKmsg();
     DumpIpAddrAndRules();
     dump_route_tables();
+    DumpHals();
 
     RunDumpsys("NETWORK DIAGNOSTICS", {"connectivity", "--diag"},
                CommandOptions::WithTimeout(10).Build());
@@ -1588,8 +1591,6 @@
     RunDumpsys("DUMPSYS", {"wifi"}, CommandOptions::WithTimeout(90).Build(),
                SEC_TO_MSEC(10));
 
-    DumpHals();
-
     printf("========================================================\n");
     printf("== dumpstate: done (id %d)\n", ds.id_);
     printf("========================================================\n");