Unified initialization and release of the global TTM state is provided
by struct ttm_global and its interfaces.

Signed-off-by: Thomas Zimmermann <tzimmerm...@suse.de>
---
 drivers/gpu/drm/cirrus/cirrus_drv.h |  5 +--
 drivers/gpu/drm/cirrus/cirrus_ttm.c | 49 +++++------------------------
 2 files changed, 11 insertions(+), 43 deletions(-)

diff --git a/drivers/gpu/drm/cirrus/cirrus_drv.h 
b/drivers/gpu/drm/cirrus/cirrus_drv.h
index a29f87e98d9d..f62774cfb68e 100644
--- a/drivers/gpu/drm/cirrus/cirrus_drv.h
+++ b/drivers/gpu/drm/cirrus/cirrus_drv.h
@@ -21,6 +21,7 @@
 #include <drm/ttm/ttm_placement.h>
 #include <drm/ttm/ttm_memory.h>
 #include <drm/ttm/ttm_module.h>
+#include <drm/ttm/ttm_global.h>
 
 #include <drm/drm_gem.h>
 
@@ -136,9 +137,9 @@ struct cirrus_device {
        int fb_mtrr;
 
        struct {
-               struct drm_global_reference mem_global_ref;
-               struct ttm_bo_global_ref bo_global_ref;
+               struct ttm_global glob;
                struct ttm_bo_device bdev;
+               bool glob_initialized;
        } ttm;
        bool mm_inited;
 };
diff --git a/drivers/gpu/drm/cirrus/cirrus_ttm.c 
b/drivers/gpu/drm/cirrus/cirrus_ttm.c
index 2e2141f26c5b..f1c5e6b8e80a 100644
--- a/drivers/gpu/drm/cirrus/cirrus_ttm.c
+++ b/drivers/gpu/drm/cirrus/cirrus_ttm.c
@@ -36,60 +36,27 @@ cirrus_bdev(struct ttm_bo_device *bd)
        return container_of(bd, struct cirrus_device, ttm.bdev);
 }
 
-static int
-cirrus_ttm_mem_global_init(struct drm_global_reference *ref)
-{
-       return ttm_mem_global_init(ref->object);
-}
-
-static void
-cirrus_ttm_mem_global_release(struct drm_global_reference *ref)
-{
-       ttm_mem_global_release(ref->object);
-}
-
 static int cirrus_ttm_global_init(struct cirrus_device *cirrus)
 {
-       struct drm_global_reference *global_ref;
-       int r;
-
-       global_ref = &cirrus->ttm.mem_global_ref;
-       global_ref->global_type = DRM_GLOBAL_TTM_MEM;
-       global_ref->size = sizeof(struct ttm_mem_global);
-       global_ref->init = &cirrus_ttm_mem_global_init;
-       global_ref->release = &cirrus_ttm_mem_global_release;
-       r = drm_global_item_ref(global_ref);
+       int r = ttm_global_init(&cirrus->ttm.glob);
        if (r != 0) {
-               DRM_ERROR("Failed setting up TTM memory accounting "
-                         "subsystem.\n");
+               DRM_ERROR("Failed setting up TTM subsystem.\n");
                return r;
        }
 
-       cirrus->ttm.bo_global_ref.mem_glob =
-               cirrus->ttm.mem_global_ref.object;
-       global_ref = &cirrus->ttm.bo_global_ref.ref;
-       global_ref->global_type = DRM_GLOBAL_TTM_BO;
-       global_ref->size = sizeof(struct ttm_bo_global);
-       global_ref->init = &ttm_bo_global_ref_init;
-       global_ref->release = &ttm_bo_global_ref_release;
-       r = drm_global_item_ref(global_ref);
-       if (r != 0) {
-               DRM_ERROR("Failed setting up TTM BO subsystem.\n");
-               drm_global_item_unref(&cirrus->ttm.mem_global_ref);
-               return r;
-       }
+       cirrus->ttm.glob_initialized = true;
+
        return 0;
 }
 
 static void
 cirrus_ttm_global_release(struct cirrus_device *cirrus)
 {
-       if (cirrus->ttm.mem_global_ref.release == NULL)
+       if (!cirrus->ttm.glob_initialized)
                return;
 
-       drm_global_item_unref(&cirrus->ttm.bo_global_ref.ref);
-       drm_global_item_unref(&cirrus->ttm.mem_global_ref);
-       cirrus->ttm.mem_global_ref.release = NULL;
+       ttm_global_init(&cirrus->ttm.glob);
+       cirrus->ttm.glob_initialized = false;
 }
 
 
@@ -237,7 +204,7 @@ int cirrus_mm_init(struct cirrus_device *cirrus)
                return ret;
 
        ret = ttm_bo_device_init(&cirrus->ttm.bdev,
-                                cirrus->ttm.bo_global_ref.ref.object,
+                                cirrus->ttm.glob.bo_ref.object,
                                 &cirrus_bo_driver,
                                 dev->anon_inode->i_mapping,
                                 DRM_FILE_PAGE_OFFSET,
-- 
2.19.1

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to