[Mesa-dev] [PATCH v2 3/3] gallium/auxiliary/vl: Add barrier/unbind after compute shader launch.

[Marek Olšák]

For the series:

Reviewed-by: Marek Olšák <marek.olsak at amd.com>

Marek

On Wed, Apr 3, 2019 at 11:01 AM Zhu, James <James.Zhu at amd.com> wrote:

> Add memory barrier sync for multiple launch cases, and unbind completed
> resources after launch.
>
> Signed-off-by: James Zhu <James.Zhu at amd.com>
> ---
>  src/gallium/auxiliary/vl/vl_compositor_cs.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/src/gallium/auxiliary/vl/vl_compositor_cs.c
> b/src/gallium/auxiliary/vl/vl_compositor_cs.c
> index c3afe61..a659fd2 100644
> --- a/src/gallium/auxiliary/vl/vl_compositor_cs.c
> +++ b/src/gallium/auxiliary/vl/vl_compositor_cs.c
> @@ -292,6 +292,10 @@ cs_launch(struct vl_compositor *c,
>     info.grid[2] = 1;
>
>     ctx->launch_grid(ctx, &info);
> +
> +   /* Make the result visible to all clients. */
> +   ctx->memory_barrier(ctx, PIPE_BARRIER_ALL);
> +
>  }
>
>  static inline struct u_rect
> @@ -389,6 +393,13 @@ draw_layers(struct vl_compositor       *c,
>
>           cs_launch(c, layer->cs, &(drawn.area));
>
> +         /* Unbind. */
> +         ctx->set_shader_images(ctx, PIPE_SHADER_COMPUTE, 0, 1, NULL);
> +         ctx->set_constant_buffer(ctx, PIPE_SHADER_COMPUTE, 0, NULL);
> +         ctx->set_sampler_views(c->pipe, PIPE_SHADER_FRAGMENT, 0,
> num_sampler_views, NULL);
> +         ctx->bind_compute_state(ctx, NULL);
> +         ctx->bind_sampler_states(c->pipe, PIPE_SHADER_COMPUTE, 0,
> num_sampler_views, NULL);
> +
>           if (dirty) {
>              struct u_rect drawn = calc_drawn_area(s, layer);
>              dirty->x0 = MIN2(drawn.x0, dirty->x0);
> --
> 2.7.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20190403/51fa57c5/attachment.html>