commit 29e307db68e44d11461c7c8b94293f76102c9745
author Ram Indani <ramindani@google.com> Mon Sep 16 21:26:03 2024 +0000
committer Sally Qi <sallyqi@google.com> Mon Sep 16 22:28:23 2024 +0000
tree b5619fa900b762e63d1bae15dba6999535df2ce6
parent 357c2ed35dce4aeefb3bfbd36d312a6c7fbc9af6

DO NOT MERGE Early return for getPresentFenceShift

When minFrameInterval is 0 then return the shift was 0.

Bug: 365442861
Merged-In: I00c1795c11f1dc38e6b4ce3ff49fae79e265f25c
Change-Id: I00c1795c11f1dc38e6b4ce3ff49fae79e265f25c

services/surfaceflinger/Scheduler/include/scheduler/FrameTargeter.h

diff --git a/services/surfaceflinger/Scheduler/include/scheduler/FrameTargeter.h b/services/surfaceflinger/Scheduler/include/scheduler/FrameTargeter.h
index 2c397bd..a54d435 100644
--- a/services/surfaceflinger/Scheduler/include/scheduler/FrameTargeter.h
+++ b/services/surfaceflinger/Scheduler/include/scheduler/FrameTargeter.h
@@ -134,8 +134,11 @@
     }
 
     size_t getPresentFenceShift(Period minFramePeriod) const {
-        const bool isTwoVsyncsAhead = targetsVsyncsAhead<2>(minFramePeriod);
         size_t shift = 0;
+        if (minFramePeriod.ns() == 0) {
+            return shift;
+        }
+        const bool isTwoVsyncsAhead = targetsVsyncsAhead<2>(minFramePeriod);
         if (isTwoVsyncsAhead) {
             shift = static_cast<size_t>(expectedFrameDuration().ns() / minFramePeriod.ns());
             if (shift >= mPresentFences.size()) {