[PATCH xserver 02/10 v2] glamor: Require that 16bpp pixmap depths match for Render copies.

Michel Dänzer michel at daenzer.net
Fri Sep 23 08:09:22 UTC 2016


On 23/09/16 04:57 PM, Eric Anholt wrote:
> The copy optimization in d37329cba42fa8e72fe4be8a7be18e512268b5bd
> replicated a bug from last time we did a copy optimization, and didn't
> get rendercheck run on it.

Actually, I'm pretty sure I did run rendercheck, but didn't notice the
regression due to the radeonsi bug affecting the same test. Please
consider removing that language from the commit log.


> This is effectively a re-cherry-pick of 510c8605641803f1f5b5d2de6d3bb422b148e0e7.
> 
> Fixes rendercheck -t blend -o src -f x4r4g4b4,x3r4g4b4
> 
> v2: Drop excessive src->depth == dst->depth check that snuck in.
> 
> Signed-off-by: Eric Anholt <eric at anholt.net>
> ---
> 
> Michel, I didn't apply your review becuase you said "second clause",
> but I removed the first of the two.

Is there any case where the drawable depths don't match, but a copy does
what's expected?


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer


More information about the xorg-devel mailing list