commit ee6365b1d1704b1eedfb8f81e471b80692d8ec43
author Ady Abraham <adyabr@google.com> Wed Mar 06 14:31:45 2024 -0800
committer Ady Abraham <adyabr@google.com> Thu Mar 07 16:27:07 2024 -0800
tree 238403d2c271a705839fd795c4751e6532fca28a
parent 77b4fb1e91144f60dddc027612c237d79fe3776f

SF: flush setRenderRate when changing mode

There is no point to try to syncronize changing the render rate
if the display mode changes. This would just cause more latency
to getting the next vsync.

Bug: 328140524
Test: presubmit
Change-Id: I78a82bee01ba793890d459564ab701d257851b49

services/surfaceflinger/Scheduler/VSyncPredictor.h

diff --git a/services/surfaceflinger/Scheduler/VSyncPredictor.h b/services/surfaceflinger/Scheduler/VSyncPredictor.h
index a1e72d1..21039f1 100644
--- a/services/surfaceflinger/Scheduler/VSyncPredictor.h
+++ b/services/surfaceflinger/Scheduler/VSyncPredictor.h
@@ -68,7 +68,7 @@
 
     void setDisplayModePtr(ftl::NonNull<DisplayModePtr>) final EXCLUDES(mMutex);
 
-    void setRenderRate(Fps) final EXCLUDES(mMutex);
+    void setRenderRate(Fps, bool applyImmediately) final EXCLUDES(mMutex);
 
     void onFrameBegin(TimePoint expectedPresentTime, TimePoint lastConfirmedPresentTime) final
             EXCLUDES(mMutex);