Commit 9764c353 authored by Dave Airlie's avatar Dave Airlie

drm/ttm: move some move binds into the drivers

This just gives the driver control over some of the bind paths.
Reviewed-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarDave Airlie <airlied@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20201020010319.1692445-2-airlied@gmail.com
parent 47170f89
...@@ -605,10 +605,15 @@ static int amdgpu_move_ram_vram(struct ttm_buffer_object *bo, bool evict, ...@@ -605,10 +605,15 @@ static int amdgpu_move_ram_vram(struct ttm_buffer_object *bo, bool evict,
} }
/* move/bind old memory to GTT space */ /* move/bind old memory to GTT space */
r = ttm_bo_move_to_new_tt_mem(bo, ctx, &tmp_mem); r = ttm_tt_populate(bo->bdev, bo->ttm, ctx);
if (unlikely(r))
return r;
r = amdgpu_ttm_backend_bind(bo->bdev, bo->ttm, &tmp_mem);
if (unlikely(r)) { if (unlikely(r)) {
goto out_cleanup; goto out_cleanup;
} }
ttm_bo_assign_mem(bo, &tmp_mem); ttm_bo_assign_mem(bo, &tmp_mem);
/* copy to VRAM */ /* copy to VRAM */
r = amdgpu_move_blit(bo, evict, new_mem, old_mem); r = amdgpu_move_blit(bo, evict, new_mem, old_mem);
......
...@@ -931,9 +931,13 @@ nouveau_bo_move_flips(struct ttm_buffer_object *bo, bool evict, ...@@ -931,9 +931,13 @@ nouveau_bo_move_flips(struct ttm_buffer_object *bo, bool evict,
if (ret) if (ret)
return ret; return ret;
ret = ttm_bo_move_to_new_tt_mem(bo, ctx, &tmp_reg); ret = ttm_tt_populate(bo->bdev, bo->ttm, ctx);
if (ret) if (unlikely(ret != 0))
goto out; return ret;
ret = nouveau_ttm_tt_bind(bo->bdev, bo->ttm, &tmp_reg);
if (unlikely(ret != 0))
return ret;
ttm_bo_assign_mem(bo, &tmp_reg); ttm_bo_assign_mem(bo, &tmp_reg);
ret = nouveau_bo_move_m2mf(bo, true, ctx, new_reg); ret = nouveau_bo_move_m2mf(bo, true, ctx, new_reg);
......
...@@ -279,10 +279,15 @@ static int radeon_move_ram_vram(struct ttm_buffer_object *bo, ...@@ -279,10 +279,15 @@ static int radeon_move_ram_vram(struct ttm_buffer_object *bo,
if (unlikely(r)) { if (unlikely(r)) {
return r; return r;
} }
r = ttm_bo_move_to_new_tt_mem(bo, ctx, &tmp_mem);
if (unlikely(r)) { r = ttm_tt_populate(bo->bdev, bo->ttm, ctx);
if (unlikely(r))
goto out_cleanup; goto out_cleanup;
}
r = radeon_ttm_tt_bind(bo->bdev, bo->ttm, &tmp_mem);
if (unlikely(r))
goto out_cleanup;
ttm_bo_assign_mem(bo, &tmp_mem); ttm_bo_assign_mem(bo, &tmp_mem);
r = radeon_move_blit(bo, true, new_mem, old_mem); r = radeon_move_blit(bo, true, new_mem, old_mem);
if (unlikely(r)) { if (unlikely(r)) {
......
...@@ -64,7 +64,6 @@ int ttm_bo_move_to_new_tt_mem(struct ttm_buffer_object *bo, ...@@ -64,7 +64,6 @@ int ttm_bo_move_to_new_tt_mem(struct ttm_buffer_object *bo,
return 0; return 0;
} }
EXPORT_SYMBOL(ttm_bo_move_to_new_tt_mem);
int ttm_bo_move_to_system(struct ttm_buffer_object *bo, int ttm_bo_move_to_system(struct ttm_buffer_object *bo,
struct ttm_operation_ctx *ctx) struct ttm_operation_ctx *ctx)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment