[PATCH] EXA: fix mixed-pixmaps issue where we could do accel with pixmap mapped

Michel Dänzer michel at daenzer.net
Tue Oct 27 03:00:31 PDT 2009


On Mon, 2009-10-26 at 19:22 +0100, Maarten Maathuis wrote: 
> On Mon, Oct 26, 2009 at 6:53 PM, Keith Packard <keithp at keithp.com> wrote:
> > Excerpts from skeggsb's message of Fri Oct 23 17:45:49 -0700 2009:
> >
> >> -    if (pExaPixmap->pDamage && exaPixmapIsOffscreen(pPixmap)) {
> >> +    if (pExaPixmap->pDamage && exaPixmapIsOffscreen(pPixmap) &&
> >> +    !pExaPixmap->offscreen) {
> >
> > This doesn't apply on master which has two cases inside the condition.
> >
> 
> Both situations involve pixmap copying, something which is not
> desirable if prepare access succeeded.

True, however master might need more changes due to the deferred upload
mechanism added there, e.g. in exaDoMigration_mixed(). I'm afraid I
don't have time right now to think this through properly.

Anyway, Ben's change should be good as is for server-1.7-branch, and any
further changes for master won't apply there anyway, so maybe it can be
fast-tracked to server-1.7-nominations.


-- 
Earthling Michel Dänzer           |                http://www.vmware.com
Libre software enthusiast         |          Debian, X and DRI developer


More information about the xorg-devel mailing list