Merge "surfaceflinger: fix fencing when composition mode switches" am: 42ddbb44ac am: 4256a72959
am: 3d91a35182
Change-Id: I3ffbf8ba6f224cdb8c4d6d5a36ad40a1860dfad4
diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 5e1de75..50ed8f7 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2037,13 +2037,22 @@
displayDevice->onSwapBuffersCompleted();
displayDevice->makeCurrent(mEGLDisplay, mEGLContext);
for (auto& layer : displayDevice->getVisibleLayersSortedByZ()) {
- sp<Fence> releaseFence = Fence::NO_FENCE;
+ // The layer buffer from the previous frame (if any) is released
+ // by HWC only when the release fence from this frame (if any) is
+ // signaled. Always get the release fence from HWC first.
+ auto hwcLayer = layer->getHwcLayer(hwcId);
+ sp<Fence> releaseFence = mHwc->getLayerReleaseFence(hwcId, hwcLayer);
+
+ // If the layer was client composited in the previous frame, we
+ // need to merge with the previous client target acquire fence.
+ // Since we do not track that, always merge with the current
+ // client target acquire fence when it is available, even though
+ // this is suboptimal.
if (layer->getCompositionType(hwcId) == HWC2::Composition::Client) {
- releaseFence = displayDevice->getClientTargetAcquireFence();
- } else {
- auto hwcLayer = layer->getHwcLayer(hwcId);
- releaseFence = mHwc->getLayerReleaseFence(hwcId, hwcLayer);
+ releaseFence = Fence::merge("LayerRelease", releaseFence,
+ displayDevice->getClientTargetAcquireFence());
}
+
layer->onLayerDisplayed(releaseFence);
}
if (hwcId >= 0) {