[PATCH 1/5] drm/xe: Drop __engine_mask

Lucas De Marchi lucas.demarchi at intel.com
Mon May 13 21:35:31 UTC 2024


On Thu, Apr 25, 2024 at 11:24:06AM GMT, Lucas De Marchi wrote:
>Not really used, it's just a copy of engine_mask, which already reads
>the fuses to mark engines as available/not-available.
>
>While at it, use XE_HW_ENGINE_BCS_MASK to span all copy engines.
>
>Signed-off-by: Lucas De Marchi <lucas.demarchi at intel.com>
>---
> drivers/gpu/drm/xe/xe_gt.c       | 3 ---
> drivers/gpu/drm/xe/xe_gt_types.h | 6 ------
> drivers/gpu/drm/xe/xe_migrate.c  | 3 +--
> drivers/gpu/drm/xe/xe_pci.c      | 6 +++---
> 4 files changed, 4 insertions(+), 14 deletions(-)
>
>diff --git a/drivers/gpu/drm/xe/xe_gt.c b/drivers/gpu/drm/xe/xe_gt.c
>index e922e77f5010..00a22cf2f5b5 100644
>--- a/drivers/gpu/drm/xe/xe_gt.c
>+++ b/drivers/gpu/drm/xe/xe_gt.c
>@@ -515,9 +515,6 @@ int xe_gt_init_hwconfig(struct xe_gt *gt)
> 	if (err)
> 		goto out_fw;
>
>-	/* XXX: Fake that we pull the engine mask from hwconfig blob */
>-	gt->info.engine_mask = gt->info.__engine_mask;
>-
> out_fw:
> 	xe_force_wake_put(gt_to_fw(gt), XE_FW_GT);
> out:
>diff --git a/drivers/gpu/drm/xe/xe_gt_types.h b/drivers/gpu/drm/xe/xe_gt_types.h
>index cfdc761ff7f4..72568414fb7d 100644
>--- a/drivers/gpu/drm/xe/xe_gt_types.h
>+++ b/drivers/gpu/drm/xe/xe_gt_types.h
>@@ -116,12 +116,6 @@ struct xe_gt {
> 		u32 reference_clock;
> 		/** @info.engine_mask: mask of engines present on GT */
> 		u64 engine_mask;
>-		/**
>-		 * @info.__engine_mask: mask of engines present on GT read from
>-		 * xe_pci.c, used to fake reading the engine_mask from the
>-		 * hwconfig blob.
>-		 */
>-		u64 __engine_mask;
> 		/** @info.gmdid: raw GMD_ID value from hardware */
> 		u32 gmdid;
> 	} info;
>diff --git a/drivers/gpu/drm/xe/xe_migrate.c b/drivers/gpu/drm/xe/xe_migrate.c
>index 9f6e9b7f11c8..59a3f24d31e6 100644
>--- a/drivers/gpu/drm/xe/xe_migrate.c
>+++ b/drivers/gpu/drm/xe/xe_migrate.c
>@@ -936,8 +936,7 @@ static bool has_service_copy_support(struct xe_gt *gt)
> 	 * all of the actual service copy engines (BCS1-BCS8) have been fused
> 	 * off.
> 	 */
>-	return gt->info.__engine_mask & GENMASK(XE_HW_ENGINE_BCS8,
>-						XE_HW_ENGINE_BCS1);
>+	return gt->info.engine_mask & XE_HW_ENGINE_BCS_MASK;

oh well, stupid me

has_service_copy_support() means bcs1 to bcs8, not bcs0.

Lucas De Marchi


More information about the Intel-xe mailing list