commit e48652674ebd4047363a79d38a6018978087223e
author Sanna Catherine de Treville Wager <scwager@google.com> Fri Jul 14 16:24:15 2017 -0700
committer Sanna Catherine de Treville Wager <scwager@google.com> Fri Jul 21 15:52:40 2017 +0000
tree 4d7b5927da48cf1504b0b32fc01a88265dea717f
parent 633adf69ae8f9aeb41bceb193afefe35e5550c7f

Separate merge from dump and call merge periodically

Every time merge is called by the thread loop,
the data is also processed and written to
PerformanceAnalysis. A call to dump retrieves
the FIFO of processed data in PerformanceAnalysis
instead of the unprocessed Reader FIFO, and prints
the result to the console.

Test: dumpsys media.log

Change-Id: Ic479f48e9e4fdf4523a8f15db514dbdd85b70434

media/libnbaio/PerformanceAnalysis.cpp

diff --git a/media/libnbaio/PerformanceAnalysis.cpp b/media/libnbaio/PerformanceAnalysis.cpp
index fb3bddc..d44dd45 100644
--- a/media/libnbaio/PerformanceAnalysis.cpp
+++ b/media/libnbaio/PerformanceAnalysis.cpp
@@ -271,7 +271,6 @@
         ALOGD("reportPerformance: mRecentHists is empty");
         return;
     }
-    ALOGD("reportPerformance: hists size %d", static_cast<int>(mRecentHists.size()));
     // TODO: more elaborate data analysis
     std::map<int, int> buckets;
     for (const auto &shortHist: mRecentHists) {
@@ -306,6 +305,7 @@
         scalingFactor = (height + maxHeight) / maxHeight;
         height /= scalingFactor;
     }
+    // TODO: print reader (author) ID
     body->appendFormat("\n%*s", leftPadding + 11, "Occurrences");
     // write histogram label line with bucket values
     body->appendFormat("\n%s", " ");