commit 1a79145897a0059ff698c494b187e5d838faea68
author yawanng <yawanng@google.com> Thu Jun 25 00:28:36 2020 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Thu Jun 25 00:28:36 2020 +0000
tree f048d31d9e159f29197bb499477d046f9c8d2f73
parent 4723ee6c2278febd59a4753e502fb1f00c952cc0
parent ab7cdc3db888586d755c7aca727b15583c3e0e04

iorap: Reduce the log spam from IORap. am: ab7cdc3db8

Original change: https://googleplex-android-review.googlesource.com/c/platform/system/iorap/+/11991331

Change-Id: I531c3d8bc7136ce339039d51f5b2dee56a65810d

src/iorapd/main.cc

diff --git a/src/iorapd/main.cc b/src/iorapd/main.cc
index 929ac54..8d806ba 100644
--- a/src/iorapd/main.cc
+++ b/src/iorapd/main.cc
@@ -33,9 +33,6 @@
   if (android::base::GetBoolProperty("iorapd.log.verbose", iorap::kIsDebugBuild)) {
     // Show verbose logs if the property is enabled or if we are a debug build.
     setenv("ANDROID_LOG_TAGS", "*:v", /*overwrite*/ 1);
-  } else {
-    // Debug log-level.
-    setenv("ANDROID_LOG_TAGS", "*:d", /*overwrite*/ 1);
   }
 
   // Logs go to system logcat.

src/maintenance/controller.cc

diff --git a/src/maintenance/controller.cc b/src/maintenance/controller.cc
index 639f402..daaf08f 100644
--- a/src/maintenance/controller.cc
+++ b/src/maintenance/controller.cc
@@ -234,7 +234,7 @@
     } else if (history.total_time_ns) {
       timestamp_limit = *history.total_time_ns;
     } else {
-      LOG(ERROR) << " No timestamp exists. Using the max value.";
+      LOG(DEBUG) << " No timestamp exists. Using the max value.";
     }
     perfetto_traces.push_back({raw_trace->file_path, timestamp_limit});
   }

src/manager/event_manager.cc

diff --git a/src/manager/event_manager.cc b/src/manager/event_manager.cc
index 74c1cfc..bae676b 100644
--- a/src/manager/event_manager.cc
+++ b/src/manager/event_manager.cc
@@ -410,7 +410,7 @@
       if (std::filesystem::exists(compiled_trace->file_path)) {
         return compiled_trace->file_path;
       } else {
-        LOG(ERROR) << "Compiled trace in sqlite doesn't exists. file_path: "
+        LOG(DEBUG) << "Compiled trace in sqlite doesn't exists. file_path: "
                    << compiled_trace->file_path;
       }
     }