[PATCH] drm/xe/pf: Use correct function to check LMEM provisioning
Ghimiray, Himal Prasad
himal.prasad.ghimiray at intel.com
Thu Dec 19 03:24:43 UTC 2024
On 17-12-2024 04:02, Michal Wajdeczko wrote:
> There is a typo in function call and instead of VF LMEM we were
> looking at VF GGTT provisioning. Fix that.
>
> Fixes: 234670cea9a2 ("drm/xe/pf: Skip fair VFs provisioning if already provisioned")
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski at intel.com>
> ---
> drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> index 65082f12f1a8..bd621df3ab91 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> @@ -2120,7 +2120,7 @@ static int pf_validate_vf_config(struct xe_gt *gt, unsigned int vfid)
> valid_any = valid_any || (valid_ggtt && is_primary);
>
> if (IS_DGFX(xe)) {
> - bool valid_lmem = pf_get_vf_config_ggtt(primary_gt, vfid);
> + bool valid_lmem = pf_get_vf_config_lmem(primary_gt, vfid);
Looks Good.
Reviewed-by: Himal Prasad Ghimiray <himal.prasad.ghimiray at intel.com>
>
> valid_any = valid_any || (valid_lmem && is_primary);
> valid_all = valid_all && valid_lmem;
More information about the Intel-xe
mailing list