[PATCH xserver 2/3] glamor: Accelerate up XY pixmap putimage a little
Keith Packard
keithp at keithp.com
Mon Oct 3 19:26:20 UTC 2016
Adam Jackson <ajax at redhat.com> writes:
> On Fri, 2016-09-30 at 22:44 -0700, Keith Packard wrote:
>
>> + /* copy mode for the first plane to clear all of the other bits */
>> + gcv[0].val = GXcopy;
>> + gcv[1].val = gc->fgPixel;
>> + gcv[2].val = gc->bgPixel;
>> + ChangeGC(NullClient, temp_gc, GCFunction|GCForeground|GCBackground, gcv);
>
> You don't need to set GCFunction here, the scratch gc already has
> GXcopy set.
Thanks.
> You _do_ need to copy GCPlanemask from the source gc, because for
> XYPixmap PutImage that determines how many planes of data were given;
> if you let it stay as FB_ALLONES when it should only be, say, 0x3,
> you'll read in 30 planes of garbage.
mi says no, fb says yes. I need to read the Xlib code (and the spec).
--
-keith
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 800 bytes
Desc: not available
URL: <https://lists.x.org/archives/xorg-devel/attachments/20161003/be0272f5/attachment.sig>
More information about the xorg-devel
mailing list