Re: [PATCH v3 5/5] drm/xe/vf: Do not disable VF migration on ATS-M

[Michal Wajdeczko <michal.wajdeczko@intel.com>]

On 10/16/2025 2:05 PM, Tomasz Lis wrote:
> The ATS-M does support VF migration, and it has graphics ver below
> the currently allowed range. Experimental support of this platform
> should therefore be available.
> 
> This change allows experimental VF migration support on ATS-M. It
> is also explicitly not allowing ADL, by adding condition on MEMIRQ.
> Supporting VF migration through MMIO interrupts would require
> some special handling in order to achieve reliability.
> 
> v2: Add MEMIRQ condition
> 
> Signed-off-by: Tomasz Lis <tomasz.lis@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_sriov_vf.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_sriov_vf.c b/drivers/gpu/drm/xe/xe_sriov_vf.c
> index 5fb042c05112..fb2b3d41c610 100644
> --- a/drivers/gpu/drm/xe/xe_sriov_vf.c
> +++ b/drivers/gpu/drm/xe/xe_sriov_vf.c
> @@ -159,10 +159,13 @@ static void vf_migration_init_early(struct xe_device *xe)
>  		return vf_disable_migration(xe,
>  					    "experimental feature not available on production builds");
>  
> -	if (GRAPHICS_VER(xe) < 20)
> -		return vf_disable_migration(xe, "requires gfx version >= 20, but only %u found",
> +	if (GRAPHICS_VER(xe) < 12)
> +		return vf_disable_migration(xe, "requires gfx version >= 12, but only %u found",
>  					    GRAPHICS_VER(xe));

maybe just drop it completely and only rely on memirq support ?

it's available from 12.50, see

static inline bool xe_device_has_memirq(struct xe_device *xe)
{
	return GRAPHICS_VERx100(xe) >= 1250;
}

>  > +	if (!xe_device_has_memirq(xe))
> +		return vf_disable_migration(xe, "requires memory-based IRQ support");
> +
>  	xe->sriov.vf.migration.enabled = true;
>  	xe_sriov_dbg(xe, "migration support enabled\n");
>  }