[Mesa-dev] [PATCH] anv/batch_chain: Handle another OOM in cmd_buffer_execbuf

[Pohjolainen, Topi]

On Tue, Mar 28, 2017 at 05:33:37PM -0700, Jason Ekstrand wrote:
> Found by inspection while rebasing other patches.
> ---
>  src/intel/vulkan/anv_batch_chain.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)

Reviewed-by: Topi Pohjolainen <topi.pohjolainen at intel.com>

> 
> diff --git a/src/intel/vulkan/anv_batch_chain.c b/src/intel/vulkan/anv_batch_chain.c
> index 5d7abc6..5a6c0ba 100644
> --- a/src/intel/vulkan/anv_batch_chain.c
> +++ b/src/intel/vulkan/anv_batch_chain.c
> @@ -1277,8 +1277,10 @@ anv_cmd_buffer_execbuf(struct anv_device *device,
>        adjust_relocations_to_state_pool(ss_pool, &(*bbo)->bo, &(*bbo)->relocs,
>                                         cmd_buffer->last_ss_pool_center);
>  
> -      anv_execbuf_add_bo(&execbuf, &(*bbo)->bo, &(*bbo)->relocs,
> -                         &cmd_buffer->pool->alloc);
> +      result = anv_execbuf_add_bo(&execbuf, &(*bbo)->bo, &(*bbo)->relocs,
> +                                  &cmd_buffer->pool->alloc);
> +      if (result != VK_SUCCESS)
> +         return result;
>     }
>  
>     /* Now that we've adjusted all of the surface state relocations, we need to
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev