drm/xe: Consolidate LRC offset calculations

Attempt to consolidate the LRC offsets calculations by aligning the
recently added wa_bb_offset with the naming scheme in the file and
also change the size stored in struct xe_lrc to not include the ring
buffer.

The former makes it somewhat visually easier to follow the layout of the
various logical blocks stored in the LRC bo, while the latter reduces the
number of sprinkled around calculations.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@igalia.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: Lucas De Marchi <lucas.demarchi@intel.com>
Reviewed-by: Lucas De Marchi <lucas.demarchi@intel.com>
Link: https://lore.kernel.org/r/20250630124711.8209-2-tvrtko.ursulin@igalia.com
Signed-off-by: Lucas De Marchi <lucas.demarchi@intel.com>

diff --git a/drivers/gpu/drm/xe/xe_lrc.c b/drivers/gpu/drm/xe/xe_lrc.c
index c92e7eff79f76bc9425965213943312d8a331922..d2ad8fe737eb383e1f3326854a538346854be319 100644 (file)
--- a/drivers/gpu/drm/xe/xe_lrc.c
+++ b/drivers/gpu/drm/xe/xe_lrc.c
@@ -717,8 +717,12 @@ static u32 __xe_lrc_ctx_timestamp_udw_offset(struct xe_lrc *lrc)
 
 static inline u32 __xe_lrc_indirect_ring_offset(struct xe_lrc *lrc)
 {
-       /* Indirect ring state page is at the very end of LRC */
-       return lrc->size - LRC_INDIRECT_RING_STATE_SIZE;
+       return xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE - LRC_INDIRECT_RING_STATE_SIZE;
+}
+
+static inline u32 __xe_lrc_wa_bb_offset(struct xe_lrc *lrc)
+{
+       return xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE;
 }
 
 #define DECL_MAP_ADDR_HELPERS(elem) \
@@ -973,11 +977,6 @@ struct wa_bb_setup {
                         u32 *batch, size_t max_size);
 };
 
-static size_t wa_bb_offset(struct xe_lrc *lrc)
-{
-       return xe_bo_size(lrc->bo) - LRC_WA_BB_SIZE;
-}
-
 static int setup_wa_bb(struct xe_lrc *lrc, struct xe_hw_engine *hwe)
 {
        const size_t max_size = LRC_WA_BB_SIZE;
@@ -993,7 +992,7 @@ static int setup_wa_bb(struct xe_lrc *lrc, struct xe_hw_engine *hwe)
                        return -ENOMEM;
                cmd = buf;
        } else {
-               cmd = lrc->bo->vmap.vaddr + wa_bb_offset(lrc);
+               cmd = lrc->bo->vmap.vaddr + __xe_lrc_wa_bb_offset(lrc);
        }
 
        remain = max_size / sizeof(*cmd);
@@ -1017,13 +1016,13 @@ static int setup_wa_bb(struct xe_lrc *lrc, struct xe_hw_engine *hwe)
 
        if (buf) {
                xe_map_memcpy_to(gt_to_xe(lrc->gt), &lrc->bo->vmap,
-                                wa_bb_offset(lrc), buf,
+                                __xe_lrc_wa_bb_offset(lrc), buf,
                                 (cmd - buf) * sizeof(*cmd));
                kfree(buf);
        }
 
        xe_lrc_write_ctx_reg(lrc, CTX_BB_PER_CTX_PTR, xe_bo_ggtt_addr(lrc->bo) +
-                            wa_bb_offset(lrc) + 1);
+                            __xe_lrc_wa_bb_offset(lrc) + 1);
 
        return 0;
 
@@ -1040,19 +1039,22 @@ static int xe_lrc_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe,
                       u32 init_flags)
 {
        struct xe_gt *gt = hwe->gt;
+       const u32 lrc_size = xe_gt_lrc_size(gt, hwe->class);
+       const u32 bo_size = ring_size + lrc_size + LRC_WA_BB_SIZE;
        struct xe_tile *tile = gt_to_tile(gt);
        struct xe_device *xe = gt_to_xe(gt);
        struct iosys_map map;
        void *init_data = NULL;
        u32 arb_enable;
-       u32 lrc_size;
        u32 bo_flags;
        int err;
 
        kref_init(&lrc->refcount);
        lrc->gt = gt;
+       lrc->size = lrc_size;
        lrc->flags = 0;
-       lrc_size = ring_size + xe_gt_lrc_size(gt, hwe->class);
+       lrc->ring.size = ring_size;
+       lrc->ring.tail = 0;
        if (xe_gt_has_indirect_ring_state(gt))
                lrc->flags |= XE_LRC_FLAG_INDIRECT_RING_STATE;
 
@@ -1065,17 +1067,12 @@ static int xe_lrc_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe,
         * FIXME: Perma-pinning LRC as we don't yet support moving GGTT address
         * via VM bind calls.
         */
-       lrc->bo = xe_bo_create_pin_map(xe, tile, NULL,
-                                      lrc_size + LRC_WA_BB_SIZE,
+       lrc->bo = xe_bo_create_pin_map(xe, tile, NULL, bo_size,
                                       ttm_bo_type_kernel,
                                       bo_flags);
        if (IS_ERR(lrc->bo))
                return PTR_ERR(lrc->bo);
 
-       lrc->size = lrc_size;
-       lrc->ring.size = ring_size;
-       lrc->ring.tail = 0;
-
        xe_hw_fence_ctx_init(&lrc->fence_ctx, hwe->gt,
                             hwe->fence_irq, hwe->name);
 
@@ -1096,10 +1093,9 @@ static int xe_lrc_init(struct xe_lrc *lrc, struct xe_hw_engine *hwe,
                xe_map_memset(xe, &map, 0, 0, LRC_PPHWSP_SIZE); /* PPHWSP */
                xe_map_memcpy_to(xe, &map, LRC_PPHWSP_SIZE,
                                 gt->default_lrc[hwe->class] + LRC_PPHWSP_SIZE,
-                                xe_gt_lrc_size(gt, hwe->class) - LRC_PPHWSP_SIZE);
+                                lrc_size - LRC_PPHWSP_SIZE);
        } else {
-               xe_map_memcpy_to(xe, &map, 0, init_data,
-                                xe_gt_lrc_size(gt, hwe->class));
+               xe_map_memcpy_to(xe, &map, 0, init_data, lrc_size);
                kfree(init_data);
        }
 
@@ -1859,8 +1855,7 @@ struct xe_lrc_snapshot *xe_lrc_snapshot_capture(struct xe_lrc *lrc)
        snapshot->seqno = xe_lrc_seqno(lrc);
        snapshot->lrc_bo = xe_bo_get(lrc->bo);
        snapshot->lrc_offset = xe_lrc_pphwsp_offset(lrc);
-       snapshot->lrc_size = xe_bo_size(lrc->bo) - snapshot->lrc_offset -
-               LRC_WA_BB_SIZE;
+       snapshot->lrc_size = lrc->size;
        snapshot->lrc_snapshot = NULL;
        snapshot->ctx_timestamp = lower_32_bits(xe_lrc_ctx_timestamp(lrc));
        snapshot->ctx_job_timestamp = xe_lrc_ctx_job_timestamp(lrc);

diff --git a/drivers/gpu/drm/xe/xe_lrc_types.h b/drivers/gpu/drm/xe/xe_lrc_types.h
index 883e550a94234cbcc22173a63e34d60d4a69a617..2c7c810798015bd357236f3a8b462e04cfe85732 100644 (file)
--- a/drivers/gpu/drm/xe/xe_lrc_types.h
+++ b/drivers/gpu/drm/xe/xe_lrc_types.h
@@ -22,7 +22,7 @@ struct xe_lrc {
         */
        struct xe_bo *bo;
 
-       /** @size: size of lrc including any indirect ring state page */
+       /** @size: size of the lrc and optional indirect ring state */
        u32 size;
 
        /** @gt: gt which this LRC belongs to */