[PATCH] drm/ttm: once more fix ttm_buffer_object_transfer

Deucher, Alexander Alexander.Deucher at amd.com
Mon Oct 30 17:06:51 UTC 2017


> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-bounces at lists.freedesktop.org] On Behalf
> Of Christian König
> Sent: Monday, October 30, 2017 10:00 AM
> To: dri-devel at lists.freedesktop.org; amd-gfx at lists.freedesktop.org
> Subject: [PATCH] drm/ttm: once more fix ttm_buffer_object_transfer
> 
> From: Christian König <christian.koenig at amd.com>
> 
> When the mutex is locked just in the moment we copy it we end up with a
> warning that we release a locked mutex.
> 
> Fix this by properly reinitializing the mutex.
> 
> Signed-off-by: Christian König <christian.koenig at amd.com>

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

Stable?

Alex

> ---
>  drivers/gpu/drm/ttm/ttm_bo_util.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/gpu/drm/ttm/ttm_bo_util.c
> b/drivers/gpu/drm/ttm/ttm_bo_util.c
> index 78cb99be7146..e7a519f1849b 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo_util.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo_util.c
> @@ -474,6 +474,7 @@ static int ttm_buffer_object_transfer(struct
> ttm_buffer_object *bo,
>  	INIT_LIST_HEAD(&fbo->lru);
>  	INIT_LIST_HEAD(&fbo->swap);
>  	INIT_LIST_HEAD(&fbo->io_reserve_lru);
> +	mutex_init(&fbo->wu_mutex);
>  	fbo->moving = NULL;
>  	drm_vma_node_reset(&fbo->vma_node);
>  	atomic_set(&fbo->cpu_writers, 0);
> --
> 2.11.0
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx


More information about the dri-devel mailing list