On Mon, Mar 6, 2017 at 11:16 PM, Iago Toral Quiroga <[email protected]>
wrote:
> ---
> src/intel/vulkan/genX_cmd_buffer.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c
> b/src/intel/vulkan/genX_cmd_buffer.c
> index 2c9aec8..8b752c4 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -640,6 +640,9 @@ genX(CmdExecuteCommands)(
>
> assert(primary->level == VK_COMMAND_BUFFER_LEVEL_PRIMARY);
>
> + if (primary->error_status != VK_SUCCESS)
> + return;
> +
> /* The secondary command buffers will assume that the PMA fix is
> disabled
> * when they begin executing. Make sure this is true.
> */
> @@ -648,6 +651,9 @@ genX(CmdExecuteCommands)(
> for (uint32_t i = 0; i < commandBufferCount; i++) {
> ANV_FROM_HANDLE(anv_cmd_buffer, secondary, pCmdBuffers[i]);
>
> + if (secondary->error_status != VK_SUCCESS)
> + return;
>
I think this can just be an assert. They needed to call EndCommandBuffer
on the secondary and would have gotten the error there.
Maybe we should also add an assert to QueueSubmit?
> +
> assert(secondary->level == VK_COMMAND_BUFFER_LEVEL_SECONDARY);
>
> if (secondary->usage_flags &
> --
> 2.7.4
>
> _______________________________________________
> mesa-dev mailing list
> [email protected]
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
