[PATCH xf86-video-intel 2/2] Fix handling of target_msc, divisor, remainder in I830DRI2ScheduleSwap()

[Mario Kleiner]

On Mar 7, 2010, at 6:18 PM, Jesse Barnes wrote:
>
> Arg, I did botch that patch.  And of course I only tested the swap
> buffers behavior and not OML's WaitMSC so I didn't catch it.  I'll
> improve the test and push the fix.


No problem. Just fyi: I noticed you added a test in  
I830DRI2ScheduleSwap() that outputs X_Warnings if a swap is scheduled  
more than 100 vblanks ahead. At least for the users of my toolkit,  
scheduling a swap more than 100 vblanks ahead wouldn't be something  
noteworthy but quite a typical case of normal use of the system. At  
the end of a work day some users would probably find ten thousands of  
such warnings in some log, assuming those warnings get logged at the  
normal log level. So this usage case is less weird than one would  
think :-)

-mario