]> git.ipfire.org Git - thirdparty/kernel/linux.git/commitdiff
drm/xe: Rename struct xe_mem_region to struct xe_vram_region
authorPiotr Piórkowski <piotr.piorkowski@intel.com>
Mon, 10 Feb 2025 08:15:10 +0000 (09:15 +0100)
committerMichal Wajdeczko <michal.wajdeczko@intel.com>
Mon, 10 Feb 2025 12:08:57 +0000 (13:08 +0100)
The xe_mem_region structure has so far been used only in the context
of VRAM regions. Also, the description of its fields clearly indicates
that it was designed for VRAM regions. This struct is strictly related
only to VRAM.
So let's be clear on this point and rename it to xe_vram_region.

Signed-off-by: Piotr Piórkowski <piotr.piorkowski@intel.com>
Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20250210081511.906452-2-piotr.piorkowski@intel.com
drivers/gpu/drm/xe/xe_bo.c
drivers/gpu/drm/xe/xe_device_types.h
drivers/gpu/drm/xe/xe_ttm_vram_mgr.c
drivers/gpu/drm/xe/xe_ttm_vram_mgr_types.h

index 6812164e1470aa21003b005ed045e869066522b5..a202d552582252d6b7628001bc7c698989628bd1 100644 (file)
@@ -143,7 +143,7 @@ mem_type_to_migrate(struct xe_device *xe, u32 mem_type)
        return tile->migrate;
 }
 
-static struct xe_mem_region *res_to_mem_region(struct ttm_resource *res)
+static struct xe_vram_region *res_to_mem_region(struct ttm_resource *res)
 {
        struct xe_device *xe = ttm_to_xe_device(res->bo->bdev);
        struct ttm_resource_manager *mgr;
@@ -179,7 +179,7 @@ static void add_vram(struct xe_device *xe, struct xe_bo *bo,
                     struct ttm_place *places, u32 bo_flags, u32 mem_type, u32 *c)
 {
        struct ttm_place place = { .mem_type = mem_type };
-       struct xe_mem_region *vram;
+       struct xe_vram_region *vram;
        u64 io_size;
 
        xe_assert(xe, *c < ARRAY_SIZE(bo->placements));
@@ -468,7 +468,7 @@ static int xe_ttm_io_mem_reserve(struct ttm_device *bdev,
                return 0;
        case XE_PL_VRAM0:
        case XE_PL_VRAM1: {
-               struct xe_mem_region *vram = res_to_mem_region(mem);
+               struct xe_vram_region *vram = res_to_mem_region(mem);
 
                if (!xe_ttm_resource_visible(mem))
                        return -EINVAL;
@@ -815,7 +815,7 @@ static int xe_bo_move(struct ttm_buffer_object *ttm_bo, bool evict,
 
                        /* Create a new VMAP once kernel BO back in VRAM */
                        if (!ret && resource_is_vram(new_mem)) {
-                               struct xe_mem_region *vram = res_to_mem_region(new_mem);
+                               struct xe_vram_region *vram = res_to_mem_region(new_mem);
                                void __iomem *new_addr = vram->mapping +
                                        (new_mem->start << PAGE_SHIFT);
 
@@ -1025,7 +1025,7 @@ static unsigned long xe_ttm_io_mem_pfn(struct ttm_buffer_object *ttm_bo,
 {
        struct xe_bo *bo = ttm_to_xe_bo(ttm_bo);
        struct xe_res_cursor cursor;
-       struct xe_mem_region *vram;
+       struct xe_vram_region *vram;
 
        if (ttm_bo->resource->mem_type == XE_PL_STOLEN)
                return xe_ttm_stolen_io_offset(bo, page_offset << PAGE_SHIFT) >> PAGE_SHIFT;
@@ -1165,7 +1165,7 @@ static int xe_ttm_access_memory(struct ttm_buffer_object *ttm_bo,
        struct xe_device *xe = ttm_to_xe_device(ttm_bo->bdev);
        struct iosys_map vmap;
        struct xe_res_cursor cursor;
-       struct xe_mem_region *vram;
+       struct xe_vram_region *vram;
        int bytes_left = len;
 
        xe_bo_assert_held(bo);
index c0e886bac1831acf72e82358cce077adfc1d0afa..1f27ed2c0448c39386b7d319ba3f396828561a6c 100644 (file)
@@ -70,11 +70,11 @@ struct xe_pxp;
                 struct xe_tile * : (tile__)->xe)
 
 /**
- * struct xe_mem_region - memory region structure
+ * struct xe_vram_region - memory region structure
  * This is used to describe a memory region in xe
  * device, such as HBM memory or CXL extension memory.
  */
-struct xe_mem_region {
+struct xe_vram_region {
        /** @io_start: IO start address of this VRAM instance */
        resource_size_t io_start;
        /**
@@ -197,7 +197,7 @@ struct xe_tile {
                 * Although VRAM is associated with a specific tile, it can
                 * still be accessed by all tiles' GTs.
                 */
-               struct xe_mem_region vram;
+               struct xe_vram_region vram;
 
                /** @mem.vram_mgr: VRAM TTM manager */
                struct xe_ttm_vram_mgr *vram_mgr;
@@ -369,7 +369,7 @@ struct xe_device {
        /** @mem: memory info for device */
        struct {
                /** @mem.vram: VRAM info for device */
-               struct xe_mem_region vram;
+               struct xe_vram_region vram;
                /** @mem.sys_mgr: system TTM manager */
                struct ttm_resource_manager sys_mgr;
        } mem;
index f4a16e5fa7700965fbd7a0e5734d61606d8a4ef6..a8c37fb4fec089ab4cca89a0dbbfc6d77a4b60fd 100644 (file)
@@ -340,7 +340,7 @@ int __xe_ttm_vram_mgr_init(struct xe_device *xe, struct xe_ttm_vram_mgr *mgr,
 int xe_ttm_vram_mgr_init(struct xe_tile *tile, struct xe_ttm_vram_mgr *mgr)
 {
        struct xe_device *xe = tile_to_xe(tile);
-       struct xe_mem_region *vram = &tile->mem.vram;
+       struct xe_vram_region *vram = &tile->mem.vram;
 
        mgr->vram = vram;
        return __xe_ttm_vram_mgr_init(xe, mgr, XE_PL_VRAM0 + tile->id,
index 2d75cf1262893fb01b30d412e533128a2cd7804b..4c52ced4ee44726697527dc2b45f957cc7779e53 100644 (file)
@@ -9,7 +9,7 @@
 #include <drm/drm_buddy.h>
 #include <drm/ttm/ttm_device.h>
 
-struct xe_mem_region;
+struct xe_vram_region;
 
 /**
  * struct xe_ttm_vram_mgr - XE TTM VRAM manager
@@ -22,7 +22,7 @@ struct xe_ttm_vram_mgr {
        /** @mm: DRM buddy allocator which manages the VRAM */
        struct drm_buddy mm;
        /** @vram: ptr to details of associated VRAM region */
-       struct xe_mem_region *vram;
+       struct xe_vram_region *vram;
        /** @visible_size: Proped size of the CPU visible portion */
        u64 visible_size;
        /** @visible_avail: CPU visible portion still unallocated */