Re: [PATCH v7 12/12] drm/xe: Remove xe_ggtt_node_allocated

[Matthew Brost <matthew.brost@intel.com>]

On Wed, Oct 15, 2025 at 09:47:21AM +0200, Maarten Lankhorst wrote:
> With the intermediate state gone, no longer useful. Just check against
> NULL.
> 

Ah, ok. I suggested earlier in the series to use xe_ggtt_node_allocated
but I see now this function is removed. Before I can give an RB I'd like
to apply the entire series to a branch to make sure I understand this
change. The mbox as is doesn't apply to drm-tip. Any chance you can
rebase this series?

Matt

> Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
> ---
>  drivers/gpu/drm/xe/display/xe_fb_pin.c     |  3 +--
>  drivers/gpu/drm/xe/xe_ggtt.c               | 14 --------------
>  drivers/gpu/drm/xe/xe_ggtt.h               |  1 -
>  drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 12 ++++++------
>  4 files changed, 7 insertions(+), 23 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_fb_pin.c b/drivers/gpu/drm/xe/display/xe_fb_pin.c
> index 784d2db5fd0db..6592b000950ff 100644
> --- a/drivers/gpu/drm/xe/display/xe_fb_pin.c
> +++ b/drivers/gpu/drm/xe/display/xe_fb_pin.c
> @@ -350,8 +350,7 @@ static void __xe_unpin_fb_vma(struct i915_vma *vma)
>  
>  	if (vma->dpt)
>  		xe_bo_unpin_map_no_vm(vma->dpt);
> -	else if (!xe_ggtt_node_allocated(vma->bo->ggtt_node[tile_id]) ||
> -		 vma->bo->ggtt_node[tile_id] != vma->node)
> +	else if (vma->bo->ggtt_node[tile_id] != vma->node)
>  		xe_ggtt_node_remove(vma->node, false);
>  
>  	ttm_bo_reserve(&vma->bo->ttm, false, false, NULL);
> diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
> index e4e8be114786a..e9ab8d9415412 100644
> --- a/drivers/gpu/drm/xe/xe_ggtt.c
> +++ b/drivers/gpu/drm/xe/xe_ggtt.c
> @@ -656,20 +656,6 @@ struct xe_ggtt_node *xe_ggtt_node_insert(struct xe_ggtt *ggtt, u32 size, u32 ali
>  	return node;
>  }
>  
> -/**
> - * xe_ggtt_node_allocated - Check if node is allocated in GGTT
> - * @node: the &xe_ggtt_node to be inspected
> - *
> - * Return: True if allocated, False otherwise.
> - */
> -bool xe_ggtt_node_allocated(const struct xe_ggtt_node *node)
> -{
> -	if (!node || !node->ggtt)
> -		return false;
> -
> -	return drm_mm_node_allocated(&node->base);
> -}
> -
>  /**
>   * xe_ggtt_map_bo - Map the BO into GGTT
>   * @ggtt: the &xe_ggtt where node will be mapped
> diff --git a/drivers/gpu/drm/xe/xe_ggtt.h b/drivers/gpu/drm/xe/xe_ggtt.h
> index d0ad541a40baf..e32a14ac63665 100644
> --- a/drivers/gpu/drm/xe/xe_ggtt.h
> +++ b/drivers/gpu/drm/xe/xe_ggtt.h
> @@ -30,7 +30,6 @@ xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
>  			      u64 size, u32 align,
>  			      xe_ggtt_transform_cb transform, void *arg);
>  void xe_ggtt_node_remove(struct xe_ggtt_node *node, bool invalidate);
> -bool xe_ggtt_node_allocated(const struct xe_ggtt_node *node);
>  void xe_ggtt_map_bo_unlocked(struct xe_ggtt *ggtt, struct xe_bo *bo);
>  int xe_ggtt_insert_bo(struct xe_ggtt *ggtt, struct xe_bo *bo, struct drm_exec *exec);
>  int xe_ggtt_insert_bo_at(struct xe_ggtt *ggtt, struct xe_bo *bo,
> 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 d71041a611d35..db47b49e5842c 100644
> --- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> +++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
> @@ -261,7 +261,7 @@ static u32 encode_config_ggtt(u32 *cfg, const struct xe_gt_sriov_config *config,
>  {
>  	struct xe_ggtt_node *node = config->ggtt_region;
>  
> -	if (!xe_ggtt_node_allocated(node))
> +	if (!node)
>  		return 0;
>  
>  	return encode_ggtt(cfg, xe_ggtt_node_addr(node), config->ggtt_size, details);
> @@ -455,7 +455,7 @@ static int pf_provision_vf_ggtt(struct xe_gt *gt, unsigned int vfid, u64 size)
>  
>  	size = round_up(size, alignment);
>  
> -	if (xe_ggtt_node_allocated(config->ggtt_region)) {
> +	if (config->ggtt_region) {
>  		err = pf_distribute_config_ggtt(tile, vfid, 0, 0);
>  		if (unlikely(err))
>  			return err;
> @@ -466,7 +466,7 @@ static int pf_provision_vf_ggtt(struct xe_gt *gt, unsigned int vfid, u64 size)
>  		if (unlikely(err))
>  			return err;
>  	}
> -	xe_gt_assert(gt, !xe_ggtt_node_allocated(config->ggtt_region));
> +	xe_gt_assert(gt, !config->ggtt_region);
>  
>  	if (!size)
>  		return 0;
> @@ -497,7 +497,7 @@ static u64 pf_get_vf_config_ggtt(struct xe_gt *gt, unsigned int vfid)
>  	struct xe_ggtt_node *node = config->ggtt_region;
>  
>  	xe_gt_assert(gt, xe_gt_is_main_type(gt));
> -	return xe_ggtt_node_allocated(node) ? config->ggtt_size : 0;
> +	return node ? config->ggtt_size : 0;
>  }
>  
>  /**
> @@ -2049,7 +2049,7 @@ int xe_gt_sriov_pf_config_release(struct xe_gt *gt, unsigned int vfid, bool forc
>  
>  static void pf_sanitize_ggtt(struct xe_ggtt_node *ggtt_region, unsigned int vfid)
>  {
> -	if (xe_ggtt_node_allocated(ggtt_region))
> +	if (ggtt_region)
>  		xe_ggtt_assign(ggtt_region, vfid);
>  }
>  
> @@ -2496,7 +2496,7 @@ int xe_gt_sriov_pf_config_print_ggtt(struct xe_gt *gt, struct drm_printer *p)
>  
>  	for (n = 1; n <= total_vfs; n++) {
>  		config = &gt->sriov.pf.vfs[n].config;
> -		if (!xe_ggtt_node_allocated(config->ggtt_region))
> +		if (!config->ggtt_region)
>  			continue;
>  
>  		string_get_size(config->ggtt_size, 1, STRING_UNITS_2,
> -- 
> 2.51.0
>