drm/xe: Add define WQ_HEADER_SIZE
authorMatthew Brost <matthew.brost@intel.com>
Fri, 28 Jul 2023 02:36:00 +0000 (19:36 -0700)
committerRodrigo Vivi <rodrigo.vivi@intel.com>
Thu, 21 Dec 2023 16:39:16 +0000 (11:39 -0500)
Previously used a a magic '+ 3', use define instead.

Suggested-by: Oded Gabbay <ogabbay@kernel.org>
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Matthew Auld <matthew.auld@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
drivers/gpu/drm/xe/xe_guc_submit.c

index 99c9b71..a2eeb3f 100644 (file)
@@ -558,8 +558,9 @@ static void wq_item_append(struct xe_engine *e)
        struct xe_guc *guc = engine_to_guc(e);
        struct xe_device *xe = guc_to_xe(guc);
        struct iosys_map map = xe_lrc_parallel_map(e->lrc);
-       u32 wqi[XE_HW_ENGINE_MAX_INSTANCE + 3];
-       u32 wqi_size = (e->width + 3) * sizeof(u32);
+#define WQ_HEADER_SIZE 4       /* Includes 1 LRC address too */
+       u32 wqi[XE_HW_ENGINE_MAX_INSTANCE + (WQ_HEADER_SIZE - 1)];
+       u32 wqi_size = (e->width + (WQ_HEADER_SIZE - 1)) * sizeof(u32);
        u32 len_dw = (wqi_size / sizeof(u32)) - 1;
        int i = 0, j;