drm/ttm: drop persistent_swap_storage from ttm_bo_init and co

Never used as parameter, the only driver actually using this is nouveau
and there it is initialized after the BO is initialized.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

16 files changed, 21 insertions, 41 deletions

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
index c2a4b7215c46..216799ccb545 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_object.c
@@ -418,8 +418,8 @@ static int amdgpu_bo_do_create(struct amdgpu_device *adev,
         amdgpu_ttm_placement_from_domain(bo, domain);
 
         r = ttm_bo_init_reserved(&adev->mman.bdev, &bo->tbo, size, type,
-                                 &bo->placement, page_align, &ctx, NULL,
-                                 acc_size, sg, resv, &amdgpu_ttm_bo_destroy);
+                                 &bo->placement, page_align, &ctx, acc_size,
+                                 sg, resv, &amdgpu_ttm_bo_destroy);
         if (unlikely(r != 0))
                 return r;
 
diff --git a/drivers/gpu/drm/ast/ast_ttm.c b/drivers/gpu/drm/ast/ast_ttm.c
index 77d2035dc7b7..211224f6bdd3 100644
--- a/drivers/gpu/drm/ast/ast_ttm.c
+++ b/drivers/gpu/drm/ast/ast_ttm.c
@@ -321,7 +321,7 @@ int ast_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&ast->ttm.bdev, &astbo->bo, size,
                           ttm_bo_type_device, &astbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, ast_bo_ttm_destroy);
         if (ret)
                 goto error;
diff --git a/drivers/gpu/drm/bochs/bochs_mm.c b/drivers/gpu/drm/bochs/bochs_mm.c
index 96edf005bfea..73722484e12b 100644
--- a/drivers/gpu/drm/bochs/bochs_mm.c
+++ b/drivers/gpu/drm/bochs/bochs_mm.c
@@ -368,7 +368,7 @@ static int bochs_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&bochs->ttm.bdev, &bochsbo->bo, size,
                           ttm_bo_type_device, &bochsbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, bochs_bo_ttm_destroy);
         if (ret)
                 return ret;
diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c b/drivers/gpu/drm/cirrus/cirrus_ttm.c
index 3413389c0fbe..6cd0233b3bf8 100644
--- a/drivers/gpu/drm/cirrus/cirrus_ttm.c
+++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c
@@ -328,7 +328,7 @@ int cirrus_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&cirrus->ttm.bdev, &cirrusbo->bo, size,
                           ttm_bo_type_device, &cirrusbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, cirrus_bo_ttm_destroy);
         if (ret)
                 return ret;
diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
index 50e317a2a4ca..8dfffdbb6b07 100644
--- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
+++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
@@ -317,7 +317,7 @@ int hibmc_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&hibmc->bdev, &hibmcbo->bo, size,
                           ttm_bo_type_device, &hibmcbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, hibmc_bo_ttm_destroy);
         if (ret) {
                 hibmc_bo_unref(&hibmcbo);
diff --git a/drivers/gpu/drm/mgag200/mgag200_ttm.c b/drivers/gpu/drm/mgag200/mgag200_ttm.c
index cd55ff5f0f0a..69beb2046008 100644
--- a/drivers/gpu/drm/mgag200/mgag200_ttm.c
+++ b/drivers/gpu/drm/mgag200/mgag200_ttm.c
@@ -324,7 +324,7 @@ int mgag200_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&mdev->ttm.bdev, &mgabo->bo, size,
                           ttm_bo_type_device, &mgabo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, mgag200_bo_ttm_destroy);
         if (ret)
                 return ret;
diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
index 5c01ccfd3066..49cc8dfcb141 100644
--- a/drivers/gpu/drm/nouveau/nouveau_bo.c
+++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
@@ -298,7 +298,7 @@ nouveau_bo_new(struct nouveau_cli *cli, u64 size, int align,
 
         ret = ttm_bo_init(&drm->ttm.bdev, &nvbo->bo, size,
                           type, &nvbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size, sg,
+                          align >> PAGE_SHIFT, false, acc_size, sg,
                           robj, nouveau_bo_del_ttm);
         if (ret) {
                 /* ttm will call nouveau_bo_del_ttm if it fails.. */
diff --git a/drivers/gpu/drm/qxl/qxl_object.c b/drivers/gpu/drm/qxl/qxl_object.c
index f6b80fe47d1f..af62824ed4cc 100644
--- a/drivers/gpu/drm/qxl/qxl_object.c
+++ b/drivers/gpu/drm/qxl/qxl_object.c
@@ -109,7 +109,7 @@ int qxl_bo_create(struct qxl_device *qdev,
         qxl_ttm_placement_from_domain(bo, domain, pinned);
 
         r = ttm_bo_init(&qdev->mman.bdev, &bo->tbo, size, type,
-                        &bo->placement, 0, !kernel, NULL, size,
+                        &bo->placement, 0, !kernel, size,
                         NULL, NULL, &qxl_ttm_bo_destroy);
         if (unlikely(r != 0)) {
                 if (r != -ERESTARTSYS)
diff --git a/drivers/gpu/drm/radeon/radeon_object.c b/drivers/gpu/drm/radeon/radeon_object.c
index 64ab11d4ea58..38431f682ed0 100644
--- a/drivers/gpu/drm/radeon/radeon_object.c
+++ b/drivers/gpu/drm/radeon/radeon_object.c
@@ -255,8 +255,8 @@ int radeon_bo_create(struct radeon_device *rdev,
         /* Kernel allocation are uninterruptible */
         down_read(&rdev->pm.mclk_lock);
         r = ttm_bo_init(&rdev->mman.bdev, &bo->tbo, size, type,
-                        &bo->placement, page_align, !kernel, NULL,
-                        acc_size, sg, resv, &radeon_ttm_bo_destroy);
+                        &bo->placement, page_align, !kernel, acc_size,
+                        sg, resv, &radeon_ttm_bo_destroy);
         up_read(&rdev->pm.mclk_lock);
         if (unlikely(r != 0)) {
                 return r;
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
index 55028745214b..4bfa109e2a66 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -1149,7 +1149,6 @@ int ttm_bo_init_reserved(struct ttm_bo_device *bdev,
                          struct ttm_placement *placement,
                          uint32_t page_alignment,
                          struct ttm_operation_ctx *ctx,
-                         struct file *persistent_swap_storage,
                          size_t acc_size,
                          struct sg_table *sg,
                          struct reservation_object *resv,
@@ -1202,7 +1201,6 @@ int ttm_bo_init_reserved(struct ttm_bo_device *bdev,
         bo->mem.bus.io_reserved_count = 0;
         bo->moving = NULL;
         bo->mem.placement = (TTM_PL_FLAG_SYSTEM | TTM_PL_FLAG_CACHED);
-        bo->persistent_swap_storage = persistent_swap_storage;
         bo->acc_size = acc_size;
         bo->sg = sg;
         if (resv) {
@@ -1261,7 +1259,6 @@ int ttm_bo_init(struct ttm_bo_device *bdev,
                 struct ttm_placement *placement,
                 uint32_t page_alignment,
                 bool interruptible,
-                struct file *persistent_swap_storage,
                 size_t acc_size,
                 struct sg_table *sg,
                 struct reservation_object *resv,
@@ -1271,8 +1268,7 @@ int ttm_bo_init(struct ttm_bo_device *bdev,
         int ret;
 
         ret = ttm_bo_init_reserved(bdev, bo, size, type, placement,
-                                   page_alignment, &ctx,
-                                   persistent_swap_storage, acc_size,
+                                   page_alignment, &ctx, acc_size,
                                    sg, resv, destroy);
         if (ret)
                 return ret;
@@ -1318,7 +1314,6 @@ int ttm_bo_create(struct ttm_bo_device *bdev,
                         struct ttm_placement *placement,
                         uint32_t page_alignment,
                         bool interruptible,
-                        struct file *persistent_swap_storage,
                         struct ttm_buffer_object **p_bo)
 {
         struct ttm_buffer_object *bo;
@@ -1331,7 +1326,7 @@ int ttm_bo_create(struct ttm_bo_device *bdev,
 
         acc_size = ttm_bo_acc_size(bdev, size, sizeof(struct ttm_buffer_object));
         ret = ttm_bo_init(bdev, bo, size, type, placement, page_alignment,
-                          interruptible, persistent_swap_storage, acc_size,
+                          interruptible, acc_size,
                           NULL, NULL, NULL);
         if (likely(ret == 0))
                 *p_bo = bo;
diff --git a/drivers/gpu/drm/virtio/virtgpu_object.c b/drivers/gpu/drm/virtio/virtgpu_object.c
index 0b90cdb3d9fe..9f2f470efd9b 100644
--- a/drivers/gpu/drm/virtio/virtgpu_object.c
+++ b/drivers/gpu/drm/virtio/virtgpu_object.c
@@ -89,7 +89,7 @@ int virtio_gpu_object_create(struct virtio_gpu_device *vgdev,
         virtio_gpu_init_ttm_placement(bo, pinned);
 
         ret = ttm_bo_init(&vgdev->mman.bdev, &bo->tbo, size, type,
-                          &bo->placement, 0, !kernel, NULL, acc_size,
+                          &bo->placement, 0, !kernel, acc_size,
                           NULL, NULL, &virtio_gpu_ttm_bo_destroy);
         /* ttm_bo_init failure will call the destroy */
         if (ret != 0)
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c
index c706ad30411b..f283324ce598 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf.c
@@ -1245,7 +1245,7 @@ int vmw_cmdbuf_set_pool_size(struct vmw_cmdbuf_man *man,
                         return -ENOMEM;
 
                 ret = ttm_bo_create(&dev_priv->bdev, size, ttm_bo_type_device,
-                                    &vmw_mob_ne_placement, 0, false, NULL,
+                                    &vmw_mob_ne_placement, 0, false,
                                     &man->cmd_space);
                 if (ret)
                         return ret;
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c b/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c
index 736ca47e28ea..d07c585e3c1d 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_mob.c
@@ -260,8 +260,7 @@ static int vmw_otable_batch_setup(struct vmw_private *dev_priv,
         ret = ttm_bo_create(&dev_priv->bdev, bo_size,
                             ttm_bo_type_device,
                             &vmw_sys_ne_placement,
-                            0, false, NULL,
-                            &batch->otable_bo);
+                            0, false, &batch->otable_bo);
 
         if (unlikely(ret != 0))
                 goto out_no_bo;
@@ -444,7 +443,7 @@ static int vmw_mob_pt_populate(struct vmw_private *dev_priv,
         ret = ttm_bo_create(&dev_priv->bdev, mob->num_pages * PAGE_SIZE,
                             ttm_bo_type_device,
                             &vmw_sys_ne_placement,
-                            0, false, NULL, &mob->pt_bo);
+                            0, false, &mob->pt_bo);
         if (unlikely(ret != 0))
                 return ret;
 
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
index 200904ff9a22..9e101450cc4d 100644
--- a/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
+++ b/drivers/gpu/drm/vmwgfx/vmwgfx_resource.c
@@ -384,8 +384,8 @@ int vmw_dmabuf_init(struct vmw_private *dev_priv,
 
         ret = ttm_bo_init(bdev, &vmw_bo->base, size,
                           ttm_bo_type_device, placement,
-                          0, interruptible,
-                          NULL, acc_size, NULL, NULL, bo_free);
+                          0, interruptible, acc_size,
+                          NULL, NULL, bo_free);
         return ret;
 }
 
diff --git a/drivers/staging/vboxvideo/vbox_ttm.c b/drivers/staging/vboxvideo/vbox_ttm.c
index d1a211b61718..2c7daa3d0f24 100644
--- a/drivers/staging/vboxvideo/vbox_ttm.c
+++ b/drivers/staging/vboxvideo/vbox_ttm.c
@@ -331,7 +331,7 @@ int vbox_bo_create(struct drm_device *dev, int size, int align,
 
         ret = ttm_bo_init(&vbox->ttm.bdev, &vboxbo->bo, size,
                           ttm_bo_type_device, &vboxbo->placement,
-                          align >> PAGE_SHIFT, false, NULL, acc_size,
+                          align >> PAGE_SHIFT, false, acc_size,
                           NULL, NULL, vbox_bo_ttm_destroy);
         if (ret)
                 goto err_free_vboxbo;
diff --git a/include/drm/ttm/ttm_bo_api.h b/include/drm/ttm/ttm_bo_api.h
index a9e0640849d8..8e2fb1ac4e0c 100644
--- a/include/drm/ttm/ttm_bo_api.h
+++ b/include/drm/ttm/ttm_bo_api.h
@@ -467,11 +467,6 @@ size_t ttm_bo_dma_acc_size(struct ttm_bo_device *bdev,
  * @flags: Initial placement flags.
  * @page_alignment: Data alignment in pages.
  * @ctx: TTM operation context for memory allocation.
- * @persistent_swap_storage: Usually the swap storage is deleted for buffers
- * pinned in physical memory. If this behaviour is not desired, this member
- * holds a pointer to a persistent shmem object. Typically, this would
- * point to the shmem object backing a GEM object if TTM is used to back a
- * GEM user interface.
  * @acc_size: Accounted size for this object.
  * @resv: Pointer to a reservation_object, or NULL to let ttm allocate one.
  * @destroy: Destroy function. Use NULL for kfree().
@@ -504,7 +499,6 @@ int ttm_bo_init_reserved(struct ttm_bo_device *bdev,
                          struct ttm_placement *placement,
                          uint32_t page_alignment,
                          struct ttm_operation_ctx *ctx,
-                         struct file *persistent_swap_storage,
                          size_t acc_size,
                          struct sg_table *sg,
                          struct reservation_object *resv,
@@ -521,7 +515,6 @@ int ttm_bo_init_reserved(struct ttm_bo_device *bdev,
  * @page_alignment: Data alignment in pages.
  * @interruptible: If needing to sleep to wait for GPU resources,
  * sleep interruptible.
- * @persistent_swap_storage: Usually the swap storage is deleted for buffers
  * pinned in physical memory. If this behaviour is not desired, this member
  * holds a pointer to a persistent shmem object. Typically, this would
  * point to the shmem object backing a GEM object if TTM is used to back a
@@ -551,8 +544,7 @@ int ttm_bo_init_reserved(struct ttm_bo_device *bdev,
 int ttm_bo_init(struct ttm_bo_device *bdev, struct ttm_buffer_object *bo,
                 unsigned long size, enum ttm_bo_type type,
                 struct ttm_placement *placement,
-                uint32_t page_alignment, bool interrubtible,
-                struct file *persistent_swap_storage, size_t acc_size,
+                uint32_t page_alignment, bool interrubtible, size_t acc_size,
                 struct sg_table *sg, struct reservation_object *resv,
                 void (*destroy) (struct ttm_buffer_object *));
 
@@ -566,11 +558,6 @@ int ttm_bo_init(struct ttm_bo_device *bdev, struct ttm_buffer_object *bo,
  * @page_alignment: Data alignment in pages.
  * @interruptible: If needing to sleep while waiting for GPU resources,
  * sleep interruptible.
- * @persistent_swap_storage: Usually the swap storage is deleted for buffers
- * pinned in physical memory. If this behaviour is not desired, this member
- * holds a pointer to a persistent shmem object. Typically, this would
- * point to the shmem object backing a GEM object if TTM is used to back a
- * GEM user interface.
  * @p_bo: On successful completion *p_bo points to the created object.
  *
  * This function allocates a ttm_buffer_object, and then calls ttm_bo_init
@@ -583,7 +570,6 @@ int ttm_bo_init(struct ttm_bo_device *bdev, struct ttm_buffer_object *bo,
 int ttm_bo_create(struct ttm_bo_device *bdev, unsigned long size,
                   enum ttm_bo_type type, struct ttm_placement *placement,
                   uint32_t page_alignment, bool interruptible,
-                  struct file *persistent_swap_storage,
                   struct ttm_buffer_object **p_bo);
 
 /**