[Mesa-dev] [PATCH v2] st/va: flush the context before calling flush_frontbuffer(v2)

[Michel Dänzer]

On 16/09/16 07:33 PM, Nayan Deshmukh wrote:
> so that the texture is rendered to back buffer before calling
> flush_frontbuffer and can be copied to a different buffer in
> the function
> 
> v2: change comment style
> 
> Signed-off-by: Nayan Deshmukh <nayan26deshmukh at gmail.com>
> ---
>  src/gallium/state_trackers/va/surface.c | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/src/gallium/state_trackers/va/surface.c b/src/gallium/state_trackers/va/surface.c
> index 00df69d..115db43 100644
> --- a/src/gallium/state_trackers/va/surface.c
> +++ b/src/gallium/state_trackers/va/surface.c
> @@ -321,10 +321,14 @@ vlVaPutSurface(VADriverContextP ctx, VASurfaceID surface_id, void* draw, short s
>        return status;
>     }
>  
> +   /* flush before calling flush_frontbuffer so that rendering is flushed
> +    * to back buffer so the texture can be copied in flush_frontbuffer
> +    */
> +   drv->pipe->flush(drv->pipe, NULL, 0);
> +
>     screen->flush_frontbuffer(screen, tex, 0, 0,
>                               vscreen->get_private(vscreen), NULL);
>  
> -   drv->pipe->flush(drv->pipe, NULL, 0);
>  
>     pipe_resource_reference(&tex, NULL);
>     pipe_surface_reference(&surf_draw, NULL);
> 

This patch and patch 3 are

Reviewed-by: Michel Dänzer <michel.daenzer at amd.com>


-- 
Earthling Michel Dänzer               |               http://www.amd.com
Libre software enthusiast             |             Mesa and X developer