drm/vram: Have VRAM MM call GEM VRAM functions directly
authorThomas Zimmermann <tzimmermann@suse.de>
Wed, 11 Sep 2019 11:09:08 +0000 (13:09 +0200)
committerThomas Zimmermann <tzimmermann@suse.de>
Thu, 12 Sep 2019 17:54:42 +0000 (19:54 +0200)
VRAM MM and GEM VRAM buffer objects are only used with each other;
connected via 3 function pointers. Simplify this code by making the
memory manager call the rsp. functions of the BOs directly; and
remove the functions from the BO's public interface.

v2:
* typos in commit message

Signed-off-by: Thomas Zimmermann <tzimmermann@suse.de>
Acked-by: Gerd Hoffmann <kraxel@redhat.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20190911110910.30698-3-tzimmermann@suse.de
drivers/gpu/drm/ast/ast_ttm.c
drivers/gpu/drm/bochs/bochs_mm.c
drivers/gpu/drm/drm_gem_vram_helper.c
drivers/gpu/drm/drm_vram_helper_common.c
drivers/gpu/drm/hisilicon/hibmc/hibmc_ttm.c
drivers/gpu/drm/mgag200/mgag200_ttm.c
drivers/gpu/drm/vboxvideo/vbox_ttm.c
include/drm/drm_gem_vram_helper.h
include/drm/drm_vram_mm_helper.h [deleted file]

index 08ba0a91759322fc387641a213c277cc9e0d4a4f..fad34106083a88c7297df95a034d743b5a9810f0 100644 (file)
@@ -41,7 +41,7 @@ int ast_mm_init(struct ast_private *ast)
 
        vmm = drm_vram_helper_alloc_mm(
                dev, pci_resource_start(dev->pdev, 0),
-               ast->vram_size, &drm_gem_vram_mm_funcs);
+               ast->vram_size);
        if (IS_ERR(vmm)) {
                ret = PTR_ERR(vmm);
                DRM_ERROR("Error initializing VRAM MM; %d\n", ret);
index 8f9bb886f7add4cafb17db3313c37dbacfedd8e0..1b74f530b07c4e90e2a3f5ca6e1dc2834b509ff9 100644 (file)
@@ -11,8 +11,7 @@ int bochs_mm_init(struct bochs_device *bochs)
        struct drm_vram_mm *vmm;
 
        vmm = drm_vram_helper_alloc_mm(bochs->dev, bochs->fb_base,
-                                      bochs->fb_size,
-                                      &drm_gem_vram_mm_funcs);
+                                      bochs->fb_size);
        return PTR_ERR_OR_ZERO(vmm);
 }
 
index 2dae03170d1b968bc2507d1954feb0727b30b3d9..b5d342423bb208f42ddaf514784e601f6ffc6bb0 100644 (file)
@@ -7,7 +7,6 @@
 #include <drm/drm_gem_vram_helper.h>
 #include <drm/drm_mode.h>
 #include <drm/drm_prime.h>
-#include <drm/drm_vram_mm_helper.h>
 #include <drm/ttm/ttm_page_alloc.h>
 
 static const struct drm_gem_object_funcs drm_gem_vram_object_funcs;
@@ -464,68 +463,25 @@ static bool drm_is_gem_vram(struct ttm_buffer_object *bo)
        return (bo->destroy == ttm_buffer_object_destroy);
 }
 
-/**
- * drm_gem_vram_bo_driver_evict_flags() - \
-       Implements &struct ttm_bo_driver.evict_flags
- * @bo:        TTM buffer object. Refers to &struct drm_gem_vram_object.bo
- * @pl:        TTM placement information.
- */
-void drm_gem_vram_bo_driver_evict_flags(struct ttm_buffer_object *bo,
-                                       struct ttm_placement *pl)
+static void drm_gem_vram_bo_driver_evict_flags(struct drm_gem_vram_object *gbo,
+                                              struct ttm_placement *pl)
 {
-       struct drm_gem_vram_object *gbo;
-
-       /* TTM may pass BOs that are not GEM VRAM BOs. */
-       if (!drm_is_gem_vram(bo))
-               return;
-
-       gbo = drm_gem_vram_of_bo(bo);
        drm_gem_vram_placement(gbo, TTM_PL_FLAG_SYSTEM);
        *pl = gbo->placement;
 }
-EXPORT_SYMBOL(drm_gem_vram_bo_driver_evict_flags);
 
-/**
- * drm_gem_vram_bo_driver_verify_access() - \
-       Implements &struct ttm_bo_driver.verify_access
- * @bo:                TTM buffer object. Refers to &struct drm_gem_vram_object.bo
- * @filp:      File pointer.
- *
- * Returns:
- * 0 on success, or
- * a negative errno code otherwise.
- */
-int drm_gem_vram_bo_driver_verify_access(struct ttm_buffer_object *bo,
-                                        struct file *filp)
+static int drm_gem_vram_bo_driver_verify_access(struct drm_gem_vram_object *gbo,
+                                               struct file *filp)
 {
-       struct drm_gem_vram_object *gbo = drm_gem_vram_of_bo(bo);
-
        return drm_vma_node_verify_access(&gbo->bo.base.vma_node,
                                          filp->private_data);
 }
-EXPORT_SYMBOL(drm_gem_vram_bo_driver_verify_access);
 
-/**
- * drm_gem_vram_bo_driver_move_notify() -
- *     Implements &struct ttm_bo_driver.move_notify
- * @bo:                TTM buffer object. Refers to &struct drm_gem_vram_object.bo
- * @evict:     True, if the BO is being evicted from graphics memory;
- *             false otherwise.
- * @new_mem:   New memory region, or NULL on destruction
- */
-void drm_gem_vram_bo_driver_move_notify(struct ttm_buffer_object *bo,
-                                       bool evict,
-                                       struct ttm_mem_reg *new_mem)
+static void drm_gem_vram_bo_driver_move_notify(struct drm_gem_vram_object *gbo,
+                                              bool evict,
+                                              struct ttm_mem_reg *new_mem)
 {
-       struct drm_gem_vram_object *gbo;
-       struct ttm_bo_kmap_obj *kmap;
-
-       /* TTM may pass BOs that are not GEM VRAM BOs. */
-       if (!drm_is_gem_vram(bo))
-               return;
-
-       gbo = drm_gem_vram_of_bo(bo);
-       kmap = &gbo->kmap;
+       struct ttm_bo_kmap_obj *kmap = &gbo->kmap;
 
        if (WARN_ON_ONCE(gbo->kmap_use_count))
                return;
@@ -535,21 +491,6 @@ void drm_gem_vram_bo_driver_move_notify(struct ttm_buffer_object *bo,
        ttm_bo_kunmap(kmap);
        kmap->virtual = NULL;
 }
-EXPORT_SYMBOL(drm_gem_vram_bo_driver_move_notify);
-
-/*
- * drm_gem_vram_mm_funcs - Functions for &struct drm_vram_mm
- *
- * Most users of @struct drm_gem_vram_object will also use
- * @struct drm_vram_mm. This instance of &struct drm_vram_mm_funcs
- * can be used to connect both.
- */
-const struct drm_vram_mm_funcs drm_gem_vram_mm_funcs = {
-       .evict_flags = drm_gem_vram_bo_driver_evict_flags,
-       .verify_access = drm_gem_vram_bo_driver_verify_access,
-       .move_notify = drm_gem_vram_bo_driver_move_notify,
-};
-EXPORT_SYMBOL(drm_gem_vram_mm_funcs);
 
 /*
  * Helpers for struct drm_gem_object_funcs
@@ -815,31 +756,44 @@ static int bo_driver_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
 static void bo_driver_evict_flags(struct ttm_buffer_object *bo,
                                  struct ttm_placement *placement)
 {
-       struct drm_vram_mm *vmm = drm_vram_mm_of_bdev(bo->bdev);
+       struct drm_gem_vram_object *gbo;
 
-       if (vmm->funcs && vmm->funcs->evict_flags)
-               vmm->funcs->evict_flags(bo, placement);
+       /* TTM may pass BOs that are not GEM VRAM BOs. */
+       if (!drm_is_gem_vram(bo))
+               return;
+
+       gbo = drm_gem_vram_of_bo(bo);
+
+       drm_gem_vram_bo_driver_evict_flags(gbo, placement);
 }
 
 static int bo_driver_verify_access(struct ttm_buffer_object *bo,
                                   struct file *filp)
 {
-       struct drm_vram_mm *vmm = drm_vram_mm_of_bdev(bo->bdev);
+       struct drm_gem_vram_object *gbo;
 
-       if (!vmm->funcs || !vmm->funcs->verify_access)
-               return 0;
-       return vmm->funcs->verify_access(bo, filp);
+       /* TTM may pass BOs that are not GEM VRAM BOs. */
+       if (!drm_is_gem_vram(bo))
+               return -EINVAL;
+
+       gbo = drm_gem_vram_of_bo(bo);
+
+       return drm_gem_vram_bo_driver_verify_access(gbo, filp);
 }
 
 static void bo_driver_move_notify(struct ttm_buffer_object *bo,
                                  bool evict,
                                  struct ttm_mem_reg *new_mem)
 {
-       struct drm_vram_mm *vmm = drm_vram_mm_of_bdev(bo->bdev);
+       struct drm_gem_vram_object *gbo;
 
-       if (!vmm->funcs || !vmm->funcs->move_notify)
+       /* TTM may pass BOs that are not GEM VRAM BOs. */
+       if (!drm_is_gem_vram(bo))
                return;
-       vmm->funcs->move_notify(bo, evict, new_mem);
+
+       gbo = drm_gem_vram_of_bo(bo);
+
+       drm_gem_vram_bo_driver_move_notify(gbo, evict, new_mem);
 }
 
 static int bo_driver_io_mem_reserve(struct ttm_bo_device *bdev,
@@ -941,21 +895,18 @@ EXPORT_SYMBOL(drm_vram_mm_debugfs_init);
  * @dev:       the DRM device
  * @vram_base: the base address of the video memory
  * @vram_size: the size of the video memory in bytes
- * @funcs:     callback functions for buffer objects
  *
  * Returns:
  * 0 on success, or
  * a negative error code otherwise.
  */
 int drm_vram_mm_init(struct drm_vram_mm *vmm, struct drm_device *dev,
-                    uint64_t vram_base, size_t vram_size,
-                    const struct drm_vram_mm_funcs *funcs)
+                    uint64_t vram_base, size_t vram_size)
 {
        int ret;
 
        vmm->vram_base = vram_base;
        vmm->vram_size = vram_size;
-       vmm->funcs = funcs;
 
        ret = ttm_bo_device_init(&vmm->bdev, &bo_driver,
                                 dev->anon_inode->i_mapping,
@@ -1009,15 +960,13 @@ EXPORT_SYMBOL(drm_vram_mm_mmap);
  * @dev:       the DRM device
  * @vram_base: the base address of the video memory
  * @vram_size: the size of the video memory in bytes
- * @funcs:     callback functions for buffer objects
  *
  * Returns:
  * The new instance of &struct drm_vram_mm on success, or
  * an ERR_PTR()-encoded errno code otherwise.
  */
 struct drm_vram_mm *drm_vram_helper_alloc_mm(
-       struct drm_device *dev, uint64_t vram_base, size_t vram_size,
-       const struct drm_vram_mm_funcs *funcs)
+       struct drm_device *dev, uint64_t vram_base, size_t vram_size)
 {
        int ret;
 
@@ -1028,7 +977,7 @@ struct drm_vram_mm *drm_vram_helper_alloc_mm(
        if (!dev->vram_mm)
                return ERR_PTR(-ENOMEM);
 
-       ret = drm_vram_mm_init(dev->vram_mm, dev, vram_base, vram_size, funcs);
+       ret = drm_vram_mm_init(dev->vram_mm, dev, vram_base, vram_size);
        if (ret)
                goto err_kfree;
 
index e9c9f9a80ba396c46c053cbb79e6fa43e183b53e..2000d9b33fd52c75d084b599ef98d8a3601208ab 100644 (file)
@@ -7,9 +7,8 @@
  *
  * This library provides &struct drm_gem_vram_object (GEM VRAM), a GEM
  * buffer object that is backed by video RAM. It can be used for
- * framebuffer devices with dedicated memory. The video RAM can be
- * managed with &struct drm_vram_mm (VRAM MM). Both data structures are
- * supposed to be used together, but can also be used individually.
+ * framebuffer devices with dedicated memory. The video RAM is managed
+ * by &struct drm_vram_mm (VRAM MM).
  *
  * With the GEM interface userspace applications create, manage and destroy
  * graphics buffers, such as an on-screen framebuffer. GEM does not provide
@@ -50,8 +49,7 @@
  *             // setup device, vram base and size
  *             // ...
  *
- *             ret = drm_vram_helper_alloc_mm(dev, vram_base, vram_size,
- *                                            &drm_gem_vram_mm_funcs);
+ *             ret = drm_vram_helper_alloc_mm(dev, vram_base, vram_size);
  *             if (ret)
  *                     return ret;
  *             return 0;
index 5d52cd748603ff01600e33278d3bae133965d146..21b684eab5c9c6785550774a37dd19b8242a4997 100644 (file)
@@ -28,7 +28,7 @@ int hibmc_mm_init(struct hibmc_drm_private *hibmc)
 
        vmm = drm_vram_helper_alloc_mm(dev,
                                       pci_resource_start(dev->pdev, 0),
-                                      hibmc->fb_size, &drm_gem_vram_mm_funcs);
+                                      hibmc->fb_size);
        if (IS_ERR(vmm)) {
                ret = PTR_ERR(vmm);
                DRM_ERROR("Error initializing VRAM MM; %d\n", ret);
index 73a6b848601cd405c77aa6f5a2f1610de08c73df..69c81ebf37456445e82eed09969912f0fbbab7da 100644 (file)
@@ -37,8 +37,7 @@ int mgag200_mm_init(struct mga_device *mdev)
        struct drm_device *dev = mdev->dev;
 
        vmm = drm_vram_helper_alloc_mm(dev, pci_resource_start(dev->pdev, 0),
-                                      mdev->mc.vram_size,
-                                      &drm_gem_vram_mm_funcs);
+                                      mdev->mc.vram_size);
        if (IS_ERR(vmm)) {
                ret = PTR_ERR(vmm);
                DRM_ERROR("Error initializing VRAM MM; %d\n", ret);
index b82595a9ed0fd337e1447f4707d83f4f378864aa..976423d0c3ccacc588fcf1e6a2a43e7b679b8736 100644 (file)
@@ -17,8 +17,7 @@ int vbox_mm_init(struct vbox_private *vbox)
        struct drm_device *dev = &vbox->ddev;
 
        vmm = drm_vram_helper_alloc_mm(dev, pci_resource_start(dev->pdev, 0),
-                                      vbox->available_vram_size,
-                                      &drm_gem_vram_mm_funcs);
+                                      vbox->available_vram_size);
        if (IS_ERR(vmm)) {
                ret = PTR_ERR(vmm);
                DRM_ERROR("Error initializing VRAM MM; %d\n", ret);
index 1513349a27b15a202a1834ef0b784bbdff006a14..fd978e0c9542a7a046e6b3f69b51bbd9ac4f0cae 100644 (file)
@@ -109,22 +109,6 @@ int drm_gem_vram_fill_create_dumb(struct drm_file *file,
                                  bool interruptible,
                                  struct drm_mode_create_dumb *args);
 
-/*
- * Helpers for struct ttm_bo_driver
- */
-
-void drm_gem_vram_bo_driver_evict_flags(struct ttm_buffer_object *bo,
-                                       struct ttm_placement *pl);
-
-void drm_gem_vram_bo_driver_move_notify(struct ttm_buffer_object *bo,
-                                       bool evict,
-                                       struct ttm_mem_reg *new_mem);
-
-int drm_gem_vram_bo_driver_verify_access(struct ttm_buffer_object *bo,
-                                        struct file *filp);
-
-extern const struct drm_vram_mm_funcs drm_gem_vram_mm_funcs;
-
 /*
  * Helpers for struct drm_driver
  */
@@ -170,8 +154,6 @@ struct drm_vram_mm {
        size_t vram_size;
 
        struct ttm_bo_device bdev;
-
-       const struct drm_vram_mm_funcs *funcs;
 };
 
 /**
@@ -190,8 +172,7 @@ static inline struct drm_vram_mm *drm_vram_mm_of_bdev(
 
 int drm_vram_mm_debugfs_init(struct drm_minor *minor);
 int drm_vram_mm_init(struct drm_vram_mm *vmm, struct drm_device *dev,
-                    uint64_t vram_base, size_t vram_size,
-                    const struct drm_vram_mm_funcs *funcs);
+                    uint64_t vram_base, size_t vram_size);
 void drm_vram_mm_cleanup(struct drm_vram_mm *vmm);
 
 int drm_vram_mm_mmap(struct file *filp, struct vm_area_struct *vma,
@@ -202,8 +183,7 @@ int drm_vram_mm_mmap(struct file *filp, struct vm_area_struct *vma,
  */
 
 struct drm_vram_mm *drm_vram_helper_alloc_mm(
-       struct drm_device *dev, uint64_t vram_base, size_t vram_size,
-       const struct drm_vram_mm_funcs *funcs);
+       struct drm_device *dev, uint64_t vram_base, size_t vram_size);
 void drm_vram_helper_release_mm(struct drm_device *dev);
 
 /*
diff --git a/include/drm/drm_vram_mm_helper.h b/include/drm/drm_vram_mm_helper.h
deleted file mode 100644 (file)
index e3b79e1..0000000
+++ /dev/null
@@ -1,32 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-
-#ifndef DRM_VRAM_MM_HELPER_H
-#define DRM_VRAM_MM_HELPER_H
-
-#include <drm/drm_file.h>
-#include <drm/drm_ioctl.h>
-#include <drm/ttm/ttm_bo_driver.h>
-
-struct drm_device;
-
-/**
- * struct drm_vram_mm_funcs - Callback functions for &struct drm_vram_mm
- * @evict_flags:       Provides an implementation for struct \
-       &ttm_bo_driver.evict_flags
- * @verify_access:     Provides an implementation for \
-       struct &ttm_bo_driver.verify_access
- * @move_notify:       Provides an implementation for
- *                     struct &ttm_bo_driver.move_notify
- *
- * These callback function integrate VRAM MM with TTM buffer objects. New
- * functions can be added if necessary.
- */
-struct drm_vram_mm_funcs {
-       void (*evict_flags)(struct ttm_buffer_object *bo,
-                           struct ttm_placement *placement);
-       int (*verify_access)(struct ttm_buffer_object *bo, struct file *filp);
-       void (*move_notify)(struct ttm_buffer_object *bo, bool evict,
-                           struct ttm_mem_reg *new_mem);
-};
-
-#endif