Re: [PATCH] drm/xe/pf: Use correct function to check LMEM provisioning

["Ghimiray, Himal Prasad" <himal.prasad.ghimiray@intel.com>]

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@intel.com>
> Cc: Piotr Piórkowski <piotr.piorkowski@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@intel.com>

>   
>   		valid_any = valid_any || (valid_lmem && is_primary);
>   		valid_all = valid_all && valid_lmem;