commit 80e4b3b732e18bafe7f95215a4adb7c0c67a53ab
author Hungyen Weng <hungyenweng@google.com> Wed Mar 11 10:04:08 2020 +0800
committer Hungyen Weng <hungyenweng@google.com> Wed Mar 11 03:23:19 2020 +0000
tree f73c3eebc24dd1c5e05dc00c44f4afb18961dc95
parent 065c4a2fe87eb5df6d1c5e40bd491c20b742d1f0

dumpstate: don't dump modem logs if verbose logging is disabled.

Bug: 151186921
Change-Id: I9e7b499d332ea4e042edf173ac0368f3c537f0f7

dumpstate/DumpstateDevice.cpp

diff --git a/dumpstate/DumpstateDevice.cpp b/dumpstate/DumpstateDevice.cpp
index 9f616df..c8f2752 100755
--- a/dumpstate/DumpstateDevice.cpp
+++ b/dumpstate/DumpstateDevice.cpp
@@ -379,12 +379,15 @@
     RunCommandToFd(fd, "Notify modem", {"/vendor/bin/modem_svc", "-s"}, CommandOptions::WithTimeout(1).Build());
 
     pthread_t modemThreadHandle = 0;
-    if (handle->numFds < 2) {
-        ALOGE("no FD for modem\n");
-    } else {
-        int fdModem = handle->data[1];
-        if (pthread_create(&modemThreadHandle, NULL, dumpModemThread, (void *)((long)fdModem)) != 0) {
-            ALOGE("could not create thread for dumpModem\n");
+    if (getVerboseLoggingEnabled()) {
+        ALOGD("Verbose logging is enabled.\n");
+        if (handle->numFds < 2) {
+            ALOGE("no FD for modem\n");
+        } else {
+            int fdModem = handle->data[1];
+            if (pthread_create(&modemThreadHandle, NULL, dumpModemThread, (void *)((long)fdModem)) != 0) {
+                ALOGE("could not create thread for dumpModem\n");
+            }
         }
     }