commit 6a9731d3ed5c5bd1cce64db28e2bc6ada25ebc85
author Rachel Lee <rnlee@google.com> Mon Jun 06 17:08:14 2022 -0700
committer Rachel Lee <rnlee@google.com> Tue Jun 07 21:05:23 2022 +0000
tree 3b782666b01764c9b39a84b90863adc205deeb78
parent 23a9ec0bb371a98db20a961c61cb8eb80b9fd71f

HW Vsync turns off correctly in Doze/AOD.

Otherwise when entering AOD mode, SF keeps HW Vsync on indefinitely,
leading to power inefficiency.

Bug: 219109873
Test: atest libsurfaceflinger_test
Test: perfetto trace
Change-Id: Idf80d4c64abe78593809be9d1ba6b7dd42d29054

services/surfaceflinger/SurfaceFlinger.cpp

diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp
index 5cf9558..906cf73 100644
--- a/services/surfaceflinger/SurfaceFlinger.cpp
+++ b/services/surfaceflinger/SurfaceFlinger.cpp
@@ -4947,7 +4947,7 @@
     if (isDisplayActiveLocked(display)) {
         mTimeStats->setPowerMode(mode);
         mRefreshRateStats->setPowerMode(mode);
-        mScheduler->setDisplayPowerState(mode == hal::PowerMode::ON);
+        mScheduler->setDisplayPowerMode(mode);
     }
 
     ALOGD("Finished setting power mode %d on display %s", mode, to_string(displayId).c_str());