commit 74fe27ef5f4340466a9008f86c8af340b3d67572
author Ady Abraham <adyabr@google.com> Fri Jul 26 10:56:54 2024 -0700
committer Ady Abraham <adyabr@google.com> Fri Jul 26 10:59:35 2024 -0700
tree 5f21ec0968afe570c153af6af0ffb358a47a928c
parent 9953e060ba211798cd5efe04afb1bcf4ef5b600a

SF: fix wouldPresentEarly when allow_n_vsyncs_in_targeter=true

wouldPresentEarly should return true if the previous frame would
not cause a backpressure.

Bug: 308858993
Test: SF unit tests
Flag: com.android.graphics.surfaceflinger.flags.allow_n_vsyncs_in_targeter
Change-Id: I0035145e06f2782cc190e329b542c2f4bbd6f0f1

services/surfaceflinger/Scheduler/src/FrameTargeter.cpp

diff --git a/services/surfaceflinger/Scheduler/src/FrameTargeter.cpp b/services/surfaceflinger/Scheduler/src/FrameTargeter.cpp
index 21f5949..8adf2a6 100644
--- a/services/surfaceflinger/Scheduler/src/FrameTargeter.cpp
+++ b/services/surfaceflinger/Scheduler/src/FrameTargeter.cpp
@@ -61,8 +61,10 @@
 
     const auto [wouldBackpressure, fence] =
             expectedSignaledPresentFence(vsyncPeriod, minFramePeriod);
-    return wouldBackpressure && fence.fenceTime->isValid() &&
-            fence.fenceTime->getSignalTime() != Fence::SIGNAL_TIME_PENDING;
+
+    return !wouldBackpressure ||
+            (fence.fenceTime->isValid() &&
+             fence.fenceTime->getSignalTime() != Fence::SIGNAL_TIME_PENDING);
 }
 
 const FenceTimePtr& FrameTarget::presentFenceForPreviousFrame() const {