commit 248bce89cac382462dac0e1bc9cea5488d5b33cf
author Ady Abraham <adyabr@google.com> Thu Sep 16 14:29:59 2021 -0700
committer Ady Abraham <adyabr@google.com> Thu Sep 16 21:30:26 2021 +0000
tree 46ef875d3a508169216301d308466449ab85ad25
parent 3605a85663c9d777a60d161fe9e5eef23a49d40a

SF: trace vsyncPeriod from onComposerHalVsync

Test: manual
Change-Id: Ie447c65eafc11e6c1c0bc32401501b1499a73e6b

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 5a881a3..c7e5dff 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -1710,7 +1710,15 @@
 
 void SurfaceFlinger::onComposerHalVsync(hal::HWDisplayId hwcDisplayId, int64_t timestamp,
                                         std::optional<hal::VsyncPeriodNanos> vsyncPeriod) {
-    ATRACE_CALL();
+    const std::string tracePeriod = [vsyncPeriod]() {
+        if (ATRACE_ENABLED() && vsyncPeriod) {
+            std::stringstream ss;
+            ss << "(" << *vsyncPeriod << ")";
+            return ss.str();
+        }
+        return std::string();
+    }();
+    ATRACE_FORMAT("onComposerHalVsync%s", tracePeriod.c_str());
 
     Mutex::Autolock lock(mStateLock);
     const auto displayId = getHwComposer().toPhysicalDisplayId(hwcDisplayId);