Skip to content

Commit c7a5aa4

Browse files
author
Andi Shyti
committed
drm/i915/gt: Do not generate the command streamer for all the CCS
We want a fixed load CCS balancing consisting in all slices sharing one single user engine. For this reason do not create the intel_engine_cs structure with its dedicated command streamer for CCS slices beyond the first. Fixes: d2eae8e ("drm/i915/dg2: Drop force_probe requirement") Signed-off-by: Andi Shyti <[email protected]> Cc: Chris Wilson <[email protected]> Cc: Joonas Lahtinen <[email protected]> Cc: Matt Roper <[email protected]> Cc: <[email protected]> # v6.2+ Acked-by: Michal Mrozek <[email protected]> Reviewed-by: Matt Roper <[email protected]> Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
1 parent f5d2904 commit c7a5aa4

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

drivers/gpu/drm/i915/gt/intel_engine_cs.c

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -908,6 +908,23 @@ static intel_engine_mask_t init_engine_mask(struct intel_gt *gt)
908908
info->engine_mask &= ~BIT(GSC0);
909909
}
910910

911+
/*
912+
* Do not create the command streamer for CCS slices beyond the first.
913+
* All the workload submitted to the first engine will be shared among
914+
* all the slices.
915+
*
916+
* Once the user will be allowed to customize the CCS mode, then this
917+
* check needs to be removed.
918+
*/
919+
if (IS_DG2(gt->i915)) {
920+
u8 first_ccs = __ffs(CCS_MASK(gt));
921+
922+
/* Mask off all the CCS engine */
923+
info->engine_mask &= ~GENMASK(CCS3, CCS0);
924+
/* Put back in the first CCS engine */
925+
info->engine_mask |= BIT(_CCS(first_ccs));
926+
}
927+
911928
return info->engine_mask;
912929
}
913930

0 commit comments

Comments
 (0)