[Mesa-dev] [Mesa-stable] [PATCH 01/14] anv/cmd_buffer: Advance the address when initializing clear colors

[Andres Gomez]

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-stable at 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