[Mesa-dev] [PATCH] st/mesa: empty buffer binding if the buffer's not really there

Marek Olšák maraeo at gmail.com
Tue Feb 16 00:41:20 UTC 2016


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

Marek

On Sun, Feb 14, 2016 at 1:25 AM, Ilia Mirkin <imirkin at alum.mit.edu> wrote:
> This can happen with 0-sized buffers.
>
> Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
> ---
>  src/mesa/state_tracker/st_atom_atomicbuf.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/state_tracker/st_atom_atomicbuf.c b/src/mesa/state_tracker/st_atom_atomicbuf.c
> index d83c396..a27dbe0 100644
> --- a/src/mesa/state_tracker/st_atom_atomicbuf.c
> +++ b/src/mesa/state_tracker/st_atom_atomicbuf.c
> @@ -58,9 +58,11 @@ st_bind_atomics(struct st_context *st,
>           st_buffer_object(binding->BufferObject);
>        struct pipe_shader_buffer sb = { 0 };
>
> -      sb.buffer = st_obj->buffer;
> -      sb.buffer_offset = binding->Offset;
> -      sb.buffer_size = st_obj->buffer->width0 - binding->Offset;
> +      if (st_obj && st_obj->buffer) {
> +         sb.buffer = st_obj->buffer;
> +         sb.buffer_offset = binding->Offset;
> +         sb.buffer_size = st_obj->buffer->width0 - binding->Offset;
> +      }
>
>        st->pipe->set_shader_buffers(st->pipe, shader_type,
>                                     atomic->Binding, 1, &sb);
> --
> 2.4.10
>
> _______________________________________________
> 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