Improvements to Intel DDX for implementation of DRI2 OML_sync_control extensions.

Jesse Barnes jbarnes at virtuousgeek.org
Thu Mar 4 10:09:01 PST 2010


Probably easiest to do the latter since I'm not sure if the current
master of xf86-video-intel is just a fast forward of my tree.

If it is though, you can just edit .git/config to point at the URL
below and git pull the latest bits.

Jesse

On Thu, 4 Mar 2010 19:05:39 +0100
Mario Kleiner <mario.kleiner at tuebingen.mpg.de> wrote:

> 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)
> 
> 


-- 
Jesse Barnes, Intel Open Source Technology Center


More information about the xorg-devel mailing list