Mesa (master): tu: Fix empty blit scissor case
GitLab Mirror
gitlab-mirror at kemper.freedesktop.org
Fri Jul 24 18:58:59 UTC 2020
Module: Mesa
Branch: master
Commit: 7ad962bf89f4f310a7dfb3dd1416c9dd24b31f43
URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=7ad962bf89f4f310a7dfb3dd1416c9dd24b31f43
Author: Connor Abbott <cwabbott0 at gmail.com>
Date: Tue Jul 21 14:36:53 2020 +0200
tu: Fix empty blit scissor case
With vertexPipelineStoresAndAtomics enabled, fixes:
dEQP-VK.tessellation.invariance.one_minus_tess_coord_component.quads_fractional_even_spacing_cw_point_mode
dEQP-VK.tessellation.invariance.tess_coord_component_range.triangles_fractional_even_spacing_ccw_point_mode
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5738>
---
src/freedreno/vulkan/tu_cmd_buffer.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)
diff --git a/src/freedreno/vulkan/tu_cmd_buffer.c b/src/freedreno/vulkan/tu_cmd_buffer.c
index d786a4589a9..03266c2287e 100644
--- a/src/freedreno/vulkan/tu_cmd_buffer.c
+++ b/src/freedreno/vulkan/tu_cmd_buffer.c
@@ -433,7 +433,19 @@ tu6_emit_render_cntl(struct tu_cmd_buffer *cmd,
static void
tu6_emit_blit_scissor(struct tu_cmd_buffer *cmd, struct tu_cs *cs, bool align)
{
+
const VkRect2D *render_area = &cmd->state.render_area;
+
+ /* Avoid assertion fails with an empty render area at (0, 0) where the
+ * subtraction below wraps around. Empty render areas should be forced to
+ * the sysmem path by use_sysmem_rendering(). It's not even clear whether
+ * an empty scissor here works, and the blob seems to force sysmem too as
+ * it sets something wrong (non-empty) for the scissor.
+ */
+ if (render_area->extent.width == 0 ||
+ render_area->extent.height == 0)
+ return;
+
uint32_t x1 = render_area->offset.x;
uint32_t y1 = render_area->offset.y;
uint32_t x2 = x1 + render_area->extent.width - 1;
@@ -571,6 +583,11 @@ use_sysmem_rendering(struct tu_cmd_buffer *cmd)
if (cmd->state.framebuffer->layers > 1)
return true;
+ /* Use sysmem for empty render areas */
+ if (cmd->state.render_area.extent.width == 0 ||
+ cmd->state.render_area.extent.height == 0)
+ return true;
+
if (cmd->has_tess)
return true;
More information about the mesa-commit
mailing list