[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