Merge "Disable hardware vsync when blanking the screen" into klp-dev (24742375) · Commits · e / os / platform_frameworks_native

services/surfaceflinger/SurfaceFlinger.cpp

+19 −6

Original line number	Diff line number	Diff line
		@@ -147,6 +147,7 @@ SurfaceFlinger::SurfaceFlinger()
		mLastTransactionTime(0),
		mBootFinished(false),
		mPrimaryHWVsyncEnabled(false),
		mHWVsyncAvailable(false),
		mDaltonize(false)
		{
		ALOGI("SurfaceFlinger is starting");
		@@ -752,16 +753,23 @@ void SurfaceFlinger::run() {

		void SurfaceFlinger::enableHardwareVsync() {
		Mutex::Autolock _l(mHWVsyncLock);
		if (!mPrimaryHWVsyncEnabled) {
		if (!mPrimaryHWVsyncEnabled && mHWVsyncAvailable) {
		mPrimaryDispSync.beginResync();
		eventControl(HWC_DISPLAY_PRIMARY, SurfaceFlinger::EVENT_VSYNC, true);
		mPrimaryHWVsyncEnabled = true;
		}
		}

		void SurfaceFlinger::resyncToHardwareVsync() {
		void SurfaceFlinger::resyncToHardwareVsync(bool makeAvailable) {
		Mutex::Autolock _l(mHWVsyncLock);

		if (makeAvailable) {
		mHWVsyncAvailable = true;
		} else if (!mHWVsyncAvailable) {
		ALOGE("resyncToHardwareVsync called when HW vsync unavailable");
		return;
		}

		const nsecs_t period =
		getHwComposer().getRefreshPeriod(HWC_DISPLAY_PRIMARY);

		@@ -775,13 +783,16 @@ void SurfaceFlinger::resyncToHardwareVsync() {
		}
		}

		void SurfaceFlinger::disableHardwareVsync() {
		void SurfaceFlinger::disableHardwareVsync(bool makeUnavailable) {
		Mutex::Autolock _l(mHWVsyncLock);
		if (mPrimaryHWVsyncEnabled) {
		eventControl(HWC_DISPLAY_PRIMARY, SurfaceFlinger::EVENT_VSYNC, false);
		mPrimaryDispSync.endResync();
		mPrimaryHWVsyncEnabled = false;
		}
		if (makeUnavailable) {
		mHWVsyncAvailable = false;
		}
		}

		void SurfaceFlinger::onVSyncReceived(int type, nsecs_t timestamp) {
		@@ -791,7 +802,7 @@ void SurfaceFlinger::onVSyncReceived(int type, nsecs_t timestamp) {
		if (needsHwVsync) {
		enableHardwareVsync();
		} else {
		disableHardwareVsync();
		disableHardwareVsync(false);
		}
		}
		}
		@@ -933,7 +944,7 @@ void SurfaceFlinger::postComposition()
		if (mPrimaryDispSync.addPresentFence(presentFence)) {
		enableHardwareVsync();
		} else {
		disableHardwareVsync();
		disableHardwareVsync(false);
		}
		}

		@@ -2178,7 +2189,7 @@ void SurfaceFlinger::onScreenAcquired(const sp<const DisplayDevice>& hw) {
		// FIXME: eventthread only knows about the main display right now
		mEventThread->onScreenAcquired();

		resyncToHardwareVsync();
		resyncToHardwareVsync(true);
		}
		}
		mVisibleRegionsDirty = true;
		@@ -2196,6 +2207,8 @@ void SurfaceFlinger::onScreenReleased(const sp<const DisplayDevice>& hw) {
		int32_t type = hw->getDisplayType();
		if (type < DisplayDevice::NUM_BUILTIN_DISPLAY_TYPES) {
		if (type == DisplayDevice::DISPLAY_PRIMARY) {
		disableHardwareVsync(true); // also cancels any in-progress resync

		// FIXME: eventthread only knows about the main display right now
		mEventThread->onScreenReleased();
		}

services/surfaceflinger/SurfaceFlinger.h

+3 −2

Original line number	Diff line number	Diff line
		@@ -383,8 +383,8 @@ private:
		* VSync
		*/
		void enableHardwareVsync();
		void disableHardwareVsync();
		void resyncToHardwareVsync();
		void disableHardwareVsync(bool makeUnavailable);
		void resyncToHardwareVsync(bool makeAvailable);

		/* ------------------------------------------------------------------------
		* Debugging & dumpsys
		@@ -467,6 +467,7 @@ private:
		// protected by mHWVsyncLock
		Mutex mHWVsyncLock;
		bool mPrimaryHWVsyncEnabled;
		bool mHWVsyncAvailable;

		/* ------------------------------------------------------------------------
		* Feature prototyping