Re: [PATCH v10 06/20] drm/xe/vsec: Do not register BMG PMT for VF

[Rodrigo Vivi <rodrigo.vivi@intel.com>]

On Fri, Sep 18, 2026 at 06:17:57AM -0700, Michael J. Ruhl wrote:
> BMG PMT is not supported for VF nodes.
> 
> If IS_SRIOV_VF() is true, do NOT register with PMT.
> 
> Signed-off-by: Michael J. Ruhl <michael.j.ruhl@intel.com>

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>

> ---
>  drivers/gpu/drm/xe/xe_vsec.c | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/xe_vsec.c b/drivers/gpu/drm/xe/xe_vsec.c
> index a9baf0bfe572..481bcbcc6372 100644
> --- a/drivers/gpu/drm/xe/xe_vsec.c
> +++ b/drivers/gpu/drm/xe/xe_vsec.c
> @@ -149,6 +149,8 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  	u32 offset;
>  	int ret;
>  
> +	xe_assert(xe, !IS_SRIOV_VF(xe));
> +
>  	ret = xe_guid_decode(guid, &mem_region, &offset);
>  	if (ret)
>  		return ret;
> @@ -157,9 +159,6 @@ int xe_pmt_telem_read(struct device *dev, u32 guid, u64 *data, loff_t user_offse
>  
>  	guard(mutex)(&xe->pmt.lock);
>  
> -	if (!xe->soc_remapper.set_telem_region)
> -		return -ENODEV;
> -
>  	/* indicate that we are not at an appropriate power level */
>  	if (!xe_pm_runtime_get_if_active(xe))
>  		return -ENODATA;
> @@ -186,6 +185,9 @@ static enum xe_vsec get_platform_info(struct xe_device *xe)
>  	if (xe->info.platform > XE_BATTLEMAGE)
>  		return XE_VSEC_UNKNOWN;
>  
> +	if (IS_SRIOV_VF(xe) && (xe->info.platform == XE_BATTLEMAGE))
> +		return XE_VSEC_UNKNOWN;
> +
>  	return vsec_platforms[xe->info.platform];
>  }
>  
> -- 
> 2.43.0
>