commit aa83158502e233bde50f78ad2a91ed9168252977
author Alec Mouri <alecmouri@google.com> Mon Jun 07 16:23:01 2021 -0700
committer Alec Mouri <alecmouri@google.com> Mon Jun 14 16:47:54 2021 -0700
tree c97bc7f94140212fc03706d8fde7bd0546544ba2
parent df6201b9495e4d07f23eeae10c100a02aca2e721

Support skipping CachedSet rendering based on invalidate() proximity

Add tunable sysprops which allow for skipping rendering a new
CachedSet if we're too close to an invalidate(), so that rendering the
CachedSet does not cause jank.

The sysprops control:
1. Whether we support this type of deferred rendering
2. How close to a rendering deadline we're allowed to be
3. Max number of times we can skip before we give up and render anyway.

Bug: 188678587
Test: builds, boots
Test: perfetto
Change-Id: I9801c25da1d9178faa8fa28b9bd8f9d180db7b11

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index a4b6fef..a109f3b 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -2040,6 +2040,7 @@
     }
 
     refreshArgs.earliestPresentTime = mScheduler->getPreviousVsyncFrom(mExpectedPresentTime);
+    refreshArgs.nextInvalidateTime = mEventQueue->nextExpectedInvalidate();
 
     mGeometryInvalid = false;