Re: [PATCH 10/13] drm/xe/pf: Reprovision scheduling to default when no VFs

["Piotr Piórkowski" <piotr.piorkowski@intel.com>]

Michal Wajdeczko <michal.wajdeczko@intel.com> wrote on pon [2026-mar-30 22:40:41 +0200]:
> Once we have disabled VFs there is no point in keeping potentially
> limited PF scheduling provisioning as we want to complete a cleanup
> of the VFs resources and return to the pure native mode, the same
> as before VFs were enabled, as soon as possible.
> 
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
> ---
>  drivers/gpu/drm/xe/xe_pci_sriov.c          |  2 ++
>  drivers/gpu/drm/xe/xe_sriov_pf_provision.c | 39 ++++++++++++++++++++++
>  drivers/gpu/drm/xe/xe_sriov_pf_provision.h |  1 +
>  3 files changed, 42 insertions(+)
> 
> diff --git a/drivers/gpu/drm/xe/xe_pci_sriov.c b/drivers/gpu/drm/xe/xe_pci_sriov.c
> index 3fd22034f03e..35e6b53e18ce 100644
> --- a/drivers/gpu/drm/xe/xe_pci_sriov.c
> +++ b/drivers/gpu/drm/xe/xe_pci_sriov.c
> @@ -191,6 +191,8 @@ static int pf_disable_vfs(struct xe_device *xe)
>  
>  	pci_disable_sriov(pdev);
>  
> +	xe_sriov_pf_reprovision_default(xe);
> +
>  	pf_reset_vfs(xe, num_vfs);
>  
>  	xe_sriov_pf_unprovision_vfs(xe, num_vfs);
> diff --git a/drivers/gpu/drm/xe/xe_sriov_pf_provision.c b/drivers/gpu/drm/xe/xe_sriov_pf_provision.c
> index abe3677d33ed..e11874d689fa 100644
> --- a/drivers/gpu/drm/xe/xe_sriov_pf_provision.c
> +++ b/drivers/gpu/drm/xe/xe_sriov_pf_provision.c
> @@ -103,6 +103,45 @@ int xe_sriov_pf_unprovision_vfs(struct xe_device *xe, unsigned int num_vfs)
>  	return 0;
>  }
>  
> +static int pf_reprovision_default(struct xe_device *xe)
> +{
> +	struct xe_gt *gt;
> +	unsigned int id;
> +	int result = 0;
> +	int err;
> +
> +	guard(mutex)(xe_sriov_pf_master_mutex(xe));
> +
> +	for_each_gt(gt, xe, id) {
> +		err = xe_gt_sriov_pf_policy_set_sched_if_idle_locked(gt, false);
> +		result = result ?: err;
> +		err = xe_gt_sriov_pf_config_set_exec_quantum_locked(gt, PFID, 0);
> +		result = result ?: err;
> +		err = xe_gt_sriov_pf_config_set_preempt_timeout_locked(gt, PFID, 0);
> +		result = result ?: err;
> +	}
> +
> +	return result;
> +}
> +
> +/**
> + * xe_sriov_pf_reprovision_default() - Reprovision default PF in auto-mode.
> + * @xe: the PF &xe_device
> + *
> + * This function can only be called on PF.
> + *
> + * Return: 0 on success or a negative error code on failure.
> + */
> +int xe_sriov_pf_reprovision_default(struct xe_device *xe)
> +{
> +	xe_assert(xe, IS_SRIOV_PF(xe));
> +
> +	if (!pf_auto_provisioning_mode(xe))
> +		return 0;
> +
> +	return pf_reprovision_default(xe);
> +}
> +
>  /**
>   * xe_sriov_pf_provision_set_mode() - Change VFs provision mode.
>   * @xe: the PF &xe_device
> diff --git a/drivers/gpu/drm/xe/xe_sriov_pf_provision.h b/drivers/gpu/drm/xe/xe_sriov_pf_provision.h
> index f26f49539697..b15e4d7ad940 100644
> --- a/drivers/gpu/drm/xe/xe_sriov_pf_provision.h
> +++ b/drivers/gpu/drm/xe/xe_sriov_pf_provision.h
> @@ -30,6 +30,7 @@ int xe_sriov_pf_provision_query_vf_vram(struct xe_device *xe, unsigned int vfid,
>  
>  int xe_sriov_pf_provision_vfs(struct xe_device *xe, unsigned int num_vfs);
>  int xe_sriov_pf_unprovision_vfs(struct xe_device *xe, unsigned int num_vfs);
> +int xe_sriov_pf_reprovision_default(struct xe_device *xe);
>  
>  int xe_sriov_pf_provision_set_mode(struct xe_device *xe, enum xe_sriov_provisioning_mode mode);

LGTM:
Reviewed-by: Piotr Piórkowski <piotr.piorkowski@intel.com>

>  
> -- 
> 2.47.1
> 

--