drm/nouveau: clean up nouveau_bo_move_ntfy
authorChristian König <christian.koenig@amd.com>
Thu, 11 Feb 2021 11:35:23 +0000 (12:35 +0100)
committerChristian König <christian.koenig@amd.com>
Tue, 16 Mar 2021 15:30:18 +0000 (16:30 +0100)
Just another leftover from a TTM cleanup.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210315191432.153826-2-christian.koenig@amd.com
drivers/gpu/drm/nouveau/nouveau_bo.c

index 281e9ed..2d5d68f 100644 (file)
@@ -886,9 +886,8 @@ nouveau_bo_move_init(struct nouveau_drm *drm)
        NV_INFO(drm, "MM: using %s for buffer copies\n", name);
 }
 
-static void
-nouveau_bo_move_ntfy(struct ttm_buffer_object *bo, bool evict,
-                    struct ttm_resource *new_reg)
+static void nouveau_bo_move_ntfy(struct ttm_buffer_object *bo,
+                                struct ttm_resource *new_reg)
 {
        struct nouveau_mem *mem = new_reg ? nouveau_mem(new_reg) : NULL;
        struct nouveau_bo *nvbo = nouveau_bo(bo);
@@ -974,7 +973,7 @@ nouveau_bo_move(struct ttm_buffer_object *bo, bool evict,
                        return ret;
        }
 
-       nouveau_bo_move_ntfy(bo, evict, new_reg);
+       nouveau_bo_move_ntfy(bo, new_reg);
        ret = ttm_bo_wait_ctx(bo, ctx);
        if (ret)
                goto out_ntfy;
@@ -1039,9 +1038,7 @@ out:
        }
 out_ntfy:
        if (ret) {
-               swap(*new_reg, bo->mem);
-               nouveau_bo_move_ntfy(bo, false, new_reg);
-               swap(*new_reg, bo->mem);
+               nouveau_bo_move_ntfy(bo, &bo->mem);
        }
        return ret;
 }
@@ -1315,7 +1312,7 @@ nouveau_bo_fence(struct nouveau_bo *nvbo, struct nouveau_fence *fence, bool excl
 static void
 nouveau_bo_delete_mem_notify(struct ttm_buffer_object *bo)
 {
-       nouveau_bo_move_ntfy(bo, false, NULL);
+       nouveau_bo_move_ntfy(bo, NULL);
 }
 
 struct ttm_device_funcs nouveau_bo_driver = {