[AMD Official Use Only - AMD Internal Distribution Only] Hi Christian, Could you help to review again?
Emily Deng Best Wishes >-----Original Message----- >From: Emily Deng <emily.d...@amd.com> >Sent: Tuesday, June 3, 2025 5:12 PM >To: amd-gfx@lists.freedesktop.org >Cc: Deng, Emily <emily.d...@amd.com> >Subject: [PATCH v3] drm/ttm: Should to return the evict error > >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.d...@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); >-- >2.34.1