commit 8eebba4486626cae984a9ec5ffb1cbe0677585db
author Vishnu Nair <vishnun@google.com> Fri Feb 25 07:57:15 2022 -0800
committer Vishnu Nair <vishnun@google.com> Fri Feb 25 08:03:03 2022 -0800
tree f6f7cc795a7e43c84e0a8a90d7842930b5e80a3b
parent 4e293cd2cc7d9a2e772666e00258972933a7a393

Sf Tracing: Set a valid/fired acquire fence

Fixes a NPE with acquire fence issues when generating layers trace
from transaction traces.

Bug: 200284593
Test: atest transactiontrace_testsuite

Change-Id: I7e07b5a84af097a53927635ec83226f850d5a9e6

services/surfaceflinger/Tracing/TransactionProtoParser.cpp

diff --git a/services/surfaceflinger/Tracing/TransactionProtoParser.cpp b/services/surfaceflinger/Tracing/TransactionProtoParser.cpp
index 1e5c3e7..4cd5ace 100644
--- a/services/surfaceflinger/Tracing/TransactionProtoParser.cpp
+++ b/services/surfaceflinger/Tracing/TransactionProtoParser.cpp
@@ -15,6 +15,7 @@
  */
 
 #include <gui/SurfaceComposerClient.h>
+#include <ui/Fence.h>
 #include <ui/Rect.h>
 
 #include "LayerProtoHelper.h"
@@ -431,6 +432,7 @@
         layer.bufferData->frameNumber = bufferProto.frame_number();
         layer.bufferData->flags = Flags<BufferData::BufferDataChange>(bufferProto.flags());
         layer.bufferData->cachedBuffer.id = bufferProto.cached_buffer_id();
+        layer.bufferData->acquireFence = Fence::NO_FENCE;
     }
 
     if (proto.what() & layer_state_t::eApiChanged) {