commit 16490fdbfbf667784ffa22247e4e6fb36eaa1052
author Ady Abraham <adyabr@google.com> Tue May 07 16:08:36 2024 -0700
committer Ady Abraham <adyabr@google.com> Tue May 07 16:08:49 2024 -0700
tree 951b4d4894b0064a6c174b78aa6e85c6ca6074ed
parent 4994e61d4378f7bba1e60a680aae872a6b651ce1

SF: do not wait for LastCompositeEventThread if it is not used

Bug: 333248925
Bug: 162235855
Test: SysUI Jank Regression: perfetto_cuj_systemui-SPLASHSCREEN_EXIT_ANIM-timeline_metrics-missed_sf_frames-mean
Change-Id: I0aa85f676c9fe789cbe31d88a529a9ec7aa72341

services/surfaceflinger/Scheduler/Scheduler.h

diff --git a/services/surfaceflinger/Scheduler/Scheduler.h b/services/surfaceflinger/Scheduler/Scheduler.h
index ccb3aa7..ccaa05f 100644
--- a/services/surfaceflinger/Scheduler/Scheduler.h
+++ b/services/surfaceflinger/Scheduler/Scheduler.h
@@ -346,7 +346,9 @@
     // Used to skip event dispatch before EventThread creation during boot.
     // TODO: b/241285191 - Reorder Scheduler initialization to avoid this.
     bool hasEventThreads() const {
-        return CC_LIKELY(mRenderEventThread && mLastCompositeEventThread);
+        return CC_LIKELY(
+                mRenderEventThread &&
+                (FlagManager::getInstance().deprecate_vsync_sf() || mLastCompositeEventThread));
     }
 
     EventThread& eventThreadFor(Cycle cycle) const {