[Mesa-dev] [PATCH 2/2] st/va: flush pipeline after post processing
Nicolai Hähnle
nhaehnle at gmail.com
Tue Nov 29 13:46:04 UTC 2016
On 28.11.2016 15:51, Christian König wrote:
> From: sguttula <suresh.guttula at amd.com>
>
> This will flush the pipeline,which will allow to share dma-buf based
> buffers.
>
> Signed-off-by: Suresh Guttula <Suresh.Guttula at amd.com>
> Reviewed-by: Christian König <christian.koenig at amd.com>
Why is there no fence? Relying on the correctness of doing a flush
without a fence seems very likely to be wrong... it might seemingly fix
a sharing issue, but once the timing changes, the other side of a buffer
sharing might still see wrong results if it isn't properly synchronized.
Cheers,
Nicolai
> ---
> src/gallium/state_trackers/va/postproc.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/state_trackers/va/postproc.c b/src/gallium/state_trackers/va/postproc.c
> index d06f016..01e240f 100644
> --- a/src/gallium/state_trackers/va/postproc.c
> +++ b/src/gallium/state_trackers/va/postproc.c
> @@ -80,6 +80,7 @@ vlVaPostProcCompositor(vlVaDriver *drv, vlVaContext *context,
> vl_compositor_set_layer_dst_area(&drv->cstate, 0, &dst_rect);
> vl_compositor_render(&drv->cstate, &drv->compositor, surfaces[0], NULL, false);
>
> + drv->pipe->flush(drv->pipe, NULL, 0);
> return VA_STATUS_SUCCESS;
> }
>
>
More information about the mesa-dev
mailing list