On Tue, Nov 04, 2008 at 06:46:09PM -0800, Eric Anholt wrote:
> When userland detected that this ioctl was supported (by version number
> check),
> it used it in a racy way -- dispatch delayed swap, wait for vblank, continue
> rendering. As there was no mechanism for it to wait for the swap to finish,
> sometimes it would render before the swap and garbage would be displayed on
> the screen.
>
> By removing the ioctl and returning -EINVAL, userland returns to its previous,
> correct rendering path of waiting for a vblank then dispatching a swap. The
> only path that could have used this ioctl correctly was page flipping, which
> relied on only one client running and emitting
> wait-for-vblank-before-rendering
> in the command stream. That path also falls back correctly, at the
> performance
> cost of not being able to queue up rendering before the flip occurs.
Much better!
This and the tasklet removal diff look pretty much the same as I came up
with on ``that other os''. All fine by me.
--
Larkinson's Law:
All laws are basically false.
-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/
--
_______________________________________________
Dri-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-devel
