Re: [PATCH] drm/xe/xe3p: Disable GuC Dynamic ICS for Xe3p

[Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>]

On 1/21/2026 6:37 AM, Balasubramani Vivekanandan wrote:
> From: Gustavo Sousa <gustavo.sousa@intel.com>
>
> The Dynamic Inhibit Context Switch (ICS) feature is not currently
> supported on Xe3p due to conflict with multi-queue feature on the GuC
> side. Disable ICS so multi-queue can be introduced for Xe3p platforms.
>
> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
> Signed-off-by: Gustavo Sousa <gustavo.sousa@intel.com>
> Signed-off-by: Balasubramani Vivekanandan <balasubramani.vivekanandan@intel.com>
> ---
>   drivers/gpu/drm/xe/xe_guc.c | 7 +++++++
>   1 file changed, 7 insertions(+)
>
> diff --git a/drivers/gpu/drm/xe/xe_guc.c b/drivers/gpu/drm/xe/xe_guc.c
> index 44360437beeb..e350ded92544 100644
> --- a/drivers/gpu/drm/xe/xe_guc.c
> +++ b/drivers/gpu/drm/xe/xe_guc.c
> @@ -608,6 +608,13 @@ static bool supports_dynamic_ics(struct xe_guc *guc)
>   	if (xe_gt_is_media_type(gt) || gt->ccs_mode > 1)
>   		return false;
>   
> +	/*
> +	 * The feature is currently not compatible with platforms with support
> +	 * to multi-queue, even if the feature is not enabled/used.
> +	 */
> +	if (GRAPHICS_VER(xe) >= 35)
> +		return false;

This has been fixed in GuC 70.50.0, so this check shouldn't be 
necessary. Are you seeing any issue with that GuC version or newer?

Daniele

> +
>   	/*
>   	 * Dynamic ICS requires GuC v70.40.1, which maps to compatibility
>   	 * version v1.18.4.