For the series:
Reviewed-by: Marek Olšák <marek.ol...@amd.com>
Marek
On Wed, Apr 3, 2019 at 11:01 AM Zhu, James <james....@amd.com> wrote:
> Add memory barrier sync for multiple launch cases, and unbind completed
> resources after launch.
>
> Signed-off-by: James Zhu <james....@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@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
