[PATCH] glamor: don't do render ops with matching source/dest (v2)

Dave Airlie airlied at gmail.com
Wed May 27 23:17:38 PDT 2015


On 28 May 2015 at 16:17, Dave Airlie <airlied at gmail.com> wrote:
> On 28 May 2015 at 16:09, Keith Packard <keithp at keithp.com> wrote:
>> Dave Airlie <airlied at gmail.com> writes:
>>
>>> From: Dave Airlie <airlied at redhat.com>
>>>
>>> XRender defines this, GL really doesn't like it.
>>>
>>> kwin 4.x and qt 4.x seem to make this happen for the
>>> gradient in the titlebar, and on radeonsi/r600 hw
>>> this draws all kinds of wrong.
>>>
>>> v2: bump this up a level, and check it earlier.
>>> (I assume the XXXX was for this case.)
>>
>> You're missing the matching case in the large_pixmap code
>
> The large pixmap code eventually calls this code, once you go crazy.
>

Though it asserts on failure, so yes I probably do need that :)

Dave.


More information about the xorg-devel mailing list