Unverified Commit 0567f18e authored by Rodrigo Vivi's avatar Rodrigo Vivi
Browse files

drm/xe: Rename xe_ggtt_node related functions 



Bring some consistency and prepare for more xe_ggtt_node related
functions to be introduced.

Reviewed-by: default avatarMatthew Brost <matthew.brost@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240821193842.352557-6-rodrigo.vivi@intel.com


Signed-off-by: default avatarRodrigo Vivi <rodrigo.vivi@intel.com>
parent 6062ea93

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

+3 −5
Original line number Diff line number Diff line
@@ -209,8 +209,7 @@ static int __xe_pin_fb_vma_ggtt(const struct intel_framebuffer *fb, 
	} else if (view->type == I915_GTT_VIEW_NORMAL) {

		u32 x, size = bo->ttm.base.size;



		ret = xe_ggtt_insert_special_node_locked(ggtt, &vma->node, size,

							 align, 0);

		ret = xe_ggtt_node_insert_locked(ggtt, &vma->node, size, align, 0);

		if (ret)

			goto out_unlock;
@@ -227,8 +226,7 @@ static int __xe_pin_fb_vma_ggtt(const struct intel_framebuffer *fb, 
		/* display seems to use tiles instead of bytes here, so convert it back.. */

		u32 size = intel_rotation_info_size(rot_info) * XE_PAGE_SIZE;



		ret = xe_ggtt_insert_special_node_locked(ggtt, &vma->node, size,

							 align, 0);

		ret = xe_ggtt_node_insert_locked(ggtt, &vma->node, size, align, 0);

		if (ret)

			goto out_unlock;
@@ -327,7 +325,7 @@ static void __xe_unpin_fb_vma(struct i915_vma *vma) 
		xe_bo_unpin_map_no_vm(vma->dpt);

	else if (!drm_mm_node_allocated(&vma->bo->ggtt_node.base) ||

		 vma->bo->ggtt_node.base.start != vma->node.base.start)

		xe_ggtt_remove_node(ggtt, &vma->node, false);

		xe_ggtt_node_remove(ggtt, &vma->node, false);



	ttm_bo_reserve(&vma->bo->ttm, false, false, NULL);

	ttm_bo_unpin(&vma->bo->ttm);

drivers/gpu/drm/xe/xe_ggtt.c

+43 −43
Original line number Diff line number Diff line
@@ -403,7 +403,7 @@ void xe_ggtt_deballoon(struct xe_ggtt *ggtt, struct xe_ggtt_node *node) 
}



/**

 * xe_ggtt_insert_special_node_locked - Locked version to insert a &xe_ggtt_node into the GGTT

 * xe_ggtt_node_insert_locked - Locked version to insert a &xe_ggtt_node into the GGTT

 * @ggtt: the &xe_ggtt where node will be inserted

 * @node: the &xe_ggtt_node to be inserted

 * @size: size of the node
@@ -414,7 +414,7 @@ void xe_ggtt_deballoon(struct xe_ggtt *ggtt, struct xe_ggtt_node *node) 
 *

 * Return: 0 on success or a negative error code on failure.

 */

int xe_ggtt_insert_special_node_locked(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

int xe_ggtt_node_insert_locked(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			       u32 size, u32 align, u32 mm_flags)

{

	return drm_mm_insert_node_generic(&ggtt->mm, &node->base, size, align, 0,
@@ -422,7 +422,7 @@ int xe_ggtt_insert_special_node_locked(struct xe_ggtt *ggtt, struct xe_ggtt_node 
}



/**

 * xe_ggtt_insert_special_node - Insert a &xe_ggtt_node into the GGTT

 * xe_ggtt_node_insert - Insert a &xe_ggtt_node into the GGTT

 * @ggtt: the &xe_ggtt where node will be inserted

 * @node: the &xe_ggtt_node to be inserted

 * @size: size of the node
@@ -430,19 +430,53 @@ int xe_ggtt_insert_special_node_locked(struct xe_ggtt *ggtt, struct xe_ggtt_node 
 *

 * Return: 0 on success or a negative error code on failure.

 */

int xe_ggtt_insert_special_node(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

int xe_ggtt_node_insert(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			u32 size, u32 align)

{

	int ret;



	mutex_lock(&ggtt->lock);

	ret = xe_ggtt_insert_special_node_locked(ggtt, node, size,

	ret = xe_ggtt_node_insert_locked(ggtt, node, size,

					 align, DRM_MM_INSERT_HIGH);

	mutex_unlock(&ggtt->lock);



	return ret;

}



/**

 * xe_ggtt_node_remove - Remove a &xe_ggtt_node from the GGTT

 * @ggtt: the &xe_ggtt where node will be removed

 * @node: the &xe_ggtt_node to be removed

 * @invalidate: if node needs invalidation upon removal

 */

void xe_ggtt_node_remove(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			 bool invalidate)

{

	struct xe_device *xe = tile_to_xe(ggtt->tile);

	bool bound;

	int idx;



	bound = drm_dev_enter(&xe->drm, &idx);

	if (bound)

		xe_pm_runtime_get_noresume(xe);



	mutex_lock(&ggtt->lock);

	if (bound)

		xe_ggtt_clear(ggtt, node->base.start, node->base.size);

	drm_mm_remove_node(&node->base);

	node->base.size = 0;

	mutex_unlock(&ggtt->lock);



	if (!bound)

		return;



	if (invalidate)

		xe_ggtt_invalidate(ggtt);



	xe_pm_runtime_put(xe);

	drm_dev_exit(idx);

}



/**

 * xe_ggtt_map_bo - Map the BO into GGTT

 * @ggtt: the &xe_ggtt where node will be mapped
@@ -522,40 +556,6 @@ int xe_ggtt_insert_bo(struct xe_ggtt *ggtt, struct xe_bo *bo) 
	return __xe_ggtt_insert_bo_at(ggtt, bo, 0, U64_MAX);

}



/**

 * xe_ggtt_remove_node - Remove a &xe_ggtt_node from the GGTT

 * @ggtt: the &xe_ggtt where node will be removed

 * @node: the &xe_ggtt_node to be removed

 * @invalidate: if node needs invalidation upon removal

 */

void xe_ggtt_remove_node(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			 bool invalidate)

{

	struct xe_device *xe = tile_to_xe(ggtt->tile);

	bool bound;

	int idx;



	bound = drm_dev_enter(&xe->drm, &idx);

	if (bound)

		xe_pm_runtime_get_noresume(xe);



	mutex_lock(&ggtt->lock);

	if (bound)

		xe_ggtt_clear(ggtt, node->base.start, node->base.size);

	drm_mm_remove_node(&node->base);

	node->base.size = 0;

	mutex_unlock(&ggtt->lock);



	if (!bound)

		return;



	if (invalidate)

		xe_ggtt_invalidate(ggtt);



	xe_pm_runtime_put(xe);

	drm_dev_exit(idx);

}



/**

 * xe_ggtt_remove_bo - Remove a BO from the GGTT

 * @ggtt: the &xe_ggtt where node will be removed
@@ -569,7 +569,7 @@ void xe_ggtt_remove_bo(struct xe_ggtt *ggtt, struct xe_bo *bo) 
	/* This BO is not currently in the GGTT */

	xe_tile_assert(ggtt->tile, bo->ggtt_node.base.size == bo->size);



	xe_ggtt_remove_node(ggtt, &bo->ggtt_node,

	xe_ggtt_node_remove(ggtt, &bo->ggtt_node,

			    bo->flags & XE_BO_FLAG_GGTT_INVALIDATE);

}

drivers/gpu/drm/xe/xe_ggtt.h

+6 −6
Original line number Diff line number Diff line
@@ -16,12 +16,12 @@ int xe_ggtt_init(struct xe_ggtt *ggtt); 
int xe_ggtt_balloon(struct xe_ggtt *ggtt, u64 start, u64 size, struct xe_ggtt_node *node);

void xe_ggtt_deballoon(struct xe_ggtt *ggtt, struct xe_ggtt_node *node);



int xe_ggtt_insert_special_node(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

int xe_ggtt_node_insert(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			u32 size, u32 align);

int xe_ggtt_insert_special_node_locked(struct xe_ggtt *ggtt,

int xe_ggtt_node_insert_locked(struct xe_ggtt *ggtt,

			       struct xe_ggtt_node *node,

			       u32 size, u32 align, u32 mm_flags);

void xe_ggtt_remove_node(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

void xe_ggtt_node_remove(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,

			 bool invalidate);

void xe_ggtt_map_bo(struct xe_ggtt *ggtt, struct xe_bo *bo);

int xe_ggtt_insert_bo(struct xe_ggtt *ggtt, struct xe_bo *bo);

drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c

+2 −2
Original line number Diff line number Diff line
@@ -382,7 +382,7 @@ static void pf_release_ggtt(struct xe_tile *tile, struct xe_ggtt_node *node) 
		 * is redundant, as PTE will be implicitly re-assigned to PF by

		 * the xe_ggtt_clear() called by below xe_ggtt_remove_node().

		 */

		xe_ggtt_remove_node(ggtt, node, false);

		xe_ggtt_node_remove(ggtt, node, false);

	}

}
@@ -418,7 +418,7 @@ static int pf_provision_vf_ggtt(struct xe_gt *gt, unsigned int vfid, u64 size) 
	if (!size)

		return 0;



	err = xe_ggtt_insert_special_node(ggtt, node, size, alignment);

	err = xe_ggtt_node_insert(ggtt, node, size, alignment);

	if (unlikely(err))

		return err;