[Mesa-dev] [PATCH] nv50, nvc0: make sure to destroy the mutex used for blits

[Ilia Mirkin]

Reviewed-by: Ilia Mirkin <imirkin at alum.mit.edu>

On Tue, Mar 8, 2016 at 1:01 PM, Samuel Pitoiset
<samuel.pitoiset at gmail.com> wrote:
> This mutex is initialized when the blitter is created, but it is never
> destroyed. This doesn't hurt anything but it makes sense to destroy it
> at blitter deletion.
>
> Signed-off-by: Samuel Pitoiset <samuel.pitoiset at gmail.com>
> ---
>  src/gallium/drivers/nouveau/nv50/nv50_surface.c | 1 +
>  src/gallium/drivers/nouveau/nvc0/nvc0_surface.c | 1 +
>  2 files changed, 2 insertions(+)
>
> diff --git a/src/gallium/drivers/nouveau/nv50/nv50_surface.c b/src/gallium/drivers/nouveau/nv50/nv50_surface.c
> index 4db73cb..84646f6 100644
> --- a/src/gallium/drivers/nouveau/nv50/nv50_surface.c
> +++ b/src/gallium/drivers/nouveau/nv50/nv50_surface.c
> @@ -1756,6 +1756,7 @@ nv50_blitter_destroy(struct nv50_screen *screen)
>        }
>     }
>
> +   pipe_mutex_destroy(blitter->mutex);
>     FREE(blitter);
>  }
>
> diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c
> index 4957796..09b3115 100644
> --- a/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c
> +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_surface.c
> @@ -1644,6 +1644,7 @@ nvc0_blitter_destroy(struct nvc0_screen *screen)
>        }
>     }
>
> +   pipe_mutex_destroy(blitter->mutex);
>     FREE(blitter);
>  }
>
> --
> 2.7.1
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev