On 6/6/25 18:55, Alex Deucher wrote: > On Wed, Jun 4, 2025 at 5:06 AM Christian König <[email protected]> > 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 <[email protected]> >> >> Reviewed-by: Christian König <[email protected]> > > 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); >>
