Improvements to Intel DDX for implementation of DRI2 OML_sync_control extensions.

Mario Kleiner mario.kleiner at tuebingen.mpg.de
Thu Mar 4 10:05:39 PST 2010


Is there a simple git way to switch to that tree, or would i need to  
checkout the whole tree again and then copy & paste chunks of code  
from the old tree to the new one manually?

-mario

On Mar 4, 2010, at 4:56 PM, Jesse Barnes wrote:

> Oh sorry, that tree is out of date since I pushed everything to
> master.  The tree to generate fixes against is
> git://git.freedesktop.org/git/xorg/driver/xf86-video-intel.
>
> Jesse
>
> On Thu, 4 Mar 2010 05:24:38 +0100
> Mario Kleiner <mario.kleiner at tuebingen.mpg.de> wrote:
>
>> They are against Jesse Barnes xf86-video-intel tree:
>>
>> <http://cgit.freedesktop.org/~jbarnes/xf86-video-intel/log/>
>> git://people.freedesktop.org/~jbarnes/xf86-video-intel master branch
>>
>> The tree is in the same state as when i wrote the patches - they are
>> following commit
>>
>> <http://cgit.freedesktop.org/~jbarnes/xf86-video-intel/commit/?
>> id=dd1930673565e70ec247f7f96ca0bdc575eecfb0>
>>
>> Ideas to fix? I'm a git newbie.
>>
>> thanks,
>> -mario
>>
>> On Mar 4, 2010, at 1:28 AM, Eric Anholt wrote:
>>
>>> On Sun, 21 Feb 2010 18:45:46 +0100, Mario Kleiner
>>> <mario.kleiner at tuebingen.mpg.de> wrote:
>>>> This series of 2 patches fixes issues in the Intel DDX
>>>> implementation of I830DRI2ScheduleWaitMSC() and
>>>> I830DRI2ScheduleSwap().
>>>>
>>>> The previous implementation should mostly work in the simple
>>>> glXSwapBuffers() case, although it may reduce the maximum
>>>> swaprate to half the video refresh interval, e.g., max 30 fps
>>>> at a refresh of 60 Hz. It won't work correctly for the
>>>> glXSwapBuffersMscOML(target_msc, divisor, remainder) case if
>>>> divisor or remainder are non-zero or target_msc is not > 2
>>>> frames ahead of the current vblank count. It doesn't handle
>>>> the (msc % divisor) == remainder case correctly.
>>>>
>>>> A similar problem exists for calls to glXWaitForMscOML().
>>>>
>>>> These patches should fix above issues, or at least make
>>>> handling more robust.
>>>>
>>>> Jesse Barnes already had a look at these patches, but due to
>>>> lack of suitable hardware, i haven't tested them myself, not
>>>> even if they compile cleanly.
>>>>
>>>> Could somebody please give them some good testing and merge
>>>> them if appropriate?
>>>
>>> These don't seem to apply, and I can't quite work out what code
>>> they're
>>> against
>>
>> *********************************************************************
>> Mario Kleiner
>> Max Planck Institute for Biological Cybernetics
>> Spemannstr. 38
>> 72076 Tuebingen
>> Germany
>>
>> e-mail: mario.kleiner at tuebingen.mpg.de
>> office: +49 (0)7071/601-1623
>> fax:    +49 (0)7071/601-616
>> www:    http://www.kyb.tuebingen.mpg.de/~kleinerm
>> *********************************************************************
>> "For a successful technology, reality must take precedence
>> over public relations, for Nature cannot be fooled."
>> (Richard Feynman)
>>
>>
>
>
> -- 
> Jesse Barnes, Intel Open Source Technology Center

*********************************************************************
Mario Kleiner
Max Planck Institute for Biological Cybernetics
Spemannstr. 38
72076 Tuebingen
Germany

e-mail: mario.kleiner at tuebingen.mpg.de
office: +49 (0)7071/601-1623
fax:    +49 (0)7071/601-616
www:    http://www.kyb.tuebingen.mpg.de/~kleinerm
*********************************************************************
"For a successful technology, reality must take precedence
over public relations, for Nature cannot be fooled."
(Richard Feynman)



More information about the xorg-devel mailing list