[Mesa-dev] [PATCH] radeonsi: avoid crash when a sampler state is bound for a buffer texture
Ilia Mirkin
imirkin at alum.mit.edu
Sun Mar 13 00:09:44 UTC 2016
On Fri, Mar 11, 2016 at 11:17 AM, Nicolai Hähnle <nhaehnle at gmail.com> wrote:
> From: Nicolai Hähnle <nicolai.haehnle at amd.com>
>
> Sampler states don't really make sense with buffer textures, but the PBO
> upload code sets one because apparently nouveau needs it. It would be
> nice to work that out at some point, but in any case being defensive
> here is a good idea.
Sampler states are set in regular GL as well if you have a regular
buffer texture too, no?
>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=94284
> Cc: mesa-stable at lists.freedesktop.org
> ---
> src/gallium/drivers/radeonsi/si_descriptors.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/gallium/drivers/radeonsi/si_descriptors.c b/src/gallium/drivers/radeonsi/si_descriptors.c
> index 9aa4877..f5ad113 100644
> --- a/src/gallium/drivers/radeonsi/si_descriptors.c
> +++ b/src/gallium/drivers/radeonsi/si_descriptors.c
> @@ -324,6 +324,7 @@ static void si_bind_sampler_states(struct pipe_context *ctx, unsigned shader,
> */
> if (samplers->views.views[i] &&
> samplers->views.views[i]->texture &&
> + samplers->views.views[i]->texture->target != PIPE_BUFFER &&
> ((struct r600_texture*)samplers->views.views[i]->texture)->fmask.size)
> continue;
>
> --
> 2.5.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
More information about the mesa-dev
mailing list