[Mesa-dev] [PATCH 23/25] anv: Set framebuffer to NULL in secondary command buffers

[Nanley Chery]

On Sat, Oct 22, 2016 at 10:50:54AM -0700, Jason Ekstrand wrote:
> Nothing that is allowed to be called within a secondary now relies on the
> framebuffer.
> ---
>  src/intel/vulkan/genX_cmd_buffer.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)

This patch is,
Reviewed-by: Nanley Chery <nanley.g.chery at intel.com>

> 
> diff --git a/src/intel/vulkan/genX_cmd_buffer.c b/src/intel/vulkan/genX_cmd_buffer.c
> index 78b9bcc..f43c643 100644
> --- a/src/intel/vulkan/genX_cmd_buffer.c
> +++ b/src/intel/vulkan/genX_cmd_buffer.c
> @@ -319,12 +319,11 @@ genX(BeginCommandBuffer)(
>  
>     if (cmd_buffer->usage_flags &
>         VK_COMMAND_BUFFER_USAGE_RENDER_PASS_CONTINUE_BIT) {
> -      cmd_buffer->state.framebuffer =
> -         anv_framebuffer_from_handle(pBeginInfo->pInheritanceInfo->framebuffer);
>        cmd_buffer->state.pass =
>           anv_render_pass_from_handle(pBeginInfo->pInheritanceInfo->renderPass);
>        cmd_buffer->state.subpass =
>           &cmd_buffer->state.pass->subpasses[pBeginInfo->pInheritanceInfo->subpass];
> +      cmd_buffer->state.framebuffer = NULL;
>  
>        genX(cmd_buffer_setup_attachments)(cmd_buffer, cmd_buffer->state.pass,
>                                           NULL, NULL);
> -- 
> 2.5.0.400.gff86faf
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev