[PATCH v7 0/5] drm/xe: Privatize struct xe_ggtt.

[PATCH v7 0/5] drm/xe: Privatize struct xe_ggtt.

[Maarten Lankhorst]

Clean up SR-IOV VF handling and pop the balloons, move struct xe_ggtt to xe_ggtt.c,
and clean up the API's slightly by removing a separate xe_gtt_node_init() step.

This makes the xe_ggtt_node_allocated obsolete, as we only create allocated nodes now.

Updated based on review feedback. Hopefully all remaining issues addressed now. :)

Maarten Lankhorst (5):
  drm/xe: Make xe_ggtt_node offset relative to starting offset
  drm/xe: Rewrite GGTT VF initialization
  drm/xe: Move struct xe_ggtt to xe_ggtt.c
  drm/xe: Make xe_ggtt_node_insert return a node
  drm/xe: Remove xe_ggtt_node_allocated

 drivers/gpu/drm/xe/display/xe_fb_pin.c      |   5 +-
 drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c |   6 +-
 drivers/gpu/drm/xe/xe_device_types.h        |   2 -
 drivers/gpu/drm/xe/xe_ggtt.c                | 359 ++++++++------------
 drivers/gpu/drm/xe/xe_ggtt.h                |  14 +-
 drivers/gpu/drm/xe/xe_ggtt_types.h          |  60 +---
 drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c  |  36 +-
 drivers/gpu/drm/xe/xe_gt_sriov_vf.c         |  35 +-
 drivers/gpu/drm/xe/xe_tile_sriov_vf.c       | 198 +----------
 drivers/gpu/drm/xe/xe_tile_sriov_vf.h       |   3 -
 1files changed, 189 insertions(+), 529 deletions(-)

-- 
2.51.0

[PATCH v7 1/5] drm/xe: Make xe_ggtt_node offset relative to starting offset

[Maarten Lankhorst]

Fix all functions that use node->start to use xe_ggtt_node_addr,
and add ggtt->start to node->start.

This will make node shifting for SR-IOV VF a one-liner, instead of
manually changing each GGTT node's base address.

Also convert some uses of mutex_lock/unlock to mutex guards.

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
---
 drivers/gpu/drm/xe/xe_ggtt.c | 53 +++++++++++++++++++++---------------
 1 file changed, 31 insertions(+), 22 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index 60665ad1415be..5c11df67b589e 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -299,7 +299,7 @@ static void __xe_ggtt_init_early(struct xe_ggtt *ggtt, u64 start, u64 size)
 {
 	ggtt->start = start;
 	ggtt->size = size;
-	drm_mm_init(&ggtt->mm, start, size);
+	drm_mm_init(&ggtt->mm, 0, size);
 }
 
 int xe_ggtt_init_kunit(struct xe_ggtt *ggtt, u32 start, u32 size)
@@ -401,7 +401,7 @@ static void xe_ggtt_initial_clear(struct xe_ggtt *ggtt)
 	/* Display may have allocated inside ggtt, so be careful with clearing here */
 	mutex_lock(&ggtt->lock);
 	drm_mm_for_each_hole(hole, &ggtt->mm, start, end)
-		xe_ggtt_clear(ggtt, start, end - start);
+		xe_ggtt_clear(ggtt, ggtt->start + start, end - start);
 
 	xe_ggtt_invalidate(ggtt);
 	mutex_unlock(&ggtt->lock);
@@ -418,7 +418,7 @@ static void ggtt_node_remove(struct xe_ggtt_node *node)
 
 	mutex_lock(&ggtt->lock);
 	if (bound)
-		xe_ggtt_clear(ggtt, node->base.start, node->base.size);
+		xe_ggtt_clear(ggtt, xe_ggtt_node_addr(node), xe_ggtt_node_size(node));
 	drm_mm_remove_node(&node->base);
 	node->base.size = 0;
 	mutex_unlock(&ggtt->lock);
@@ -570,16 +570,17 @@ int xe_ggtt_node_insert_balloon_locked(struct xe_ggtt_node *node, u64 start, u64
 	xe_tile_assert(ggtt->tile, IS_ALIGNED(start, XE_PAGE_SIZE));
 	xe_tile_assert(ggtt->tile, IS_ALIGNED(end, XE_PAGE_SIZE));
 	xe_tile_assert(ggtt->tile, !drm_mm_node_allocated(&node->base));
+	xe_tile_assert(ggtt->tile, start >= ggtt->start);
 	lockdep_assert_held(&ggtt->lock);
 
 	node->base.color = 0;
-	node->base.start = start;
+	node->base.start = start - ggtt->start;
 	node->base.size = end - start;
 
 	err = drm_mm_reserve_node(&ggtt->mm, &node->base);
 
 	if (xe_tile_WARN(ggtt->tile, err, "Failed to balloon GGTT %#llx-%#llx (%pe)\n",
-			 node->base.start, node->base.start + node->base.size, ERR_PTR(err)))
+			 xe_ggtt_node_addr(node), xe_ggtt_node_addr(node) + node->base.size, ERR_PTR(err)))
 		return err;
 
 	xe_ggtt_dump_node(ggtt, &node->base, "balloon");
@@ -770,7 +771,7 @@ static void xe_ggtt_map_bo(struct xe_ggtt *ggtt, struct xe_ggtt_node *node,
 	if (XE_WARN_ON(!node))
 		return;
 
-	start = node->base.start;
+	start = xe_ggtt_node_addr(node);
 	end = start + xe_bo_size(bo);
 
 	if (!xe_bo_is_vram(bo) && !xe_bo_is_stolen(bo)) {
@@ -891,6 +892,14 @@ static int __xe_ggtt_insert_bo_at(struct xe_ggtt *ggtt, struct xe_bo *bo,
 	}
 
 	mutex_lock(&ggtt->lock);
+	xe_tile_assert(ggtt->tile, start >= ggtt->start || !start);
+	xe_tile_assert(ggtt->tile, end >= ggtt->start);
+
+	if (start)
+		start -= ggtt->start;
+
+	end -= ggtt->start;
+
 	err = drm_mm_insert_node_in_range(&ggtt->mm, &bo->ggtt_node[tile_id]->base,
 					  xe_bo_size(bo), alignment, 0, start, end, 0);
 	if (err) {
@@ -1002,16 +1011,17 @@ static u64 xe_encode_vfid_pte(u16 vfid)
 	return FIELD_PREP(GGTT_PTE_VFID, vfid) | XE_PAGE_PRESENT;
 }
 
-static void xe_ggtt_assign_locked(struct xe_ggtt *ggtt, const struct drm_mm_node *node, u16 vfid)
+static void xe_ggtt_assign_locked(const struct xe_ggtt_node *node, u16 vfid)
 {
-	u64 start = node->start;
-	u64 size = node->size;
+	struct xe_ggtt *ggtt = node->ggtt;
+	u64 start = xe_ggtt_node_addr(node);
+	u64 size = xe_ggtt_node_size(node);
 	u64 end = start + size - 1;
 	u64 pte = xe_encode_vfid_pte(vfid);
 
 	lockdep_assert_held(&ggtt->lock);
 
-	if (!drm_mm_node_allocated(node))
+	if (!xe_ggtt_node_allocated(node))
 		return;
 
 	while (start < end) {
@@ -1033,9 +1043,8 @@ static void xe_ggtt_assign_locked(struct xe_ggtt *ggtt, const struct drm_mm_node
  */
 void xe_ggtt_assign(const struct xe_ggtt_node *node, u16 vfid)
 {
-	mutex_lock(&node->ggtt->lock);
-	xe_ggtt_assign_locked(node->ggtt, &node->base, vfid);
-	mutex_unlock(&node->ggtt->lock);
+	guard(mutex)(&node->ggtt->lock);
+	xe_ggtt_assign_locked(node, vfid);
 }
 
 /**
@@ -1057,14 +1066,14 @@ int xe_ggtt_node_save(struct xe_ggtt_node *node, void *dst, size_t size, u16 vfi
 	if (!node)
 		return -ENOENT;
 
-	guard(mutex)(&node->ggtt->lock);
+	ggtt = node->ggtt;
+	guard(mutex)(&ggtt->lock);
 
 	if (xe_ggtt_node_pt_size(node) != size)
 		return -EINVAL;
 
-	ggtt = node->ggtt;
-	start = node->base.start;
-	end = start + node->base.size - 1;
+	start = xe_ggtt_node_addr(node);
+	end = start + xe_ggtt_node_size(node) - 1;
 
 	while (start < end) {
 		pte = ggtt->pt_ops->ggtt_get_pte(ggtt, start);
@@ -1097,14 +1106,14 @@ int xe_ggtt_node_load(struct xe_ggtt_node *node, const void *src, size_t size, u
 	if (!node)
 		return -ENOENT;
 
-	guard(mutex)(&node->ggtt->lock);
+	ggtt = node->ggtt;
+	guard(mutex)(&ggtt->lock);
 
 	if (xe_ggtt_node_pt_size(node) != size)
 		return -EINVAL;
 
-	ggtt = node->ggtt;
-	start = node->base.start;
-	end = start + node->base.size - 1;
+	start = xe_ggtt_node_addr(node);
+	end = start + xe_ggtt_node_size(node) - 1;
 
 	while (start < end) {
 		vfid_pte = u64_replace_bits(*buf++, vfid, GGTT_PTE_VFID);
@@ -1211,7 +1220,7 @@ u64 xe_ggtt_read_pte(struct xe_ggtt *ggtt, u64 offset)
  */
 u64 xe_ggtt_node_addr(const struct xe_ggtt_node *node)
 {
-	return node->base.start;
+	return node->base.start + node->ggtt->start;
 }
 
 /**
-- 
2.51.0

[PATCH v7 2/5] drm/xe: Rewrite GGTT VF initialization

[Maarten Lankhorst]

The previous code was using a complicated system with 2 balloons to
set GGTT size and adjust GGTT offset. While it works, it's overly
complicated.

A better approach is to set the offset and size when initializing GGTT,
this removes the need for adding balloons. The resize function only
needs readjust ggtt->start to have GGTT at the new offset.

This removes the need to manipulate the internals of xe_ggtt outside
of xe_ggtt, and cleans up a lot of now unneeded code.

Co-developed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
---
Changelog:
- Fix docbook description.
- Simplify the vf_get_ggtt_info() flow and remove the shift calculation.
---
 drivers/gpu/drm/xe/xe_device_types.h  |   2 -
 drivers/gpu/drm/xe/xe_ggtt.c          | 165 +++++----------------
 drivers/gpu/drm/xe/xe_ggtt.h          |   5 +-
 drivers/gpu/drm/xe/xe_gt_sriov_vf.c   |  35 +++--
 drivers/gpu/drm/xe/xe_tile_sriov_vf.c | 198 +-------------------------
 drivers/gpu/drm/xe/xe_tile_sriov_vf.h |   3 -
 6 files changed, 59 insertions(+), 349 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_device_types.h b/drivers/gpu/drm/xe/xe_device_types.h
index 34feef79fa4e7..0409fa98835e8 100644
--- a/drivers/gpu/drm/xe/xe_device_types.h
+++ b/drivers/gpu/drm/xe/xe_device_types.h
@@ -228,8 +228,6 @@ struct xe_tile {
 			struct xe_lmtt lmtt;
 		} pf;
 		struct {
-			/** @sriov.vf.ggtt_balloon: GGTT regions excluded from use. */
-			struct xe_ggtt_node *ggtt_balloon[2];
 			/** @sriov.vf.self_config: VF configuration data */
 			struct xe_tile_sriov_vf_selfconfig self_config;
 		} vf;
diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index 5c11df67b589e..6ab655073d042 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -70,8 +70,8 @@
  * struct xe_ggtt_node - A node in GGTT.
  *
  * This struct needs to be initialized (only-once) with xe_ggtt_node_init() before any node
- * insertion, reservation, or 'ballooning'.
- * It will, then, be finalized by either xe_ggtt_node_remove() or xe_ggtt_node_deballoon().
+ * insertion or reservation.
+ * It will, then, be finalized by xe_ggtt_node_remove().
  */
 struct xe_ggtt_node {
 	/** @ggtt: Back pointer to xe_ggtt where this region will be inserted at */
@@ -347,9 +347,15 @@ int xe_ggtt_init_early(struct xe_ggtt *ggtt)
 		ggtt_start = wopcm;
 		ggtt_size = (gsm_size / 8) * (u64)XE_PAGE_SIZE - ggtt_start;
 	} else {
-		/* GGTT is expected to be 4GiB */
-		ggtt_start = wopcm;
-		ggtt_size = SZ_4G - ggtt_start;
+		ggtt_start = xe_tile_sriov_vf_ggtt_base(ggtt->tile);
+		ggtt_size = xe_tile_sriov_vf_ggtt(ggtt->tile);
+
+		if (ggtt_start < wopcm ||
+		    ggtt_start + ggtt_size > GUC_GGTT_TOP) {
+			xe_tile_err(ggtt->tile, "Invalid GGTT configuration: %#llx-%#llx\n",
+				    ggtt_start, ggtt_start + ggtt_size - 1);
+			return -ERANGE;
+		}
 	}
 
 	ggtt->gsm = ggtt->tile->mmio.regs + SZ_8M;
@@ -377,17 +383,7 @@ int xe_ggtt_init_early(struct xe_ggtt *ggtt)
 	if (err)
 		return err;
 
-	err = devm_add_action_or_reset(xe->drm.dev, dev_fini_ggtt, ggtt);
-	if (err)
-		return err;
-
-	if (IS_SRIOV_VF(xe)) {
-		err = xe_tile_sriov_vf_prepare_ggtt(ggtt->tile);
-		if (err)
-			return err;
-	}
-
-	return 0;
+	return devm_add_action_or_reset(xe->drm.dev, dev_fini_ggtt, ggtt);
 }
 ALLOW_ERROR_INJECTION(xe_ggtt_init_early, ERRNO); /* See xe_pci_probe() */
 
@@ -538,120 +534,28 @@ static void xe_ggtt_invalidate(struct xe_ggtt *ggtt)
 	ggtt_invalidate_gt_tlb(ggtt->tile->media_gt);
 }
 
-static void xe_ggtt_dump_node(struct xe_ggtt *ggtt,
-			      const struct drm_mm_node *node, const char *description)
-{
-	char buf[10];
-
-	if (IS_ENABLED(CONFIG_DRM_XE_DEBUG)) {
-		string_get_size(node->size, 1, STRING_UNITS_2, buf, sizeof(buf));
-		xe_tile_dbg(ggtt->tile, "GGTT %#llx-%#llx (%s) %s\n",
-			    node->start, node->start + node->size, buf, description);
-	}
-}
-
-/**
- * xe_ggtt_node_insert_balloon_locked - prevent allocation of specified GGTT addresses
- * @node: the &xe_ggtt_node to hold reserved GGTT node
- * @start: the starting GGTT address of the reserved region
- * @end: then end GGTT address of the reserved region
- *
- * To be used in cases where ggtt->lock is already taken.
- * Use xe_ggtt_node_remove_balloon_locked() to release a reserved GGTT node.
- *
- * Return: 0 on success or a negative error code on failure.
- */
-int xe_ggtt_node_insert_balloon_locked(struct xe_ggtt_node *node, u64 start, u64 end)
-{
-	struct xe_ggtt *ggtt = node->ggtt;
-	int err;
-
-	xe_tile_assert(ggtt->tile, start < end);
-	xe_tile_assert(ggtt->tile, IS_ALIGNED(start, XE_PAGE_SIZE));
-	xe_tile_assert(ggtt->tile, IS_ALIGNED(end, XE_PAGE_SIZE));
-	xe_tile_assert(ggtt->tile, !drm_mm_node_allocated(&node->base));
-	xe_tile_assert(ggtt->tile, start >= ggtt->start);
-	lockdep_assert_held(&ggtt->lock);
-
-	node->base.color = 0;
-	node->base.start = start - ggtt->start;
-	node->base.size = end - start;
-
-	err = drm_mm_reserve_node(&ggtt->mm, &node->base);
-
-	if (xe_tile_WARN(ggtt->tile, err, "Failed to balloon GGTT %#llx-%#llx (%pe)\n",
-			 xe_ggtt_node_addr(node), xe_ggtt_node_addr(node) + node->base.size, ERR_PTR(err)))
-		return err;
-
-	xe_ggtt_dump_node(ggtt, &node->base, "balloon");
-	return 0;
-}
-
 /**
- * xe_ggtt_node_remove_balloon_locked - release a reserved GGTT region
- * @node: the &xe_ggtt_node with reserved GGTT region
- *
- * To be used in cases where ggtt->lock is already taken.
- * See xe_ggtt_node_insert_balloon_locked() for details.
- */
-void xe_ggtt_node_remove_balloon_locked(struct xe_ggtt_node *node)
-{
-	if (!xe_ggtt_node_allocated(node))
-		return;
-
-	lockdep_assert_held(&node->ggtt->lock);
-
-	xe_ggtt_dump_node(node->ggtt, &node->base, "remove-balloon");
-
-	drm_mm_remove_node(&node->base);
-}
-
-static void xe_ggtt_assert_fit(struct xe_ggtt *ggtt, u64 start, u64 size)
-{
-	struct xe_tile *tile = ggtt->tile;
-
-	xe_tile_assert(tile, start >= ggtt->start);
-	xe_tile_assert(tile, start + size <= ggtt->start + ggtt->size);
-}
-
-/**
- * xe_ggtt_shift_nodes_locked - Shift GGTT nodes to adjust for a change in usable address range.
+ * xe_ggtt_shift_nodes() - Shift GGTT nodes to adjust for a change in usable address range.
  * @ggtt: the &xe_ggtt struct instance
- * @shift: change to the location of area provisioned for current VF
+ * @new_start: new location of area provisioned for current VF
+ *
+ * Ensure that all struct &xe_ggtt_node are moved to the @new_start base address
+ * by changing the base offset of the GGTT.
  *
- * This function moves all nodes from the GGTT VM, to a temp list. These nodes are expected
- * to represent allocations in range formerly assigned to current VF, before the range changed.
- * When the GGTT VM is completely clear of any nodes, they are re-added with shifted offsets.
+ * This function may be called multiple times during recovery, but if
+ * @new_start is unchanged from the current base, it's a noop.
  *
- * The function has no ability of failing - because it shifts existing nodes, without
- * any additional processing. If the nodes were successfully existing at the old address,
- * they will do the same at the new one. A fail inside this function would indicate that
- * the list of nodes was either already damaged, or that the shift brings the address range
- * outside of valid bounds. Both cases justify an assert rather than error code.
+ * @new_start should be a value between xe_wopcm_size() and #GUC_GGTT_TOP.
  */
-void xe_ggtt_shift_nodes_locked(struct xe_ggtt *ggtt, s64 shift)
+void xe_ggtt_shift_nodes(struct xe_ggtt *ggtt, u64 new_start)
 {
-	struct xe_tile *tile __maybe_unused = ggtt->tile;
-	struct drm_mm_node *node, *tmpn;
-	LIST_HEAD(temp_list_head);
-
-	lockdep_assert_held(&ggtt->lock);
+	guard(mutex)(&ggtt->lock);
 
-	if (IS_ENABLED(CONFIG_DRM_XE_DEBUG))
-		drm_mm_for_each_node_safe(node, tmpn, &ggtt->mm)
-			xe_ggtt_assert_fit(ggtt, node->start + shift, node->size);
+	xe_tile_assert(ggtt->tile, new_start >= xe_wopcm_size(tile_to_xe(ggtt->tile)));
+	xe_tile_assert(ggtt->tile, new_start + ggtt->size <= GUC_GGTT_TOP);
 
-	drm_mm_for_each_node_safe(node, tmpn, &ggtt->mm) {
-		drm_mm_remove_node(node);
-		list_add(&node->node_list, &temp_list_head);
-	}
-
-	list_for_each_entry_safe(node, tmpn, &temp_list_head, node_list) {
-		list_del(&node->node_list);
-		node->start += shift;
-		drm_mm_reserve_node(&ggtt->mm, node);
-		xe_tile_assert(tile, drm_mm_node_allocated(node));
-	}
+	/* pairs with READ_ONCE in xe_ggtt_node_addr() */
+	WRITE_ONCE(ggtt->start, new_start);
 }
 
 static int xe_ggtt_node_insert_locked(struct xe_ggtt_node *node,
@@ -692,12 +596,8 @@ int xe_ggtt_node_insert(struct xe_ggtt_node *node, u32 size, u32 align)
  *
  * This function will allocate the struct %xe_ggtt_node and return its pointer.
  * This struct will then be freed after the node removal upon xe_ggtt_node_remove()
- * or xe_ggtt_node_remove_balloon_locked().
- *
- * Having %xe_ggtt_node struct allocated doesn't mean that the node is already
- * allocated in GGTT. Only xe_ggtt_node_insert(), allocation through
- * xe_ggtt_node_insert_transform(), or xe_ggtt_node_insert_balloon_locked() will ensure the node is inserted or reserved
- * in GGTT.
+ * Having %xe_ggtt_node struct allocated doesn't mean that the node is already allocated
+ * in GGTT. Only xe_ggtt_node_insert() will ensure the node is inserted or reserved in GGTT.
  *
  * Return: A pointer to %xe_ggtt_node struct on success. An ERR_PTR otherwise.
  **/
@@ -718,9 +618,9 @@ struct xe_ggtt_node *xe_ggtt_node_init(struct xe_ggtt *ggtt)
  * xe_ggtt_node_fini - Forcebly finalize %xe_ggtt_node struct
  * @node: the &xe_ggtt_node to be freed
  *
- * If anything went wrong with either xe_ggtt_node_insert(), xe_ggtt_node_insert_locked(),
- * or xe_ggtt_node_insert_balloon_locked(); and this @node is not going to be reused, then,
- * this function needs to be called to free the %xe_ggtt_node struct
+ * If anything went wrong with either xe_ggtt_node_insert() and this @node is
+ * not going to be reused, then this function needs to be called to free the
+ * %xe_ggtt_node struct
  **/
 void xe_ggtt_node_fini(struct xe_ggtt_node *node)
 {
@@ -1220,7 +1120,8 @@ u64 xe_ggtt_read_pte(struct xe_ggtt *ggtt, u64 offset)
  */
 u64 xe_ggtt_node_addr(const struct xe_ggtt_node *node)
 {
-	return node->base.start + node->ggtt->start;
+	/* pairs with WRITE_ONCE in xe_ggtt_shift_nodes() */
+	return node->base.start + READ_ONCE(node->ggtt->start);
 }
 
 /**
diff --git a/drivers/gpu/drm/xe/xe_ggtt.h b/drivers/gpu/drm/xe/xe_ggtt.h
index 70d5e07ac4b66..49ea8e7ecc105 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.h
+++ b/drivers/gpu/drm/xe/xe_ggtt.h
@@ -19,10 +19,7 @@ int xe_ggtt_init(struct xe_ggtt *ggtt);
 
 struct xe_ggtt_node *xe_ggtt_node_init(struct xe_ggtt *ggtt);
 void xe_ggtt_node_fini(struct xe_ggtt_node *node);
-int xe_ggtt_node_insert_balloon_locked(struct xe_ggtt_node *node,
-				       u64 start, u64 size);
-void xe_ggtt_node_remove_balloon_locked(struct xe_ggtt_node *node);
-void xe_ggtt_shift_nodes_locked(struct xe_ggtt *ggtt, s64 shift);
+void xe_ggtt_shift_nodes(struct xe_ggtt *ggtt, u64 new_base);
 u64 xe_ggtt_start(struct xe_ggtt *ggtt);
 u64 xe_ggtt_size(struct xe_ggtt *ggtt);
 
diff --git a/drivers/gpu/drm/xe/xe_gt_sriov_vf.c b/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
index 30e8c2cf5f09a..527ded3c9c22f 100644
--- a/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
+++ b/drivers/gpu/drm/xe/xe_gt_sriov_vf.c
@@ -488,16 +488,12 @@ u32 xe_gt_sriov_vf_gmdid(struct xe_gt *gt)
 static int vf_get_ggtt_info(struct xe_gt *gt)
 {
 	struct xe_tile *tile = gt_to_tile(gt);
-	struct xe_ggtt *ggtt = tile->mem.ggtt;
 	struct xe_guc *guc = &gt->uc.guc;
 	u64 start, size, ggtt_size;
-	s64 shift;
 	int err;
 
 	xe_gt_assert(gt, IS_SRIOV_VF(gt_to_xe(gt)));
 
-	guard(mutex)(&ggtt->lock);
-
 	err = guc_action_query_single_klv64(guc, GUC_KLV_VF_CFG_GGTT_START_KEY, &start);
 	if (unlikely(err))
 		return err;
@@ -509,8 +505,21 @@ static int vf_get_ggtt_info(struct xe_gt *gt)
 	if (!size)
 		return -ENODATA;
 
+	xe_tile_sriov_vf_ggtt_base_store(tile, start);
 	ggtt_size = xe_tile_sriov_vf_ggtt(tile);
-	if (ggtt_size && ggtt_size != size) {
+	if (!ggtt_size) {
+		/*
+		 * This function is called once during xe_guc_init_noalloc(),
+		 * at which point ggtt_size = 0 and we have to initialize everything,
+		 * and GGTT is not yet initialized.
+		 *
+		 * Return early as there's nothing to fixup.
+		 */
+		xe_tile_sriov_vf_ggtt_store(tile, size);
+		return 0;
+	}
+
+	if (ggtt_size != size) {
 		xe_gt_sriov_err(gt, "Unexpected GGTT reassignment: %lluK != %lluK\n",
 				size / SZ_1K, ggtt_size / SZ_1K);
 		return -EREMCHG;
@@ -519,15 +528,13 @@ static int vf_get_ggtt_info(struct xe_gt *gt)
 	xe_gt_sriov_dbg_verbose(gt, "GGTT %#llx-%#llx = %lluK\n",
 				start, start + size - 1, size / SZ_1K);
 
-	shift = start - (s64)xe_tile_sriov_vf_ggtt_base(tile);
-	xe_tile_sriov_vf_ggtt_base_store(tile, start);
-	xe_tile_sriov_vf_ggtt_store(tile, size);
-
-	if (shift && shift != start) {
-		xe_gt_sriov_info(gt, "Shifting GGTT base by %lld to 0x%016llx\n",
-				 shift, start);
-		xe_tile_sriov_vf_fixup_ggtt_nodes_locked(gt_to_tile(gt), shift);
-	}
+	/*
+	 * This function can be called repeatedly from post migration fixups,
+	 * at which point we inform the GGTT of the new base address.
+	 * xe_ggtt_shift_nodes() may be called multiple times for each migration,
+	 * but will be a noop if the base is unchanged.
+	 */
+	xe_ggtt_shift_nodes(tile->mem.ggtt, start);
 
 	if (xe_sriov_vf_migration_supported(gt_to_xe(gt))) {
 		WRITE_ONCE(gt->sriov.vf.migration.ggtt_need_fixes, false);
diff --git a/drivers/gpu/drm/xe/xe_tile_sriov_vf.c b/drivers/gpu/drm/xe/xe_tile_sriov_vf.c
index c9bac2cfdd044..24293521e0904 100644
--- a/drivers/gpu/drm/xe/xe_tile_sriov_vf.c
+++ b/drivers/gpu/drm/xe/xe_tile_sriov_vf.c
@@ -14,173 +14,12 @@
 #include "xe_tile_sriov_vf.h"
 #include "xe_wopcm.h"
 
-static int vf_init_ggtt_balloons(struct xe_tile *tile)
-{
-	struct xe_ggtt *ggtt = tile->mem.ggtt;
-
-	xe_tile_assert(tile, IS_SRIOV_VF(tile_to_xe(tile)));
-
-	tile->sriov.vf.ggtt_balloon[0] = xe_ggtt_node_init(ggtt);
-	if (IS_ERR(tile->sriov.vf.ggtt_balloon[0]))
-		return PTR_ERR(tile->sriov.vf.ggtt_balloon[0]);
-
-	tile->sriov.vf.ggtt_balloon[1] = xe_ggtt_node_init(ggtt);
-	if (IS_ERR(tile->sriov.vf.ggtt_balloon[1])) {
-		xe_ggtt_node_fini(tile->sriov.vf.ggtt_balloon[0]);
-		return PTR_ERR(tile->sriov.vf.ggtt_balloon[1]);
-	}
-
-	return 0;
-}
-
-/**
- * xe_tile_sriov_vf_balloon_ggtt_locked - Insert balloon nodes to limit used GGTT address range.
- * @tile: the &xe_tile struct instance
- *
- * Return: 0 on success or a negative error code on failure.
- */
-static int xe_tile_sriov_vf_balloon_ggtt_locked(struct xe_tile *tile)
-{
-	u64 ggtt_base = tile->sriov.vf.self_config.ggtt_base;
-	u64 ggtt_size = tile->sriov.vf.self_config.ggtt_size;
-	struct xe_device *xe = tile_to_xe(tile);
-	u64 wopcm = xe_wopcm_size(xe);
-	u64 start, end;
-	int err;
-
-	xe_tile_assert(tile, IS_SRIOV_VF(xe));
-	xe_tile_assert(tile, ggtt_size);
-	lockdep_assert_held(&tile->mem.ggtt->lock);
-
-	/*
-	 * VF can only use part of the GGTT as allocated by the PF:
-	 *
-	 *      WOPCM                                  GUC_GGTT_TOP
-	 *      |<------------ Total GGTT size ------------------>|
-	 *
-	 *           VF GGTT base -->|<- size ->|
-	 *
-	 *      +--------------------+----------+-----------------+
-	 *      |////////////////////|   block  |\\\\\\\\\\\\\\\\\|
-	 *      +--------------------+----------+-----------------+
-	 *
-	 *      |<--- balloon[0] --->|<-- VF -->|<-- balloon[1] ->|
-	 */
-
-	if (ggtt_base < wopcm || ggtt_base > GUC_GGTT_TOP ||
-	    ggtt_size > GUC_GGTT_TOP - ggtt_base) {
-		xe_sriov_err(xe, "tile%u: Invalid GGTT configuration: %#llx-%#llx\n",
-			     tile->id, ggtt_base, ggtt_base + ggtt_size - 1);
-		return -ERANGE;
-	}
-
-	start = wopcm;
-	end = ggtt_base;
-	if (end != start) {
-		err = xe_ggtt_node_insert_balloon_locked(tile->sriov.vf.ggtt_balloon[0],
-							 start, end);
-		if (err)
-			return err;
-	}
-
-	start = ggtt_base + ggtt_size;
-	end = GUC_GGTT_TOP;
-	if (end != start) {
-		err = xe_ggtt_node_insert_balloon_locked(tile->sriov.vf.ggtt_balloon[1],
-							 start, end);
-		if (err) {
-			xe_ggtt_node_remove_balloon_locked(tile->sriov.vf.ggtt_balloon[0]);
-			return err;
-		}
-	}
-
-	return 0;
-}
-
-static int vf_balloon_ggtt(struct xe_tile *tile)
-{
-	struct xe_ggtt *ggtt = tile->mem.ggtt;
-	int err;
-
-	mutex_lock(&ggtt->lock);
-	err = xe_tile_sriov_vf_balloon_ggtt_locked(tile);
-	mutex_unlock(&ggtt->lock);
-
-	return err;
-}
-
-/**
- * xe_tile_sriov_vf_deballoon_ggtt_locked - Remove balloon nodes.
- * @tile: the &xe_tile struct instance
- */
-void xe_tile_sriov_vf_deballoon_ggtt_locked(struct xe_tile *tile)
-{
-	xe_tile_assert(tile, IS_SRIOV_VF(tile_to_xe(tile)));
-
-	xe_ggtt_node_remove_balloon_locked(tile->sriov.vf.ggtt_balloon[1]);
-	xe_ggtt_node_remove_balloon_locked(tile->sriov.vf.ggtt_balloon[0]);
-}
-
-static void vf_deballoon_ggtt(struct xe_tile *tile)
-{
-	mutex_lock(&tile->mem.ggtt->lock);
-	xe_tile_sriov_vf_deballoon_ggtt_locked(tile);
-	mutex_unlock(&tile->mem.ggtt->lock);
-}
-
-static void vf_fini_ggtt_balloons(struct xe_tile *tile)
-{
-	xe_tile_assert(tile, IS_SRIOV_VF(tile_to_xe(tile)));
-
-	xe_ggtt_node_fini(tile->sriov.vf.ggtt_balloon[1]);
-	xe_ggtt_node_fini(tile->sriov.vf.ggtt_balloon[0]);
-}
-
-static void cleanup_ggtt(struct drm_device *drm, void *arg)
-{
-	struct xe_tile *tile = arg;
-
-	vf_deballoon_ggtt(tile);
-	vf_fini_ggtt_balloons(tile);
-}
-
-/**
- * xe_tile_sriov_vf_prepare_ggtt - Prepare a VF's GGTT configuration.
- * @tile: the &xe_tile
- *
- * This function is for VF use only.
- *
- * Return: 0 on success or a negative error code on failure.
- */
-int xe_tile_sriov_vf_prepare_ggtt(struct xe_tile *tile)
-{
-	struct xe_device *xe = tile_to_xe(tile);
-	int err;
-
-	err = vf_init_ggtt_balloons(tile);
-	if (err)
-		return err;
-
-	err = vf_balloon_ggtt(tile);
-	if (err) {
-		vf_fini_ggtt_balloons(tile);
-		return err;
-	}
-
-	return drmm_add_action_or_reset(&xe->drm, cleanup_ggtt, tile);
-}
-
 /**
  * DOC: GGTT nodes shifting during VF post-migration recovery
  *
  * The first fixup applied to the VF KMD structures as part of post-migration
  * recovery is shifting nodes within &xe_ggtt instance. The nodes are moved
  * from range previously assigned to this VF, into newly provisioned area.
- * The changes include balloons, which are resized accordingly.
- *
- * The balloon nodes are there to eliminate unavailable ranges from use: one
- * reserves the GGTT area below the range for current VF, and another one
- * reserves area above.
  *
  * Below is a GGTT layout of example VF, with a certain address range assigned to
  * said VF, and inaccessible areas above and below:
@@ -198,10 +37,6 @@ int xe_tile_sriov_vf_prepare_ggtt(struct xe_tile *tile)
  *
  *  |<------- inaccessible for VF ------->|<VF owned>|<-- inaccessible for VF ->|
  *
- * GGTT nodes used for tracking allocations:
- *
- *      |<---------- balloon ------------>|<- nodes->|<----- balloon ------>|
- *
  * After the migration, GGTT area assigned to the VF might have shifted, either
  * to lower or to higher address. But we expect the total size and extra areas to
  * be identical, as migration can only happen between matching platforms.
@@ -219,37 +54,12 @@ int xe_tile_sriov_vf_prepare_ggtt(struct xe_tile *tile)
  * So the VF has a new slice of GGTT assigned, and during migration process, the
  * memory content was copied to that new area. But the &xe_ggtt nodes are still
  * tracking allocations using the old addresses. The nodes within VF owned area
- * have to be shifted, and balloon nodes need to be resized to properly mask out
- * areas not owned by the VF.
- *
- * Fixed &xe_ggtt nodes used for tracking allocations:
- *
- *     |<------ balloon ------>|<- nodes->|<----------- balloon ----------->|
+ * have to be shifted, and the start offset for GGTT adjusted.
  *
- * Due to use of GPU profiles, we do not expect the old and new GGTT ares to
+ * Due to use of GPU profiles, we do not expect the old and new GGTT areas to
  * overlap; but our node shifting will fix addresses properly regardless.
  */
 
-/**
- * xe_tile_sriov_vf_fixup_ggtt_nodes_locked - Shift GGTT allocations to match assigned range.
- * @tile: the &xe_tile struct instance
- * @shift: the shift value
- *
- * Since Global GTT is not virtualized, each VF has an assigned range
- * within the global space. This range might have changed during migration,
- * which requires all memory addresses pointing to GGTT to be shifted.
- */
-void xe_tile_sriov_vf_fixup_ggtt_nodes_locked(struct xe_tile *tile, s64 shift)
-{
-	struct xe_ggtt *ggtt = tile->mem.ggtt;
-
-	lockdep_assert_held(&ggtt->lock);
-
-	xe_tile_sriov_vf_deballoon_ggtt_locked(tile);
-	xe_ggtt_shift_nodes_locked(ggtt, shift);
-	xe_tile_sriov_vf_balloon_ggtt_locked(tile);
-}
-
 /**
  * xe_tile_sriov_vf_lmem - VF LMEM configuration.
  * @tile: the &xe_tile
@@ -330,7 +140,7 @@ u64 xe_tile_sriov_vf_ggtt_base(struct xe_tile *tile)
 
 	xe_tile_assert(tile, IS_SRIOV_VF(tile_to_xe(tile)));
 
-	return config->ggtt_base;
+	return READ_ONCE(config->ggtt_base);
 }
 
 /**
@@ -346,5 +156,5 @@ void xe_tile_sriov_vf_ggtt_base_store(struct xe_tile *tile, u64 ggtt_base)
 
 	xe_tile_assert(tile, IS_SRIOV_VF(tile_to_xe(tile)));
 
-	config->ggtt_base = ggtt_base;
+	WRITE_ONCE(config->ggtt_base, ggtt_base);
 }
diff --git a/drivers/gpu/drm/xe/xe_tile_sriov_vf.h b/drivers/gpu/drm/xe/xe_tile_sriov_vf.h
index 749f41504883c..f2bbc4fc57347 100644
--- a/drivers/gpu/drm/xe/xe_tile_sriov_vf.h
+++ b/drivers/gpu/drm/xe/xe_tile_sriov_vf.h
@@ -10,9 +10,6 @@
 
 struct xe_tile;
 
-int xe_tile_sriov_vf_prepare_ggtt(struct xe_tile *tile);
-void xe_tile_sriov_vf_deballoon_ggtt_locked(struct xe_tile *tile);
-void xe_tile_sriov_vf_fixup_ggtt_nodes_locked(struct xe_tile *tile, s64 shift);
 u64 xe_tile_sriov_vf_ggtt(struct xe_tile *tile);
 void xe_tile_sriov_vf_ggtt_store(struct xe_tile *tile, u64 ggtt_size);
 u64 xe_tile_sriov_vf_ggtt_base(struct xe_tile *tile);
-- 
2.51.0

Re: [PATCH v7 2/5] drm/xe: Rewrite GGTT VF initialization

[Michal Wajdeczko]

On 1/28/2026 5:57 PM, Maarten Lankhorst wrote:
> The previous code was using a complicated system with 2 balloons to
> set GGTT size and adjust GGTT offset. While it works, it's overly
> complicated.
> 
> A better approach is to set the offset and size when initializing GGTT,
> this removes the need for adding balloons. The resize function only
> needs readjust ggtt->start to have GGTT at the new offset.
> 
> This removes the need to manipulate the internals of xe_ggtt outside
> of xe_ggtt, and cleans up a lot of now unneeded code.
> 
> Co-developed-by: Matthew Brost <matthew.brost@intel.com>
> Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>

> ---
> Changelog:
> - Fix docbook description.
> - Simplify the vf_get_ggtt_info() flow and remove the shift calculation.

[PATCH v7 3/5] drm/xe: Move struct xe_ggtt to xe_ggtt.c

[Maarten Lankhorst]

No users left outside of xe_ggtt.c, so we can make the struct private.

This prevents us from accidentally touching it before init.

Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
---
 drivers/gpu/drm/xe/xe_ggtt.c       | 55 +++++++++++++++++++++++++++
 drivers/gpu/drm/xe/xe_ggtt.h       |  1 +
 drivers/gpu/drm/xe/xe_ggtt_types.h | 60 +-----------------------------
 3 files changed, 58 insertions(+), 58 deletions(-)

diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index 6ab655073d042..d0789877e2cdc 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -84,6 +84,61 @@ struct xe_ggtt_node {
 	bool invalidate_on_remove;
 };
 
+/**
+ * struct xe_ggtt_pt_ops - GGTT Page table operations
+ * Which can vary from platform to platform.
+ */
+struct xe_ggtt_pt_ops {
+	/** @pte_encode_flags: Encode PTE flags for a given BO */
+	u64 (*pte_encode_flags)(struct xe_bo *bo, u16 pat_index);
+
+	/** @ggtt_set_pte: Directly write into GGTT's PTE */
+	xe_ggtt_set_pte_fn ggtt_set_pte;
+
+	/** @ggtt_get_pte: Directly read from GGTT's PTE */
+	u64 (*ggtt_get_pte)(struct xe_ggtt *ggtt, u64 addr);
+};
+
+/**
+ * struct xe_ggtt - Main GGTT struct
+ *
+ * In general, each tile can contains its own Global Graphics Translation Table
+ * (GGTT) instance.
+ */
+struct xe_ggtt {
+	/** @tile: Back pointer to tile where this GGTT belongs */
+	struct xe_tile *tile;
+        /** @start: Start offset of GGTT */
+	u64 start;
+	/** @size: Total usable size of this GGTT */
+	u64 size;
+
+#define XE_GGTT_FLAGS_64K BIT(0)
+	/**
+	 * @flags: Flags for this GGTT
+	 * Acceptable flags:
+	 * - %XE_GGTT_FLAGS_64K - if PTE size is 64K. Otherwise, regular is 4K.
+	 */
+	unsigned int flags;
+	/** @scratch: Internal object allocation used as a scratch page */
+	struct xe_bo *scratch;
+	/** @lock: Mutex lock to protect GGTT data */
+	struct mutex lock;
+	/**
+	 *  @gsm: The iomem pointer to the actual location of the translation
+	 * table located in the GSM for easy PTE manipulation
+	 */
+	u64 __iomem *gsm;
+	/** @pt_ops: Page Table operations per platform */
+	const struct xe_ggtt_pt_ops *pt_ops;
+	/** @mm: The memory manager used to manage individual GGTT allocations */
+	struct drm_mm mm;
+	/** @access_count: counts GGTT writes */
+	unsigned int access_count;
+	/** @wq: Dedicated unordered work queue to process node removals */
+	struct workqueue_struct *wq;
+};
+
 static u64 xelp_ggtt_pte_flags(struct xe_bo *bo, u16 pat_index)
 {
 	u64 pte = XE_PAGE_PRESENT;
diff --git a/drivers/gpu/drm/xe/xe_ggtt.h b/drivers/gpu/drm/xe/xe_ggtt.h
index 49ea8e7ecc105..403eb5c0db49d 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.h
+++ b/drivers/gpu/drm/xe/xe_ggtt.h
@@ -9,6 +9,7 @@
 #include "xe_ggtt_types.h"
 
 struct drm_printer;
+struct xe_bo;
 struct xe_tile;
 struct drm_exec;
 
diff --git a/drivers/gpu/drm/xe/xe_ggtt_types.h b/drivers/gpu/drm/xe/xe_ggtt_types.h
index d82b71a198bc2..cf754e4d502ad 100644
--- a/drivers/gpu/drm/xe/xe_ggtt_types.h
+++ b/drivers/gpu/drm/xe/xe_ggtt_types.h
@@ -6,72 +6,16 @@
 #ifndef _XE_GGTT_TYPES_H_
 #define _XE_GGTT_TYPES_H_
 
+#include <linux/types.h>
 #include <drm/drm_mm.h>
 
-#include "xe_pt_types.h"
-
-struct xe_bo;
+struct xe_ggtt;
 struct xe_ggtt_node;
-struct xe_gt;
-
-/**
- * struct xe_ggtt - Main GGTT struct
- *
- * In general, each tile can contains its own Global Graphics Translation Table
- * (GGTT) instance.
- */
-struct xe_ggtt {
-	/** @tile: Back pointer to tile where this GGTT belongs */
-	struct xe_tile *tile;
-	/** @start: Start offset of GGTT */
-	u64 start;
-	/** @size: Total usable size of this GGTT */
-	u64 size;
-
-#define XE_GGTT_FLAGS_64K BIT(0)
-	/**
-	 * @flags: Flags for this GGTT
-	 * Acceptable flags:
-	 * - %XE_GGTT_FLAGS_64K - if PTE size is 64K. Otherwise, regular is 4K.
-	 */
-	unsigned int flags;
-	/** @scratch: Internal object allocation used as a scratch page */
-	struct xe_bo *scratch;
-	/** @lock: Mutex lock to protect GGTT data */
-	struct mutex lock;
-	/**
-	 *  @gsm: The iomem pointer to the actual location of the translation
-	 * table located in the GSM for easy PTE manipulation
-	 */
-	u64 __iomem *gsm;
-	/** @pt_ops: Page Table operations per platform */
-	const struct xe_ggtt_pt_ops *pt_ops;
-	/** @mm: The memory manager used to manage individual GGTT allocations */
-	struct drm_mm mm;
-	/** @access_count: counts GGTT writes */
-	unsigned int access_count;
-	/** @wq: Dedicated unordered work queue to process node removals */
-	struct workqueue_struct *wq;
-};
 
 typedef void (*xe_ggtt_set_pte_fn)(struct xe_ggtt *ggtt, u64 addr, u64 pte);
 typedef void (*xe_ggtt_transform_cb)(struct xe_ggtt *ggtt,
 				     struct xe_ggtt_node *node,
 				     u64 pte_flags,
 				     xe_ggtt_set_pte_fn set_pte, void *arg);
-/**
- * struct xe_ggtt_pt_ops - GGTT Page table operations
- * Which can vary from platform to platform.
- */
-struct xe_ggtt_pt_ops {
-	/** @pte_encode_flags: Encode PTE flags for a given BO */
-	u64 (*pte_encode_flags)(struct xe_bo *bo, u16 pat_index);
-
-	/** @ggtt_set_pte: Directly write into GGTT's PTE */
-	xe_ggtt_set_pte_fn ggtt_set_pte;
-
-	/** @ggtt_get_pte: Directly read from GGTT's PTE */
-	u64 (*ggtt_get_pte)(struct xe_ggtt *ggtt, u64 addr);
-};
 
 #endif
-- 
2.51.0

[PATCH v7 4/5] drm/xe: Make xe_ggtt_node_insert return a node

[Maarten Lankhorst]

This extra step is easier to handle inside xe_ggtt.c and makes
xe_ggtt_node_allocated a simple null check instead, as the intermediate
state 'allocated but not inserted' is no longer used.

Privatize xe_ggtt_node_fini() and init() as they're no longer used
outside of xe_ggtt.c

Reviewed-by: Matthew Brost <matthew.brost@intel.com> #v1
Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
---
 drivers/gpu/drm/xe/display/xe_fb_pin.c      |  2 +-
 drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c |  6 +-
 drivers/gpu/drm/xe/xe_ggtt.c                | 97 +++++++++------------
 drivers/gpu/drm/xe/xe_ggtt.h                |  7 +-
 drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c  | 24 +----
 5 files changed, 48 insertions(+), 88 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_fb_pin.c b/drivers/gpu/drm/xe/display/xe_fb_pin.c
index d2c4e94180fa3..5ff583699325c 100644
--- a/drivers/gpu/drm/xe/display/xe_fb_pin.c
+++ b/drivers/gpu/drm/xe/display/xe_fb_pin.c
@@ -256,7 +256,7 @@ static int __xe_pin_fb_vma_ggtt(const struct intel_framebuffer *fb,
 		size = intel_rotation_info_size(&view->rotated) * XE_PAGE_SIZE;
 
 	pte = xe_ggtt_encode_pte_flags(ggtt, bo, xe->pat.idx[XE_CACHE_NONE]);
-	vma->node = xe_ggtt_node_insert_transform(ggtt, bo, pte,
+	vma->node = xe_ggtt_insert_node_transform(ggtt, bo, pte,
 						  ALIGN(size, align), align,
 						  view->type == I915_GTT_VIEW_NORMAL ?
 						  NULL : write_ggtt_rotated_node,
diff --git a/drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c b/drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c
index acddbedcf17cb..51e1e04001ace 100644
--- a/drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c
+++ b/drivers/gpu/drm/xe/tests/xe_guc_buf_kunit.c
@@ -38,12 +38,8 @@ static struct xe_bo *replacement_xe_managed_bo_create_pin_map(struct xe_device *
 	if (flags & XE_BO_FLAG_GGTT) {
 		struct xe_ggtt *ggtt = tile->mem.ggtt;
 
-		bo->ggtt_node[tile->id] = xe_ggtt_node_init(ggtt);
+		bo->ggtt_node[tile->id] = xe_ggtt_insert_node(ggtt, xe_bo_size(bo), SZ_4K);
 		KUNIT_ASSERT_NOT_ERR_OR_NULL(test, bo->ggtt_node[tile->id]);
-
-		KUNIT_ASSERT_EQ(test, 0,
-				xe_ggtt_node_insert(bo->ggtt_node[tile->id],
-						    xe_bo_size(bo), SZ_4K));
 	}
 
 	return bo;
diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index d0789877e2cdc..ca9ae02659b1a 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -69,9 +69,8 @@
 /**
  * struct xe_ggtt_node - A node in GGTT.
  *
- * This struct needs to be initialized (only-once) with xe_ggtt_node_init() before any node
- * insertion or reservation.
- * It will, then, be finalized by xe_ggtt_node_remove().
+ * This struct is allocated with xe_ggtt_insert_node(,_transform) or xe_ggtt_insert_bo(,_at).
+ * It will be deallocated using xe_ggtt_node_remove().
  */
 struct xe_ggtt_node {
 	/** @ggtt: Back pointer to xe_ggtt where this region will be inserted at */
@@ -458,6 +457,11 @@ static void xe_ggtt_initial_clear(struct xe_ggtt *ggtt)
 	mutex_unlock(&ggtt->lock);
 }
 
+static void ggtt_node_fini(struct xe_ggtt_node *node)
+{
+	kfree(node);
+}
+
 static void ggtt_node_remove(struct xe_ggtt_node *node)
 {
 	struct xe_ggtt *ggtt = node->ggtt;
@@ -483,7 +487,7 @@ static void ggtt_node_remove(struct xe_ggtt_node *node)
 	drm_dev_exit(idx);
 
 free_node:
-	xe_ggtt_node_fini(node);
+	ggtt_node_fini(node);
 }
 
 static void ggtt_node_remove_work_func(struct work_struct *work)
@@ -613,50 +617,14 @@ void xe_ggtt_shift_nodes(struct xe_ggtt *ggtt, u64 new_start)
 	WRITE_ONCE(ggtt->start, new_start);
 }
 
-static int xe_ggtt_node_insert_locked(struct xe_ggtt_node *node,
+static int xe_ggtt_insert_node_locked(struct xe_ggtt_node *node,
 				      u32 size, u32 align, u32 mm_flags)
 {
 	return drm_mm_insert_node_generic(&node->ggtt->mm, &node->base, size, align, 0,
 					  mm_flags);
 }
 
-/**
- * xe_ggtt_node_insert - Insert a &xe_ggtt_node into the GGTT
- * @node: the &xe_ggtt_node to be inserted
- * @size: size of the node
- * @align: alignment constrain of the node
- *
- * It cannot be called without first having called xe_ggtt_init() once.
- *
- * Return: 0 on success or a negative error code on failure.
- */
-int xe_ggtt_node_insert(struct xe_ggtt_node *node, u32 size, u32 align)
-{
-	int ret;
-
-	if (!node || !node->ggtt)
-		return -ENOENT;
-
-	mutex_lock(&node->ggtt->lock);
-	ret = xe_ggtt_node_insert_locked(node, size, align,
-					 DRM_MM_INSERT_HIGH);
-	mutex_unlock(&node->ggtt->lock);
-
-	return ret;
-}
-
-/**
- * xe_ggtt_node_init - Initialize %xe_ggtt_node struct
- * @ggtt: the &xe_ggtt where the new node will later be inserted/reserved.
- *
- * This function will allocate the struct %xe_ggtt_node and return its pointer.
- * This struct will then be freed after the node removal upon xe_ggtt_node_remove()
- * Having %xe_ggtt_node struct allocated doesn't mean that the node is already allocated
- * in GGTT. Only xe_ggtt_node_insert() will ensure the node is inserted or reserved in GGTT.
- *
- * Return: A pointer to %xe_ggtt_node struct on success. An ERR_PTR otherwise.
- **/
-struct xe_ggtt_node *xe_ggtt_node_init(struct xe_ggtt *ggtt)
+static struct xe_ggtt_node *ggtt_node_init(struct xe_ggtt *ggtt)
 {
 	struct xe_ggtt_node *node = kzalloc(sizeof(*node), GFP_NOFS);
 
@@ -670,16 +638,31 @@ struct xe_ggtt_node *xe_ggtt_node_init(struct xe_ggtt *ggtt)
 }
 
 /**
- * xe_ggtt_node_fini - Forcebly finalize %xe_ggtt_node struct
- * @node: the &xe_ggtt_node to be freed
+ * xe_ggtt_insert_node - Insert a &xe_ggtt_node into the GGTT
+ * @ggtt: the &xe_ggtt into which the node should be inserted.
+ * @size: size of the node
+ * @align: alignment constrain of the node
  *
- * If anything went wrong with either xe_ggtt_node_insert() and this @node is
- * not going to be reused, then this function needs to be called to free the
- * %xe_ggtt_node struct
- **/
-void xe_ggtt_node_fini(struct xe_ggtt_node *node)
+ * Return: &xe_ggtt_node on success or a ERR_PTR on failure.
+ */
+struct xe_ggtt_node *xe_ggtt_insert_node(struct xe_ggtt *ggtt, u32 size, u32 align)
 {
-	kfree(node);
+	struct xe_ggtt_node *node;
+	int ret;
+
+	node = ggtt_node_init(ggtt);
+	if (IS_ERR(node))
+		return node;
+
+	guard(mutex)(&ggtt->lock);
+	ret = xe_ggtt_insert_node_locked(node, size, align,
+					 DRM_MM_INSERT_HIGH);
+	if (ret) {
+		ggtt_node_fini(node);
+		return ERR_PTR(ret);
+	}
+
+	return node;
 }
 
 /**
@@ -767,7 +750,7 @@ void xe_ggtt_map_bo_unlocked(struct xe_ggtt *ggtt, struct xe_bo *bo)
 }
 
 /**
- * xe_ggtt_node_insert_transform - Insert a newly allocated &xe_ggtt_node into the GGTT
+ * xe_ggtt_insert_node_transform - Insert a newly allocated &xe_ggtt_node into the GGTT
  * @ggtt: the &xe_ggtt where the node will inserted/reserved.
  * @bo: The bo to be transformed
  * @pte_flags: The extra GGTT flags to add to mapping.
@@ -781,7 +764,7 @@ void xe_ggtt_map_bo_unlocked(struct xe_ggtt *ggtt, struct xe_bo *bo)
  *
  * Return: A pointer to %xe_ggtt_node struct on success. An ERR_PTR otherwise.
  */
-struct xe_ggtt_node *xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
+struct xe_ggtt_node *xe_ggtt_insert_node_transform(struct xe_ggtt *ggtt,
 						   struct xe_bo *bo, u64 pte_flags,
 						   u64 size, u32 align,
 						   xe_ggtt_transform_cb transform, void *arg)
@@ -789,7 +772,7 @@ struct xe_ggtt_node *xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
 	struct xe_ggtt_node *node;
 	int ret;
 
-	node = xe_ggtt_node_init(ggtt);
+	node = ggtt_node_init(ggtt);
 	if (IS_ERR(node))
 		return ERR_CAST(node);
 
@@ -798,7 +781,7 @@ struct xe_ggtt_node *xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
 		goto err;
 	}
 
-	ret = xe_ggtt_node_insert_locked(node, size, align, 0);
+	ret = xe_ggtt_insert_node_locked(node, size, align, 0);
 	if (ret)
 		goto err_unlock;
 
@@ -813,7 +796,7 @@ struct xe_ggtt_node *xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
 err_unlock:
 	mutex_unlock(&ggtt->lock);
 err:
-	xe_ggtt_node_fini(node);
+	ggtt_node_fini(node);
 	return ERR_PTR(ret);
 }
 
@@ -839,7 +822,7 @@ static int __xe_ggtt_insert_bo_at(struct xe_ggtt *ggtt, struct xe_bo *bo,
 
 	xe_pm_runtime_get_noresume(tile_to_xe(ggtt->tile));
 
-	bo->ggtt_node[tile_id] = xe_ggtt_node_init(ggtt);
+	bo->ggtt_node[tile_id] = ggtt_node_init(ggtt);
 	if (IS_ERR(bo->ggtt_node[tile_id])) {
 		err = PTR_ERR(bo->ggtt_node[tile_id]);
 		bo->ggtt_node[tile_id] = NULL;
@@ -858,7 +841,7 @@ static int __xe_ggtt_insert_bo_at(struct xe_ggtt *ggtt, struct xe_bo *bo,
 	err = drm_mm_insert_node_in_range(&ggtt->mm, &bo->ggtt_node[tile_id]->base,
 					  xe_bo_size(bo), alignment, 0, start, end, 0);
 	if (err) {
-		xe_ggtt_node_fini(bo->ggtt_node[tile_id]);
+		ggtt_node_fini(bo->ggtt_node[tile_id]);
 		bo->ggtt_node[tile_id] = NULL;
 	} else {
 		u16 cache_mode = bo->flags & XE_BO_FLAG_NEEDS_UC ? XE_CACHE_NONE : XE_CACHE_WB;
diff --git a/drivers/gpu/drm/xe/xe_ggtt.h b/drivers/gpu/drm/xe/xe_ggtt.h
index 403eb5c0db49d..9e6210c6f44ea 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.h
+++ b/drivers/gpu/drm/xe/xe_ggtt.h
@@ -18,15 +18,14 @@ int xe_ggtt_init_early(struct xe_ggtt *ggtt);
 int xe_ggtt_init_kunit(struct xe_ggtt *ggtt, u32 reserved, u32 size);
 int xe_ggtt_init(struct xe_ggtt *ggtt);
 
-struct xe_ggtt_node *xe_ggtt_node_init(struct xe_ggtt *ggtt);
-void xe_ggtt_node_fini(struct xe_ggtt_node *node);
 void xe_ggtt_shift_nodes(struct xe_ggtt *ggtt, u64 new_base);
 u64 xe_ggtt_start(struct xe_ggtt *ggtt);
 u64 xe_ggtt_size(struct xe_ggtt *ggtt);
 
-int xe_ggtt_node_insert(struct xe_ggtt_node *node, u32 size, u32 align);
 struct xe_ggtt_node *
-xe_ggtt_node_insert_transform(struct xe_ggtt *ggtt,
+xe_ggtt_insert_node(struct xe_ggtt *ggtt, u32 size, u32 align);
+struct xe_ggtt_node *
+xe_ggtt_insert_node_transform(struct xe_ggtt *ggtt,
 			      struct xe_bo *bo, u64 pte,
 			      u64 size, u32 align,
 			      xe_ggtt_transform_cb transform, void *arg);
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 23601ce79348a..3fe664cd3b88c 100644
--- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
+++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
@@ -482,23 +482,9 @@ static int pf_distribute_config_ggtt(struct xe_tile *tile, unsigned int vfid, u6
 	return err ?: err2;
 }
 
-static void pf_release_ggtt(struct xe_tile *tile, struct xe_ggtt_node *node)
-{
-	if (xe_ggtt_node_allocated(node)) {
-		/*
-		 * explicit GGTT PTE assignment to the PF using xe_ggtt_assign()
-		 * 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_node_remove(node, false);
-	} else {
-		xe_ggtt_node_fini(node);
-	}
-}
-
 static void pf_release_vf_config_ggtt(struct xe_gt *gt, struct xe_gt_sriov_config *config)
 {
-	pf_release_ggtt(gt_to_tile(gt), config->ggtt_region);
+	xe_ggtt_node_remove(config->ggtt_region, false);
 	config->ggtt_region = NULL;
 }
 
@@ -533,14 +519,10 @@ static int pf_provision_vf_ggtt(struct xe_gt *gt, unsigned int vfid, u64 size)
 	if (!size)
 		return 0;
 
-	node = xe_ggtt_node_init(ggtt);
+	node = xe_ggtt_insert_node(ggtt, size, alignment);
 	if (IS_ERR(node))
 		return PTR_ERR(node);
 
-	err = xe_ggtt_node_insert(node, size, alignment);
-	if (unlikely(err))
-		goto err;
-
 	xe_ggtt_assign(node, vfid);
 	xe_gt_sriov_dbg_verbose(gt, "VF%u assigned GGTT %llx-%llx\n",
 				vfid, xe_ggtt_node_addr(node), xe_ggtt_node_addr(node) + size - 1);
@@ -552,7 +534,7 @@ static int pf_provision_vf_ggtt(struct xe_gt *gt, unsigned int vfid, u64 size)
 	config->ggtt_region = node;
 	return 0;
 err:
-	pf_release_ggtt(tile, node);
+	xe_ggtt_node_remove(node, false);
 	return err;
 }
 
-- 
2.51.0

[PATCH v7 5/5] drm/xe: Remove xe_ggtt_node_allocated

[Maarten Lankhorst]

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.

Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
---
Changelog:
- Update commit message to explain the change in xe_fb_pin.c
---
 drivers/gpu/drm/xe/display/xe_fb_pin.c     |  3 +--
 drivers/gpu/drm/xe/xe_ggtt.c               | 17 -----------------
 drivers/gpu/drm/xe/xe_ggtt.h               |  1 -
 drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c | 12 ++++++------
 4 files changed, 7 insertions(+), 26 deletions(-)

diff --git a/drivers/gpu/drm/xe/display/xe_fb_pin.c b/drivers/gpu/drm/xe/display/xe_fb_pin.c
index 5ff583699325c..e1d29b6ba043e 100644
--- a/drivers/gpu/drm/xe/display/xe_fb_pin.c
+++ b/drivers/gpu/drm/xe/display/xe_fb_pin.c
@@ -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);
diff --git a/drivers/gpu/drm/xe/xe_ggtt.c b/drivers/gpu/drm/xe/xe_ggtt.c
index ca9ae02659b1a..ae9c68bca9a4d 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.c
+++ b/drivers/gpu/drm/xe/xe_ggtt.c
@@ -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
@@ -959,9 +945,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;
diff --git a/drivers/gpu/drm/xe/xe_ggtt.h b/drivers/gpu/drm/xe/xe_ggtt.h
index 9e6210c6f44ea..c864cc975a695 100644
--- a/drivers/gpu/drm/xe/xe_ggtt.h
+++ b/drivers/gpu/drm/xe/xe_ggtt.h
@@ -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);
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 3fe664cd3b88c..888193e1d2c50 100644
--- a/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
+++ b/drivers/gpu/drm/xe/xe_gt_sriov_pf_config.c
@@ -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,
-- 
2.51.0

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

[Michal Wajdeczko]

On 1/28/2026 5:57 PM, Maarten Lankhorst wrote:
> 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.
> 
> Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>

Reviewed-by: Michal Wajdeczko <michal.wajdeczko@intel.com>

> ---
> Changelog:
> - Update commit message to explain the change in xe_fb_pin.c
> ---

✗ CI.checkpatch: warning for drm/xe: Privatize struct xe_ggtt. (rev5)

[Patchwork]

== Series Details ==

Series: drm/xe: Privatize struct xe_ggtt. (rev5)
URL   : https://patchwork.freedesktop.org/series/160006/
State : warning

== Summary ==

+ KERNEL=/kernel
+ git clone https://gitlab.freedesktop.org/drm/maintainer-tools mt
Cloning into 'mt'...
warning: redirecting to https://gitlab.freedesktop.org/drm/maintainer-tools.git/
+ git -C mt rev-list -n1 origin/master
1f57ba1afceae32108bd24770069f764d940a0e4
+ cd /kernel
+ git config --global --add safe.directory /kernel
+ git log -n1
commit 21d73f39e9a7d7c580f0f4d851cb83b9da631fb8
Author: Maarten Lankhorst <dev@lankhorst.se>
Date:   Wed Jan 28 17:57:23 2026 +0100

    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.
    
    Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>
+ /mt/dim checkpatch 8059f097e25f736bb3da09af6a9b283079abfd4f drm-intel
53ec1c1456e6 drm/xe: Make xe_ggtt_node offset relative to starting offset
-:64: WARNING:LONG_LINE: line length of 107 exceeds 100 columns
#64: FILE: drivers/gpu/drm/xe/xe_ggtt.c:583:
+			 xe_ggtt_node_addr(node), xe_ggtt_node_addr(node) + node->base.size, ERR_PTR(err)))

total: 0 errors, 1 warnings, 0 checks, 141 lines checked
e7b03f702d9c drm/xe: Rewrite GGTT VF initialization
-:17: WARNING:BAD_SIGN_OFF: Co-developed-by and Signed-off-by: name/email do not match
#17: 
Co-developed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Maarten Lankhorst <dev@lankhorst.se>

total: 0 errors, 1 warnings, 0 checks, 539 lines checked
6eaac9407d43 drm/xe: Move struct xe_ggtt to xe_ggtt.c
-:46: ERROR:CODE_INDENT: code indent should use tabs where possible
#46: FILE: drivers/gpu/drm/xe/xe_ggtt.c:111:
+        /** @start: Start offset of GGTT */$

total: 1 errors, 0 warnings, 0 checks, 142 lines checked
714f82824f0b drm/xe: Make xe_ggtt_node_insert return a node
21d73f39e9a7 drm/xe: Remove xe_ggtt_node_allocated

✓ CI.KUnit: success for drm/xe: Privatize struct xe_ggtt. (rev5)

[Patchwork]

== Series Details ==

Series: drm/xe: Privatize struct xe_ggtt. (rev5)
URL   : https://patchwork.freedesktop.org/series/160006/
State : success

== Summary ==

+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[20:27:56] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:28:00] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[20:28:39] Starting KUnit Kernel (1/1)...
[20:28:39] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:28:39] ================== guc_buf (11 subtests) ===================
[20:28:39] [PASSED] test_smallest
[20:28:39] [PASSED] test_largest
[20:28:39] [PASSED] test_granular
[20:28:39] [PASSED] test_unique
[20:28:39] [PASSED] test_overlap
[20:28:39] [PASSED] test_reusable
[20:28:39] [PASSED] test_too_big
[20:28:39] [PASSED] test_flush
[20:28:39] [PASSED] test_lookup
[20:28:39] [PASSED] test_data
[20:28:39] [PASSED] test_class
[20:28:39] ===================== [PASSED] guc_buf =====================
[20:28:39] =================== guc_dbm (7 subtests) ===================
[20:28:39] [PASSED] test_empty
[20:28:39] [PASSED] test_default
[20:28:39] ======================== test_size  ========================
[20:28:39] [PASSED] 4
[20:28:39] [PASSED] 8
[20:28:39] [PASSED] 32
[20:28:39] [PASSED] 256
[20:28:39] ==================== [PASSED] test_size ====================
[20:28:39] ======================= test_reuse  ========================
[20:28:39] [PASSED] 4
[20:28:39] [PASSED] 8
[20:28:39] [PASSED] 32
[20:28:39] [PASSED] 256
[20:28:39] =================== [PASSED] test_reuse ====================
[20:28:39] =================== test_range_overlap  ====================
[20:28:39] [PASSED] 4
[20:28:39] [PASSED] 8
[20:28:39] [PASSED] 32
[20:28:39] [PASSED] 256
[20:28:39] =============== [PASSED] test_range_overlap ================
[20:28:39] =================== test_range_compact  ====================
[20:28:39] [PASSED] 4
[20:28:39] [PASSED] 8
[20:28:39] [PASSED] 32
[20:28:39] [PASSED] 256
[20:28:39] =============== [PASSED] test_range_compact ================
[20:28:39] ==================== test_range_spare  =====================
[20:28:39] [PASSED] 4
[20:28:39] [PASSED] 8
[20:28:39] [PASSED] 32
[20:28:39] [PASSED] 256
[20:28:39] ================ [PASSED] test_range_spare =================
[20:28:39] ===================== [PASSED] guc_dbm =====================
[20:28:39] =================== guc_idm (6 subtests) ===================
[20:28:39] [PASSED] bad_init
[20:28:39] [PASSED] no_init
[20:28:39] [PASSED] init_fini
[20:28:39] [PASSED] check_used
[20:28:39] [PASSED] check_quota
[20:28:39] [PASSED] check_all
[20:28:39] ===================== [PASSED] guc_idm =====================
[20:28:39] ================== no_relay (3 subtests) ===================
[20:28:39] [PASSED] xe_drops_guc2pf_if_not_ready
[20:28:39] [PASSED] xe_drops_guc2vf_if_not_ready
[20:28:39] [PASSED] xe_rejects_send_if_not_ready
[20:28:39] ==================== [PASSED] no_relay =====================
[20:28:39] ================== pf_relay (14 subtests) ==================
[20:28:39] [PASSED] pf_rejects_guc2pf_too_short
[20:28:39] [PASSED] pf_rejects_guc2pf_too_long
[20:28:39] [PASSED] pf_rejects_guc2pf_no_payload
[20:28:39] [PASSED] pf_fails_no_payload
[20:28:39] [PASSED] pf_fails_bad_origin
[20:28:39] [PASSED] pf_fails_bad_type
[20:28:39] [PASSED] pf_txn_reports_error
[20:28:39] [PASSED] pf_txn_sends_pf2guc
[20:28:39] [PASSED] pf_sends_pf2guc
[20:28:39] [SKIPPED] pf_loopback_nop
[20:28:39] [SKIPPED] pf_loopback_echo
[20:28:39] [SKIPPED] pf_loopback_fail
[20:28:39] [SKIPPED] pf_loopback_busy
[20:28:39] [SKIPPED] pf_loopback_retry
[20:28:39] ==================== [PASSED] pf_relay =====================
[20:28:39] ================== vf_relay (3 subtests) ===================
[20:28:39] [PASSED] vf_rejects_guc2vf_too_short
[20:28:39] [PASSED] vf_rejects_guc2vf_too_long
[20:28:39] [PASSED] vf_rejects_guc2vf_no_payload
[20:28:39] ==================== [PASSED] vf_relay =====================
[20:28:39] ================ pf_gt_config (6 subtests) =================
[20:28:39] [PASSED] fair_contexts_1vf
[20:28:39] [PASSED] fair_doorbells_1vf
[20:28:39] [PASSED] fair_ggtt_1vf
[20:28:39] ====================== fair_contexts  ======================
[20:28:39] [PASSED] 1 VF
[20:28:39] [PASSED] 2 VFs
[20:28:39] [PASSED] 3 VFs
[20:28:39] [PASSED] 4 VFs
[20:28:39] [PASSED] 5 VFs
[20:28:39] [PASSED] 6 VFs
[20:28:39] [PASSED] 7 VFs
[20:28:39] [PASSED] 8 VFs
[20:28:39] [PASSED] 9 VFs
[20:28:39] [PASSED] 10 VFs
[20:28:39] [PASSED] 11 VFs
[20:28:39] [PASSED] 12 VFs
[20:28:39] [PASSED] 13 VFs
[20:28:39] [PASSED] 14 VFs
[20:28:39] [PASSED] 15 VFs
[20:28:39] [PASSED] 16 VFs
[20:28:39] [PASSED] 17 VFs
[20:28:39] [PASSED] 18 VFs
[20:28:39] [PASSED] 19 VFs
[20:28:39] [PASSED] 20 VFs
[20:28:39] [PASSED] 21 VFs
[20:28:39] [PASSED] 22 VFs
[20:28:39] [PASSED] 23 VFs
[20:28:39] [PASSED] 24 VFs
[20:28:39] [PASSED] 25 VFs
[20:28:39] [PASSED] 26 VFs
[20:28:39] [PASSED] 27 VFs
[20:28:39] [PASSED] 28 VFs
[20:28:39] [PASSED] 29 VFs
[20:28:39] [PASSED] 30 VFs
[20:28:39] [PASSED] 31 VFs
[20:28:39] [PASSED] 32 VFs
[20:28:39] [PASSED] 33 VFs
[20:28:39] [PASSED] 34 VFs
[20:28:39] [PASSED] 35 VFs
[20:28:39] [PASSED] 36 VFs
[20:28:39] [PASSED] 37 VFs
[20:28:39] [PASSED] 38 VFs
[20:28:39] [PASSED] 39 VFs
[20:28:39] [PASSED] 40 VFs
[20:28:39] [PASSED] 41 VFs
[20:28:39] [PASSED] 42 VFs
[20:28:39] [PASSED] 43 VFs
[20:28:39] [PASSED] 44 VFs
[20:28:39] [PASSED] 45 VFs
[20:28:39] [PASSED] 46 VFs
[20:28:39] [PASSED] 47 VFs
[20:28:39] [PASSED] 48 VFs
[20:28:39] [PASSED] 49 VFs
[20:28:39] [PASSED] 50 VFs
[20:28:39] [PASSED] 51 VFs
[20:28:39] [PASSED] 52 VFs
[20:28:39] [PASSED] 53 VFs
[20:28:39] [PASSED] 54 VFs
[20:28:39] [PASSED] 55 VFs
[20:28:39] [PASSED] 56 VFs
[20:28:39] [PASSED] 57 VFs
[20:28:39] [PASSED] 58 VFs
[20:28:39] [PASSED] 59 VFs
[20:28:39] [PASSED] 60 VFs
[20:28:39] [PASSED] 61 VFs
[20:28:39] [PASSED] 62 VFs
[20:28:39] [PASSED] 63 VFs
[20:28:39] ================== [PASSED] fair_contexts ==================
[20:28:39] ===================== fair_doorbells  ======================
[20:28:39] [PASSED] 1 VF
[20:28:39] [PASSED] 2 VFs
[20:28:39] [PASSED] 3 VFs
[20:28:39] [PASSED] 4 VFs
[20:28:39] [PASSED] 5 VFs
[20:28:39] [PASSED] 6 VFs
[20:28:39] [PASSED] 7 VFs
[20:28:39] [PASSED] 8 VFs
[20:28:39] [PASSED] 9 VFs
[20:28:39] [PASSED] 10 VFs
[20:28:39] [PASSED] 11 VFs
[20:28:39] [PASSED] 12 VFs
[20:28:39] [PASSED] 13 VFs
[20:28:39] [PASSED] 14 VFs
[20:28:39] [PASSED] 15 VFs
[20:28:39] [PASSED] 16 VFs
[20:28:39] [PASSED] 17 VFs
[20:28:39] [PASSED] 18 VFs
[20:28:39] [PASSED] 19 VFs
[20:28:39] [PASSED] 20 VFs
[20:28:39] [PASSED] 21 VFs
[20:28:39] [PASSED] 22 VFs
[20:28:39] [PASSED] 23 VFs
[20:28:39] [PASSED] 24 VFs
[20:28:39] [PASSED] 25 VFs
[20:28:39] [PASSED] 26 VFs
[20:28:39] [PASSED] 27 VFs
[20:28:39] [PASSED] 28 VFs
[20:28:39] [PASSED] 29 VFs
[20:28:39] [PASSED] 30 VFs
[20:28:39] [PASSED] 31 VFs
[20:28:39] [PASSED] 32 VFs
[20:28:39] [PASSED] 33 VFs
[20:28:39] [PASSED] 34 VFs
[20:28:39] [PASSED] 35 VFs
[20:28:39] [PASSED] 36 VFs
[20:28:39] [PASSED] 37 VFs
[20:28:39] [PASSED] 38 VFs
[20:28:39] [PASSED] 39 VFs
[20:28:39] [PASSED] 40 VFs
[20:28:39] [PASSED] 41 VFs
[20:28:39] [PASSED] 42 VFs
[20:28:39] [PASSED] 43 VFs
[20:28:39] [PASSED] 44 VFs
[20:28:39] [PASSED] 45 VFs
[20:28:39] [PASSED] 46 VFs
[20:28:39] [PASSED] 47 VFs
[20:28:39] [PASSED] 48 VFs
[20:28:39] [PASSED] 49 VFs
[20:28:39] [PASSED] 50 VFs
[20:28:39] [PASSED] 51 VFs
[20:28:39] [PASSED] 52 VFs
[20:28:39] [PASSED] 53 VFs
[20:28:39] [PASSED] 54 VFs
[20:28:39] [PASSED] 55 VFs
[20:28:39] [PASSED] 56 VFs
[20:28:39] [PASSED] 57 VFs
[20:28:39] [PASSED] 58 VFs
[20:28:39] [PASSED] 59 VFs
[20:28:39] [PASSED] 60 VFs
[20:28:39] [PASSED] 61 VFs
[20:28:39] [PASSED] 62 VFs
[20:28:39] [PASSED] 63 VFs
[20:28:39] ================= [PASSED] fair_doorbells ==================
[20:28:39] ======================== fair_ggtt  ========================
[20:28:39] [PASSED] 1 VF
[20:28:39] [PASSED] 2 VFs
[20:28:39] [PASSED] 3 VFs
[20:28:39] [PASSED] 4 VFs
[20:28:39] [PASSED] 5 VFs
[20:28:39] [PASSED] 6 VFs
[20:28:39] [PASSED] 7 VFs
[20:28:39] [PASSED] 8 VFs
[20:28:39] [PASSED] 9 VFs
[20:28:39] [PASSED] 10 VFs
[20:28:39] [PASSED] 11 VFs
[20:28:39] [PASSED] 12 VFs
[20:28:39] [PASSED] 13 VFs
[20:28:39] [PASSED] 14 VFs
[20:28:39] [PASSED] 15 VFs
[20:28:39] [PASSED] 16 VFs
[20:28:39] [PASSED] 17 VFs
[20:28:39] [PASSED] 18 VFs
[20:28:39] [PASSED] 19 VFs
[20:28:39] [PASSED] 20 VFs
[20:28:39] [PASSED] 21 VFs
[20:28:39] [PASSED] 22 VFs
[20:28:39] [PASSED] 23 VFs
[20:28:39] [PASSED] 24 VFs
[20:28:39] [PASSED] 25 VFs
[20:28:39] [PASSED] 26 VFs
[20:28:39] [PASSED] 27 VFs
[20:28:39] [PASSED] 28 VFs
[20:28:39] [PASSED] 29 VFs
[20:28:39] [PASSED] 30 VFs
[20:28:39] [PASSED] 31 VFs
[20:28:39] [PASSED] 32 VFs
[20:28:39] [PASSED] 33 VFs
[20:28:39] [PASSED] 34 VFs
[20:28:39] [PASSED] 35 VFs
[20:28:39] [PASSED] 36 VFs
[20:28:39] [PASSED] 37 VFs
[20:28:39] [PASSED] 38 VFs
[20:28:39] [PASSED] 39 VFs
[20:28:39] [PASSED] 40 VFs
[20:28:39] [PASSED] 41 VFs
[20:28:39] [PASSED] 42 VFs
[20:28:39] [PASSED] 43 VFs
[20:28:39] [PASSED] 44 VFs
[20:28:39] [PASSED] 45 VFs
[20:28:39] [PASSED] 46 VFs
[20:28:39] [PASSED] 47 VFs
[20:28:39] [PASSED] 48 VFs
[20:28:39] [PASSED] 49 VFs
[20:28:39] [PASSED] 50 VFs
[20:28:39] [PASSED] 51 VFs
[20:28:39] [PASSED] 52 VFs
[20:28:39] [PASSED] 53 VFs
[20:28:39] [PASSED] 54 VFs
[20:28:39] [PASSED] 55 VFs
[20:28:39] [PASSED] 56 VFs
[20:28:39] [PASSED] 57 VFs
[20:28:39] [PASSED] 58 VFs
[20:28:39] [PASSED] 59 VFs
[20:28:39] [PASSED] 60 VFs
[20:28:39] [PASSED] 61 VFs
[20:28:39] [PASSED] 62 VFs
[20:28:39] [PASSED] 63 VFs
[20:28:39] ==================== [PASSED] fair_ggtt ====================
[20:28:39] ================== [PASSED] pf_gt_config ===================
[20:28:39] ===================== lmtt (1 subtest) =====================
[20:28:39] ======================== test_ops  =========================
[20:28:39] [PASSED] 2-level
[20:28:39] [PASSED] multi-level
[20:28:39] ==================== [PASSED] test_ops =====================
[20:28:39] ====================== [PASSED] lmtt =======================
[20:28:39] ================= pf_service (11 subtests) =================
[20:28:39] [PASSED] pf_negotiate_any
[20:28:39] [PASSED] pf_negotiate_base_match
[20:28:39] [PASSED] pf_negotiate_base_newer
[20:28:39] [PASSED] pf_negotiate_base_next
[20:28:39] [SKIPPED] pf_negotiate_base_older
[20:28:39] [PASSED] pf_negotiate_base_prev
[20:28:39] [PASSED] pf_negotiate_latest_match
[20:28:39] [PASSED] pf_negotiate_latest_newer
[20:28:39] [PASSED] pf_negotiate_latest_next
[20:28:39] [SKIPPED] pf_negotiate_latest_older
[20:28:39] [SKIPPED] pf_negotiate_latest_prev
[20:28:39] =================== [PASSED] pf_service ====================
[20:28:39] ================= xe_guc_g2g (2 subtests) ==================
[20:28:39] ============== xe_live_guc_g2g_kunit_default  ==============
[20:28:39] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[20:28:39] ============== xe_live_guc_g2g_kunit_allmem  ===============
[20:28:39] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[20:28:39] =================== [SKIPPED] xe_guc_g2g ===================
[20:28:39] =================== xe_mocs (2 subtests) ===================
[20:28:39] ================ xe_live_mocs_kernel_kunit  ================
[20:28:39] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[20:28:39] ================ xe_live_mocs_reset_kunit  =================
[20:28:39] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[20:28:39] ==================== [SKIPPED] xe_mocs =====================
[20:28:39] ================= xe_migrate (2 subtests) ==================
[20:28:39] ================= xe_migrate_sanity_kunit  =================
[20:28:39] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[20:28:39] ================== xe_validate_ccs_kunit  ==================
[20:28:39] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[20:28:39] =================== [SKIPPED] xe_migrate ===================
[20:28:39] ================== xe_dma_buf (1 subtest) ==================
[20:28:39] ==================== xe_dma_buf_kunit  =====================
[20:28:39] ================ [SKIPPED] xe_dma_buf_kunit ================
[20:28:39] =================== [SKIPPED] xe_dma_buf ===================
[20:28:39] ================= xe_bo_shrink (1 subtest) =================
[20:28:39] =================== xe_bo_shrink_kunit  ====================
[20:28:39] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[20:28:39] ================== [SKIPPED] xe_bo_shrink ==================
[20:28:39] ==================== xe_bo (2 subtests) ====================
[20:28:39] ================== xe_ccs_migrate_kunit  ===================
[20:28:39] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[20:28:39] ==================== xe_bo_evict_kunit  ====================
[20:28:39] =============== [SKIPPED] xe_bo_evict_kunit ================
[20:28:39] ===================== [SKIPPED] xe_bo ======================
[20:28:39] ==================== args (13 subtests) ====================
[20:28:39] [PASSED] count_args_test
[20:28:39] [PASSED] call_args_example
[20:28:39] [PASSED] call_args_test
[20:28:39] [PASSED] drop_first_arg_example
[20:28:39] [PASSED] drop_first_arg_test
[20:28:39] [PASSED] first_arg_example
[20:28:39] [PASSED] first_arg_test
[20:28:39] [PASSED] last_arg_example
[20:28:39] [PASSED] last_arg_test
[20:28:39] [PASSED] pick_arg_example
[20:28:39] [PASSED] if_args_example
[20:28:39] [PASSED] if_args_test
[20:28:39] [PASSED] sep_comma_example
[20:28:39] ====================== [PASSED] args =======================
[20:28:39] =================== xe_pci (3 subtests) ====================
[20:28:39] ==================== check_graphics_ip  ====================
[20:28:39] [PASSED] 12.00 Xe_LP
[20:28:39] [PASSED] 12.10 Xe_LP+
[20:28:39] [PASSED] 12.55 Xe_HPG
[20:28:39] [PASSED] 12.60 Xe_HPC
[20:28:39] [PASSED] 12.70 Xe_LPG
[20:28:39] [PASSED] 12.71 Xe_LPG
[20:28:39] [PASSED] 12.74 Xe_LPG+
[20:28:39] [PASSED] 20.01 Xe2_HPG
[20:28:39] [PASSED] 20.02 Xe2_HPG
[20:28:39] [PASSED] 20.04 Xe2_LPG
[20:28:39] [PASSED] 30.00 Xe3_LPG
[20:28:39] [PASSED] 30.01 Xe3_LPG
[20:28:39] [PASSED] 30.03 Xe3_LPG
[20:28:39] [PASSED] 30.04 Xe3_LPG
[20:28:39] [PASSED] 30.05 Xe3_LPG
[20:28:39] [PASSED] 35.11 Xe3p_XPC
[20:28:39] ================ [PASSED] check_graphics_ip ================
[20:28:39] ===================== check_media_ip  ======================
[20:28:39] [PASSED] 12.00 Xe_M
[20:28:39] [PASSED] 12.55 Xe_HPM
[20:28:39] [PASSED] 13.00 Xe_LPM+
[20:28:39] [PASSED] 13.01 Xe2_HPM
[20:28:39] [PASSED] 20.00 Xe2_LPM
[20:28:39] [PASSED] 30.00 Xe3_LPM
[20:28:39] [PASSED] 30.02 Xe3_LPM
[20:28:39] [PASSED] 35.00 Xe3p_LPM
[20:28:39] [PASSED] 35.03 Xe3p_HPM
[20:28:39] ================= [PASSED] check_media_ip ==================
[20:28:39] =================== check_platform_desc  ===================
[20:28:39] [PASSED] 0x9A60 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A68 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A70 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A40 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A49 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A59 (TIGERLAKE)
[20:28:39] [PASSED] 0x9A78 (TIGERLAKE)
[20:28:39] [PASSED] 0x9AC0 (TIGERLAKE)
[20:28:39] [PASSED] 0x9AC9 (TIGERLAKE)
[20:28:39] [PASSED] 0x9AD9 (TIGERLAKE)
[20:28:39] [PASSED] 0x9AF8 (TIGERLAKE)
[20:28:39] [PASSED] 0x4C80 (ROCKETLAKE)
[20:28:39] [PASSED] 0x4C8A (ROCKETLAKE)
[20:28:39] [PASSED] 0x4C8B (ROCKETLAKE)
[20:28:39] [PASSED] 0x4C8C (ROCKETLAKE)
[20:28:39] [PASSED] 0x4C90 (ROCKETLAKE)
[20:28:39] [PASSED] 0x4C9A (ROCKETLAKE)
[20:28:39] [PASSED] 0x4680 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4682 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4688 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x468A (ALDERLAKE_S)
[20:28:39] [PASSED] 0x468B (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4690 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4692 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4693 (ALDERLAKE_S)
[20:28:39] [PASSED] 0x46A0 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46A1 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46A2 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46A3 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46A6 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46A8 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46AA (ALDERLAKE_P)
[20:28:39] [PASSED] 0x462A (ALDERLAKE_P)
[20:28:39] [PASSED] 0x4626 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x4628 (ALDERLAKE_P)
stty: 'standard input': Inappropriate ioctl for device
[20:28:39] [PASSED] 0x46B0 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46B1 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46B2 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46B3 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46C0 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46C1 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46C2 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46C3 (ALDERLAKE_P)
[20:28:39] [PASSED] 0x46D0 (ALDERLAKE_N)
[20:28:39] [PASSED] 0x46D1 (ALDERLAKE_N)
[20:28:39] [PASSED] 0x46D2 (ALDERLAKE_N)
[20:28:39] [PASSED] 0x46D3 (ALDERLAKE_N)
[20:28:39] [PASSED] 0x46D4 (ALDERLAKE_N)
[20:28:39] [PASSED] 0xA721 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7A1 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7A9 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7AC (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7AD (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA720 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7A0 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7A8 (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7AA (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA7AB (ALDERLAKE_P)
[20:28:39] [PASSED] 0xA780 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA781 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA782 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA783 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA788 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA789 (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA78A (ALDERLAKE_S)
[20:28:39] [PASSED] 0xA78B (ALDERLAKE_S)
[20:28:39] [PASSED] 0x4905 (DG1)
[20:28:39] [PASSED] 0x4906 (DG1)
[20:28:39] [PASSED] 0x4907 (DG1)
[20:28:39] [PASSED] 0x4908 (DG1)
[20:28:39] [PASSED] 0x4909 (DG1)
[20:28:39] [PASSED] 0x56C0 (DG2)
[20:28:39] [PASSED] 0x56C2 (DG2)
[20:28:39] [PASSED] 0x56C1 (DG2)
[20:28:39] [PASSED] 0x7D51 (METEORLAKE)
[20:28:39] [PASSED] 0x7DD1 (METEORLAKE)
[20:28:39] [PASSED] 0x7D41 (METEORLAKE)
[20:28:39] [PASSED] 0x7D67 (METEORLAKE)
[20:28:39] [PASSED] 0xB640 (METEORLAKE)
[20:28:39] [PASSED] 0x56A0 (DG2)
[20:28:39] [PASSED] 0x56A1 (DG2)
[20:28:39] [PASSED] 0x56A2 (DG2)
[20:28:39] [PASSED] 0x56BE (DG2)
[20:28:39] [PASSED] 0x56BF (DG2)
[20:28:39] [PASSED] 0x5690 (DG2)
[20:28:39] [PASSED] 0x5691 (DG2)
[20:28:39] [PASSED] 0x5692 (DG2)
[20:28:39] [PASSED] 0x56A5 (DG2)
[20:28:39] [PASSED] 0x56A6 (DG2)
[20:28:39] [PASSED] 0x56B0 (DG2)
[20:28:39] [PASSED] 0x56B1 (DG2)
[20:28:39] [PASSED] 0x56BA (DG2)
[20:28:39] [PASSED] 0x56BB (DG2)
[20:28:39] [PASSED] 0x56BC (DG2)
[20:28:39] [PASSED] 0x56BD (DG2)
[20:28:39] [PASSED] 0x5693 (DG2)
[20:28:39] [PASSED] 0x5694 (DG2)
[20:28:39] [PASSED] 0x5695 (DG2)
[20:28:39] [PASSED] 0x56A3 (DG2)
[20:28:39] [PASSED] 0x56A4 (DG2)
[20:28:39] [PASSED] 0x56B2 (DG2)
[20:28:39] [PASSED] 0x56B3 (DG2)
[20:28:39] [PASSED] 0x5696 (DG2)
[20:28:39] [PASSED] 0x5697 (DG2)
[20:28:39] [PASSED] 0xB69 (PVC)
[20:28:39] [PASSED] 0xB6E (PVC)
[20:28:39] [PASSED] 0xBD4 (PVC)
[20:28:39] [PASSED] 0xBD5 (PVC)
[20:28:39] [PASSED] 0xBD6 (PVC)
[20:28:39] [PASSED] 0xBD7 (PVC)
[20:28:39] [PASSED] 0xBD8 (PVC)
[20:28:39] [PASSED] 0xBD9 (PVC)
[20:28:39] [PASSED] 0xBDA (PVC)
[20:28:39] [PASSED] 0xBDB (PVC)
[20:28:39] [PASSED] 0xBE0 (PVC)
[20:28:39] [PASSED] 0xBE1 (PVC)
[20:28:39] [PASSED] 0xBE5 (PVC)
[20:28:39] [PASSED] 0x7D40 (METEORLAKE)
[20:28:39] [PASSED] 0x7D45 (METEORLAKE)
[20:28:39] [PASSED] 0x7D55 (METEORLAKE)
[20:28:39] [PASSED] 0x7D60 (METEORLAKE)
[20:28:39] [PASSED] 0x7DD5 (METEORLAKE)
[20:28:39] [PASSED] 0x6420 (LUNARLAKE)
[20:28:39] [PASSED] 0x64A0 (LUNARLAKE)
[20:28:39] [PASSED] 0x64B0 (LUNARLAKE)
[20:28:39] [PASSED] 0xE202 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE209 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE20B (BATTLEMAGE)
[20:28:39] [PASSED] 0xE20C (BATTLEMAGE)
[20:28:39] [PASSED] 0xE20D (BATTLEMAGE)
[20:28:39] [PASSED] 0xE210 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE211 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE212 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE216 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE220 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE221 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE222 (BATTLEMAGE)
[20:28:39] [PASSED] 0xE223 (BATTLEMAGE)
[20:28:39] [PASSED] 0xB080 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB081 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB082 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB083 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB084 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB085 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB086 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB087 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB08F (PANTHERLAKE)
[20:28:39] [PASSED] 0xB090 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB0A0 (PANTHERLAKE)
[20:28:39] [PASSED] 0xB0B0 (PANTHERLAKE)
[20:28:39] [PASSED] 0xFD80 (PANTHERLAKE)
[20:28:39] [PASSED] 0xFD81 (PANTHERLAKE)
[20:28:39] [PASSED] 0xD740 (NOVALAKE_S)
[20:28:39] [PASSED] 0xD741 (NOVALAKE_S)
[20:28:39] [PASSED] 0xD742 (NOVALAKE_S)
[20:28:39] [PASSED] 0xD743 (NOVALAKE_S)
[20:28:39] [PASSED] 0xD744 (NOVALAKE_S)
[20:28:39] [PASSED] 0xD745 (NOVALAKE_S)
[20:28:39] [PASSED] 0x674C (CRESCENTISLAND)
[20:28:39] =============== [PASSED] check_platform_desc ===============
[20:28:39] ===================== [PASSED] xe_pci ======================
[20:28:39] =================== xe_rtp (2 subtests) ====================
[20:28:39] =============== xe_rtp_process_to_sr_tests  ================
[20:28:39] [PASSED] coalesce-same-reg
[20:28:39] [PASSED] no-match-no-add
[20:28:39] [PASSED] match-or
[20:28:39] [PASSED] match-or-xfail
[20:28:39] [PASSED] no-match-no-add-multiple-rules
[20:28:39] [PASSED] two-regs-two-entries
[20:28:39] [PASSED] clr-one-set-other
[20:28:39] [PASSED] set-field
[20:28:39] [PASSED] conflict-duplicate
[20:28:39] [PASSED] conflict-not-disjoint
[20:28:39] [PASSED] conflict-reg-type
[20:28:39] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[20:28:39] ================== xe_rtp_process_tests  ===================
[20:28:39] [PASSED] active1
[20:28:39] [PASSED] active2
[20:28:39] [PASSED] active-inactive
[20:28:39] [PASSED] inactive-active
[20:28:39] [PASSED] inactive-1st_or_active-inactive
[20:28:39] [PASSED] inactive-2nd_or_active-inactive
[20:28:39] [PASSED] inactive-last_or_active-inactive
[20:28:39] [PASSED] inactive-no_or_active-inactive
[20:28:39] ============== [PASSED] xe_rtp_process_tests ===============
[20:28:39] ===================== [PASSED] xe_rtp ======================
[20:28:39] ==================== xe_wa (1 subtest) =====================
[20:28:39] ======================== xe_wa_gt  =========================
[20:28:39] [PASSED] TIGERLAKE B0
[20:28:39] [PASSED] DG1 A0
[20:28:39] [PASSED] DG1 B0
[20:28:39] [PASSED] ALDERLAKE_S A0
[20:28:39] [PASSED] ALDERLAKE_S B0
[20:28:39] [PASSED] ALDERLAKE_S C0
[20:28:39] [PASSED] ALDERLAKE_S D0
[20:28:39] [PASSED] ALDERLAKE_P A0
[20:28:39] [PASSED] ALDERLAKE_P B0
[20:28:39] [PASSED] ALDERLAKE_P C0
[20:28:39] [PASSED] ALDERLAKE_S RPLS D0
[20:28:39] [PASSED] ALDERLAKE_P RPLU E0
[20:28:39] [PASSED] DG2 G10 C0
[20:28:39] [PASSED] DG2 G11 B1
[20:28:39] [PASSED] DG2 G12 A1
[20:28:39] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[20:28:39] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[20:28:39] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[20:28:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[20:28:39] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[20:28:39] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[20:28:39] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[20:28:39] ==================== [PASSED] xe_wa_gt =====================
[20:28:39] ====================== [PASSED] xe_wa ======================
[20:28:39] ============================================================
[20:28:39] Testing complete. Ran 512 tests: passed: 494, skipped: 18
[20:28:39] Elapsed time: 43.740s total, 4.298s configuring, 38.875s building, 0.550s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[20:28:39] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:28:41] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[20:29:11] Starting KUnit Kernel (1/1)...
[20:29:11] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:29:11] ============ drm_test_pick_cmdline (2 subtests) ============
[20:29:11] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[20:29:11] =============== drm_test_pick_cmdline_named  ===============
[20:29:11] [PASSED] NTSC
[20:29:11] [PASSED] NTSC-J
[20:29:11] [PASSED] PAL
[20:29:11] [PASSED] PAL-M
[20:29:11] =========== [PASSED] drm_test_pick_cmdline_named ===========
[20:29:11] ============== [PASSED] drm_test_pick_cmdline ==============
[20:29:11] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[20:29:11] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[20:29:11] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[20:29:11] =========== drm_validate_clone_mode (2 subtests) ===========
[20:29:11] ============== drm_test_check_in_clone_mode  ===============
[20:29:11] [PASSED] in_clone_mode
[20:29:11] [PASSED] not_in_clone_mode
[20:29:11] ========== [PASSED] drm_test_check_in_clone_mode ===========
[20:29:11] =============== drm_test_check_valid_clones  ===============
[20:29:11] [PASSED] not_in_clone_mode
[20:29:11] [PASSED] valid_clone
[20:29:11] [PASSED] invalid_clone
[20:29:11] =========== [PASSED] drm_test_check_valid_clones ===========
[20:29:11] ============= [PASSED] drm_validate_clone_mode =============
[20:29:11] ============= drm_validate_modeset (1 subtest) =============
[20:29:11] [PASSED] drm_test_check_connector_changed_modeset
[20:29:11] ============== [PASSED] drm_validate_modeset ===============
[20:29:11] ====== drm_test_bridge_get_current_state (2 subtests) ======
[20:29:11] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[20:29:11] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[20:29:11] ======== [PASSED] drm_test_bridge_get_current_state ========
[20:29:11] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[20:29:11] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[20:29:11] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[20:29:11] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[20:29:11] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[20:29:11] ============== drm_bridge_alloc (2 subtests) ===============
[20:29:11] [PASSED] drm_test_drm_bridge_alloc_basic
[20:29:11] [PASSED] drm_test_drm_bridge_alloc_get_put
[20:29:11] ================ [PASSED] drm_bridge_alloc =================
[20:29:11] ================== drm_buddy (9 subtests) ==================
[20:29:11] [PASSED] drm_test_buddy_alloc_limit
[20:29:11] [PASSED] drm_test_buddy_alloc_optimistic
[20:29:11] [PASSED] drm_test_buddy_alloc_pessimistic
[20:29:11] [PASSED] drm_test_buddy_alloc_pathological
[20:29:11] [PASSED] drm_test_buddy_alloc_contiguous
[20:29:11] [PASSED] drm_test_buddy_alloc_clear
[20:29:12] [PASSED] drm_test_buddy_alloc_range_bias
[20:29:12] [PASSED] drm_test_buddy_fragmentation_performance
[20:29:12] [PASSED] drm_test_buddy_alloc_exceeds_max_order
[20:29:12] ==================== [PASSED] drm_buddy ====================
[20:29:12] ============= drm_cmdline_parser (40 subtests) =============
[20:29:12] [PASSED] drm_test_cmdline_force_d_only
[20:29:12] [PASSED] drm_test_cmdline_force_D_only_dvi
[20:29:12] [PASSED] drm_test_cmdline_force_D_only_hdmi
[20:29:12] [PASSED] drm_test_cmdline_force_D_only_not_digital
[20:29:12] [PASSED] drm_test_cmdline_force_e_only
[20:29:12] [PASSED] drm_test_cmdline_res
[20:29:12] [PASSED] drm_test_cmdline_res_vesa
[20:29:12] [PASSED] drm_test_cmdline_res_vesa_rblank
[20:29:12] [PASSED] drm_test_cmdline_res_rblank
[20:29:12] [PASSED] drm_test_cmdline_res_bpp
[20:29:12] [PASSED] drm_test_cmdline_res_refresh
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[20:29:12] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[20:29:12] [PASSED] drm_test_cmdline_res_margins_force_on
[20:29:12] [PASSED] drm_test_cmdline_res_vesa_margins
[20:29:12] [PASSED] drm_test_cmdline_name
[20:29:12] [PASSED] drm_test_cmdline_name_bpp
[20:29:12] [PASSED] drm_test_cmdline_name_option
[20:29:12] [PASSED] drm_test_cmdline_name_bpp_option
[20:29:12] [PASSED] drm_test_cmdline_rotate_0
[20:29:12] [PASSED] drm_test_cmdline_rotate_90
[20:29:12] [PASSED] drm_test_cmdline_rotate_180
[20:29:12] [PASSED] drm_test_cmdline_rotate_270
[20:29:12] [PASSED] drm_test_cmdline_hmirror
[20:29:12] [PASSED] drm_test_cmdline_vmirror
[20:29:12] [PASSED] drm_test_cmdline_margin_options
[20:29:12] [PASSED] drm_test_cmdline_multiple_options
[20:29:12] [PASSED] drm_test_cmdline_bpp_extra_and_option
[20:29:12] [PASSED] drm_test_cmdline_extra_and_option
[20:29:12] [PASSED] drm_test_cmdline_freestanding_options
[20:29:12] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[20:29:12] [PASSED] drm_test_cmdline_panel_orientation
[20:29:12] ================ drm_test_cmdline_invalid  =================
[20:29:12] [PASSED] margin_only
[20:29:12] [PASSED] interlace_only
[20:29:12] [PASSED] res_missing_x
[20:29:12] [PASSED] res_missing_y
[20:29:12] [PASSED] res_bad_y
[20:29:12] [PASSED] res_missing_y_bpp
[20:29:12] [PASSED] res_bad_bpp
[20:29:12] [PASSED] res_bad_refresh
[20:29:12] [PASSED] res_bpp_refresh_force_on_off
[20:29:12] [PASSED] res_invalid_mode
[20:29:12] [PASSED] res_bpp_wrong_place_mode
[20:29:12] [PASSED] name_bpp_refresh
[20:29:12] [PASSED] name_refresh
[20:29:12] [PASSED] name_refresh_wrong_mode
[20:29:12] [PASSED] name_refresh_invalid_mode
[20:29:12] [PASSED] rotate_multiple
[20:29:12] [PASSED] rotate_invalid_val
[20:29:12] [PASSED] rotate_truncated
[20:29:12] [PASSED] invalid_option
[20:29:12] [PASSED] invalid_tv_option
[20:29:12] [PASSED] truncated_tv_option
[20:29:12] ============ [PASSED] drm_test_cmdline_invalid =============
[20:29:12] =============== drm_test_cmdline_tv_options  ===============
[20:29:12] [PASSED] NTSC
[20:29:12] [PASSED] NTSC_443
[20:29:12] [PASSED] NTSC_J
[20:29:12] [PASSED] PAL
[20:29:12] [PASSED] PAL_M
[20:29:12] [PASSED] PAL_N
[20:29:12] [PASSED] SECAM
[20:29:12] [PASSED] MONO_525
[20:29:12] [PASSED] MONO_625
[20:29:12] =========== [PASSED] drm_test_cmdline_tv_options ===========
[20:29:12] =============== [PASSED] drm_cmdline_parser ================
[20:29:12] ========== drmm_connector_hdmi_init (20 subtests) ==========
[20:29:12] [PASSED] drm_test_connector_hdmi_init_valid
[20:29:12] [PASSED] drm_test_connector_hdmi_init_bpc_8
[20:29:12] [PASSED] drm_test_connector_hdmi_init_bpc_10
[20:29:12] [PASSED] drm_test_connector_hdmi_init_bpc_12
[20:29:12] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[20:29:12] [PASSED] drm_test_connector_hdmi_init_bpc_null
[20:29:12] [PASSED] drm_test_connector_hdmi_init_formats_empty
[20:29:12] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[20:29:12] === drm_test_connector_hdmi_init_formats_yuv420_allowed  ===
[20:29:12] [PASSED] supported_formats=0x9 yuv420_allowed=1
[20:29:12] [PASSED] supported_formats=0x9 yuv420_allowed=0
[20:29:12] [PASSED] supported_formats=0x3 yuv420_allowed=1
[20:29:12] [PASSED] supported_formats=0x3 yuv420_allowed=0
[20:29:12] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[20:29:12] [PASSED] drm_test_connector_hdmi_init_null_ddc
[20:29:12] [PASSED] drm_test_connector_hdmi_init_null_product
[20:29:12] [PASSED] drm_test_connector_hdmi_init_null_vendor
[20:29:12] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[20:29:12] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[20:29:12] [PASSED] drm_test_connector_hdmi_init_product_valid
[20:29:12] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[20:29:12] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[20:29:12] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[20:29:12] ========= drm_test_connector_hdmi_init_type_valid  =========
[20:29:12] [PASSED] HDMI-A
[20:29:12] [PASSED] HDMI-B
[20:29:12] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[20:29:12] ======== drm_test_connector_hdmi_init_type_invalid  ========
[20:29:12] [PASSED] Unknown
[20:29:12] [PASSED] VGA
[20:29:12] [PASSED] DVI-I
[20:29:12] [PASSED] DVI-D
[20:29:12] [PASSED] DVI-A
[20:29:12] [PASSED] Composite
[20:29:12] [PASSED] SVIDEO
[20:29:12] [PASSED] LVDS
[20:29:12] [PASSED] Component
[20:29:12] [PASSED] DIN
[20:29:12] [PASSED] DP
[20:29:12] [PASSED] TV
[20:29:12] [PASSED] eDP
[20:29:12] [PASSED] Virtual
[20:29:12] [PASSED] DSI
[20:29:12] [PASSED] DPI
[20:29:12] [PASSED] Writeback
[20:29:12] [PASSED] SPI
[20:29:12] [PASSED] USB
[20:29:12] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[20:29:12] ============ [PASSED] drmm_connector_hdmi_init =============
[20:29:12] ============= drmm_connector_init (3 subtests) =============
[20:29:12] [PASSED] drm_test_drmm_connector_init
[20:29:12] [PASSED] drm_test_drmm_connector_init_null_ddc
[20:29:12] ========= drm_test_drmm_connector_init_type_valid  =========
[20:29:12] [PASSED] Unknown
[20:29:12] [PASSED] VGA
[20:29:12] [PASSED] DVI-I
[20:29:12] [PASSED] DVI-D
[20:29:12] [PASSED] DVI-A
[20:29:12] [PASSED] Composite
[20:29:12] [PASSED] SVIDEO
[20:29:12] [PASSED] LVDS
[20:29:12] [PASSED] Component
[20:29:12] [PASSED] DIN
[20:29:12] [PASSED] DP
[20:29:12] [PASSED] HDMI-A
[20:29:12] [PASSED] HDMI-B
[20:29:12] [PASSED] TV
[20:29:12] [PASSED] eDP
[20:29:12] [PASSED] Virtual
[20:29:12] [PASSED] DSI
[20:29:12] [PASSED] DPI
[20:29:12] [PASSED] Writeback
[20:29:12] [PASSED] SPI
[20:29:12] [PASSED] USB
[20:29:12] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[20:29:12] =============== [PASSED] drmm_connector_init ===============
[20:29:12] ========= drm_connector_dynamic_init (6 subtests) ==========
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_init
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_init_properties
[20:29:12] ===== drm_test_drm_connector_dynamic_init_type_valid  ======
[20:29:12] [PASSED] Unknown
[20:29:12] [PASSED] VGA
[20:29:12] [PASSED] DVI-I
[20:29:12] [PASSED] DVI-D
[20:29:12] [PASSED] DVI-A
[20:29:12] [PASSED] Composite
[20:29:12] [PASSED] SVIDEO
[20:29:12] [PASSED] LVDS
[20:29:12] [PASSED] Component
[20:29:12] [PASSED] DIN
[20:29:12] [PASSED] DP
[20:29:12] [PASSED] HDMI-A
[20:29:12] [PASSED] HDMI-B
[20:29:12] [PASSED] TV
[20:29:12] [PASSED] eDP
[20:29:12] [PASSED] Virtual
[20:29:12] [PASSED] DSI
[20:29:12] [PASSED] DPI
[20:29:12] [PASSED] Writeback
[20:29:12] [PASSED] SPI
[20:29:12] [PASSED] USB
[20:29:12] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[20:29:12] ======== drm_test_drm_connector_dynamic_init_name  =========
[20:29:12] [PASSED] Unknown
[20:29:12] [PASSED] VGA
[20:29:12] [PASSED] DVI-I
[20:29:12] [PASSED] DVI-D
[20:29:12] [PASSED] DVI-A
[20:29:12] [PASSED] Composite
[20:29:12] [PASSED] SVIDEO
[20:29:12] [PASSED] LVDS
[20:29:12] [PASSED] Component
[20:29:12] [PASSED] DIN
[20:29:12] [PASSED] DP
[20:29:12] [PASSED] HDMI-A
[20:29:12] [PASSED] HDMI-B
[20:29:12] [PASSED] TV
[20:29:12] [PASSED] eDP
[20:29:12] [PASSED] Virtual
[20:29:12] [PASSED] DSI
[20:29:12] [PASSED] DPI
[20:29:12] [PASSED] Writeback
[20:29:12] [PASSED] SPI
[20:29:12] [PASSED] USB
[20:29:12] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[20:29:12] =========== [PASSED] drm_connector_dynamic_init ============
[20:29:12] ==== drm_connector_dynamic_register_early (4 subtests) =====
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[20:29:12] ====== [PASSED] drm_connector_dynamic_register_early =======
[20:29:12] ======= drm_connector_dynamic_register (7 subtests) ========
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[20:29:12] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[20:29:12] ========= [PASSED] drm_connector_dynamic_register ==========
[20:29:12] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[20:29:12] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[20:29:12] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[20:29:12] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[20:29:12] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[20:29:12] ========== drm_test_get_tv_mode_from_name_valid  ===========
[20:29:12] [PASSED] NTSC
[20:29:12] [PASSED] NTSC-443
[20:29:12] [PASSED] NTSC-J
[20:29:12] [PASSED] PAL
[20:29:12] [PASSED] PAL-M
[20:29:12] [PASSED] PAL-N
[20:29:12] [PASSED] SECAM
[20:29:12] [PASSED] Mono
[20:29:12] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[20:29:12] [PASSED] drm_test_get_tv_mode_from_name_truncated
[20:29:12] ============ [PASSED] drm_get_tv_mode_from_name ============
[20:29:12] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[20:29:12] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[20:29:12] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid  =
[20:29:12] [PASSED] VIC 96
[20:29:12] [PASSED] VIC 97
[20:29:12] [PASSED] VIC 101
[20:29:12] [PASSED] VIC 102
[20:29:12] [PASSED] VIC 106
[20:29:12] [PASSED] VIC 107
[20:29:12] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[20:29:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[20:29:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[20:29:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[20:29:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[20:29:12] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[20:29:12] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[20:29:12] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[20:29:12] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name  ====
[20:29:12] [PASSED] Automatic
[20:29:12] [PASSED] Full
[20:29:12] [PASSED] Limited 16:235
[20:29:12] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[20:29:12] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[20:29:12] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[20:29:12] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[20:29:12] === drm_test_drm_hdmi_connector_get_output_format_name  ====
[20:29:12] [PASSED] RGB
[20:29:12] [PASSED] YUV 4:2:0
[20:29:12] [PASSED] YUV 4:2:2
[20:29:12] [PASSED] YUV 4:4:4
[20:29:12] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[20:29:12] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[20:29:12] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[20:29:12] ============= drm_damage_helper (21 subtests) ==============
[20:29:12] [PASSED] drm_test_damage_iter_no_damage
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_src_moved
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_not_visible
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[20:29:12] [PASSED] drm_test_damage_iter_no_damage_no_fb
[20:29:12] [PASSED] drm_test_damage_iter_simple_damage
[20:29:12] [PASSED] drm_test_damage_iter_single_damage
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_outside_src
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_src_moved
[20:29:12] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[20:29:12] [PASSED] drm_test_damage_iter_damage
[20:29:12] [PASSED] drm_test_damage_iter_damage_one_intersect
[20:29:12] [PASSED] drm_test_damage_iter_damage_one_outside
[20:29:12] [PASSED] drm_test_damage_iter_damage_src_moved
[20:29:12] [PASSED] drm_test_damage_iter_damage_not_visible
[20:29:12] ================ [PASSED] drm_damage_helper ================
[20:29:12] ============== drm_dp_mst_helper (3 subtests) ==============
[20:29:12] ============== drm_test_dp_mst_calc_pbn_mode  ==============
[20:29:12] [PASSED] Clock 154000 BPP 30 DSC disabled
[20:29:12] [PASSED] Clock 234000 BPP 30 DSC disabled
[20:29:12] [PASSED] Clock 297000 BPP 24 DSC disabled
[20:29:12] [PASSED] Clock 332880 BPP 24 DSC enabled
[20:29:12] [PASSED] Clock 324540 BPP 24 DSC enabled
[20:29:12] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[20:29:12] ============== drm_test_dp_mst_calc_pbn_div  ===============
[20:29:12] [PASSED] Link rate 2000000 lane count 4
[20:29:12] [PASSED] Link rate 2000000 lane count 2
[20:29:12] [PASSED] Link rate 2000000 lane count 1
[20:29:12] [PASSED] Link rate 1350000 lane count 4
[20:29:12] [PASSED] Link rate 1350000 lane count 2
[20:29:12] [PASSED] Link rate 1350000 lane count 1
[20:29:12] [PASSED] Link rate 1000000 lane count 4
[20:29:12] [PASSED] Link rate 1000000 lane count 2
[20:29:12] [PASSED] Link rate 1000000 lane count 1
[20:29:12] [PASSED] Link rate 810000 lane count 4
[20:29:12] [PASSED] Link rate 810000 lane count 2
[20:29:12] [PASSED] Link rate 810000 lane count 1
[20:29:12] [PASSED] Link rate 540000 lane count 4
[20:29:12] [PASSED] Link rate 540000 lane count 2
[20:29:12] [PASSED] Link rate 540000 lane count 1
[20:29:12] [PASSED] Link rate 270000 lane count 4
[20:29:12] [PASSED] Link rate 270000 lane count 2
[20:29:12] [PASSED] Link rate 270000 lane count 1
[20:29:12] [PASSED] Link rate 162000 lane count 4
[20:29:12] [PASSED] Link rate 162000 lane count 2
[20:29:12] [PASSED] Link rate 162000 lane count 1
[20:29:12] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[20:29:12] ========= drm_test_dp_mst_sideband_msg_req_decode  =========
[20:29:12] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[20:29:12] [PASSED] DP_POWER_UP_PHY with port number
[20:29:12] [PASSED] DP_POWER_DOWN_PHY with port number
[20:29:12] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[20:29:12] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[20:29:12] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[20:29:12] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[20:29:12] [PASSED] DP_QUERY_PAYLOAD with port number
[20:29:12] [PASSED] DP_QUERY_PAYLOAD with VCPI
[20:29:12] [PASSED] DP_REMOTE_DPCD_READ with port number
[20:29:12] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[20:29:12] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[20:29:12] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[20:29:12] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[20:29:12] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[20:29:12] [PASSED] DP_REMOTE_I2C_READ with port number
[20:29:12] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[20:29:12] [PASSED] DP_REMOTE_I2C_READ with transactions array
[20:29:12] [PASSED] DP_REMOTE_I2C_WRITE with port number
[20:29:12] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[20:29:12] [PASSED] DP_REMOTE_I2C_WRITE with data array
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[20:29:12] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[20:29:12] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[20:29:12] ================ [PASSED] drm_dp_mst_helper ================
[20:29:12] ================== drm_exec (7 subtests) ===================
[20:29:12] [PASSED] sanitycheck
[20:29:12] [PASSED] test_lock
[20:29:12] [PASSED] test_lock_unlock
[20:29:12] [PASSED] test_duplicates
[20:29:12] [PASSED] test_prepare
[20:29:12] [PASSED] test_prepare_array
[20:29:12] [PASSED] test_multiple_loops
[20:29:12] ==================== [PASSED] drm_exec =====================
[20:29:12] =========== drm_format_helper_test (17 subtests) ===========
[20:29:12] ============== drm_test_fb_xrgb8888_to_gray8  ==============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[20:29:12] ============= drm_test_fb_xrgb8888_to_rgb332  ==============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[20:29:12] ============= drm_test_fb_xrgb8888_to_rgb565  ==============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[20:29:12] ============ drm_test_fb_xrgb8888_to_xrgb1555  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[20:29:12] ============ drm_test_fb_xrgb8888_to_argb1555  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[20:29:12] ============ drm_test_fb_xrgb8888_to_rgba5551  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[20:29:12] ============= drm_test_fb_xrgb8888_to_rgb888  ==============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[20:29:12] ============= drm_test_fb_xrgb8888_to_bgr888  ==============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[20:29:12] ============ drm_test_fb_xrgb8888_to_argb8888  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[20:29:12] =========== drm_test_fb_xrgb8888_to_xrgb2101010  ===========
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[20:29:12] =========== drm_test_fb_xrgb8888_to_argb2101010  ===========
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[20:29:12] ============== drm_test_fb_xrgb8888_to_mono  ===============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[20:29:12] ==================== drm_test_fb_swab  =====================
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ================ [PASSED] drm_test_fb_swab =================
[20:29:12] ============ drm_test_fb_xrgb8888_to_xbgr8888  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[20:29:12] ============ drm_test_fb_xrgb8888_to_abgr8888  =============
[20:29:12] [PASSED] single_pixel_source_buffer
[20:29:12] [PASSED] single_pixel_clip_rectangle
[20:29:12] [PASSED] well_known_colors
[20:29:12] [PASSED] destination_pitch
[20:29:12] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[20:29:12] ================= drm_test_fb_clip_offset  =================
[20:29:12] [PASSED] pass through
[20:29:12] [PASSED] horizontal offset
[20:29:12] [PASSED] vertical offset
[20:29:12] [PASSED] horizontal and vertical offset
[20:29:12] [PASSED] horizontal offset (custom pitch)
[20:29:12] [PASSED] vertical offset (custom pitch)
[20:29:12] [PASSED] horizontal and vertical offset (custom pitch)
[20:29:12] ============= [PASSED] drm_test_fb_clip_offset =============
[20:29:12] =================== drm_test_fb_memcpy  ====================
[20:29:12] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[20:29:12] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[20:29:12] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[20:29:12] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[20:29:12] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[20:29:12] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[20:29:12] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[20:29:12] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[20:29:12] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[20:29:12] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[20:29:12] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[20:29:12] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[20:29:12] =============== [PASSED] drm_test_fb_memcpy ================
[20:29:12] ============= [PASSED] drm_format_helper_test ==============
[20:29:12] ================= drm_format (18 subtests) =================
[20:29:12] [PASSED] drm_test_format_block_width_invalid
[20:29:12] [PASSED] drm_test_format_block_width_one_plane
[20:29:12] [PASSED] drm_test_format_block_width_two_plane
[20:29:12] [PASSED] drm_test_format_block_width_three_plane
[20:29:12] [PASSED] drm_test_format_block_width_tiled
[20:29:12] [PASSED] drm_test_format_block_height_invalid
[20:29:12] [PASSED] drm_test_format_block_height_one_plane
[20:29:12] [PASSED] drm_test_format_block_height_two_plane
[20:29:12] [PASSED] drm_test_format_block_height_three_plane
[20:29:12] [PASSED] drm_test_format_block_height_tiled
[20:29:12] [PASSED] drm_test_format_min_pitch_invalid
[20:29:12] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[20:29:12] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[20:29:12] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[20:29:12] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[20:29:12] [PASSED] drm_test_format_min_pitch_two_plane
[20:29:12] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[20:29:12] [PASSED] drm_test_format_min_pitch_tiled
[20:29:12] =================== [PASSED] drm_format ====================
[20:29:12] ============== drm_framebuffer (10 subtests) ===============
[20:29:12] ========== drm_test_framebuffer_check_src_coords  ==========
[20:29:12] [PASSED] Success: source fits into fb
[20:29:12] [PASSED] Fail: overflowing fb with x-axis coordinate
[20:29:12] [PASSED] Fail: overflowing fb with y-axis coordinate
[20:29:12] [PASSED] Fail: overflowing fb with source width
[20:29:12] [PASSED] Fail: overflowing fb with source height
[20:29:12] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[20:29:12] [PASSED] drm_test_framebuffer_cleanup
[20:29:12] =============== drm_test_framebuffer_create  ===============
[20:29:12] [PASSED] ABGR8888 normal sizes
[20:29:12] [PASSED] ABGR8888 max sizes
[20:29:12] [PASSED] ABGR8888 pitch greater than min required
[20:29:12] [PASSED] ABGR8888 pitch less than min required
[20:29:12] [PASSED] ABGR8888 Invalid width
[20:29:12] [PASSED] ABGR8888 Invalid buffer handle
[20:29:12] [PASSED] No pixel format
[20:29:12] [PASSED] ABGR8888 Width 0
[20:29:12] [PASSED] ABGR8888 Height 0
[20:29:12] [PASSED] ABGR8888 Out of bound height * pitch combination
[20:29:12] [PASSED] ABGR8888 Large buffer offset
[20:29:12] [PASSED] ABGR8888 Buffer offset for inexistent plane
[20:29:12] [PASSED] ABGR8888 Invalid flag
[20:29:12] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[20:29:12] [PASSED] ABGR8888 Valid buffer modifier
[20:29:12] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[20:29:12] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] NV12 Normal sizes
[20:29:12] [PASSED] NV12 Max sizes
[20:29:12] [PASSED] NV12 Invalid pitch
[20:29:12] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[20:29:12] [PASSED] NV12 different  modifier per-plane
[20:29:12] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[20:29:12] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] NV12 Modifier for inexistent plane
[20:29:12] [PASSED] NV12 Handle for inexistent plane
[20:29:12] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[20:29:12] [PASSED] YVU420 Normal sizes
[20:29:12] [PASSED] YVU420 Max sizes
[20:29:12] [PASSED] YVU420 Invalid pitch
[20:29:12] [PASSED] YVU420 Different pitches
[20:29:12] [PASSED] YVU420 Different buffer offsets/pitches
[20:29:12] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[20:29:12] [PASSED] YVU420 Valid modifier
[20:29:12] [PASSED] YVU420 Different modifiers per plane
[20:29:12] [PASSED] YVU420 Modifier for inexistent plane
[20:29:12] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[20:29:12] [PASSED] X0L2 Normal sizes
[20:29:12] [PASSED] X0L2 Max sizes
[20:29:12] [PASSED] X0L2 Invalid pitch
[20:29:12] [PASSED] X0L2 Pitch greater than minimum required
[20:29:12] [PASSED] X0L2 Handle for inexistent plane
[20:29:12] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[20:29:12] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[20:29:12] [PASSED] X0L2 Valid modifier
[20:29:12] [PASSED] X0L2 Modifier for inexistent plane
[20:29:12] =========== [PASSED] drm_test_framebuffer_create ===========
[20:29:12] [PASSED] drm_test_framebuffer_free
[20:29:12] [PASSED] drm_test_framebuffer_init
[20:29:12] [PASSED] drm_test_framebuffer_init_bad_format
[20:29:12] [PASSED] drm_test_framebuffer_init_dev_mismatch
[20:29:12] [PASSED] drm_test_framebuffer_lookup
[20:29:12] [PASSED] drm_test_framebuffer_lookup_inexistent
[20:29:12] [PASSED] drm_test_framebuffer_modifiers_not_supported
[20:29:12] ================= [PASSED] drm_framebuffer =================
[20:29:12] ================ drm_gem_shmem (8 subtests) ================
[20:29:12] [PASSED] drm_gem_shmem_test_obj_create
[20:29:12] [PASSED] drm_gem_shmem_test_obj_create_private
[20:29:12] [PASSED] drm_gem_shmem_test_pin_pages
[20:29:12] [PASSED] drm_gem_shmem_test_vmap
[20:29:12] [PASSED] drm_gem_shmem_test_get_sg_table
[20:29:12] [PASSED] drm_gem_shmem_test_get_pages_sgt
[20:29:12] [PASSED] drm_gem_shmem_test_madvise
[20:29:12] [PASSED] drm_gem_shmem_test_purge
[20:29:12] ================== [PASSED] drm_gem_shmem ==================
[20:29:12] === drm_atomic_helper_connector_hdmi_check (27 subtests) ===
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[20:29:12] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420  =======
[20:29:12] [PASSED] Automatic
[20:29:12] [PASSED] Full
[20:29:12] [PASSED] Limited 16:235
[20:29:12] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[20:29:12] [PASSED] drm_test_check_disable_connector
[20:29:12] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[20:29:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[20:29:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[20:29:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[20:29:12] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[20:29:12] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[20:29:12] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[20:29:12] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[20:29:12] [PASSED] drm_test_check_output_bpc_dvi
[20:29:12] [PASSED] drm_test_check_output_bpc_format_vic_1
[20:29:12] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[20:29:12] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[20:29:12] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[20:29:12] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[20:29:12] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[20:29:12] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[20:29:12] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[20:29:12] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[20:29:12] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[20:29:12] [PASSED] drm_test_check_broadcast_rgb_value
[20:29:12] [PASSED] drm_test_check_bpc_8_value
[20:29:12] [PASSED] drm_test_check_bpc_10_value
[20:29:12] [PASSED] drm_test_check_bpc_12_value
[20:29:12] [PASSED] drm_test_check_format_value
[20:29:12] [PASSED] drm_test_check_tmds_char_value
[20:29:12] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[20:29:12] = drm_atomic_helper_connector_hdmi_mode_valid (4 subtests) =
[20:29:12] [PASSED] drm_test_check_mode_valid
[20:29:12] [PASSED] drm_test_check_mode_valid_reject
[20:29:12] [PASSED] drm_test_check_mode_valid_reject_rate
[20:29:12] [PASSED] drm_test_check_mode_valid_reject_max_clock
[20:29:12] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[20:29:12] = drm_atomic_helper_connector_hdmi_infoframes (5 subtests) =
[20:29:12] [PASSED] drm_test_check_infoframes
[20:29:12] [PASSED] drm_test_check_reject_avi_infoframe
[20:29:12] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_8
[20:29:12] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_10
[20:29:12] [PASSED] drm_test_check_reject_audio_infoframe
[20:29:12] === [PASSED] drm_atomic_helper_connector_hdmi_infoframes ===
[20:29:12] ================= drm_managed (2 subtests) =================
[20:29:12] [PASSED] drm_test_managed_release_action
[20:29:12] [PASSED] drm_test_managed_run_action
[20:29:12] =================== [PASSED] drm_managed ===================
[20:29:12] =================== drm_mm (6 subtests) ====================
[20:29:12] [PASSED] drm_test_mm_init
[20:29:12] [PASSED] drm_test_mm_debug
[20:29:12] [PASSED] drm_test_mm_align32
[20:29:12] [PASSED] drm_test_mm_align64
[20:29:12] [PASSED] drm_test_mm_lowest
[20:29:12] [PASSED] drm_test_mm_highest
[20:29:12] ===================== [PASSED] drm_mm ======================
[20:29:12] ============= drm_modes_analog_tv (5 subtests) =============
[20:29:12] [PASSED] drm_test_modes_analog_tv_mono_576i
[20:29:12] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[20:29:12] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[20:29:12] [PASSED] drm_test_modes_analog_tv_pal_576i
[20:29:12] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[20:29:12] =============== [PASSED] drm_modes_analog_tv ===============
[20:29:12] ============== drm_plane_helper (2 subtests) ===============
[20:29:12] =============== drm_test_check_plane_state  ================
[20:29:12] [PASSED] clipping_simple
[20:29:12] [PASSED] clipping_rotate_reflect
[20:29:12] [PASSED] positioning_simple
[20:29:12] [PASSED] upscaling
[20:29:12] [PASSED] downscaling
[20:29:12] [PASSED] rounding1
[20:29:12] [PASSED] rounding2
[20:29:12] [PASSED] rounding3
[20:29:12] [PASSED] rounding4
[20:29:12] =========== [PASSED] drm_test_check_plane_state ============
[20:29:12] =========== drm_test_check_invalid_plane_state  ============
[20:29:12] [PASSED] positioning_invalid
[20:29:12] [PASSED] upscaling_invalid
[20:29:12] [PASSED] downscaling_invalid
[20:29:12] ======= [PASSED] drm_test_check_invalid_plane_state ========
[20:29:12] ================ [PASSED] drm_plane_helper =================
[20:29:12] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[20:29:12] ====== drm_test_connector_helper_tv_get_modes_check  =======
[20:29:12] [PASSED] None
[20:29:12] [PASSED] PAL
[20:29:12] [PASSED] NTSC
[20:29:12] [PASSED] Both, NTSC Default
[20:29:12] [PASSED] Both, PAL Default
[20:29:12] [PASSED] Both, NTSC Default, with PAL on command-line
[20:29:12] [PASSED] Both, PAL Default, with NTSC on command-line
[20:29:12] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[20:29:12] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[20:29:12] ================== drm_rect (9 subtests) ===================
[20:29:12] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[20:29:12] [PASSED] drm_test_rect_clip_scaled_not_clipped
[20:29:12] [PASSED] drm_test_rect_clip_scaled_clipped
[20:29:12] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[20:29:12] ================= drm_test_rect_intersect  =================
[20:29:12] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[20:29:12] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[20:29:12] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[20:29:12] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[20:29:12] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[20:29:12] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[20:29:12] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[20:29:12] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[20:29:12] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[20:29:12] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[20:29:12] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[20:29:12] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[20:29:12] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[20:29:12] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[20:29:12] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
stty: 'standard input': Inappropriate ioctl for device
[20:29:12] ============= [PASSED] drm_test_rect_intersect =============
[20:29:12] ================ drm_test_rect_calc_hscale  ================
[20:29:12] [PASSED] normal use
[20:29:12] [PASSED] out of max range
[20:29:12] [PASSED] out of min range
[20:29:12] [PASSED] zero dst
[20:29:12] [PASSED] negative src
[20:29:12] [PASSED] negative dst
[20:29:12] ============ [PASSED] drm_test_rect_calc_hscale ============
[20:29:12] ================ drm_test_rect_calc_vscale  ================
[20:29:12] [PASSED] normal use
[20:29:12] [PASSED] out of max range
[20:29:12] [PASSED] out of min range
[20:29:12] [PASSED] zero dst
[20:29:12] [PASSED] negative src
[20:29:12] [PASSED] negative dst
[20:29:12] ============ [PASSED] drm_test_rect_calc_vscale ============
[20:29:12] ================== drm_test_rect_rotate  ===================
[20:29:12] [PASSED] reflect-x
[20:29:12] [PASSED] reflect-y
[20:29:12] [PASSED] rotate-0
[20:29:12] [PASSED] rotate-90
[20:29:12] [PASSED] rotate-180
[20:29:12] [PASSED] rotate-270
[20:29:12] ============== [PASSED] drm_test_rect_rotate ===============
[20:29:12] ================ drm_test_rect_rotate_inv  =================
[20:29:12] [PASSED] reflect-x
[20:29:12] [PASSED] reflect-y
[20:29:12] [PASSED] rotate-0
[20:29:12] [PASSED] rotate-90
[20:29:12] [PASSED] rotate-180
[20:29:12] [PASSED] rotate-270
[20:29:12] ============ [PASSED] drm_test_rect_rotate_inv =============
[20:29:12] ==================== [PASSED] drm_rect =====================
[20:29:12] ============ drm_sysfb_modeset_test (1 subtest) ============
[20:29:12] ============ drm_test_sysfb_build_fourcc_list  =============
[20:29:12] [PASSED] no native formats
[20:29:12] [PASSED] XRGB8888 as native format
[20:29:12] [PASSED] remove duplicates
[20:29:12] [PASSED] convert alpha formats
[20:29:12] [PASSED] random formats
[20:29:12] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[20:29:12] ============= [PASSED] drm_sysfb_modeset_test ==============
[20:29:12] ================== drm_fixp (2 subtests) ===================
[20:29:12] [PASSED] drm_test_int2fixp
[20:29:12] [PASSED] drm_test_sm2fixp
[20:29:12] ==================== [PASSED] drm_fixp =====================
[20:29:12] ============================================================
[20:29:12] Testing complete. Ran 630 tests: passed: 630
[20:29:12] Elapsed time: 32.406s total, 1.624s configuring, 30.314s building, 0.405s running

+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[20:29:12] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[20:29:14] Building KUnit Kernel ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
Building with:
$ make all compile_commands.json scripts_gdb ARCH=um O=.kunit --jobs=25
[20:29:23] Starting KUnit Kernel (1/1)...
[20:29:23] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[20:29:23] ================= ttm_device (5 subtests) ==================
[20:29:23] [PASSED] ttm_device_init_basic
[20:29:23] [PASSED] ttm_device_init_multiple
[20:29:23] [PASSED] ttm_device_fini_basic
[20:29:23] [PASSED] ttm_device_init_no_vma_man
[20:29:23] ================== ttm_device_init_pools  ==================
[20:29:23] [PASSED] No DMA allocations, no DMA32 required
[20:29:23] [PASSED] DMA allocations, DMA32 required
[20:29:23] [PASSED] No DMA allocations, DMA32 required
[20:29:23] [PASSED] DMA allocations, no DMA32 required
[20:29:23] ============== [PASSED] ttm_device_init_pools ==============
[20:29:23] =================== [PASSED] ttm_device ====================
[20:29:23] ================== ttm_pool (8 subtests) ===================
[20:29:23] ================== ttm_pool_alloc_basic  ===================
[20:29:23] [PASSED] One page
[20:29:23] [PASSED] More than one page
[20:29:23] [PASSED] Above the allocation limit
[20:29:23] [PASSED] One page, with coherent DMA mappings enabled
[20:29:23] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[20:29:23] ============== [PASSED] ttm_pool_alloc_basic ===============
[20:29:23] ============== ttm_pool_alloc_basic_dma_addr  ==============
[20:29:23] [PASSED] One page
[20:29:23] [PASSED] More than one page
[20:29:23] [PASSED] Above the allocation limit
[20:29:23] [PASSED] One page, with coherent DMA mappings enabled
[20:29:23] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[20:29:23] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[20:29:23] [PASSED] ttm_pool_alloc_order_caching_match
[20:29:23] [PASSED] ttm_pool_alloc_caching_mismatch
[20:29:23] [PASSED] ttm_pool_alloc_order_mismatch
[20:29:23] [PASSED] ttm_pool_free_dma_alloc
[20:29:23] [PASSED] ttm_pool_free_no_dma_alloc
[20:29:23] [PASSED] ttm_pool_fini_basic
[20:29:23] ==================== [PASSED] ttm_pool =====================
[20:29:23] ================ ttm_resource (8 subtests) =================
[20:29:23] ================= ttm_resource_init_basic  =================
[20:29:23] [PASSED] Init resource in TTM_PL_SYSTEM
[20:29:23] [PASSED] Init resource in TTM_PL_VRAM
[20:29:23] [PASSED] Init resource in a private placement
[20:29:23] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[20:29:23] ============= [PASSED] ttm_resource_init_basic =============
[20:29:23] [PASSED] ttm_resource_init_pinned
[20:29:23] [PASSED] ttm_resource_fini_basic
[20:29:23] [PASSED] ttm_resource_manager_init_basic
[20:29:23] [PASSED] ttm_resource_manager_usage_basic
[20:29:23] [PASSED] ttm_resource_manager_set_used_basic
[20:29:23] [PASSED] ttm_sys_man_alloc_basic
[20:29:23] [PASSED] ttm_sys_man_free_basic
[20:29:23] ================== [PASSED] ttm_resource ===================
[20:29:23] =================== ttm_tt (15 subtests) ===================
[20:29:23] ==================== ttm_tt_init_basic  ====================
[20:29:23] [PASSED] Page-aligned size
[20:29:23] [PASSED] Extra pages requested
[20:29:23] ================ [PASSED] ttm_tt_init_basic ================
[20:29:23] [PASSED] ttm_tt_init_misaligned
[20:29:23] [PASSED] ttm_tt_fini_basic
[20:29:23] [PASSED] ttm_tt_fini_sg
[20:29:23] [PASSED] ttm_tt_fini_shmem
[20:29:23] [PASSED] ttm_tt_create_basic
[20:29:23] [PASSED] ttm_tt_create_invalid_bo_type
[20:29:23] [PASSED] ttm_tt_create_ttm_exists
[20:29:23] [PASSED] ttm_tt_create_failed
[20:29:23] [PASSED] ttm_tt_destroy_basic
[20:29:23] [PASSED] ttm_tt_populate_null_ttm
[20:29:23] [PASSED] ttm_tt_populate_populated_ttm
[20:29:23] [PASSED] ttm_tt_unpopulate_basic
[20:29:23] [PASSED] ttm_tt_unpopulate_empty_ttm
[20:29:23] [PASSED] ttm_tt_swapin_basic
[20:29:23] ===================== [PASSED] ttm_tt ======================
[20:29:23] =================== ttm_bo (14 subtests) ===================
[20:29:23] =========== ttm_bo_reserve_optimistic_no_ticket  ===========
[20:29:23] [PASSED] Cannot be interrupted and sleeps
[20:29:23] [PASSED] Cannot be interrupted, locks straight away
[20:29:23] [PASSED] Can be interrupted, sleeps
[20:29:23] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[20:29:23] [PASSED] ttm_bo_reserve_locked_no_sleep
[20:29:23] [PASSED] ttm_bo_reserve_no_wait_ticket
[20:29:23] [PASSED] ttm_bo_reserve_double_resv
[20:29:23] [PASSED] ttm_bo_reserve_interrupted
[20:29:23] [PASSED] ttm_bo_reserve_deadlock
[20:29:23] [PASSED] ttm_bo_unreserve_basic
[20:29:23] [PASSED] ttm_bo_unreserve_pinned
[20:29:23] [PASSED] ttm_bo_unreserve_bulk
[20:29:23] [PASSED] ttm_bo_fini_basic
[20:29:23] [PASSED] ttm_bo_fini_shared_resv
[20:29:23] [PASSED] ttm_bo_pin_basic
[20:29:23] [PASSED] ttm_bo_pin_unpin_resource
[20:29:23] [PASSED] ttm_bo_multiple_pin_one_unpin
[20:29:23] ===================== [PASSED] ttm_bo ======================
[20:29:23] ============== ttm_bo_validate (21 subtests) ===============
[20:29:23] ============== ttm_bo_init_reserved_sys_man  ===============
[20:29:23] [PASSED] Buffer object for userspace
[20:29:23] [PASSED] Kernel buffer object
[20:29:23] [PASSED] Shared buffer object
[20:29:23] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[20:29:23] ============== ttm_bo_init_reserved_mock_man  ==============
[20:29:23] [PASSED] Buffer object for userspace
[20:29:23] [PASSED] Kernel buffer object
[20:29:23] [PASSED] Shared buffer object
[20:29:23] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[20:29:23] [PASSED] ttm_bo_init_reserved_resv
[20:29:23] ================== ttm_bo_validate_basic  ==================
[20:29:23] [PASSED] Buffer object for userspace
[20:29:23] [PASSED] Kernel buffer object
[20:29:23] [PASSED] Shared buffer object
[20:29:23] ============== [PASSED] ttm_bo_validate_basic ==============
[20:29:23] [PASSED] ttm_bo_validate_invalid_placement
[20:29:23] ============= ttm_bo_validate_same_placement  ==============
[20:29:23] [PASSED] System manager
[20:29:23] [PASSED] VRAM manager
[20:29:23] ========= [PASSED] ttm_bo_validate_same_placement ==========
[20:29:23] [PASSED] ttm_bo_validate_failed_alloc
[20:29:23] [PASSED] ttm_bo_validate_pinned
[20:29:23] [PASSED] ttm_bo_validate_busy_placement
[20:29:23] ================ ttm_bo_validate_multihop  =================
[20:29:23] [PASSED] Buffer object for userspace
[20:29:23] [PASSED] Kernel buffer object
[20:29:23] [PASSED] Shared buffer object
[20:29:23] ============ [PASSED] ttm_bo_validate_multihop =============
[20:29:23] ========== ttm_bo_validate_no_placement_signaled  ==========
[20:29:23] [PASSED] Buffer object in system domain, no page vector
[20:29:23] [PASSED] Buffer object in system domain with an existing page vector
[20:29:23] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[20:29:23] ======== ttm_bo_validate_no_placement_not_signaled  ========
[20:29:23] [PASSED] Buffer object for userspace
[20:29:23] [PASSED] Kernel buffer object
[20:29:23] [PASSED] Shared buffer object
[20:29:23] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[20:29:23] [PASSED] ttm_bo_validate_move_fence_signaled
[20:29:23] ========= ttm_bo_validate_move_fence_not_signaled  =========
[20:29:23] [PASSED] Waits for GPU
[20:29:23] [PASSED] Tries to lock straight away
[20:29:23] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[20:29:23] [PASSED] ttm_bo_validate_happy_evict
[20:29:23] [PASSED] ttm_bo_validate_all_pinned_evict
[20:29:23] [PASSED] ttm_bo_validate_allowed_only_evict
[20:29:23] [PASSED] ttm_bo_validate_deleted_evict
[20:29:23] [PASSED] ttm_bo_validate_busy_domain_evict
[20:29:23] [PASSED] ttm_bo_validate_evict_gutting
[20:29:23] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[20:29:23] ================= [PASSED] ttm_bo_validate =================
[20:29:23] ============================================================
[20:29:23] Testing complete. Ran 101 tests: passed: 101
[20:29:23] Elapsed time: 11.176s total, 1.643s configuring, 9.266s building, 0.228s running

+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel

✗ Xe.CI.BAT: failure for drm/xe: Privatize struct xe_ggtt. (rev5)

[Patchwork]

[-- Attachment #1: Type: text/plain, Size: 2766 bytes --]

== Series Details ==

Series: drm/xe: Privatize struct xe_ggtt. (rev5)
URL   : https://patchwork.freedesktop.org/series/160006/
State : failure

== Summary ==

CI Bug Log - changes from xe-4465-8059f097e25f736bb3da09af6a9b283079abfd4f_BAT -> xe-pw-160006v5_BAT
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with xe-pw-160006v5_BAT absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in xe-pw-160006v5_BAT, please notify your bug team (I915-ci-infra@lists.freedesktop.org) to allow them
  to document this new failure mode, which will reduce false positives in CI.

  

Participating hosts (11 -> 12)
------------------------------

  Additional (1): bat-bmg-3 

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in xe-pw-160006v5_BAT:

### IGT changes ###

#### Possible regressions ####

  * igt@xe_module_load@load:
    - bat-bmg-1:          [PASS][1] -> [ABORT][2]
   [1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4465-8059f097e25f736bb3da09af6a9b283079abfd4f/bat-bmg-1/igt@xe_module_load@load.html
   [2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160006v5/bat-bmg-1/igt@xe_module_load@load.html

  
Known issues
------------

  Here are the changes found in xe-pw-160006v5_BAT that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@xe_peer2peer@read@read-gpua-vram01-gpub-vram01-p2p:
    - bat-bmg-3:          NOTRUN -> [SKIP][3] ([Intel XE#6566]) +3 other tests skip
   [3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160006v5/bat-bmg-3/igt@xe_peer2peer@read@read-gpua-vram01-gpub-vram01-p2p.html

  
#### Possible fixes ####

  * igt@xe_waitfence@engine:
    - bat-dg2-oem2:       [FAIL][4] ([Intel XE#6519]) -> [PASS][5]
   [4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4465-8059f097e25f736bb3da09af6a9b283079abfd4f/bat-dg2-oem2/igt@xe_waitfence@engine.html
   [5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160006v5/bat-dg2-oem2/igt@xe_waitfence@engine.html

  
  [Intel XE#6519]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6519
  [Intel XE#6566]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6566


Build changes
-------------

  * IGT: IGT_8721 -> IGT_8723
  * Linux: xe-4465-8059f097e25f736bb3da09af6a9b283079abfd4f -> xe-pw-160006v5

  IGT_8721: 3707bb4267de22a18d61b232c4ab5fbaf61db90c @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git
  IGT_8723: 8723
  xe-4465-8059f097e25f736bb3da09af6a9b283079abfd4f: 8059f097e25f736bb3da09af6a9b283079abfd4f
  xe-pw-160006v5: 160006v5

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160006v5/index.html

[-- Attachment #2: Type: text/html, Size: 3407 bytes --]