Re: [PATCH] drm/amdgpu: Ignore KFD eviction fences invalidating preemptible DMABuf imports

[Felix Kuehling]

Am 2023-04-12 um 18:25 schrieb Eric Huang:

It is to avoid redundant eviction for KFD's DMAbuf import
bo when dmaunmapping DMAbuf. The DMAbuf import bo has
been set as AMDGPU_PL_PREEMPT in KFD when mapping.

Signed-off-by: Eric Huang <jinhuieric.hu...@amd.com>

Reviewed-by: Felix Kuehling <felix.kuehl...@amd.com>

I'd like to get an Acked-by from Christian as well before submitting this.

Thanks,
  Felix

---
  drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c | 7 ++++++-
  1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
index 2430f3e9f3a7..64795fe9eecb 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c
@@ -526,7 +526,12 @@ static int amdgpu_bo_move(struct ttm_buffer_object *bo, 
bool evict,
        if ((old_mem->mem_type == TTM_PL_TT ||
             old_mem->mem_type == AMDGPU_PL_PREEMPT) &&
            new_mem->mem_type == TTM_PL_SYSTEM) {
-               r = ttm_bo_wait_ctx(bo, ctx);
+               if (old_mem->mem_type == AMDGPU_PL_PREEMPT)
+                       r = amdgpu_bo_sync_wait(abo,
+                                               AMDGPU_FENCE_OWNER_KFD,
+                                               ctx->interruptible);
+               else
+                       r = ttm_bo_wait_ctx(bo, ctx);
                if (r)
                        return r;