summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Vishnu Nair <vishnun@google.com> 2023-07-21 14:20:27 +0000
committer Vishnu Nair <vishnun@google.com> 2023-07-21 14:20:27 +0000
commitd9a640b306eb1754311c867825f52ffb8bcc460c (patch)
treec37567fc58eacc9ca7e97b41b533ca9d9986f3cb
parent52d56fd14ec7e8aa5b0532dbb1330b5ef5cc5248 (diff)
[sf] Only trace rebuildLayerStacks if there are changes
Clean up perfetto traces to only show tags when they are interesting. Bug: 292225846 Test: open trace in perfetto Change-Id: I1e3744abb08c7a3b34244db8ce8be5299cf86b19
-rw-r--r--services/surfaceflinger/CompositionEngine/src/Output.cpp5
1 files changed, 2 insertions, 3 deletions
diff --git a/services/surfaceflinger/CompositionEngine/src/Output.cpp b/services/surfaceflinger/CompositionEngine/src/Output.cpp
index 1205a2ce71..b1b5708473 100644
--- a/services/surfaceflinger/CompositionEngine/src/Output.cpp
+++ b/services/surfaceflinger/CompositionEngine/src/Output.cpp
@@ -469,15 +469,14 @@ void Output::uncacheBuffers(std::vector<uint64_t> const& bufferIdsToUncache) {
void Output::rebuildLayerStacks(const compositionengine::CompositionRefreshArgs& refreshArgs,
LayerFESet& layerFESet) {
- ATRACE_CALL();
- ALOGV(__FUNCTION__);
-
auto& outputState = editState();
// Do nothing if this output is not enabled or there is no need to perform this update
if (!outputState.isEnabled || CC_LIKELY(!refreshArgs.updatingOutputGeometryThisFrame)) {
return;
}
+ ATRACE_CALL();
+ ALOGV(__FUNCTION__);
// Process the layers to determine visibility and coverage
compositionengine::Output::CoverageState coverage{layerFESet};