[sf] Clean up transaction handling trace tags
- Only add a tag if a transaction is not ready to be applied
- Add sufficient info on why its not ready and which layer was affected
- Remove unused tags
Test: perfetto traces
Bug: 277799011
Change-Id: I956823699721aaa2260eefd3762ec60a20790140
diff --git a/services/surfaceflinger/FrontEnd/TransactionHandler.cpp b/services/surfaceflinger/FrontEnd/TransactionHandler.cpp
index a209cad..9cbe0bb 100644
--- a/services/surfaceflinger/FrontEnd/TransactionHandler.cpp
+++ b/services/surfaceflinger/FrontEnd/TransactionHandler.cpp
@@ -21,6 +21,7 @@
#include <cutils/trace.h>
#include <utils/Log.h>
+#include <utils/Trace.h>
#include "TransactionHandler.h"
@@ -73,12 +74,13 @@
void TransactionHandler::applyUnsignaledBufferTransaction(
std::vector<TransactionState>& transactions, TransactionFlushState& flushState) {
- // only apply an unsignaled buffer transaction if it's the first one
- if (!transactions.empty()) {
+ if (!flushState.queueWithUnsignaledBuffer) {
return;
}
- if (!flushState.queueWithUnsignaledBuffer) {
+ // only apply an unsignaled buffer transaction if it's the first one
+ if (!transactions.empty()) {
+ ATRACE_NAME("fence unsignaled");
return;
}