diff options
| author | Nanley Chery <[email protected]> | 2016-09-30 10:28:18 -0700 |
|---|---|---|
| committer | Nanley Chery <[email protected]> | 2016-11-18 12:12:34 -0800 |
| commit | 6ff4c24fdd3ea078da5eba7c71fcbdf4c481d440 (patch) | |
| tree | 12a6980bbad043dc6fc3bfde0b1f9a7c99816189 /src/intel | |
| parent | 63318d34acd4a5edb271d57adf3b01e2e52552f8 (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/intel')
| -rw-r--r-- | src/intel/vulkan/gen8_cmd_buffer.c | 3 |
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 + |