[Mesa-stable] [Mesa-dev][PATCH 2/2] eg/compute: Drop reference to kernel_param bo in destructor

[Mark Janes]

Hi Jan,


Jan Vesely <jan.vesely at rutgers.edu> writes:

> CC: <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Jan Vesely <jan.vesely at rutgers.edu>
> ---
>  src/gallium/drivers/r600/evergreen_compute.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/r600/evergreen_compute.c b/src/gallium/drivers/r600/evergreen_compute.c
> index 027930b586..5070243914 100644
> --- a/src/gallium/drivers/r600/evergreen_compute.c
> +++ b/src/gallium/drivers/r600/evergreen_compute.c
> @@ -463,6 +463,7 @@ static void evergreen_delete_compute_state(struct pipe_context *ctx, void *state
>  #ifdef HAVE_OPENCL
>  		radeon_shader_binary_clean(&shader->binary);
>  		pipe_resource_reference(&shader->code_bo, NULL);

The stable branches do not have this ^^^ pipe_resource_reference call,
so the patch does not apply.  Can you make a proper backport of the fix
to clarify your intentions for stable?

> +		pipe_resource_reference(&shader->kernel_param, NULL);
>  #endif
>  		r600_destroy_shader(&shader->bc);
>  	}
> -- 
> 2.17.0
>
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable