Re: [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 
> ---
>  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(, NULL);
> pipe_surface_reference(_draw, NULL);
> 

This patch and patch 3 are

Reviewed-by: Michel Dänzer 


-- 
Earthling Michel Dänzer   |   http://www.amd.com
Libre software enthusiast | Mesa and X developer
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

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

[Nayan Deshmukh]

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 
---
 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(, NULL);
pipe_surface_reference(_draw, NULL);
-- 
2.7.4

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev