commit 0d5c60edf11fdded738acf3f78a0c667599cd60d
author Jamie Gennis <jgennis@google.com> Wed Oct 09 17:49:37 2013 -0700
committer Jamie Gennis <jgennis@google.com> Wed Oct 09 17:56:42 2013 -0700
tree 1505be73365a9d0a8bca96e682f33a08020afcf7
parent 98c197d826f95a82de41e16350f58a70a7ad23c6

DispSync: don't compensate for wakeup latency

This change removes the wakeup latency compensation from the software-generated
vsync events.  Choreographer can't handle timestamps in the future, so don't
aim for early wake-ups with the expectation that the actual wake-up will be
late.

Bug: 11153576

services/surfaceflinger/DispSync.cpp

diff --git a/services/surfaceflinger/DispSync.cpp b/services/surfaceflinger/DispSync.cpp
index 7e67138..bff304e 100644
--- a/services/surfaceflinger/DispSync.cpp
+++ b/services/surfaceflinger/DispSync.cpp
@@ -110,7 +110,7 @@
                 }
 
                 nextEventTime = computeNextEventTimeLocked(now);
-                targetTime = nextEventTime - mWakeupLatency;
+                targetTime = nextEventTime;
 
                 bool isWakeup = false;
 
@@ -228,7 +228,7 @@
             nsecs_t t = computeListenerNextEventTimeLocked(mEventListeners[i],
                     ref);
 
-            if (t - mWakeupLatency < now) {
+            if (t < now) {
                 CallbackInvocation ci;
                 ci.mCallback = mEventListeners[i].mCallback;
                 ci.mEventTime = t;