drm/vram-helper: Remove interruptible flag from public interface

The flag 'interruptible', which is passed to various functions,
is always set to be false. Remove it and hard-code the value.

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Suggested-by: Daniel Vetter <daniel@ffwll.ch>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Acked-by: Sam Ravnborg <sam@ravnborg.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20200106125745.13797-7-tzimmermann@suse.de

diff --git a/drivers/gpu/drm/ast/ast_mode.c b/drivers/gpu/drm/ast/ast_mode.c
index cde1cae..43572eb 100644 (file)
--- a/drivers/gpu/drm/ast/ast_mode.c
+++ b/drivers/gpu/drm/ast/ast_mode.c
@@ -1145,7 +1145,7 @@ static int ast_cursor_init(struct drm_device *dev)
 
        for (i = 0; i < ARRAY_SIZE(ast->cursor.gbo); ++i) {
                gbo = drm_gem_vram_create(dev, &dev->vram_mm->bdev,
-                                         size, 0, false);
+                                         size, 0);
                if (IS_ERR(gbo)) {
                        ret = PTR_ERR(gbo);
                        goto err_drm_gem_vram_put;

diff --git a/drivers/gpu/drm/drm_gem_vram_helper.c b/drivers/gpu/drm/drm_gem_vram_helper.c
index 51ee1f7..bc181c0 100644 (file)
--- a/drivers/gpu/drm/drm_gem_vram_helper.c
+++ b/drivers/gpu/drm/drm_gem_vram_helper.c
@@ -94,8 +94,7 @@ static void drm_gem_vram_placement(struct drm_gem_vram_object *gbo,
 static int drm_gem_vram_init(struct drm_device *dev,
                             struct ttm_bo_device *bdev,
                             struct drm_gem_vram_object *gbo,
-                            size_t size, unsigned long pg_align,
-                            bool interruptible)
+                            size_t size, unsigned long pg_align)
 {
        int ret;
        size_t acc_size;
@@ -112,7 +111,7 @@ static int drm_gem_vram_init(struct drm_device *dev,
        drm_gem_vram_placement(gbo, TTM_PL_FLAG_VRAM | TTM_PL_FLAG_SYSTEM);
 
        ret = ttm_bo_init(bdev, &gbo->bo, size, ttm_bo_type_device,
-                         &gbo->placement, pg_align, interruptible, acc_size,
+                         &gbo->placement, pg_align, false, acc_size,
                          NULL, NULL, ttm_buffer_object_destroy);
        if (ret)
                goto err_drm_gem_object_release;
@@ -130,7 +129,6 @@ err_drm_gem_object_release:
  * @bdev:              the TTM BO device backing the object
  * @size:              the buffer size in bytes
  * @pg_align:          the buffer's alignment in multiples of the page size
- * @interruptible:     sleep interruptible if waiting for memory
  *
  * Returns:
  * A new instance of &struct drm_gem_vram_object on success, or
@@ -139,8 +137,7 @@ err_drm_gem_object_release:
 struct drm_gem_vram_object *drm_gem_vram_create(struct drm_device *dev,
                                                struct ttm_bo_device *bdev,
                                                size_t size,
-                                               unsigned long pg_align,
-                                               bool interruptible)
+                                               unsigned long pg_align)
 {
        struct drm_gem_vram_object *gbo;
        int ret;
@@ -149,7 +146,7 @@ struct drm_gem_vram_object *drm_gem_vram_create(struct drm_device *dev,
        if (!gbo)
                return ERR_PTR(-ENOMEM);
 
-       ret = drm_gem_vram_init(dev, bdev, gbo, size, pg_align, interruptible);
+       ret = drm_gem_vram_init(dev, bdev, gbo, size, pg_align);
        if (ret < 0)
                goto err_kfree;
 
@@ -486,7 +483,6 @@ EXPORT_SYMBOL(drm_gem_vram_vunmap);
  * @bdev:              the TTM BO device managing the buffer object
  * @pg_align:          the buffer's alignment in multiples of the page size
  * @pitch_align:       the scanline's alignment in powers of 2
- * @interruptible:     sleep interruptible if waiting for memory
  * @args:              the arguments as provided to \
                                &struct drm_driver.dumb_create
  *
@@ -504,7 +500,6 @@ int drm_gem_vram_fill_create_dumb(struct drm_file *file,
                                  struct ttm_bo_device *bdev,
                                  unsigned long pg_align,
                                  unsigned long pitch_align,
-                                 bool interruptible,
                                  struct drm_mode_create_dumb *args)
 {
        size_t pitch, size;
@@ -524,7 +519,7 @@ int drm_gem_vram_fill_create_dumb(struct drm_file *file,
        if (!size)
                return -EINVAL;
 
-       gbo = drm_gem_vram_create(dev, bdev, size, pg_align, interruptible);
+       gbo = drm_gem_vram_create(dev, bdev, size, pg_align);
        if (IS_ERR(gbo))
                return PTR_ERR(gbo);
 
@@ -620,7 +615,7 @@ int drm_gem_vram_driver_dumb_create(struct drm_file *file,
                return -EINVAL;
 
        return drm_gem_vram_fill_create_dumb(file, dev, &dev->vram_mm->bdev,
-                                            0, 0, false, args);
+                                            0, 0, args);
 }
 EXPORT_SYMBOL(drm_gem_vram_driver_dumb_create);
 

diff --git a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
index 0af5d96..2f668b7 100644 (file)
--- a/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
+++ b/drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
@@ -51,7 +51,7 @@ int hibmc_dumb_create(struct drm_file *file, struct drm_device *dev,
                      struct drm_mode_create_dumb *args)
 {
        return drm_gem_vram_fill_create_dumb(file, dev, &dev->vram_mm->bdev,
-                                            0, 16, false, args);
+                                            0, 16, args);
 }
 
 const struct drm_mode_config_funcs hibmc_mode_funcs = {

diff --git a/drivers/gpu/drm/mgag200/mgag200_cursor.c b/drivers/gpu/drm/mgag200/mgag200_cursor.c
index 5444cf1..dd54fd5 100644 (file)
--- a/drivers/gpu/drm/mgag200/mgag200_cursor.c
+++ b/drivers/gpu/drm/mgag200/mgag200_cursor.c
@@ -209,7 +209,7 @@ int mgag200_cursor_init(struct mga_device *mdev)
 
        for (i = 0; i < ncursors; ++i) {
                gbo = drm_gem_vram_create(dev, &dev->vram_mm->bdev,
-                                         size, 0, false);
+                                         size, 0);
                if (IS_ERR(gbo)) {
                        ret = PTR_ERR(gbo);
                        goto err_drm_gem_vram_put;

diff --git a/drivers/gpu/drm/mgag200/mgag200_drv.c b/drivers/gpu/drm/mgag200/mgag200_drv.c
index 3473e9e..613fdc2 100644 (file)
--- a/drivers/gpu/drm/mgag200/mgag200_drv.c
+++ b/drivers/gpu/drm/mgag200/mgag200_drv.c
@@ -121,7 +121,7 @@ int mgag200_driver_dumb_create(struct drm_file *file,
                pg_align = PFN_UP(mdev->mc.vram_size);
 
        return drm_gem_vram_fill_create_dumb(file, dev, &dev->vram_mm->bdev,
-                                            pg_align, 0, false, args);
+                                            pg_align, 0, args);
 }
 
 static struct drm_driver driver = {

diff --git a/include/drm/drm_gem_vram_helper.h b/include/drm/drm_gem_vram_helper.h
index b86d038..219474c 100644 (file)
--- a/include/drm/drm_gem_vram_helper.h
+++ b/include/drm/drm_gem_vram_helper.h
@@ -95,8 +95,7 @@ static inline struct drm_gem_vram_object *drm_gem_vram_of_gem(
 struct drm_gem_vram_object *drm_gem_vram_create(struct drm_device *dev,
                                                struct ttm_bo_device *bdev,
                                                size_t size,
-                                               unsigned long pg_align,
-                                               bool interruptible);
+                                               unsigned long pg_align);
 void drm_gem_vram_put(struct drm_gem_vram_object *gbo);
 u64 drm_gem_vram_mmap_offset(struct drm_gem_vram_object *gbo);
 s64 drm_gem_vram_offset(struct drm_gem_vram_object *gbo);
@@ -113,7 +112,6 @@ int drm_gem_vram_fill_create_dumb(struct drm_file *file,
                                  struct ttm_bo_device *bdev,
                                  unsigned long pg_align,
                                  unsigned long pitch_align,
-                                 bool interruptible,
                                  struct drm_mode_create_dumb *args);
 
 /*