svga: Fix a leftover debug hack
authorThomas Hellstrom <thellstrom@vmware.com>
Mon, 15 Jan 2018 11:51:27 +0000 (12:51 +0100)
committerThomas Hellstrom <thellstrom@vmware.com>
Tue, 20 Feb 2018 09:12:19 +0000 (10:12 +0100)
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" <mesa-stable@lists.freedesktop.org>
Fixes: 9d81ab7376 (svga: Relax the format checks for copy_region_vgpu10 somewhat)
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=104625
Reported-by: Grazvydas Ignotas <notasas@gmail.com>
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
src/gallium/drivers/svga/svga_pipe_blit.c

index e98113c38543c7d429b02cdca6a56adceaf19906..4b65a69a9e23a2fac0beadab29b1640d2be969ed 100644 (file)
@@ -351,8 +351,8 @@ can_blit_via_surface_copy(struct svga_context *svga,
 
       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));
-      return FALSE;
+          !util_is_format_compatible(src_desc, dst_desc))
+         return false;
    }
 
    if (svga->render_condition && blit_info->render_condition_enable)