Jason, this nominated patch landed without mentioning any specific
stable queue.
From what I'm seeing, it depends on 3735af04152b which didn't make it
for 17.2 so I'm dropping it for that queue, unless you want to provide
a backport.
Let me know what you think.
On Mon, 2017-11-13 at 08:12 -0800, Jason Ekstrand wrote:
> Found by inspection
>
> Cc: mesa-sta...@lists.freedesktop.org
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/src/intel/vulkan/genX_cmd_buffer.c
> index fbb5706..2564976 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -557,12 +557,13 @@ init_fast_clear_state_entry(struct anv_cmd_buffer
> *cmd_buffer,
> /* Other combinations of auxiliary buffers and platforms require specific
> * values in the clear value dword(s).
> */
> + struct anv_address addr =
> + get_fast_clear_state_address(cmd_buffer->device, image, aspect, level,
> + FAST_CLEAR_STATE_FIELD_CLEAR_COLOR);
> unsigned i = 0;
> for (; i < cmd_buffer->device->isl_dev.ss.clear_value_size; i += 4) {
> anv_batch_emit(&cmd_buffer->batch, GENX(MI_STORE_DATA_IMM), sdi) {
> - sdi.Address =
> - get_fast_clear_state_address(cmd_buffer->device, image, aspect,
> level,
> - FAST_CLEAR_STATE_FIELD_CLEAR_COLOR);
> + sdi.Address = addr;
>
> if (GEN_GEN >= 9) {
> /* MCS buffers on SKL+ can only have 1/0 clear colors. */
> @@ -586,6 +587,8 @@ init_fast_clear_state_entry(struct anv_cmd_buffer
> *cmd_buffer,
> sdi.ImmediateData = 0;
> }
> }
> +
> + addr += 4;
> }
> }
>
--
Br,
Andres
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
