[Mesa-dev] [PATCH 2/3] st/vdpau: flush the context before calling flush_frontbuffer

[Michel Dänzer]

On 14/09/16 02:34 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
> 
> Signed-off-by: Nayan Deshmukh <nayan26deshmukh at gmail.com>
> ---
>  src/gallium/state_trackers/vdpau/presentation.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/src/gallium/state_trackers/vdpau/presentation.c b/src/gallium/state_trackers/vdpau/presentation.c
> index 2862eaf..f35d73a 100644
> --- a/src/gallium/state_trackers/vdpau/presentation.c
> +++ b/src/gallium/state_trackers/vdpau/presentation.c
> @@ -271,11 +271,14 @@ vlVdpPresentationQueueDisplay(VdpPresentationQueue presentation_queue,
>     }
>  
>     vscreen->set_next_timestamp(vscreen, earliest_presentation_time);
> -   pipe->screen->flush_frontbuffer(pipe->screen, tex, 0, 0,
> -                                   vscreen->get_private(vscreen), NULL);
>  
> +   // flush before calling flush_frontbuffer so that rendering is flushed
> +   //  to back buffer so the texture can be copied in flush_frontbuffer

Please use C-style /* ... */ comments in C code.

With that fixed,

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


Same for patch 3.


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