commit 009619f98ce51e113dbbefb5722d4b585e719dbd
author Chavi Weingarten <chaviw@google.com> Thu Sep 15 20:27:25 2022 +0000
committer Chavi Weingarten <chaviw@google.com> Thu Sep 15 20:27:25 2022 +0000
tree eca873f0d384e8b63cf5c1e6669acb142058f774
parent cc8be8c3a1c938cd8a1226608e4b830b470f25ff

Add more tracing info when setting buffer

Test: Traces
Bug: 244218818
Change-Id: I150d703986f24fd6ad3a3bc3c6ea65c5f717cba7

services/surfaceflinger/Layer.cpp

diff --git a/services/surfaceflinger/Layer.cpp b/services/surfaceflinger/Layer.cpp
index 08b71c2..91e32f3 100644
--- a/services/surfaceflinger/Layer.cpp
+++ b/services/surfaceflinger/Layer.cpp
@@ -3177,8 +3177,7 @@
                       const BufferData& bufferData, nsecs_t postTime, nsecs_t desiredPresentTime,
                       bool isAutoTimestamp, std::optional<nsecs_t> dequeueTime,
                       const FrameTimelineInfo& info) {
-    ATRACE_CALL();
-
+    ATRACE_FORMAT("setBuffer %s - hasBuffer=%s", getDebugName(), (buffer ? "true" : "false"));
     if (!buffer) {
         return false;
     }
@@ -3187,6 +3186,7 @@
             bufferData.flags.test(BufferData::BufferDataChange::frameNumberChanged);
     const uint64_t frameNumber =
             frameNumberChanged ? bufferData.frameNumber : mDrawingState.frameNumber + 1;
+    ATRACE_FORMAT_INSTANT("setBuffer %s - %" PRIu64, getDebugName(), frameNumber);
 
     if (mDrawingState.buffer) {
         mReleasePreviousBuffer = true;