qdutils: Refactor idletimeout fallback mechanism.
1. Use kernel timer interrupt to signal about the timeout to
Invalidator thread and fallback to GPU composition.
2. This implementation avoids waking up the Invalidator thread for
every idletimeout value during continuous update.
Change-Id: I4370f10a0ca06b5cb1e7bdcb047e9c8daec51afb
diff --git a/libhwcomposer/hwc_mdpcomp.cpp b/libhwcomposer/hwc_mdpcomp.cpp
index 6ff1aa3..c773c91 100644
--- a/libhwcomposer/hwc_mdpcomp.cpp
+++ b/libhwcomposer/hwc_mdpcomp.cpp
@@ -38,6 +38,7 @@
IdleInvalidator *MDPComp::idleInvalidator = NULL;
bool MDPComp::sIdleFallBack = false;
+bool MDPComp::sHandleTimeout = false;
bool MDPComp::sDebugLogs = false;
bool MDPComp::sEnabled = false;
bool MDPComp::sEnableMixedMode = true;
@@ -171,7 +172,12 @@
ALOGE("%s: received empty data in timer callback", __FUNCTION__);
return;
}
-
+ Locker::Autolock _l(ctx->mDrawLock);
+ // Handle timeout event only if the previous composition is MDP or MIXED.
+ if(!sHandleTimeout) {
+ ALOGD_IF(isDebug(), "%s:Do not handle this timeout", __FUNCTION__);
+ return;
+ }
if(!ctx->proc) {
ALOGE("%s: HWC proc not registered", __FUNCTION__);
return;
@@ -1619,9 +1625,10 @@
return true;
}
- /* reset Invalidator */
- if(idleInvalidator && !sIdleFallBack && mCurrentFrame.mdpCount)
- idleInvalidator->handleUpdateEvent();
+ // Set the Handle timeout to true for MDP or MIXED composition.
+ if(idleInvalidator && !sIdleFallBack && mCurrentFrame.mdpCount) {
+ sHandleTimeout = true;
+ }
overlay::Overlay& ov = *ctx->mOverlay;
LayerProp *layerProp = ctx->layerProp[mDpy];
@@ -1865,9 +1872,10 @@
return true;
}
- /* reset Invalidator */
- if(idleInvalidator && !sIdleFallBack && mCurrentFrame.mdpCount)
- idleInvalidator->handleUpdateEvent();
+ // Set the Handle timeout to true for MDP or MIXED composition.
+ if(idleInvalidator && !sIdleFallBack && mCurrentFrame.mdpCount) {
+ sHandleTimeout = true;
+ }
overlay::Overlay& ov = *ctx->mOverlay;
LayerProp *layerProp = ctx->layerProp[mDpy];