Commit 08d05c73 authored by Maarten Lankhorst's avatar Maarten Lankhorst
Browse files

drm/xe: Remove xe_ggtt_node_allocated 



With the intermediate state gone, no longer useful. Just check against
NULL where needed.

After looking carefully, the check for allocated in xe_fb_pin.c is
unneeded. vma->node is never NULL. The check is specifically only
to check if vma->node == the bo's root tile ggtt_obj.

Reviewed-by: default avatarMichal Wajdeczko <michal.wajdeczko@intel.com>
Link: https://patch.msgid.link/20260206112108.1453809-12-dev@lankhorst.se


Signed-off-by: default avatarMaarten Lankhorst <dev@lankhorst.se>
parent a4eac88e

drivers/gpu/drm/xe/display/xe_fb_pin.c

+1 −2
Original line number Diff line number Diff line
@@ -352,8 +352,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);

drivers/gpu/drm/xe/xe_ggtt.c

+0 −17
Original line number Diff line number Diff line
@@ -665,20 +665,6 @@ struct xe_ggtt_node *xe_ggtt_insert_node(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_node_pt_size() - Get the size of page table entries needed to map a GGTT node.

 * @node: the &xe_ggtt_node
@@ -971,9 +957,6 @@ static void xe_ggtt_assign_locked(const struct xe_ggtt_node *node, u16 vfid) 


	lockdep_assert_held(&ggtt->lock);



	if (!xe_ggtt_node_allocated(node))

		return;



	while (start < end) {

		ggtt->pt_ops->ggtt_set_pte(ggtt, start, pte);

		start += XE_PAGE_SIZE;

drivers/gpu/drm/xe/xe_ggtt.h

+0 −1
Original line number Diff line number Diff line
@@ -30,7 +30,6 @@ xe_ggtt_insert_node_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);

size_t xe_ggtt_node_pt_size(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);

drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c

+6 −6
Original line number Diff line number Diff line
@@ -279,7 +279,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), xe_ggtt_node_size(node), details);
@@ -503,7 +503,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;
@@ -514,7 +514,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;
@@ -544,7 +544,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) ? xe_ggtt_node_size(node) : 0;

	return node ? xe_ggtt_node_size(node) : 0;

}



/**
@@ -2558,7 +2558,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);

}
@@ -3017,7 +3017,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(xe_ggtt_node_size(config->ggtt_region), 1, STRING_UNITS_2,