From ecf2e0f5cf0dcb2792bd4582f1c8e1e1ab2a365d Mon Sep 17 00:00:00 2001 From: Alec Mouri Date: Tue, 26 Mar 2019 13:26:17 -0700 Subject: Remove app vsync pausing Bug: 128849305 Change-Id: I97c7ab9a2555f229783cd1c5352d883a627ab193 Test: jank tests Test: systrace --- services/surfaceflinger/SurfaceFlinger.cpp | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) (limited to 'services/surfaceflinger/SurfaceFlinger.cpp') diff --git a/services/surfaceflinger/SurfaceFlinger.cpp b/services/surfaceflinger/SurfaceFlinger.cpp index bd3f156b82..6e05a19ad7 100644 --- a/services/surfaceflinger/SurfaceFlinger.cpp +++ b/services/surfaceflinger/SurfaceFlinger.cpp @@ -922,9 +922,6 @@ void SurfaceFlinger::setDesiredActiveConfig(const ActiveConfigInfo& info) { mDesiredActiveConfig.event = mDesiredActiveConfig.event | prevConfig; if (!mDesiredActiveConfigChanged) { - // This is the first time we set the desired - mScheduler->pauseVsyncCallback(mAppConnectionHandle, true); - // This will trigger HWC refresh without resetting the idle timer. repaintEverythingForHWC(); } @@ -1000,8 +997,6 @@ bool SurfaceFlinger::performSetActiveConfig() { // display is not valid or we are already in the requested mode // on both cases there is nothing left to do std::lock_guard lock(mActiveConfigLock); - mScheduler->pauseVsyncCallback(mAppConnectionHandle, false); - mDesiredActiveConfig.event = Scheduler::ConfigEvent::None; mDesiredActiveConfigChanged = false; ATRACE_INT("DesiredActiveConfigChanged", mDesiredActiveConfigChanged); return false; @@ -1012,6 +1007,7 @@ bool SurfaceFlinger::performSetActiveConfig() { // Make sure the desired config is still allowed if (!isDisplayConfigAllowed(desiredActiveConfig.configId)) { std::lock_guard lock(mActiveConfigLock); + mDesiredActiveConfig.event = Scheduler::ConfigEvent::None; mDesiredActiveConfig.configId = display->getActiveConfig(); return false; } -- cgit v1.2.3-59-g8ed1b