commit 0311fdd6d2d5eca00e84bd95b1d1a296171ca038
author Adam Wright <adamdwright@google.com> Thu Feb 16 19:05:45 2023 +0000
committer Ady Abraham <adyabr@google.com> Fri Feb 17 16:26:20 2023 +0000
tree 78d79cfee5af33b50f1d2e5cef457783054726be
parent 5fc135bf77a1d4379aca32d2b400cd3ff864d6fc

Revert "SF: pass a render rate to VsyncTracker instead of a divisor"

Revert submission 21422016

Reason for revert: DM P0 (b/269561042)

Reverted changes: /q/submissionid:21422016

Bug: 269561042
Change-Id: I118dd75ea379673d649d3c75960f18ef45b3749f
(cherry picked from commit 3d21d3afa1056efacf702b74449f9e2e7b5701f1)
Merged-In: I118dd75ea379673d649d3c75960f18ef45b3749f

services/surfaceflinger/Scheduler/Scheduler.cpp

diff --git a/services/surfaceflinger/Scheduler/Scheduler.cpp b/services/surfaceflinger/Scheduler/Scheduler.cpp
index dab01ba..17cdff9 100644
--- a/services/surfaceflinger/Scheduler/Scheduler.cpp
+++ b/services/surfaceflinger/Scheduler/Scheduler.cpp
@@ -418,7 +418,11 @@
     ALOGV("%s %s (%s)", __func__, to_string(mode.fps).c_str(),
           to_string(mode.modePtr->getFps()).c_str());
 
-    mVsyncSchedule->getTracker().setRenderRate(renderFrameRate);
+    const auto divisor = RefreshRateSelector::getFrameRateDivisor(mode.modePtr->getFps(), mode.fps);
+    LOG_ALWAYS_FATAL_IF(divisor == 0, "%s <> %s -- not divisors", to_string(mode.fps).c_str(),
+                        to_string(mode.fps).c_str());
+
+    mVsyncSchedule->getTracker().setDivisor(static_cast<unsigned>(divisor));
 }
 
 void Scheduler::resync() {