Re: [PATCH v3] drm/ttm: Should to return the evict error

["Christian König" <christian.koenig@amd.com>]

On 6/3/25 11:11, Emily Deng wrote:
> For the evict fail case, the evict error should be returned.
> 
> v2: Consider ENOENT case.
> 
> v3: Abort directly when the eviction failed for some reason (except for -ENOENT)
>  and not wait for the move to finish
> 
> Signed-off-by: Emily Deng <Emily.Deng@amd.com>

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

> ---
>  drivers/gpu/drm/ttm/ttm_resource.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/ttm/ttm_resource.c b/drivers/gpu/drm/ttm/ttm_resource.c
> index 7e5a60c55813..bb84528276cd 100644
> --- a/drivers/gpu/drm/ttm/ttm_resource.c
> +++ b/drivers/gpu/drm/ttm/ttm_resource.c
> @@ -558,6 +558,9 @@ int ttm_resource_manager_evict_all(struct ttm_device *bdev,
>  		cond_resched();
>  	} while (!ret);
>  
> +	if (ret && ret != -ENOENT)
> +		return ret;
> +
>  	spin_lock(&man->move_lock);
>  	fence = dma_fence_get(man->move);
>  	spin_unlock(&man->move_lock);