summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohnnLee <johnnlee@google.com>2021-06-03 11:38:08 +0800
committerJohnnLee <johnnlee@google.com>2021-06-03 11:58:27 +0800
commitb0e331d53616ac605a1e8d6a1c50b0187ab5a204 (patch)
treee9f21223c33364427711eb31255c53364a7d77dc
parent2d11c2807b420507adf4f0c224b50180ea4277c2 (diff)
downloaddisplay-b0e331d53616ac605a1e8d6a1c50b0187ab5a204.tar.gz
Revert "hwc2: Skip SDE composition if all client layers are invalid"
This reverts commit 0a38f67f7c7e38f7c25b873796a303fd96ce7745. Change-Id: I101732abd705886b020b6dfbe182915720a8c09b
-rw-r--r--sdm/libs/hwc2/hwc_display.cpp12
1 files changed, 1 insertions, 11 deletions
diff --git a/sdm/libs/hwc2/hwc_display.cpp b/sdm/libs/hwc2/hwc_display.cpp
index 894d9224..a4b83441 100644
--- a/sdm/libs/hwc2/hwc_display.cpp
+++ b/sdm/libs/hwc2/hwc_display.cpp
@@ -1,5 +1,5 @@
/*
- * Copyright (c) 2014-2021, The Linux Foundation. All rights reserved.
+ * Copyright (c) 2014-2020, The Linux Foundation. All rights reserved.
* Not a Contribution.
*
* Copyright 2015 The Android Open Source Project
@@ -654,7 +654,6 @@ void HWCDisplay::BuildLayerStack() {
layer_stack_ = LayerStack();
display_rect_ = LayerRect();
metadata_refresh_rate_ = 0;
- bool has_valid_client_layer = false;
layer_stack_.flags.animating = animating_;
layer_stack_.flags.fast_path = fast_path_enabled_ && fast_path_composition_;
@@ -740,7 +739,6 @@ void HWCDisplay::BuildLayerStack() {
if (layer->flags.skip) {
layer_stack_.flags.skip_present = true;
- has_valid_client_layer |= IsValid(layer->dst_rect);
}
// TODO(user): Move to a getter if this is needed at other places
@@ -789,14 +787,6 @@ void HWCDisplay::BuildLayerStack() {
layer_stack_.layers.push_back(layer);
}
- // If all client layers are invalid, skip all layers
- if (layer_stack_.flags.skip_present && !has_valid_client_layer) {
- DLOGE("Not a single valid client layer present, skipping all");
- for (auto layer : layer_stack_.layers) {
- layer->flags.skip = true;
- }
- }
-
// If layer stack needs Client composition, HWC display gets into InternalValidate state. If
// validation gets reset by any other thread in this state, enforce Geometry change to ensure
// that Client target gets composed by SF.