9d81ab73768db6647c116f4710f7a93d8d41e3b1 adds this:
+ if (blit_info->src.resource->format != blit_info->src.format ||
+ blit_info->dst.resource->format != blit_info->dst.format ||
+ !util_is_format_compatible(src_desc, dst_desc));
which looks wrong and does nothing if I read it right. Perhaps something else was intended?
Thanks. This looks like a leftover debug hack.
author Thomas Hellstrom <firstname.lastname@example.org>
svga: Fix a leftover debug hack
Fix what appears to be a leftover debug hack.
The hack would force the driver to take a different blit path; possibly,
although unverified, reverting to software blits.
Tested using piglit tests/quick. No related regressions.
Cc: "17.2 17.3 18.0" <email@example.com>
Fixes: 9d81ab7376 (svga: Relax the format checks for copy_region_vgpu10 somewhat)
Reported-by: Grazvydas Ignotas <firstname.lastname@example.org>
Signed-off-by: Thomas Hellstrom <email@example.com>
Reviewed-by: Brian Paul <firstname.lastname@example.org>