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

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

On 6/6/25 18:55, Alex Deucher wrote:
> On Wed, Jun 4, 2025 at 5:06 AM Christian König <christian.koenig@amd.com> wrote:
>>
>> 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>
> 
> Did you push this to drm-misc?


Not yet, I wanted to take care of it this morning.

Christian.

> 
> Alex
> 
>>
>>> ---
>>>  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);
>>