[PATCH 1/4] EXA: Don't use UTS/DFS directly for Put/GetImage when there's a system copy.

Michel Dänzer michel at daenzer.net
Tue Dec 29 06:23:55 PST 2009


[ Adding Thomas to CC in case he wants to comment as well ]

On Tue, 2009-12-29 at 12:41 +0100, Maarten Maathuis wrote: 
> 2009/12/28 Michel Dänzer <michel at daenzer.net>:
> > On Mon, 2009-12-28 at 13:37 +0100, Maarten Maathuis wrote:
> >> What kind of situation are you trying to improve here?
> >
> > Same as for the UploadToScreen case in exaHWCopyNtoN(); avoiding GPU
> > copy readbacks when the GPU copy isn't directly accessible.
> 
> I wonder how many apps do CreatePixmap, PutImage, CopyArea,
> DestroyPixmap, because in that scenario i don't like the change (the
> UTS path CopyNtoN was pretty rare to be used).

Would this change affect that case at all? pExaPixmap->pDamage should be
NULL in exaDoPutImage.

Note that even in a similar scenario where this change does make a
difference, it only adds one memcpy from and to cacheable memory.


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


More information about the xorg-devel mailing list