[PATCH] drm/xe/pf: Update the LMTT when freeing VF GT config

Piotr Piórkowski piotr.piorkowski at intel.com
Fri May 31 11:37:09 UTC 2024


Michal Wajdeczko <michal.wajdeczko at intel.com> wrote on pon [2024-maj-27 13:54:08 +0200]:
> The LMTT must be updated whenever we change the VF LMEM configuration.
> We missed that step when freeing the whole VF GT config, which could
> result in stale PTE in LMTT or LMTT PT object leaks. Fix that.
> 
> Fixes: ac6598aed1b3 ("drm/xe/pf: Add support to configure SR-IOV VFs")
> Signed-off-by: Michal Wajdeczko <michal.wajdeczko at intel.com>
> ---
>  drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> 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 f678cd1ad9c5..f49fc2917f93 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> @@ -1842,6 +1842,7 @@ static void pf_release_vf_config(struct xe_gt *gt, unsigned int vfid)
>  	if (!xe_gt_is_media_type(gt)) {
>  		pf_release_vf_config_ggtt(gt, config);
>  		pf_release_vf_config_lmem(gt, config);
> +		pf_update_vf_lmtt(gt_to_xe(gt), vfid);
>  	}
>  	pf_release_config_ctxs(gt, config);
>  	pf_release_config_dbs(gt, config);

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

> -- 
> 2.43.0
> 

-- 


More information about the Intel-xe mailing list