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

["Vivekanandan, Balasubramani" <balasubramani.vivekanandan@intel.com>]

On 21.01.2026 13:06, Daniele Ceraolo Spurio wrote:
> 
> 
> 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?

okay, I wasn't aware it is fixed in 70.50.0. Thanks.
We don't need this check then.

Regards,
Bala

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