xf86-video-intel: src/i830_dri.c

[Jesse Barnes]

 src/i830_dri.c |   23 ++++++++++-------------
 1 file changed, 10 insertions(+), 13 deletions(-)

New commits:
commit b71ca26a026a356763c51c763bcdd8024fc4a783
Author: Jesse Barnes <jbarnes at virtuousgeek.org>
Date:   Mon Mar 8 11:27:42 2010 -0800

    DRI2: fixup ScheduleWaitMSC similarly to ScheduleSwap
    
    My merge of Mario's patch for this was botched.  Fix it up so that OML
    waits work correctly, and remove a bogus warning from ScheduleSwap.
    
    Reported-by: Mario Kleiner <mario.kleiner at tuebingen.mpg.de>
    Signed-off-by: Jesse Barnes <jbarnes at virtuousgeek.org>

diff --git a/src/i830_dri.c b/src/i830_dri.c
index 96d41e7..88e9d90 100644
--- a/src/i830_dri.c
+++ b/src/i830_dri.c
@@ -689,13 +689,6 @@ I830DRI2ScheduleSwap(ClientPtr client, DrawablePtr draw, DRI2BufferPtr front,
 	if (*target_msc > 0)
 		*target_msc -= flip;
 
-	if ((*target_msc != 1) && (*target_msc > vbl.reply.sequence) &&
-	    ((*target_msc - vbl.reply.sequence) > 100))
-	    xf86DrvMsg(scrn->scrnIndex, X_WARNING,
-		       "vblank event >100 frames away: cur %ld, target %ld\n",
-		       (unsigned long)vbl.reply.sequence,
-		       (unsigned long)*target_msc);
-
 	/*
 	 * If divisor is zero, or current_msc is smaller than target_msc
 	 * we just need to make sure target_msc passes before initiating
@@ -891,7 +884,7 @@ I830DRI2ScheduleWaitMSC(ClientPtr client, DrawablePtr draw, CARD64 target_msc,
 	 * we just need to make sure target_msc passes  before waking up the
 	 * client.
 	 */
-	if (divisor == 0) {
+	if (divisor == 0 || current_msc < target_msc) {
 		vbl.request.type = DRM_VBLANK_ABSOLUTE | DRM_VBLANK_EVENT;
 		if (pipe > 0)
 			vbl.request.type |= DRM_VBLANK_SECONDARY;
@@ -917,13 +910,17 @@ I830DRI2ScheduleWaitMSC(ClientPtr client, DrawablePtr draw, CARD64 target_msc,
 	if (pipe > 0)
 		vbl.request.type |= DRM_VBLANK_SECONDARY;
 
+	vbl.request.sequence = current_msc - (current_msc % divisor) +
+	    remainder;
+
 	/*
-	 * If the calculated  remainder and the condition isn't satisified, it
-	 * means we've passed the last point where seq % divisor == remainder,
-	 * so we need to wait for the next time that will happen.
+	 * If calculated remainder is larger than requested remainder,
+	 * it means we've passed the last point where
+	 * seq % divisor == remainder, so we need to wait for the next time
+	 * that will happen.
 	 */
-	if ((current_msc % divisor) != remainder)
-		vbl.request.sequence += divisor;
+	if ((current_msc % divisor) > remainder)
+	    vbl.request.sequence += divisor;
 
 	vbl.request.signal = (unsigned long)wait_info;
 	ret = drmWaitVBlank(intel->drmSubFD, &vbl);