drm/i915: Rename INTEL_REGION_LMEM with INTEL_REGION_LMEM_0
authorAndi Shyti <andi.shyti@linux.intel.com>
Fri, 18 Mar 2022 23:39:31 +0000 (01:39 +0200)
committerMatthew Auld <matthew.auld@intel.com>
Mon, 21 Mar 2022 08:37:33 +0000 (08:37 +0000)
With the upcoming multitile support each tile will have its own
local memory. Mark the current LMEM with the suffix '0' to
emphasise that it belongs to the root tile.

Suggested-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Andi Shyti <andi.shyti@linux.intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Reviewed-by: Andrzej Hajda <andrzej.hajda@intel.com>
Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220318233938.149744-2-andi.shyti@linux.intel.com
drivers/gpu/drm/i915/display/intel_fb.c
drivers/gpu/drm/i915/display/intel_fb_pin.c
drivers/gpu/drm/i915/display/intel_plane_initial.c
drivers/gpu/drm/i915/gem/i915_gem_lmem.c
drivers/gpu/drm/i915/gem/selftests/i915_gem_dmabuf.c
drivers/gpu/drm/i915/gem/selftests/i915_gem_migrate.c
drivers/gpu/drm/i915/gt/intel_gt.c
drivers/gpu/drm/i915/intel_memory_region.c
drivers/gpu/drm/i915/intel_memory_region.h

index 23cfe2e5ce2a4ff2ee4694c9c1ee13ee52569b6d..421f7238da05e0f7229fb5a4fe34478154e3c79d 100644 (file)
@@ -1981,7 +1981,7 @@ intel_user_framebuffer_create(struct drm_device *dev,
 
        /* object is backed with LMEM for discrete */
        i915 = to_i915(obj->base.dev);
-       if (HAS_LMEM(i915) && !i915_gem_object_can_migrate(obj, INTEL_REGION_LMEM)) {
+       if (HAS_LMEM(i915) && !i915_gem_object_can_migrate(obj, INTEL_REGION_LMEM_0)) {
                /* object is "remote", not in local memory */
                i915_gem_object_put(obj);
                return ERR_PTR(-EREMOTE);
index a307b4993bcf3ac276e5a81aecac9e9a4d8a1120..bd6e7c98e751dfc255deab6d4404d477d63b7672 100644 (file)
@@ -140,7 +140,7 @@ retry:
        if (!ret && phys_cursor)
                ret = i915_gem_object_attach_phys(obj, alignment);
        else if (!ret && HAS_LMEM(dev_priv))
-               ret = i915_gem_object_migrate(obj, &ww, INTEL_REGION_LMEM);
+               ret = i915_gem_object_migrate(obj, &ww, INTEL_REGION_LMEM_0);
        /* TODO: Do we need to sync when migration becomes async? */
        if (!ret)
                ret = i915_gem_object_pin_pages(obj);
index 5c0cd0775a14a1f8b14e7224958ceb44f7e384a4..41ff7bf23e3b27cfa4ae8800ccb4ef5d07befd52 100644 (file)
@@ -72,7 +72,7 @@ initial_plane_vma(struct drm_i915_private *i915,
                }
 
                phys_base = pte & I915_GTT_PAGE_MASK;
-               mem = i915->mm.regions[INTEL_REGION_LMEM];
+               mem = i915->mm.regions[INTEL_REGION_LMEM_0];
 
                /*
                 * We don't currently expect this to ever be placed in the
index 444f8268b9c5f40e36256e5d350f8bcf468bcc3a..47e43dc3a174fd3f972acfe8e45f87e723d3ce78 100644 (file)
@@ -100,7 +100,7 @@ __i915_gem_object_create_lmem_with_ps(struct drm_i915_private *i915,
                                      resource_size_t page_size,
                                      unsigned int flags)
 {
-       return i915_gem_object_create_region(i915->mm.regions[INTEL_REGION_LMEM],
+       return i915_gem_object_create_region(i915->mm.regions[INTEL_REGION_LMEM_0],
                                             size, page_size, flags);
 }
 
@@ -135,6 +135,6 @@ i915_gem_object_create_lmem(struct drm_i915_private *i915,
                            resource_size_t size,
                            unsigned int flags)
 {
-       return i915_gem_object_create_region(i915->mm.regions[INTEL_REGION_LMEM],
+       return i915_gem_object_create_region(i915->mm.regions[INTEL_REGION_LMEM_0],
                                             size, 0, flags);
 }
index b071a58dd6daac39f3c7ca4c95124c8a17761319..a342fd387d4ef3643781bfbc47035abb9248beaa 100644 (file)
@@ -88,7 +88,7 @@ out:
 static int igt_dmabuf_import_same_driver_lmem(void *arg)
 {
        struct drm_i915_private *i915 = arg;
-       struct intel_memory_region *lmem = i915->mm.regions[INTEL_REGION_LMEM];
+       struct intel_memory_region *lmem = i915->mm.regions[INTEL_REGION_LMEM_0];
        struct drm_i915_gem_object *obj;
        struct drm_gem_object *import;
        struct dma_buf *dmabuf;
@@ -252,10 +252,10 @@ static int igt_dmabuf_import_same_driver_lmem_smem(void *arg)
        struct drm_i915_private *i915 = arg;
        struct intel_memory_region *regions[2];
 
-       if (!i915->mm.regions[INTEL_REGION_LMEM])
+       if (!i915->mm.regions[INTEL_REGION_LMEM_0])
                return 0;
 
-       regions[0] = i915->mm.regions[INTEL_REGION_LMEM];
+       regions[0] = i915->mm.regions[INTEL_REGION_LMEM_0];
        regions[1] = i915->mm.regions[INTEL_REGION_SMEM];
        return igt_dmabuf_import_same_driver(i915, regions, 2);
 }
index d534141b2cf7cbf186cdefaacae574d16f88cb28..2c63daf932de7554676e5c483f38aaf0898c1d4f 100644 (file)
@@ -92,17 +92,17 @@ static int igt_create_migrate(struct intel_gt *gt, enum intel_region_id src,
 
 static int igt_smem_create_migrate(void *arg)
 {
-       return igt_create_migrate(arg, INTEL_REGION_LMEM, INTEL_REGION_SMEM);
+       return igt_create_migrate(arg, INTEL_REGION_LMEM_0, INTEL_REGION_SMEM);
 }
 
 static int igt_lmem_create_migrate(void *arg)
 {
-       return igt_create_migrate(arg, INTEL_REGION_SMEM, INTEL_REGION_LMEM);
+       return igt_create_migrate(arg, INTEL_REGION_SMEM, INTEL_REGION_LMEM_0);
 }
 
 static int igt_same_create_migrate(void *arg)
 {
-       return igt_create_migrate(arg, INTEL_REGION_LMEM, INTEL_REGION_LMEM);
+       return igt_create_migrate(arg, INTEL_REGION_LMEM_0, INTEL_REGION_LMEM_0);
 }
 
 static int lmem_pages_migrate_one(struct i915_gem_ww_ctx *ww,
@@ -152,7 +152,7 @@ static int lmem_pages_migrate_one(struct i915_gem_ww_ctx *ww,
                }
 
        } else {
-               err = i915_gem_object_migrate(obj, ww, INTEL_REGION_LMEM);
+               err = i915_gem_object_migrate(obj, ww, INTEL_REGION_LMEM_0);
                if (err) {
                        pr_err("Object failed migration to lmem\n");
                        if (err)
index 57ca1e6b6203ddad19b99992352bf33055e93627..ca875ba3e2a9d0e657f9d6f823b87a4d0a6bbb6c 100644 (file)
@@ -76,7 +76,7 @@ int intel_gt_probe_lmem(struct intel_gt *gt)
                return err;
        }
 
-       id = INTEL_REGION_LMEM;
+       id = INTEL_REGION_LMEM_0;
 
        mem->id = id;
 
index 1c841f68169a3df9a7cc1971ab4f2c4eb4904a4f..6a6ee2f200f8781b1c5857cc363ae3d8eb5728b3 100644 (file)
@@ -17,7 +17,7 @@ static const struct {
                .class = INTEL_MEMORY_SYSTEM,
                .instance = 0,
        },
-       [INTEL_REGION_LMEM] = {
+       [INTEL_REGION_LMEM_0] = {
                .class = INTEL_MEMORY_LOCAL,
                .instance = 0,
        },
index 56f266020285857057c90bf8abb2f5907529a876..92a821c259412b23492ec30c46933cae210d86cf 100644 (file)
@@ -29,14 +29,14 @@ enum intel_memory_type {
 
 enum intel_region_id {
        INTEL_REGION_SMEM = 0,
-       INTEL_REGION_LMEM,
+       INTEL_REGION_LMEM_0,
        INTEL_REGION_STOLEN_SMEM,
        INTEL_REGION_STOLEN_LMEM,
        INTEL_REGION_UNKNOWN, /* Should be last */
 };
 
 #define REGION_SMEM     BIT(INTEL_REGION_SMEM)
-#define REGION_LMEM     BIT(INTEL_REGION_LMEM)
+#define REGION_LMEM     BIT(INTEL_REGION_LMEM_0)
 #define REGION_STOLEN_SMEM   BIT(INTEL_REGION_STOLEN_SMEM)
 #define REGION_STOLEN_LMEM   BIT(INTEL_REGION_STOLEN_LMEM)