commit e08a0920030d2fd7ccf250ce90ffd5281b28ae46
author Ady Abraham <adyabr@google.com> Thu Feb 16 18:07:41 2023 +0000
committer Ady Abraham <adyabr@google.com> Thu Feb 23 22:47:27 2023 +0000
tree a1daff8b6851fa4d03768a82ebc389a14e45e5ca
parent 57ddefa7417cb67b4d92beaefd08ca2be827817d

Choreographer: add a trace for vsync callback

Bug: 269620055
Change-Id: I472104c4752c5b71b194cd581cbf1e0a6a9098d2
Test: manual

libs/gui/Choreographer.cpp

diff --git a/libs/gui/Choreographer.cpp b/libs/gui/Choreographer.cpp
index 99bf6ba..46fb068 100644
--- a/libs/gui/Choreographer.cpp
+++ b/libs/gui/Choreographer.cpp
@@ -15,8 +15,10 @@
  */
 
 // #define LOG_NDEBUG 0
+#define ATRACE_TAG ATRACE_TAG_GRAPHICS
 
 #include <gui/Choreographer.h>
+#include <gui/TraceUtils.h>
 #include <jni.h>
 
 #undef LOG_TAG
@@ -297,6 +299,8 @@
     mLastVsyncEventData = vsyncEventData;
     for (const auto& cb : callbacks) {
         if (cb.vsyncCallback != nullptr) {
+            ATRACE_FORMAT("AChoreographer_vsyncCallback %" PRId64,
+                          vsyncEventData.preferredVsyncId());
             const ChoreographerFrameCallbackDataImpl frameCallbackData =
                     createFrameCallbackData(timestamp);
             registerStartTime();
@@ -306,8 +310,10 @@
                              cb.data);
             mInCallback = false;
         } else if (cb.callback64 != nullptr) {
+            ATRACE_FORMAT("AChoreographer_frameCallback64");
             cb.callback64(timestamp, cb.data);
         } else if (cb.callback != nullptr) {
+            ATRACE_FORMAT("AChoreographer_frameCallback");
             cb.callback(timestamp, cb.data);
         }
     }