[Mesa-stable] [Mesa-dev][PATCH 2/2] eg/compute: Drop reference to kernel_param bo in destructor
Dylan Baker
dylan at pnwbakers.com
Tue May 8 17:10:01 UTC 2018
Indeed. This is currently not queued for 18.1 as it doesn't apply cleanly. The
18.1-proposed branch at git://people.freedesktop.org/~dbaker/mesa is where it
needs to apply if that's what needs to happen.
Thanks,
Dylan
Quoting Mark Janes (2018-05-08 09:28:24)
> 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
> _______________________________________________
> mesa-stable mailing list
> mesa-stable at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-stable
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: signature
URL: <https://lists.freedesktop.org/archives/mesa-stable/attachments/20180508/12ee2774/attachment.sig>
More information about the mesa-stable
mailing list