[Mesa-dev] [PATCH 02/14] anv/cmd_buffer: Take bo_offset into account in fast clear state addresses

[Lionel Landwerlin]

Oh dear...

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 13/11/17 16:12, Jason Ekstrand wrote:
> Otherwise, if the image is not bound to the start of the buffer, we're
> going to be reading and writing its fast clear state in the wrong spot.
>
> Cc: mesa-stable at lists.freedesktop.org
> ---
>   src/intel/vulkan/genX_cmd_buffer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c
> index 2564976..9eb6074 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -461,7 +461,7 @@ get_fast_clear_state_address(const struct anv_device *device,
>   
>      return (struct anv_address) {
>         .bo = image->planes[plane].bo,
> -      .offset = offset,
> +      .offset = image->planes[plane].bo_offset + offset,
>      };
>   }
>