From: Kenneth Graunke Date: Thu, 29 Sep 2016 18:53:17 +0000 (-0700) Subject: anv: Clamp depth buffer dimensions to be at least 1. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=dcca706b4e2d2086b90d77cced0cff6ce4eac953;p=mesa.git anv: Clamp depth buffer dimensions to be at least 1. When there are no framebuffer attachments, fb->width and fb->height will be 0. Subtracting 1 results in 4294967295 which is too large for the field, causing genxml assertions when trying to create the packet. In this case, we can just program it to 1. Caught by dEQP-VK.tessellation.tesscoord.triangles_equal_spacing. Signed-off-by: Kenneth Graunke Reviewed-by: Jason Ekstrand --- diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c index 6131cfb60e3..0c638d6b02d 100644 --- a/src/intel/vulkan/genX_cmd_buffer.c +++ b/src/intel/vulkan/genX_cmd_buffer.c @@ -2159,8 +2159,8 @@ cmd_buffer_emit_depth_stencil(struct anv_cmd_buffer *cmd_buffer) db.SurfaceType = SURFTYPE_2D; } db.SurfaceFormat = D32_FLOAT; - db.Width = fb->width - 1; - db.Height = fb->height - 1; + db.Width = MAX2(fb->width, 1) - 1; + db.Height = MAX2(fb->height, 1) - 1; db.StencilWriteEnable = has_stencil; } }