[PATCH] drm/ttm: fix missing parameter change for ttm_bo_cleanup_refs

Christian König christian.koenig at amd.com
Fri Jan 19 08:49:36 UTC 2018


Am 19.01.2018 um 07:42 schrieb Roger He:
> Missed in the patche:
> dc94777 drm/ttm: enable swapout for reserved BOs during allocation.
> don't unreserve the BO if it is not reserved by itself.
>
> Signed-off-by: Roger He <Hongbo.He at amd.com>

Reviewed-by: Christian König <christian.koenig at amd.com>.

I already wondered if we shouldn't use "locked" somewhere else as well,
Christian.

> ---
>   drivers/gpu/drm/ttm/ttm_bo.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
> index 893003f..2fef09a 100644
> --- a/drivers/gpu/drm/ttm/ttm_bo.c
> +++ b/drivers/gpu/drm/ttm/ttm_bo.c
> @@ -1727,7 +1727,7 @@ int ttm_bo_swapout(struct ttm_bo_global *glob, struct ttm_operation_ctx *ctx)
>   	kref_get(&bo->list_kref);
>   
>   	if (!list_empty(&bo->ddestroy)) {
> -		ret = ttm_bo_cleanup_refs(bo, false, false, true);
> +		ret = ttm_bo_cleanup_refs(bo, false, false, locked);
>   		kref_put(&bo->list_kref, ttm_bo_release_list);
>   		return ret;
>   	}



More information about the dri-devel mailing list