drm/xe: add XE_BO_CREATE_VRAM_MASK
authorMatthew Auld <matthew.auld@intel.com>
Thu, 23 Mar 2023 11:59:21 +0000 (11:59 +0000)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Tue, 19 Dec 2023 23:30:26 +0000 (18:30 -0500)
So we don't have to keep repeating VRAM0 | VRAM1. Also if there are ever
more instances, then we have less places to update.

Suggested-by: José Roberto de Souza <jose.souza@intel.com>
Signed-off-by: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: José Roberto de Souza <jose.souza@intel.com>
Reviewed-by: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_bo.c
drivers/gpu/drm/xe/xe_bo.h

index 42a5978ecc7427263b5779f8816bb2b45e5dcc31..448c4305480c3b8cc51dec7cd92e3dc153a41642 100644 (file)
@@ -999,8 +999,7 @@ struct xe_bo *__xe_bo_create_locked(struct xe_device *xe, struct xe_bo *bo,
                        return bo;
        }
 
-       if (flags & (XE_BO_CREATE_VRAM0_BIT | XE_BO_CREATE_VRAM1_BIT |
-                    XE_BO_CREATE_STOLEN_BIT) &&
+       if (flags & (XE_BO_CREATE_VRAM_MASK | XE_BO_CREATE_STOLEN_BIT) &&
            !(flags & XE_BO_CREATE_IGNORE_MIN_PAGE_SIZE_BIT) &&
            xe->info.vram_flags & XE_VRAM_FLAGS_NEED64K) {
                size = ALIGN(size, SZ_64K);
@@ -1064,8 +1063,7 @@ static int __xe_bo_fixed_placement(struct xe_device *xe,
        place->fpfn = start >> PAGE_SHIFT;
        place->lpfn = end >> PAGE_SHIFT;
 
-       switch (flags & (XE_BO_CREATE_STOLEN_BIT |
-               XE_BO_CREATE_VRAM0_BIT |XE_BO_CREATE_VRAM1_BIT)) {
+       switch (flags & (XE_BO_CREATE_STOLEN_BIT | XE_BO_CREATE_VRAM_MASK)) {
        case XE_BO_CREATE_VRAM0_BIT:
                place->mem_type = XE_PL_VRAM0;
                break;
@@ -1771,7 +1769,7 @@ bool xe_bo_needs_ccs_pages(struct xe_bo *bo)
 {
        return bo->ttm.type == ttm_bo_type_device &&
                !(bo->flags & XE_BO_CREATE_SYSTEM_BIT) &&
-               (bo->flags & (XE_BO_CREATE_VRAM0_BIT | XE_BO_CREATE_VRAM1_BIT));
+               (bo->flags & XE_BO_CREATE_VRAM_MASK);
 }
 
 /**
index e38894c1255d701f2c760fb5fa63f38789d52621..7b2104d1bda77415f4495d39d99d6855e26b7734 100644 (file)
@@ -17,6 +17,8 @@
 #define XE_BO_CREATE_SYSTEM_BIT                BIT(1)
 #define XE_BO_CREATE_VRAM0_BIT         BIT(2)
 #define XE_BO_CREATE_VRAM1_BIT         BIT(3)
+#define XE_BO_CREATE_VRAM_MASK         (XE_BO_CREATE_VRAM0_BIT | \
+                                        XE_BO_CREATE_VRAM1_BIT)
 /* -- */
 #define XE_BO_CREATE_STOLEN_BIT                BIT(4)
 #define XE_BO_CREATE_VRAM_IF_DGFX(gt) \