HWC2: Skip validate/present if display is off (de793924) · Commits · e / os / platform_frameworks_native

services/surfaceflinger/SurfaceFlinger.cpp

+15 −2

Original line number	Diff line number	Diff line
		@@ -1022,7 +1022,12 @@ void SurfaceFlinger::doDebugFlashRegions()
		}

		for (size_t displayId = 0; displayId < mDisplays.size(); ++displayId) {
		status_t result = mDisplays[displayId]->prepareFrame(*mHwc);
		auto& displayDevice = mDisplays[displayId];
		if (!displayDevice->isDisplayOn()) {
		continue;
		}

		status_t result = displayDevice->prepareFrame(*mHwc);
		ALOGE_IF(result != NO_ERROR, "prepareFrame for display %zd failed:"
		" %d (%s)", displayId, result, strerror(-result));
		}
		@@ -1248,7 +1253,12 @@ void SurfaceFlinger::setUpHWComposer() {
		}

		for (size_t displayId = 0; displayId < mDisplays.size(); ++displayId) {
		status_t result = mDisplays[displayId]->prepareFrame(*mHwc);
		auto& displayDevice = mDisplays[displayId];
		if (!displayDevice->isDisplayOn()) {
		continue;
		}

		status_t result = displayDevice->prepareFrame(*mHwc);
		ALOGE_IF(result != NO_ERROR, "prepareFrame for display %zd failed:"
		" %d (%s)", displayId, result, strerror(-result));
		}
		@@ -1286,6 +1296,9 @@ void SurfaceFlinger::postFramebuffer()

		for (size_t displayId = 0; displayId < mDisplays.size(); ++displayId) {
		auto& displayDevice = mDisplays[displayId];
		if (!displayDevice->isDisplayOn()) {
		continue;
		}
		const auto hwcId = displayDevice->getHwcDisplayId();
		if (hwcId >= 0) {
		mHwc->commit(hwcId);