summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorNanley Chery <[email protected]>2016-09-30 10:28:18 -0700
committerNanley Chery <[email protected]>2016-11-18 12:12:34 -0800
commit6ff4c24fdd3ea078da5eba7c71fcbdf4c481d440 (patch)
tree12a6980bbad043dc6fc3bfde0b1f9a7c99816189 /src
parent63318d34acd4a5edb271d57adf3b01e2e52552f8 (diff)
anv/cmd_buffer: Clarify HZ rectangle behavior
This behavior differs from what's described in the PRMs and was observed by analyzing CTS test results. Signed-off-by: Nanley Chery <[email protected]> Reviewed-by: Jason Ekstrand <[email protected]>
Diffstat (limited to 'src')
-rw-r--r--src/intel/vulkan/gen8_cmd_buffer.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/src/intel/vulkan/gen8_cmd_buffer.c b/src/intel/vulkan/gen8_cmd_buffer.c
index f1dfe7b8203..0b1d5bfe7cd 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 +