[Mesa-dev] [PATCH v2 1/3] anv/cmd_buffer: Clarify HZ rectangle behavior

Nanley Chery nanleychery at gmail.com
Wed Oct 19 17:47:15 UTC 2016


This behavior differs from what's described in the PRMs and was
observed by analyzing CTS test results.

Signed-off-by: Nanley Chery <nanley.g.chery at intel.com>
---
 src/intel/vulkan/gen8_cmd_buffer.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/intel/vulkan/gen8_cmd_buffer.c b/src/intel/vulkan/gen8_cmd_buffer.c
index 0548a5e..930b99e 100644
--- a/src/intel/vulkan/gen8_cmd_buffer.c
+++ b/src/intel/vulkan/gen8_cmd_buffer.c
@@ -481,9 +481,10 @@ genX(cmd_buffer_emit_hz_op)(struct anv_cmd_buffer *cmd_buffer,
          hzp.ClearRectangleXMax = align_u32(iview->extent.width, 8);
          hzp.ClearRectangleYMax = align_u32(iview->extent.height, 4);
       } else {
-         /* This clear rectangle is aligned */
+         /* Contrary to the HW docs both fields are inclusive */
          hzp.ClearRectangleXMin = cmd_state->render_area.offset.x;
          hzp.ClearRectangleYMin = cmd_state->render_area.offset.y;
+         /* Contrary to the HW docs both fields are exclusive */
          hzp.ClearRectangleXMax = cmd_state->render_area.offset.x +
             align_u32(cmd_state->render_area.extent.width, px_dim.width);
          hzp.ClearRectangleYMax = cmd_state->render_area.offset.y +
-- 
2.10.0



More information about the mesa-dev mailing list