* [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore
@ 2026-02-10 10:59 Satyanarayana K V P
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
` (5 more replies)
0 siblings, 6 replies; 12+ messages in thread
From: Satyanarayana K V P @ 2026-02-10 10:59 UTC (permalink / raw)
To: intel-xe; +Cc: Satyanarayana K V P
CCS save/restore batch buffers are attached during BO allocation and
detached during BO teardown. The shrinker triggers xe_bo_move(), which is
used for both allocation and deletion paths.
When BO allocation and shrinking occur concurrently, a circular locking
dependency involving fs_reclaim and swap_guard can occur, leading to a
deadlock such as:
======================================================
WARNING: possible circular locking dependency detected
------------------------------------------------------
CPU0 CPU1
---- ----
lock(fs_reclaim);
lock(&sa_manager->swap_guard);
lock(fs_reclaim);
lock(&sa_manager->swap_guard);
*** DEADLOCK ***
=====================================================
To avoid this, the BB pointer and SA are allocated using xe_bb_alloc()
before taking lock and SA is initialized using xe_bb_init() preventing
reclaim from being invoked in this context.
V2 -> V3:
- Created new functions xe_sa_bo_alloc(), xe_sa_bo_release() and
xe_sa_bo_init(). (Thomas)
- Created new functions xe_bb_alloc(), xe_bb_release and
xe_bb_init(). (Thomas)
- Updated guard() to scoped_guard() in xe_migrate_ccs_rw_copy(). (Thomas)
- Removed timeout logic from drm_suballoc_init(). (Thomas & Christian).
V1 -> V2:
- Split drm_suballoc_new() into drm_suballoc_alloc() and
drm_suballoc_init() (Thomas).
- Used drm_suballoc_alloc() and drm_suballoc_init() for BB allocation
(Thomas).
Satyanarayana K V P (3):
drm/sa: Split drm_suballoc_new() into SA alloc and init helpers
drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation
drm/xe/sa: Add lockdep annotations for SA manager swap_guard
drivers/gpu/drm/drm_suballoc.c | 110 ++++++++++++++++++++++++++------
drivers/gpu/drm/xe/xe_bb.c | 49 +++++++++-----
drivers/gpu/drm/xe/xe_bb.h | 7 +-
drivers/gpu/drm/xe/xe_migrate.c | 96 +++++++++++++++-------------
drivers/gpu/drm/xe/xe_sa.c | 46 +++++++++++++
drivers/gpu/drm/xe/xe_sa.h | 3 +
include/drm/drm_suballoc.h | 8 +++
7 files changed, 238 insertions(+), 81 deletions(-)
--
2.43.0
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
@ 2026-02-10 10:59 ` Satyanarayana K V P
2026-02-10 12:09 ` Thomas Hellström
2026-02-11 0:20 ` Matthew Brost
2026-02-10 10:59 ` [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation Satyanarayana K V P
` (4 subsequent siblings)
5 siblings, 2 replies; 12+ messages in thread
From: Satyanarayana K V P @ 2026-02-10 10:59 UTC (permalink / raw)
To: intel-xe
Cc: Satyanarayana K V P, Matthew Brost, Thomas Hellström,
Michal Wajdeczko, Matthew Auld, Christian König, dri-devel
drm_suballoc_new() currently both allocates the SA object using kmalloc()
and searches for a suitable hole in the sub-allocator for the requested
size. If SA allocation is done by holding sub-allocator mutex, this design
can lead to reclaim safety issues.
By splitting the kmalloc() step outside of the critical section, we allow
the memory allocation to use GFP_KERNEL (reclaim-safe) while ensuring that
the initialization step that holds reclaim-tainted locks (sub-allocator
mutex) operates in a reclaim-unsafe context with pre-allocated memory.
This separation prevents potential deadlocks where memory reclaim could
attempt to acquire locks that are already held during the sub-allocator
operations.
Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
Suggested-by: Matthew Brost <matthew.brost@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Christian König <christian.koenig@amd.com>
Cc: dri-devel@lists.freedesktop.org
---
V2 -> V3:
- Updated commit message (Matt, Thomas & Christian).
- Removed timeout logic from drm_suballoc_init(). (Thomas & Christian).
V1 -> V2:
- Splitted drm_suballoc_new() into drm_suballoc_alloc() and
drm_suballoc_init() (Thomas).
---
drivers/gpu/drm/drm_suballoc.c | 110 ++++++++++++++++++++++++++-------
include/drm/drm_suballoc.h | 8 +++
2 files changed, 97 insertions(+), 21 deletions(-)
diff --git a/drivers/gpu/drm/drm_suballoc.c b/drivers/gpu/drm/drm_suballoc.c
index 879ea33dbbc4..b97ffcd98d45 100644
--- a/drivers/gpu/drm/drm_suballoc.c
+++ b/drivers/gpu/drm/drm_suballoc.c
@@ -123,7 +123,7 @@ static void drm_suballoc_remove_locked(struct drm_suballoc *sa)
list_del_init(&sa->olist);
list_del_init(&sa->flist);
dma_fence_put(sa->fence);
- kfree(sa);
+ drm_suballoc_release(sa);
}
static void drm_suballoc_try_free(struct drm_suballoc_manager *sa_manager)
@@ -293,45 +293,74 @@ static bool drm_suballoc_next_hole(struct drm_suballoc_manager *sa_manager,
}
/**
- * drm_suballoc_new() - Make a suballocation.
+ * drm_suballoc_alloc() - Allocate uninitialized suballoc object.
+ * @gfp: gfp flags used for memory allocation.
+ *
+ * Allocate memory for an uninitialized suballoc object. Intended usage is
+ * allocate memory for suballoc object outside of a reclaim tainted context
+ * and then be initialized at a later time in a reclaim tainted context.
+ *
+ * @drm_suballoc_release should be used to release the memory if returned
+ * suballoc object is in uninitialized state.
+ *
+ * Return: a new uninitialized suballoc object, or an ERR_PTR(-ENOMEM).
+ */
+struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp)
+{
+ struct drm_suballoc *sa;
+
+ sa = kmalloc(sizeof(*sa), gfp);
+ if (!sa)
+ return ERR_PTR(-ENOMEM);
+
+ return sa;
+}
+EXPORT_SYMBOL(drm_suballoc_alloc);
+
+/**
+ * drm_suballoc_release() - Release memory for suballocation.
+ * @sa: The struct drm_suballoc.
+ */
+void drm_suballoc_release(struct drm_suballoc *sa)
+{
+ kfree(sa);
+}
+EXPORT_SYMBOL(drm_suballoc_release);
+
+/**
+ * drm_suballoc_init() - Initialize a suballocation.
* @sa_manager: pointer to the sa_manager
+ * @sa: The struct drm_suballoc.
* @size: number of bytes we want to suballocate.
- * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL but
- * the argument is provided for suballocations from reclaim context or
- * where the caller wants to avoid pipelining rather than wait for
- * reclaim.
* @intr: Whether to perform waits interruptible. This should typically
* always be true, unless the caller needs to propagate a
* non-interruptible context from above layers.
* @align: Alignment. Must not exceed the default manager alignment.
* If @align is zero, then the manager alignment is used.
*
- * Try to make a suballocation of size @size, which will be rounded
- * up to the alignment specified in specified in drm_suballoc_manager_init().
+ * Try to make a suballocation on a pre-allocated suballoc object of size @size,
+ * which will be rounded up to the alignment specified in specified in
+ * drm_suballoc_manager_init().
*
- * Return: a new suballocated bo, or an ERR_PTR.
+ * Return: zero on success, errno on failure.
*/
-struct drm_suballoc *
-drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
- gfp_t gfp, bool intr, size_t align)
+int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
+ struct drm_suballoc *sa, size_t size,
+ bool intr, size_t align)
{
struct dma_fence *fences[DRM_SUBALLOC_MAX_QUEUES];
unsigned int tries[DRM_SUBALLOC_MAX_QUEUES];
unsigned int count;
int i, r;
- struct drm_suballoc *sa;
if (WARN_ON_ONCE(align > sa_manager->align))
- return ERR_PTR(-EINVAL);
+ return -EINVAL;
if (WARN_ON_ONCE(size > sa_manager->size || !size))
- return ERR_PTR(-EINVAL);
+ return -EINVAL;
if (!align)
align = sa_manager->align;
- sa = kmalloc(sizeof(*sa), gfp);
- if (!sa)
- return ERR_PTR(-ENOMEM);
sa->manager = sa_manager;
sa->fence = NULL;
INIT_LIST_HEAD(&sa->olist);
@@ -348,7 +377,7 @@ drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
if (drm_suballoc_try_alloc(sa_manager, sa,
size, align)) {
spin_unlock(&sa_manager->wq.lock);
- return sa;
+ return 0;
}
/* see if we can skip over some allocations */
@@ -385,8 +414,47 @@ drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
} while (!r);
spin_unlock(&sa_manager->wq.lock);
- kfree(sa);
- return ERR_PTR(r);
+ return r;
+}
+EXPORT_SYMBOL(drm_suballoc_init);
+
+/**
+ * drm_suballoc_new() - Make a suballocation.
+ * @sa_manager: pointer to the sa_manager
+ * @size: number of bytes we want to suballocate.
+ * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL but
+ * the argument is provided for suballocations from reclaim context or
+ * where the caller wants to avoid pipelining rather than wait for
+ * reclaim.
+ * @intr: Whether to perform waits interruptible. This should typically
+ * always be true, unless the caller needs to propagate a
+ * non-interruptible context from above layers.
+ * @align: Alignment. Must not exceed the default manager alignment.
+ * If @align is zero, then the manager alignment is used.
+ *
+ * Try to make a suballocation of size @size, which will be rounded
+ * up to the alignment specified in specified in drm_suballoc_manager_init().
+ *
+ * Return: a new suballocated bo, or an ERR_PTR.
+ */
+struct drm_suballoc *
+drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
+ gfp_t gfp, bool intr, size_t align)
+{
+ struct drm_suballoc *sa;
+ int err;
+
+ sa = drm_suballoc_alloc(gfp);
+ if (IS_ERR(sa))
+ return sa;
+
+ err = drm_suballoc_init(sa_manager, sa, size, intr, align);
+ if (err) {
+ drm_suballoc_release(sa);
+ return ERR_PTR(err);
+ }
+
+ return sa;
}
EXPORT_SYMBOL(drm_suballoc_new);
diff --git a/include/drm/drm_suballoc.h b/include/drm/drm_suballoc.h
index 7ba72a81a808..b8d1d5449fd8 100644
--- a/include/drm/drm_suballoc.h
+++ b/include/drm/drm_suballoc.h
@@ -53,6 +53,14 @@ void drm_suballoc_manager_init(struct drm_suballoc_manager *sa_manager,
void drm_suballoc_manager_fini(struct drm_suballoc_manager *sa_manager);
+struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp);
+
+void drm_suballoc_release(struct drm_suballoc *sa);
+
+int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
+ struct drm_suballoc *sa, size_t size, bool intr,
+ size_t align);
+
struct drm_suballoc *
drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
gfp_t gfp, bool intr, size_t align);
--
2.43.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
@ 2026-02-10 10:59 ` Satyanarayana K V P
2026-02-10 13:02 ` Michal Wajdeczko
2026-02-10 10:59 ` [PATCH v3 3/3] drm/xe/sa: Add lockdep annotations for SA manager swap_guard Satyanarayana K V P
` (3 subsequent siblings)
5 siblings, 1 reply; 12+ messages in thread
From: Satyanarayana K V P @ 2026-02-10 10:59 UTC (permalink / raw)
To: intel-xe
Cc: Satyanarayana K V P, Matthew Brost, Michal Wajdeczko,
Matthew Auld, Thomas Hellström
CCS save/restore batch buffers are attached during BO allocation and
detached during BO teardown. The shrinker triggers xe_bo_move(), which is
used for both allocation and deletion paths.
When BO allocation and shrinking occur concurrently, a circular locking
dependency involving fs_reclaim and swap_guard can occur, leading to a
deadlock such as:
======================================================
WARNING: possible circular locking dependency detected
------------------------------------------------------
CPU0 CPU1
---- ----
lock(fs_reclaim);
lock(&sa_manager->swap_guard);
lock(fs_reclaim);
lock(&sa_manager->swap_guard);
*** DEADLOCK ***
=====================================================
To avoid this, the BB pointer and SA are allocated using xe_bb_alloc()
before taking lock and SA is initialized using xe_bb_init() preventing
reclaim from being invoked in this context.
Fixes: 864690cf4dd62 ("drm/xe/vf: Attach and detach CCS copy commands with BO")
Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
---
V2 -> V3:
- Created new functions xe_sa_bo_alloc(), xe_sa_bo_release() and
xe_sa_bo_init(). (Thomas)
- Created new functions xe_bb_alloc(), xe_bb_release and
xe_bb_init(). (Thomas)
- Updated guard() to scoped_guard() in xe_migrate_ccs_rw_copy(). (Thomas)
V1 -> V2:
- Used drm_suballoc_alloc() and drm_suballoc_init() for BB allocation
(Thomas).
---
drivers/gpu/drm/xe/xe_bb.c | 49 +++++++++++------
drivers/gpu/drm/xe/xe_bb.h | 7 ++-
drivers/gpu/drm/xe/xe_migrate.c | 96 ++++++++++++++++++---------------
drivers/gpu/drm/xe/xe_sa.c | 40 ++++++++++++++
drivers/gpu/drm/xe/xe_sa.h | 3 ++
5 files changed, 135 insertions(+), 60 deletions(-)
diff --git a/drivers/gpu/drm/xe/xe_bb.c b/drivers/gpu/drm/xe/xe_bb.c
index 8b678297aaa2..631ae564e719 100644
--- a/drivers/gpu/drm/xe/xe_bb.c
+++ b/drivers/gpu/drm/xe/xe_bb.c
@@ -59,16 +59,43 @@ struct xe_bb *xe_bb_new(struct xe_gt *gt, u32 dwords, bool usm)
return ERR_PTR(err);
}
-struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
- enum xe_sriov_vf_ccs_rw_ctxs ctx_id)
+struct xe_bb *xe_bb_alloc(struct xe_gt *gt)
{
struct xe_bb *bb = kmalloc(sizeof(*bb), GFP_KERNEL);
struct xe_device *xe = gt_to_xe(gt);
- struct xe_sa_manager *bb_pool;
int err;
if (!bb)
return ERR_PTR(-ENOMEM);
+
+ bb->bo = xe_sa_bo_alloc(GFP_KERNEL);
+ if (IS_ERR(bb->bo)) {
+ drm_err(&xe->drm, "Sub-allocator memory allocation failed with %ld\n",
+ PTR_ERR(bb->bo));
+ err = PTR_ERR(bb->bo);
+ goto err;
+ }
+
+ return bb;
+
+err:
+ kfree(bb);
+ return ERR_PTR(err);
+}
+
+void xe_bb_release(struct xe_bb *bb)
+{
+ if (bb->bo)
+ xe_sa_bo_release(bb->bo);
+
+ kfree(bb);
+}
+
+int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
+ struct xe_sa_manager *bb_pool, u32 dwords)
+{
+ int err;
+
/*
* We need to allocate space for the requested number of dwords &
* one additional MI_BATCH_BUFFER_END dword. Since the whole SA
@@ -76,22 +103,14 @@ struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
* is not over written when the last chunk of SA is allocated for BB.
* So, this extra DW acts as a guard here.
*/
-
- bb_pool = xe->sriov.vf.ccs.contexts[ctx_id].mem.ccs_bb_pool;
- bb->bo = xe_sa_bo_new(bb_pool, 4 * (dwords + 1));
-
- if (IS_ERR(bb->bo)) {
- err = PTR_ERR(bb->bo);
- goto err;
- }
+ err = xe_sa_bo_init(bb_pool, bb->bo, 4 * (dwords + 1));
+ if (err)
+ return err;
bb->cs = xe_sa_bo_cpu_addr(bb->bo);
bb->len = 0;
- return bb;
-err:
- kfree(bb);
- return ERR_PTR(err);
+ return 0;
}
static struct xe_sched_job *
diff --git a/drivers/gpu/drm/xe/xe_bb.h b/drivers/gpu/drm/xe/xe_bb.h
index 2a8adc9a6dee..3eb80925bfd1 100644
--- a/drivers/gpu/drm/xe/xe_bb.h
+++ b/drivers/gpu/drm/xe/xe_bb.h
@@ -13,11 +13,14 @@ struct dma_fence;
struct xe_gt;
struct xe_exec_queue;
struct xe_sched_job;
+struct xe_sa_manager;
enum xe_sriov_vf_ccs_rw_ctxs;
struct xe_bb *xe_bb_new(struct xe_gt *gt, u32 dwords, bool usm);
-struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
- enum xe_sriov_vf_ccs_rw_ctxs ctx_id);
+struct xe_bb *xe_bb_alloc(struct xe_gt *gt);
+void xe_bb_release(struct xe_bb *bb);
+int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
+ struct xe_sa_manager *bb_pool, u32 dwords);
struct xe_sched_job *xe_bb_create_job(struct xe_exec_queue *q,
struct xe_bb *bb);
struct xe_sched_job *xe_bb_create_migration_job(struct xe_exec_queue *q,
diff --git a/drivers/gpu/drm/xe/xe_migrate.c b/drivers/gpu/drm/xe/xe_migrate.c
index 078a9bc2821d..c858eaa70e3e 100644
--- a/drivers/gpu/drm/xe/xe_migrate.c
+++ b/drivers/gpu/drm/xe/xe_migrate.c
@@ -1148,65 +1148,75 @@ int xe_migrate_ccs_rw_copy(struct xe_tile *tile, struct xe_exec_queue *q,
size -= src_L0;
}
- bb_pool = ctx->mem.ccs_bb_pool;
- guard(mutex) (xe_sa_bo_swap_guard(bb_pool));
- xe_sa_bo_swap_shadow(bb_pool);
-
- bb = xe_bb_ccs_new(gt, batch_size, read_write);
+ bb = xe_bb_alloc(gt);
if (IS_ERR(bb)) {
- drm_err(&xe->drm, "BB allocation failed.\n");
err = PTR_ERR(bb);
return err;
}
- batch_size_allocated = batch_size;
- size = xe_bo_size(src_bo);
- batch_size = 0;
+ bb_pool = ctx->mem.ccs_bb_pool;
+ scoped_guard(mutex, xe_sa_bo_swap_guard(bb_pool)) {
+ xe_sa_bo_swap_shadow(bb_pool);
+
+ err = xe_bb_init(gt, bb, bb_pool, batch_size);
+ if (err) {
+ drm_err(&xe->drm, "BB allocation failed.\n");
+ xe_bb_release(bb);
+ return err;
+ }
- /*
- * Emit PTE and copy commands here.
- * The CCS copy command can only support limited size. If the size to be
- * copied is more than the limit, divide copy into chunks. So, calculate
- * sizes here again before copy command is emitted.
- */
- while (size) {
- batch_size += 10; /* Flush + ggtt addr + 2 NOP */
- u32 flush_flags = 0;
- u64 ccs_ofs, ccs_size;
- u32 ccs_pt;
+ batch_size_allocated = batch_size;
+ size = xe_bo_size(src_bo);
+ batch_size = 0;
- u32 avail_pts = max_mem_transfer_per_pass(xe) / LEVEL0_PAGE_TABLE_ENCODE_SIZE;
+ /*
+ * Emit PTE and copy commands here.
+ * The CCS copy command can only support limited size. If the size to be
+ * copied is more than the limit, divide copy into chunks. So, calculate
+ * sizes here again before copy command is emitted.
+ */
- src_L0 = xe_migrate_res_sizes(m, &src_it);
+ while (size) {
+ batch_size += 10; /* Flush + ggtt addr + 2 NOP */
+ u32 flush_flags = 0;
+ u64 ccs_ofs, ccs_size;
+ u32 ccs_pt;
- batch_size += pte_update_size(m, false, src, &src_it, &src_L0,
- &src_L0_ofs, &src_L0_pt, 0, 0,
- avail_pts);
+ u32 avail_pts = max_mem_transfer_per_pass(xe) /
+ LEVEL0_PAGE_TABLE_ENCODE_SIZE;
- ccs_size = xe_device_ccs_bytes(xe, src_L0);
- batch_size += pte_update_size(m, 0, NULL, &ccs_it, &ccs_size, &ccs_ofs,
- &ccs_pt, 0, avail_pts, avail_pts);
- xe_assert(xe, IS_ALIGNED(ccs_it.start, PAGE_SIZE));
- batch_size += EMIT_COPY_CCS_DW;
+ src_L0 = xe_migrate_res_sizes(m, &src_it);
+
+ batch_size += pte_update_size(m, false, src, &src_it, &src_L0,
+ &src_L0_ofs, &src_L0_pt, 0, 0,
+ avail_pts);
+
+ ccs_size = xe_device_ccs_bytes(xe, src_L0);
+ batch_size += pte_update_size(m, 0, NULL, &ccs_it, &ccs_size, &ccs_ofs,
+ &ccs_pt, 0, avail_pts, avail_pts);
+ xe_assert(xe, IS_ALIGNED(ccs_it.start, PAGE_SIZE));
+ batch_size += EMIT_COPY_CCS_DW;
- emit_pte(m, bb, src_L0_pt, false, true, &src_it, src_L0, src);
+ emit_pte(m, bb, src_L0_pt, false, true, &src_it, src_L0, src);
- emit_pte(m, bb, ccs_pt, false, false, &ccs_it, ccs_size, src);
+ emit_pte(m, bb, ccs_pt, false, false, &ccs_it, ccs_size, src);
- bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
- flush_flags = xe_migrate_ccs_copy(m, bb, src_L0_ofs, src_is_pltt,
- src_L0_ofs, dst_is_pltt,
- src_L0, ccs_ofs, true);
- bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
+ bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
+ flush_flags = xe_migrate_ccs_copy(m, bb, src_L0_ofs, src_is_pltt,
+ src_L0_ofs, dst_is_pltt,
+ src_L0, ccs_ofs, true);
+ bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
- size -= src_L0;
- }
+ size -= src_L0;
+ }
- xe_assert(xe, (batch_size_allocated == bb->len));
- src_bo->bb_ccs[read_write] = bb;
+ xe_assert(xe, (batch_size_allocated == bb->len));
+ src_bo->bb_ccs[read_write] = bb;
+
+ xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
+ xe_sa_bo_sync_shadow(bb->bo);
+ }
- xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
- xe_sa_bo_sync_shadow(bb->bo);
return 0;
}
diff --git a/drivers/gpu/drm/xe/xe_sa.c b/drivers/gpu/drm/xe/xe_sa.c
index b738102575d4..59d0187b3e82 100644
--- a/drivers/gpu/drm/xe/xe_sa.c
+++ b/drivers/gpu/drm/xe/xe_sa.c
@@ -175,6 +175,46 @@ struct drm_suballoc *__xe_sa_bo_new(struct xe_sa_manager *sa_manager, u32 size,
return drm_suballoc_new(&sa_manager->base, size, gfp, true, 0);
}
+/**
+ * xe_sa_bo_alloc - Allocate uninitialized suballoc object.
+ * @gfp: gfp flags used for memory allocation.
+ *
+ * Allocate memory for an uninitialized suballoc object. Intended usage is
+ * allocate memory for suballoc object outside of a reclaim tainted context
+ * and then be initialized at a later time in a reclaim tainted context.
+ *
+ * Return: a new uninitialized suballoc object, or an ERR_PTR(-ENOMEM).
+ */
+
+struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp)
+{
+ return drm_suballoc_alloc(gfp);
+}
+
+/**
+ * xe_sa_bo_release - Release memory for suballocation.
+ * @sa: The struct drm_suballoc.
+ */
+void xe_sa_bo_release(struct drm_suballoc *sa)
+{
+ drm_suballoc_release(sa);
+}
+
+/**
+ * xe_sa_bo_init - Initialize a suballocation.
+ * @sa_manager: pointer to the sa_manager
+ * @sa: The struct drm_suballoc.
+ * @size: number of bytes we want to suballocate.
+ *
+ * Try to make a suballocation on a pre-allocated suballoc object of size @size.
+ *
+ * Return: zero on success, errno on failure.
+ */
+int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct drm_suballoc *sa, u32 size)
+{
+ return drm_suballoc_init(&sa_manager->base, sa, size, true, 0);
+}
+
/**
* xe_sa_bo_flush_write() - Copy the data from the sub-allocation to the GPU memory.
* @sa_bo: the &drm_suballoc to flush
diff --git a/drivers/gpu/drm/xe/xe_sa.h b/drivers/gpu/drm/xe/xe_sa.h
index 05e9a4e00e78..19d4b698a7d7 100644
--- a/drivers/gpu/drm/xe/xe_sa.h
+++ b/drivers/gpu/drm/xe/xe_sa.h
@@ -38,6 +38,9 @@ static inline struct drm_suballoc *xe_sa_bo_new(struct xe_sa_manager *sa_manager
return __xe_sa_bo_new(sa_manager, size, GFP_KERNEL);
}
+struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp);
+void xe_sa_bo_release(struct drm_suballoc *sa);
+int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct drm_suballoc *sa, u32 size);
void xe_sa_bo_flush_write(struct drm_suballoc *sa_bo);
void xe_sa_bo_sync_read(struct drm_suballoc *sa_bo);
void xe_sa_bo_free(struct drm_suballoc *sa_bo, struct dma_fence *fence);
--
2.43.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH v3 3/3] drm/xe/sa: Add lockdep annotations for SA manager swap_guard
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
2026-02-10 10:59 ` [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation Satyanarayana K V P
@ 2026-02-10 10:59 ` Satyanarayana K V P
2026-02-10 11:06 ` ✓ CI.KUnit: success for Fix fs_reclaim deadlock caused by CCS save/restore (rev3) Patchwork
` (2 subsequent siblings)
5 siblings, 0 replies; 12+ messages in thread
From: Satyanarayana K V P @ 2026-02-10 10:59 UTC (permalink / raw)
To: intel-xe
Cc: Satyanarayana K V P, Matthew Brost, Michal Wajdeczko,
Matthew Auld, Thomas Hellström
Annotate the SA manager init path to model taking swap_guard while under
reclaim context. This helps lockdep catch potential circular dependencies
between fs_reclaim and swap_guard in debug builds. Without this annotation,
lockdep is unware of this chain until the shrinker runs.
Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
Suggested-by: Matthew Brost <matthew.brost@intel.com>
Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Matthew Auld <matthew.auld@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
---
V2 -> V3:
- Updated commit message (Matt).
V1 -> V2:
- None.
---
drivers/gpu/drm/xe/xe_sa.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/drivers/gpu/drm/xe/xe_sa.c b/drivers/gpu/drm/xe/xe_sa.c
index 59d0187b3e82..794f46272ab5 100644
--- a/drivers/gpu/drm/xe/xe_sa.c
+++ b/drivers/gpu/drm/xe/xe_sa.c
@@ -89,6 +89,12 @@ struct xe_sa_manager *__xe_sa_bo_manager_init(struct xe_tile *tile, u32 size,
if (ret)
return ERR_PTR(ret);
+ if (IS_ENABLED(CONFIG_PROVE_LOCKING)) {
+ fs_reclaim_acquire(GFP_KERNEL);
+ might_lock(&sa_manager->swap_guard);
+ fs_reclaim_release(GFP_KERNEL);
+ }
+
shadow = xe_managed_bo_create_pin_map(xe, tile, size,
XE_BO_FLAG_VRAM_IF_DGFX(tile) |
XE_BO_FLAG_GGTT |
--
2.43.0
^ permalink raw reply related [flat|nested] 12+ messages in thread
* ✓ CI.KUnit: success for Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
` (2 preceding siblings ...)
2026-02-10 10:59 ` [PATCH v3 3/3] drm/xe/sa: Add lockdep annotations for SA manager swap_guard Satyanarayana K V P
@ 2026-02-10 11:06 ` Patchwork
2026-02-10 11:59 ` ✓ Xe.CI.BAT: " Patchwork
2026-02-10 14:29 ` ✗ Xe.CI.FULL: failure " Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2026-02-10 11:06 UTC (permalink / raw)
To: Satyanarayana K V P; +Cc: intel-xe
== Series Details ==
Series: Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
URL : https://patchwork.freedesktop.org/series/160841/
State : success
== Summary ==
+ trap cleanup EXIT
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/xe/.kunitconfig
[11:04:48] Configuring KUnit Kernel ...
Generating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:04:52] 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=48
[11:05:24] Starting KUnit Kernel (1/1)...
[11:05:24] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:05:24] ================== guc_buf (11 subtests) ===================
[11:05:24] [PASSED] test_smallest
[11:05:24] [PASSED] test_largest
[11:05:24] [PASSED] test_granular
[11:05:24] [PASSED] test_unique
[11:05:24] [PASSED] test_overlap
[11:05:24] [PASSED] test_reusable
[11:05:24] [PASSED] test_too_big
[11:05:24] [PASSED] test_flush
[11:05:24] [PASSED] test_lookup
[11:05:24] [PASSED] test_data
[11:05:24] [PASSED] test_class
[11:05:24] ===================== [PASSED] guc_buf =====================
[11:05:24] =================== guc_dbm (7 subtests) ===================
[11:05:24] [PASSED] test_empty
[11:05:24] [PASSED] test_default
[11:05:24] ======================== test_size ========================
[11:05:24] [PASSED] 4
[11:05:24] [PASSED] 8
[11:05:24] [PASSED] 32
[11:05:24] [PASSED] 256
[11:05:24] ==================== [PASSED] test_size ====================
[11:05:24] ======================= test_reuse ========================
[11:05:24] [PASSED] 4
[11:05:24] [PASSED] 8
[11:05:24] [PASSED] 32
[11:05:24] [PASSED] 256
[11:05:24] =================== [PASSED] test_reuse ====================
[11:05:24] =================== test_range_overlap ====================
[11:05:24] [PASSED] 4
[11:05:24] [PASSED] 8
[11:05:24] [PASSED] 32
[11:05:24] [PASSED] 256
[11:05:24] =============== [PASSED] test_range_overlap ================
[11:05:24] =================== test_range_compact ====================
[11:05:24] [PASSED] 4
[11:05:24] [PASSED] 8
[11:05:24] [PASSED] 32
[11:05:24] [PASSED] 256
[11:05:24] =============== [PASSED] test_range_compact ================
[11:05:24] ==================== test_range_spare =====================
[11:05:24] [PASSED] 4
[11:05:24] [PASSED] 8
[11:05:24] [PASSED] 32
[11:05:24] [PASSED] 256
[11:05:24] ================ [PASSED] test_range_spare =================
[11:05:24] ===================== [PASSED] guc_dbm =====================
[11:05:24] =================== guc_idm (6 subtests) ===================
[11:05:24] [PASSED] bad_init
[11:05:24] [PASSED] no_init
[11:05:24] [PASSED] init_fini
[11:05:24] [PASSED] check_used
[11:05:24] [PASSED] check_quota
[11:05:24] [PASSED] check_all
[11:05:24] ===================== [PASSED] guc_idm =====================
[11:05:24] ================== no_relay (3 subtests) ===================
[11:05:24] [PASSED] xe_drops_guc2pf_if_not_ready
[11:05:24] [PASSED] xe_drops_guc2vf_if_not_ready
[11:05:24] [PASSED] xe_rejects_send_if_not_ready
[11:05:24] ==================== [PASSED] no_relay =====================
[11:05:24] ================== pf_relay (14 subtests) ==================
[11:05:24] [PASSED] pf_rejects_guc2pf_too_short
[11:05:24] [PASSED] pf_rejects_guc2pf_too_long
[11:05:24] [PASSED] pf_rejects_guc2pf_no_payload
[11:05:24] [PASSED] pf_fails_no_payload
[11:05:24] [PASSED] pf_fails_bad_origin
[11:05:24] [PASSED] pf_fails_bad_type
[11:05:24] [PASSED] pf_txn_reports_error
[11:05:24] [PASSED] pf_txn_sends_pf2guc
[11:05:24] [PASSED] pf_sends_pf2guc
[11:05:24] [SKIPPED] pf_loopback_nop
[11:05:24] [SKIPPED] pf_loopback_echo
[11:05:24] [SKIPPED] pf_loopback_fail
[11:05:24] [SKIPPED] pf_loopback_busy
[11:05:24] [SKIPPED] pf_loopback_retry
[11:05:24] ==================== [PASSED] pf_relay =====================
[11:05:24] ================== vf_relay (3 subtests) ===================
[11:05:24] [PASSED] vf_rejects_guc2vf_too_short
[11:05:24] [PASSED] vf_rejects_guc2vf_too_long
[11:05:24] [PASSED] vf_rejects_guc2vf_no_payload
[11:05:24] ==================== [PASSED] vf_relay =====================
[11:05:24] ================ pf_gt_config (6 subtests) =================
[11:05:24] [PASSED] fair_contexts_1vf
[11:05:24] [PASSED] fair_doorbells_1vf
[11:05:24] [PASSED] fair_ggtt_1vf
[11:05:24] ====================== fair_contexts ======================
[11:05:24] [PASSED] 1 VF
[11:05:24] [PASSED] 2 VFs
[11:05:24] [PASSED] 3 VFs
[11:05:24] [PASSED] 4 VFs
[11:05:24] [PASSED] 5 VFs
[11:05:24] [PASSED] 6 VFs
[11:05:24] [PASSED] 7 VFs
[11:05:24] [PASSED] 8 VFs
[11:05:24] [PASSED] 9 VFs
[11:05:24] [PASSED] 10 VFs
[11:05:24] [PASSED] 11 VFs
[11:05:24] [PASSED] 12 VFs
[11:05:24] [PASSED] 13 VFs
[11:05:24] [PASSED] 14 VFs
[11:05:24] [PASSED] 15 VFs
[11:05:24] [PASSED] 16 VFs
[11:05:24] [PASSED] 17 VFs
[11:05:24] [PASSED] 18 VFs
[11:05:24] [PASSED] 19 VFs
[11:05:24] [PASSED] 20 VFs
[11:05:24] [PASSED] 21 VFs
[11:05:24] [PASSED] 22 VFs
[11:05:24] [PASSED] 23 VFs
[11:05:24] [PASSED] 24 VFs
[11:05:24] [PASSED] 25 VFs
[11:05:24] [PASSED] 26 VFs
[11:05:24] [PASSED] 27 VFs
[11:05:24] [PASSED] 28 VFs
[11:05:24] [PASSED] 29 VFs
[11:05:24] [PASSED] 30 VFs
[11:05:24] [PASSED] 31 VFs
[11:05:24] [PASSED] 32 VFs
[11:05:24] [PASSED] 33 VFs
[11:05:24] [PASSED] 34 VFs
[11:05:24] [PASSED] 35 VFs
[11:05:24] [PASSED] 36 VFs
[11:05:24] [PASSED] 37 VFs
[11:05:24] [PASSED] 38 VFs
[11:05:24] [PASSED] 39 VFs
[11:05:24] [PASSED] 40 VFs
[11:05:24] [PASSED] 41 VFs
[11:05:24] [PASSED] 42 VFs
[11:05:24] [PASSED] 43 VFs
[11:05:24] [PASSED] 44 VFs
[11:05:24] [PASSED] 45 VFs
[11:05:24] [PASSED] 46 VFs
[11:05:24] [PASSED] 47 VFs
[11:05:24] [PASSED] 48 VFs
[11:05:24] [PASSED] 49 VFs
[11:05:24] [PASSED] 50 VFs
[11:05:24] [PASSED] 51 VFs
[11:05:24] [PASSED] 52 VFs
[11:05:24] [PASSED] 53 VFs
[11:05:24] [PASSED] 54 VFs
[11:05:24] [PASSED] 55 VFs
[11:05:24] [PASSED] 56 VFs
[11:05:24] [PASSED] 57 VFs
[11:05:24] [PASSED] 58 VFs
[11:05:24] [PASSED] 59 VFs
[11:05:24] [PASSED] 60 VFs
[11:05:24] [PASSED] 61 VFs
[11:05:24] [PASSED] 62 VFs
[11:05:24] [PASSED] 63 VFs
[11:05:24] ================== [PASSED] fair_contexts ==================
[11:05:24] ===================== fair_doorbells ======================
[11:05:24] [PASSED] 1 VF
[11:05:24] [PASSED] 2 VFs
[11:05:24] [PASSED] 3 VFs
[11:05:24] [PASSED] 4 VFs
[11:05:24] [PASSED] 5 VFs
[11:05:24] [PASSED] 6 VFs
[11:05:24] [PASSED] 7 VFs
[11:05:24] [PASSED] 8 VFs
[11:05:24] [PASSED] 9 VFs
[11:05:24] [PASSED] 10 VFs
[11:05:24] [PASSED] 11 VFs
[11:05:24] [PASSED] 12 VFs
[11:05:24] [PASSED] 13 VFs
[11:05:24] [PASSED] 14 VFs
[11:05:24] [PASSED] 15 VFs
[11:05:24] [PASSED] 16 VFs
[11:05:24] [PASSED] 17 VFs
[11:05:24] [PASSED] 18 VFs
[11:05:24] [PASSED] 19 VFs
[11:05:24] [PASSED] 20 VFs
[11:05:24] [PASSED] 21 VFs
[11:05:24] [PASSED] 22 VFs
[11:05:24] [PASSED] 23 VFs
[11:05:24] [PASSED] 24 VFs
[11:05:24] [PASSED] 25 VFs
[11:05:24] [PASSED] 26 VFs
[11:05:24] [PASSED] 27 VFs
[11:05:24] [PASSED] 28 VFs
[11:05:24] [PASSED] 29 VFs
[11:05:24] [PASSED] 30 VFs
[11:05:24] [PASSED] 31 VFs
[11:05:24] [PASSED] 32 VFs
[11:05:24] [PASSED] 33 VFs
[11:05:24] [PASSED] 34 VFs
[11:05:24] [PASSED] 35 VFs
[11:05:24] [PASSED] 36 VFs
[11:05:24] [PASSED] 37 VFs
[11:05:24] [PASSED] 38 VFs
[11:05:24] [PASSED] 39 VFs
[11:05:24] [PASSED] 40 VFs
[11:05:24] [PASSED] 41 VFs
[11:05:24] [PASSED] 42 VFs
[11:05:24] [PASSED] 43 VFs
[11:05:24] [PASSED] 44 VFs
[11:05:24] [PASSED] 45 VFs
[11:05:24] [PASSED] 46 VFs
[11:05:24] [PASSED] 47 VFs
[11:05:24] [PASSED] 48 VFs
[11:05:24] [PASSED] 49 VFs
[11:05:24] [PASSED] 50 VFs
[11:05:24] [PASSED] 51 VFs
[11:05:24] [PASSED] 52 VFs
[11:05:24] [PASSED] 53 VFs
[11:05:24] [PASSED] 54 VFs
[11:05:24] [PASSED] 55 VFs
[11:05:24] [PASSED] 56 VFs
[11:05:24] [PASSED] 57 VFs
[11:05:24] [PASSED] 58 VFs
[11:05:24] [PASSED] 59 VFs
[11:05:24] [PASSED] 60 VFs
[11:05:24] [PASSED] 61 VFs
[11:05:24] [PASSED] 62 VFs
[11:05:24] [PASSED] 63 VFs
[11:05:24] ================= [PASSED] fair_doorbells ==================
[11:05:24] ======================== fair_ggtt ========================
[11:05:24] [PASSED] 1 VF
[11:05:24] [PASSED] 2 VFs
[11:05:24] [PASSED] 3 VFs
[11:05:24] [PASSED] 4 VFs
[11:05:24] [PASSED] 5 VFs
[11:05:24] [PASSED] 6 VFs
[11:05:24] [PASSED] 7 VFs
[11:05:24] [PASSED] 8 VFs
[11:05:24] [PASSED] 9 VFs
[11:05:24] [PASSED] 10 VFs
[11:05:24] [PASSED] 11 VFs
[11:05:24] [PASSED] 12 VFs
[11:05:24] [PASSED] 13 VFs
[11:05:24] [PASSED] 14 VFs
[11:05:24] [PASSED] 15 VFs
[11:05:24] [PASSED] 16 VFs
[11:05:24] [PASSED] 17 VFs
[11:05:24] [PASSED] 18 VFs
[11:05:24] [PASSED] 19 VFs
[11:05:24] [PASSED] 20 VFs
[11:05:24] [PASSED] 21 VFs
[11:05:24] [PASSED] 22 VFs
[11:05:24] [PASSED] 23 VFs
[11:05:24] [PASSED] 24 VFs
[11:05:24] [PASSED] 25 VFs
[11:05:24] [PASSED] 26 VFs
[11:05:24] [PASSED] 27 VFs
[11:05:24] [PASSED] 28 VFs
[11:05:24] [PASSED] 29 VFs
[11:05:24] [PASSED] 30 VFs
[11:05:24] [PASSED] 31 VFs
[11:05:24] [PASSED] 32 VFs
[11:05:24] [PASSED] 33 VFs
[11:05:24] [PASSED] 34 VFs
[11:05:24] [PASSED] 35 VFs
[11:05:24] [PASSED] 36 VFs
[11:05:24] [PASSED] 37 VFs
[11:05:24] [PASSED] 38 VFs
[11:05:24] [PASSED] 39 VFs
[11:05:24] [PASSED] 40 VFs
[11:05:24] [PASSED] 41 VFs
[11:05:24] [PASSED] 42 VFs
[11:05:24] [PASSED] 43 VFs
[11:05:24] [PASSED] 44 VFs
[11:05:24] [PASSED] 45 VFs
[11:05:24] [PASSED] 46 VFs
[11:05:24] [PASSED] 47 VFs
[11:05:24] [PASSED] 48 VFs
[11:05:24] [PASSED] 49 VFs
[11:05:24] [PASSED] 50 VFs
[11:05:24] [PASSED] 51 VFs
[11:05:24] [PASSED] 52 VFs
[11:05:24] [PASSED] 53 VFs
[11:05:24] [PASSED] 54 VFs
[11:05:24] [PASSED] 55 VFs
[11:05:24] [PASSED] 56 VFs
[11:05:24] [PASSED] 57 VFs
[11:05:24] [PASSED] 58 VFs
[11:05:24] [PASSED] 59 VFs
[11:05:24] [PASSED] 60 VFs
[11:05:24] [PASSED] 61 VFs
[11:05:24] [PASSED] 62 VFs
[11:05:24] [PASSED] 63 VFs
[11:05:24] ==================== [PASSED] fair_ggtt ====================
[11:05:24] ================== [PASSED] pf_gt_config ===================
[11:05:24] ===================== lmtt (1 subtest) =====================
[11:05:24] ======================== test_ops =========================
[11:05:24] [PASSED] 2-level
[11:05:24] [PASSED] multi-level
[11:05:24] ==================== [PASSED] test_ops =====================
[11:05:24] ====================== [PASSED] lmtt =======================
[11:05:24] ================= pf_service (11 subtests) =================
[11:05:24] [PASSED] pf_negotiate_any
[11:05:24] [PASSED] pf_negotiate_base_match
[11:05:24] [PASSED] pf_negotiate_base_newer
[11:05:24] [PASSED] pf_negotiate_base_next
[11:05:24] [SKIPPED] pf_negotiate_base_older
[11:05:24] [PASSED] pf_negotiate_base_prev
[11:05:24] [PASSED] pf_negotiate_latest_match
[11:05:24] [PASSED] pf_negotiate_latest_newer
[11:05:24] [PASSED] pf_negotiate_latest_next
[11:05:24] [SKIPPED] pf_negotiate_latest_older
[11:05:24] [SKIPPED] pf_negotiate_latest_prev
[11:05:24] =================== [PASSED] pf_service ====================
[11:05:24] ================= xe_guc_g2g (2 subtests) ==================
[11:05:24] ============== xe_live_guc_g2g_kunit_default ==============
[11:05:24] ========= [SKIPPED] xe_live_guc_g2g_kunit_default ==========
[11:05:24] ============== xe_live_guc_g2g_kunit_allmem ===============
[11:05:24] ========== [SKIPPED] xe_live_guc_g2g_kunit_allmem ==========
[11:05:24] =================== [SKIPPED] xe_guc_g2g ===================
[11:05:24] =================== xe_mocs (2 subtests) ===================
[11:05:24] ================ xe_live_mocs_kernel_kunit ================
[11:05:24] =========== [SKIPPED] xe_live_mocs_kernel_kunit ============
[11:05:24] ================ xe_live_mocs_reset_kunit =================
[11:05:24] ============ [SKIPPED] xe_live_mocs_reset_kunit ============
[11:05:24] ==================== [SKIPPED] xe_mocs =====================
[11:05:24] ================= xe_migrate (2 subtests) ==================
[11:05:24] ================= xe_migrate_sanity_kunit =================
[11:05:24] ============ [SKIPPED] xe_migrate_sanity_kunit =============
[11:05:24] ================== xe_validate_ccs_kunit ==================
[11:05:24] ============= [SKIPPED] xe_validate_ccs_kunit ==============
[11:05:24] =================== [SKIPPED] xe_migrate ===================
[11:05:24] ================== xe_dma_buf (1 subtest) ==================
[11:05:24] ==================== xe_dma_buf_kunit =====================
[11:05:24] ================ [SKIPPED] xe_dma_buf_kunit ================
[11:05:24] =================== [SKIPPED] xe_dma_buf ===================
[11:05:24] ================= xe_bo_shrink (1 subtest) =================
[11:05:24] =================== xe_bo_shrink_kunit ====================
[11:05:24] =============== [SKIPPED] xe_bo_shrink_kunit ===============
[11:05:24] ================== [SKIPPED] xe_bo_shrink ==================
[11:05:24] ==================== xe_bo (2 subtests) ====================
[11:05:24] ================== xe_ccs_migrate_kunit ===================
[11:05:24] ============== [SKIPPED] xe_ccs_migrate_kunit ==============
[11:05:24] ==================== xe_bo_evict_kunit ====================
[11:05:24] =============== [SKIPPED] xe_bo_evict_kunit ================
[11:05:24] ===================== [SKIPPED] xe_bo ======================
[11:05:24] ==================== args (13 subtests) ====================
[11:05:24] [PASSED] count_args_test
[11:05:24] [PASSED] call_args_example
[11:05:24] [PASSED] call_args_test
[11:05:24] [PASSED] drop_first_arg_example
[11:05:24] [PASSED] drop_first_arg_test
[11:05:24] [PASSED] first_arg_example
[11:05:24] [PASSED] first_arg_test
[11:05:24] [PASSED] last_arg_example
[11:05:24] [PASSED] last_arg_test
[11:05:24] [PASSED] pick_arg_example
[11:05:24] [PASSED] if_args_example
[11:05:24] [PASSED] if_args_test
[11:05:24] [PASSED] sep_comma_example
[11:05:24] ====================== [PASSED] args =======================
[11:05:24] =================== xe_pci (3 subtests) ====================
[11:05:24] ==================== check_graphics_ip ====================
[11:05:24] [PASSED] 12.00 Xe_LP
[11:05:24] [PASSED] 12.10 Xe_LP+
[11:05:24] [PASSED] 12.55 Xe_HPG
[11:05:24] [PASSED] 12.60 Xe_HPC
[11:05:24] [PASSED] 12.70 Xe_LPG
[11:05:24] [PASSED] 12.71 Xe_LPG
[11:05:24] [PASSED] 12.74 Xe_LPG+
[11:05:24] [PASSED] 20.01 Xe2_HPG
[11:05:24] [PASSED] 20.02 Xe2_HPG
[11:05:24] [PASSED] 20.04 Xe2_LPG
[11:05:24] [PASSED] 30.00 Xe3_LPG
[11:05:24] [PASSED] 30.01 Xe3_LPG
[11:05:24] [PASSED] 30.03 Xe3_LPG
[11:05:24] [PASSED] 30.04 Xe3_LPG
[11:05:24] [PASSED] 30.05 Xe3_LPG
[11:05:24] [PASSED] 35.11 Xe3p_XPC
[11:05:24] ================ [PASSED] check_graphics_ip ================
[11:05:24] ===================== check_media_ip ======================
[11:05:24] [PASSED] 12.00 Xe_M
[11:05:24] [PASSED] 12.55 Xe_HPM
[11:05:24] [PASSED] 13.00 Xe_LPM+
[11:05:24] [PASSED] 13.01 Xe2_HPM
[11:05:24] [PASSED] 20.00 Xe2_LPM
[11:05:24] [PASSED] 30.00 Xe3_LPM
[11:05:24] [PASSED] 30.02 Xe3_LPM
[11:05:24] [PASSED] 35.00 Xe3p_LPM
[11:05:24] [PASSED] 35.03 Xe3p_HPM
[11:05:24] ================= [PASSED] check_media_ip ==================
[11:05:24] =================== check_platform_desc ===================
[11:05:24] [PASSED] 0x9A60 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A68 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A70 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A40 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A49 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A59 (TIGERLAKE)
[11:05:24] [PASSED] 0x9A78 (TIGERLAKE)
[11:05:24] [PASSED] 0x9AC0 (TIGERLAKE)
[11:05:24] [PASSED] 0x9AC9 (TIGERLAKE)
[11:05:24] [PASSED] 0x9AD9 (TIGERLAKE)
[11:05:24] [PASSED] 0x9AF8 (TIGERLAKE)
[11:05:24] [PASSED] 0x4C80 (ROCKETLAKE)
[11:05:24] [PASSED] 0x4C8A (ROCKETLAKE)
[11:05:24] [PASSED] 0x4C8B (ROCKETLAKE)
[11:05:24] [PASSED] 0x4C8C (ROCKETLAKE)
[11:05:24] [PASSED] 0x4C90 (ROCKETLAKE)
[11:05:24] [PASSED] 0x4C9A (ROCKETLAKE)
[11:05:24] [PASSED] 0x4680 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4682 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4688 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x468A (ALDERLAKE_S)
[11:05:24] [PASSED] 0x468B (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4690 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4692 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4693 (ALDERLAKE_S)
[11:05:24] [PASSED] 0x46A0 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46A1 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46A2 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46A3 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46A6 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46A8 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46AA (ALDERLAKE_P)
[11:05:24] [PASSED] 0x462A (ALDERLAKE_P)
[11:05:24] [PASSED] 0x4626 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x4628 (ALDERLAKE_P)
stty: 'standard input': Inappropriate ioctl for device
[11:05:24] [PASSED] 0x46B0 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46B1 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46B2 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46B3 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46C0 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46C1 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46C2 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46C3 (ALDERLAKE_P)
[11:05:24] [PASSED] 0x46D0 (ALDERLAKE_N)
[11:05:24] [PASSED] 0x46D1 (ALDERLAKE_N)
[11:05:24] [PASSED] 0x46D2 (ALDERLAKE_N)
[11:05:24] [PASSED] 0x46D3 (ALDERLAKE_N)
[11:05:24] [PASSED] 0x46D4 (ALDERLAKE_N)
[11:05:24] [PASSED] 0xA721 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7A1 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7A9 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7AC (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7AD (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA720 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7A0 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7A8 (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7AA (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA7AB (ALDERLAKE_P)
[11:05:24] [PASSED] 0xA780 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA781 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA782 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA783 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA788 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA789 (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA78A (ALDERLAKE_S)
[11:05:24] [PASSED] 0xA78B (ALDERLAKE_S)
[11:05:24] [PASSED] 0x4905 (DG1)
[11:05:24] [PASSED] 0x4906 (DG1)
[11:05:24] [PASSED] 0x4907 (DG1)
[11:05:24] [PASSED] 0x4908 (DG1)
[11:05:24] [PASSED] 0x4909 (DG1)
[11:05:24] [PASSED] 0x56C0 (DG2)
[11:05:24] [PASSED] 0x56C2 (DG2)
[11:05:24] [PASSED] 0x56C1 (DG2)
[11:05:24] [PASSED] 0x7D51 (METEORLAKE)
[11:05:24] [PASSED] 0x7DD1 (METEORLAKE)
[11:05:24] [PASSED] 0x7D41 (METEORLAKE)
[11:05:24] [PASSED] 0x7D67 (METEORLAKE)
[11:05:24] [PASSED] 0xB640 (METEORLAKE)
[11:05:24] [PASSED] 0x56A0 (DG2)
[11:05:24] [PASSED] 0x56A1 (DG2)
[11:05:24] [PASSED] 0x56A2 (DG2)
[11:05:24] [PASSED] 0x56BE (DG2)
[11:05:24] [PASSED] 0x56BF (DG2)
[11:05:24] [PASSED] 0x5690 (DG2)
[11:05:24] [PASSED] 0x5691 (DG2)
[11:05:24] [PASSED] 0x5692 (DG2)
[11:05:24] [PASSED] 0x56A5 (DG2)
[11:05:24] [PASSED] 0x56A6 (DG2)
[11:05:24] [PASSED] 0x56B0 (DG2)
[11:05:24] [PASSED] 0x56B1 (DG2)
[11:05:24] [PASSED] 0x56BA (DG2)
[11:05:24] [PASSED] 0x56BB (DG2)
[11:05:24] [PASSED] 0x56BC (DG2)
[11:05:24] [PASSED] 0x56BD (DG2)
[11:05:24] [PASSED] 0x5693 (DG2)
[11:05:24] [PASSED] 0x5694 (DG2)
[11:05:24] [PASSED] 0x5695 (DG2)
[11:05:24] [PASSED] 0x56A3 (DG2)
[11:05:24] [PASSED] 0x56A4 (DG2)
[11:05:24] [PASSED] 0x56B2 (DG2)
[11:05:24] [PASSED] 0x56B3 (DG2)
[11:05:24] [PASSED] 0x5696 (DG2)
[11:05:24] [PASSED] 0x5697 (DG2)
[11:05:24] [PASSED] 0xB69 (PVC)
[11:05:24] [PASSED] 0xB6E (PVC)
[11:05:24] [PASSED] 0xBD4 (PVC)
[11:05:24] [PASSED] 0xBD5 (PVC)
[11:05:24] [PASSED] 0xBD6 (PVC)
[11:05:24] [PASSED] 0xBD7 (PVC)
[11:05:24] [PASSED] 0xBD8 (PVC)
[11:05:24] [PASSED] 0xBD9 (PVC)
[11:05:24] [PASSED] 0xBDA (PVC)
[11:05:24] [PASSED] 0xBDB (PVC)
[11:05:24] [PASSED] 0xBE0 (PVC)
[11:05:24] [PASSED] 0xBE1 (PVC)
[11:05:24] [PASSED] 0xBE5 (PVC)
[11:05:24] [PASSED] 0x7D40 (METEORLAKE)
[11:05:24] [PASSED] 0x7D45 (METEORLAKE)
[11:05:24] [PASSED] 0x7D55 (METEORLAKE)
[11:05:24] [PASSED] 0x7D60 (METEORLAKE)
[11:05:24] [PASSED] 0x7DD5 (METEORLAKE)
[11:05:24] [PASSED] 0x6420 (LUNARLAKE)
[11:05:24] [PASSED] 0x64A0 (LUNARLAKE)
[11:05:24] [PASSED] 0x64B0 (LUNARLAKE)
[11:05:24] [PASSED] 0xE202 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE209 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE20B (BATTLEMAGE)
[11:05:24] [PASSED] 0xE20C (BATTLEMAGE)
[11:05:24] [PASSED] 0xE20D (BATTLEMAGE)
[11:05:24] [PASSED] 0xE210 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE211 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE212 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE216 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE220 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE221 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE222 (BATTLEMAGE)
[11:05:24] [PASSED] 0xE223 (BATTLEMAGE)
[11:05:24] [PASSED] 0xB080 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB081 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB082 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB083 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB084 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB085 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB086 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB087 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB08F (PANTHERLAKE)
[11:05:24] [PASSED] 0xB090 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB0A0 (PANTHERLAKE)
[11:05:24] [PASSED] 0xB0B0 (PANTHERLAKE)
[11:05:24] [PASSED] 0xFD80 (PANTHERLAKE)
[11:05:24] [PASSED] 0xFD81 (PANTHERLAKE)
[11:05:24] [PASSED] 0xD740 (NOVALAKE_S)
[11:05:24] [PASSED] 0xD741 (NOVALAKE_S)
[11:05:24] [PASSED] 0xD742 (NOVALAKE_S)
[11:05:24] [PASSED] 0xD743 (NOVALAKE_S)
[11:05:24] [PASSED] 0xD744 (NOVALAKE_S)
[11:05:24] [PASSED] 0xD745 (NOVALAKE_S)
[11:05:24] [PASSED] 0x674C (CRESCENTISLAND)
[11:05:24] =============== [PASSED] check_platform_desc ===============
[11:05:24] ===================== [PASSED] xe_pci ======================
[11:05:24] =================== xe_rtp (2 subtests) ====================
[11:05:24] =============== xe_rtp_process_to_sr_tests ================
[11:05:24] [PASSED] coalesce-same-reg
[11:05:24] [PASSED] no-match-no-add
[11:05:24] [PASSED] match-or
[11:05:24] [PASSED] match-or-xfail
[11:05:24] [PASSED] no-match-no-add-multiple-rules
[11:05:24] [PASSED] two-regs-two-entries
[11:05:24] [PASSED] clr-one-set-other
[11:05:24] [PASSED] set-field
[11:05:24] [PASSED] conflict-duplicate
[11:05:24] [PASSED] conflict-not-disjoint
[11:05:24] [PASSED] conflict-reg-type
[11:05:24] =========== [PASSED] xe_rtp_process_to_sr_tests ============
[11:05:24] ================== xe_rtp_process_tests ===================
[11:05:24] [PASSED] active1
[11:05:24] [PASSED] active2
[11:05:24] [PASSED] active-inactive
[11:05:24] [PASSED] inactive-active
[11:05:24] [PASSED] inactive-1st_or_active-inactive
[11:05:24] [PASSED] inactive-2nd_or_active-inactive
[11:05:24] [PASSED] inactive-last_or_active-inactive
[11:05:24] [PASSED] inactive-no_or_active-inactive
[11:05:24] ============== [PASSED] xe_rtp_process_tests ===============
[11:05:24] ===================== [PASSED] xe_rtp ======================
[11:05:24] ==================== xe_wa (1 subtest) =====================
[11:05:24] ======================== xe_wa_gt =========================
[11:05:24] [PASSED] TIGERLAKE B0
[11:05:24] [PASSED] DG1 A0
[11:05:24] [PASSED] DG1 B0
[11:05:24] [PASSED] ALDERLAKE_S A0
[11:05:24] [PASSED] ALDERLAKE_S B0
[11:05:24] [PASSED] ALDERLAKE_S C0
[11:05:24] [PASSED] ALDERLAKE_S D0
[11:05:24] [PASSED] ALDERLAKE_P A0
[11:05:24] [PASSED] ALDERLAKE_P B0
[11:05:24] [PASSED] ALDERLAKE_P C0
[11:05:24] [PASSED] ALDERLAKE_S RPLS D0
[11:05:24] [PASSED] ALDERLAKE_P RPLU E0
[11:05:24] [PASSED] DG2 G10 C0
[11:05:24] [PASSED] DG2 G11 B1
[11:05:24] [PASSED] DG2 G12 A1
[11:05:24] [PASSED] METEORLAKE 12.70(Xe_LPG) A0 13.00(Xe_LPM+) A0
[11:05:24] [PASSED] METEORLAKE 12.71(Xe_LPG) A0 13.00(Xe_LPM+) A0
[11:05:24] [PASSED] METEORLAKE 12.74(Xe_LPG+) A0 13.00(Xe_LPM+) A0
[11:05:24] [PASSED] LUNARLAKE 20.04(Xe2_LPG) A0 20.00(Xe2_LPM) A0
[11:05:24] [PASSED] LUNARLAKE 20.04(Xe2_LPG) B0 20.00(Xe2_LPM) A0
[11:05:24] [PASSED] BATTLEMAGE 20.01(Xe2_HPG) A0 13.01(Xe2_HPM) A1
[11:05:24] [PASSED] PANTHERLAKE 30.00(Xe3_LPG) A0 30.00(Xe3_LPM) A0
[11:05:24] ==================== [PASSED] xe_wa_gt =====================
[11:05:24] ====================== [PASSED] xe_wa ======================
[11:05:24] ============================================================
[11:05:24] Testing complete. Ran 512 tests: passed: 494, skipped: 18
[11:05:24] Elapsed time: 36.319s total, 4.215s configuring, 31.587s building, 0.480s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/tests/.kunitconfig
[11:05:24] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:05:26] 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=48
[11:05:51] Starting KUnit Kernel (1/1)...
[11:05:51] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:05:51] ============ drm_test_pick_cmdline (2 subtests) ============
[11:05:51] [PASSED] drm_test_pick_cmdline_res_1920_1080_60
[11:05:51] =============== drm_test_pick_cmdline_named ===============
[11:05:51] [PASSED] NTSC
[11:05:51] [PASSED] NTSC-J
[11:05:51] [PASSED] PAL
[11:05:51] [PASSED] PAL-M
[11:05:51] =========== [PASSED] drm_test_pick_cmdline_named ===========
[11:05:51] ============== [PASSED] drm_test_pick_cmdline ==============
[11:05:51] == drm_test_atomic_get_connector_for_encoder (1 subtest) ===
[11:05:51] [PASSED] drm_test_drm_atomic_get_connector_for_encoder
[11:05:51] ==== [PASSED] drm_test_atomic_get_connector_for_encoder ====
[11:05:51] =========== drm_validate_clone_mode (2 subtests) ===========
[11:05:51] ============== drm_test_check_in_clone_mode ===============
[11:05:51] [PASSED] in_clone_mode
[11:05:51] [PASSED] not_in_clone_mode
[11:05:51] ========== [PASSED] drm_test_check_in_clone_mode ===========
[11:05:51] =============== drm_test_check_valid_clones ===============
[11:05:51] [PASSED] not_in_clone_mode
[11:05:51] [PASSED] valid_clone
[11:05:51] [PASSED] invalid_clone
[11:05:51] =========== [PASSED] drm_test_check_valid_clones ===========
[11:05:51] ============= [PASSED] drm_validate_clone_mode =============
[11:05:51] ============= drm_validate_modeset (1 subtest) =============
[11:05:51] [PASSED] drm_test_check_connector_changed_modeset
[11:05:51] ============== [PASSED] drm_validate_modeset ===============
[11:05:51] ====== drm_test_bridge_get_current_state (2 subtests) ======
[11:05:51] [PASSED] drm_test_drm_bridge_get_current_state_atomic
[11:05:51] [PASSED] drm_test_drm_bridge_get_current_state_legacy
[11:05:51] ======== [PASSED] drm_test_bridge_get_current_state ========
[11:05:51] ====== drm_test_bridge_helper_reset_crtc (3 subtests) ======
[11:05:51] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic
[11:05:51] [PASSED] drm_test_drm_bridge_helper_reset_crtc_atomic_disabled
[11:05:51] [PASSED] drm_test_drm_bridge_helper_reset_crtc_legacy
[11:05:51] ======== [PASSED] drm_test_bridge_helper_reset_crtc ========
[11:05:51] ============== drm_bridge_alloc (2 subtests) ===============
[11:05:51] [PASSED] drm_test_drm_bridge_alloc_basic
[11:05:51] [PASSED] drm_test_drm_bridge_alloc_get_put
[11:05:51] ================ [PASSED] drm_bridge_alloc =================
[11:05:51] ============= drm_cmdline_parser (40 subtests) =============
[11:05:51] [PASSED] drm_test_cmdline_force_d_only
[11:05:51] [PASSED] drm_test_cmdline_force_D_only_dvi
[11:05:51] [PASSED] drm_test_cmdline_force_D_only_hdmi
[11:05:51] [PASSED] drm_test_cmdline_force_D_only_not_digital
[11:05:51] [PASSED] drm_test_cmdline_force_e_only
[11:05:51] [PASSED] drm_test_cmdline_res
[11:05:51] [PASSED] drm_test_cmdline_res_vesa
[11:05:51] [PASSED] drm_test_cmdline_res_vesa_rblank
[11:05:51] [PASSED] drm_test_cmdline_res_rblank
[11:05:51] [PASSED] drm_test_cmdline_res_bpp
[11:05:51] [PASSED] drm_test_cmdline_res_refresh
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_margins
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_off
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_analog
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_force_on_digital
[11:05:51] [PASSED] drm_test_cmdline_res_bpp_refresh_interlaced_margins_force_on
[11:05:51] [PASSED] drm_test_cmdline_res_margins_force_on
[11:05:51] [PASSED] drm_test_cmdline_res_vesa_margins
[11:05:51] [PASSED] drm_test_cmdline_name
[11:05:51] [PASSED] drm_test_cmdline_name_bpp
[11:05:51] [PASSED] drm_test_cmdline_name_option
[11:05:51] [PASSED] drm_test_cmdline_name_bpp_option
[11:05:51] [PASSED] drm_test_cmdline_rotate_0
[11:05:51] [PASSED] drm_test_cmdline_rotate_90
[11:05:51] [PASSED] drm_test_cmdline_rotate_180
[11:05:51] [PASSED] drm_test_cmdline_rotate_270
[11:05:51] [PASSED] drm_test_cmdline_hmirror
[11:05:51] [PASSED] drm_test_cmdline_vmirror
[11:05:51] [PASSED] drm_test_cmdline_margin_options
[11:05:51] [PASSED] drm_test_cmdline_multiple_options
[11:05:51] [PASSED] drm_test_cmdline_bpp_extra_and_option
[11:05:51] [PASSED] drm_test_cmdline_extra_and_option
[11:05:51] [PASSED] drm_test_cmdline_freestanding_options
[11:05:51] [PASSED] drm_test_cmdline_freestanding_force_e_and_options
[11:05:51] [PASSED] drm_test_cmdline_panel_orientation
[11:05:51] ================ drm_test_cmdline_invalid =================
[11:05:51] [PASSED] margin_only
[11:05:51] [PASSED] interlace_only
[11:05:51] [PASSED] res_missing_x
[11:05:51] [PASSED] res_missing_y
[11:05:51] [PASSED] res_bad_y
[11:05:51] [PASSED] res_missing_y_bpp
[11:05:51] [PASSED] res_bad_bpp
[11:05:51] [PASSED] res_bad_refresh
[11:05:51] [PASSED] res_bpp_refresh_force_on_off
[11:05:51] [PASSED] res_invalid_mode
[11:05:51] [PASSED] res_bpp_wrong_place_mode
[11:05:51] [PASSED] name_bpp_refresh
[11:05:51] [PASSED] name_refresh
[11:05:51] [PASSED] name_refresh_wrong_mode
[11:05:51] [PASSED] name_refresh_invalid_mode
[11:05:51] [PASSED] rotate_multiple
[11:05:51] [PASSED] rotate_invalid_val
[11:05:51] [PASSED] rotate_truncated
[11:05:51] [PASSED] invalid_option
[11:05:51] [PASSED] invalid_tv_option
[11:05:51] [PASSED] truncated_tv_option
[11:05:51] ============ [PASSED] drm_test_cmdline_invalid =============
[11:05:51] =============== drm_test_cmdline_tv_options ===============
[11:05:51] [PASSED] NTSC
[11:05:51] [PASSED] NTSC_443
[11:05:51] [PASSED] NTSC_J
[11:05:51] [PASSED] PAL
[11:05:51] [PASSED] PAL_M
[11:05:51] [PASSED] PAL_N
[11:05:51] [PASSED] SECAM
[11:05:51] [PASSED] MONO_525
[11:05:51] [PASSED] MONO_625
[11:05:51] =========== [PASSED] drm_test_cmdline_tv_options ===========
[11:05:51] =============== [PASSED] drm_cmdline_parser ================
[11:05:51] ========== drmm_connector_hdmi_init (20 subtests) ==========
[11:05:51] [PASSED] drm_test_connector_hdmi_init_valid
[11:05:51] [PASSED] drm_test_connector_hdmi_init_bpc_8
[11:05:51] [PASSED] drm_test_connector_hdmi_init_bpc_10
[11:05:51] [PASSED] drm_test_connector_hdmi_init_bpc_12
[11:05:51] [PASSED] drm_test_connector_hdmi_init_bpc_invalid
[11:05:51] [PASSED] drm_test_connector_hdmi_init_bpc_null
[11:05:51] [PASSED] drm_test_connector_hdmi_init_formats_empty
[11:05:51] [PASSED] drm_test_connector_hdmi_init_formats_no_rgb
[11:05:51] === drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[11:05:51] [PASSED] supported_formats=0x9 yuv420_allowed=1
[11:05:51] [PASSED] supported_formats=0x9 yuv420_allowed=0
[11:05:51] [PASSED] supported_formats=0x3 yuv420_allowed=1
[11:05:51] [PASSED] supported_formats=0x3 yuv420_allowed=0
[11:05:51] === [PASSED] drm_test_connector_hdmi_init_formats_yuv420_allowed ===
[11:05:51] [PASSED] drm_test_connector_hdmi_init_null_ddc
[11:05:51] [PASSED] drm_test_connector_hdmi_init_null_product
[11:05:51] [PASSED] drm_test_connector_hdmi_init_null_vendor
[11:05:51] [PASSED] drm_test_connector_hdmi_init_product_length_exact
[11:05:51] [PASSED] drm_test_connector_hdmi_init_product_length_too_long
[11:05:51] [PASSED] drm_test_connector_hdmi_init_product_valid
[11:05:51] [PASSED] drm_test_connector_hdmi_init_vendor_length_exact
[11:05:51] [PASSED] drm_test_connector_hdmi_init_vendor_length_too_long
[11:05:51] [PASSED] drm_test_connector_hdmi_init_vendor_valid
[11:05:51] ========= drm_test_connector_hdmi_init_type_valid =========
[11:05:51] [PASSED] HDMI-A
[11:05:51] [PASSED] HDMI-B
[11:05:51] ===== [PASSED] drm_test_connector_hdmi_init_type_valid =====
[11:05:51] ======== drm_test_connector_hdmi_init_type_invalid ========
[11:05:51] [PASSED] Unknown
[11:05:51] [PASSED] VGA
[11:05:51] [PASSED] DVI-I
[11:05:51] [PASSED] DVI-D
[11:05:51] [PASSED] DVI-A
[11:05:51] [PASSED] Composite
[11:05:51] [PASSED] SVIDEO
[11:05:51] [PASSED] LVDS
[11:05:51] [PASSED] Component
[11:05:51] [PASSED] DIN
[11:05:51] [PASSED] DP
[11:05:51] [PASSED] TV
[11:05:51] [PASSED] eDP
[11:05:51] [PASSED] Virtual
[11:05:51] [PASSED] DSI
[11:05:51] [PASSED] DPI
[11:05:51] [PASSED] Writeback
[11:05:51] [PASSED] SPI
[11:05:51] [PASSED] USB
[11:05:51] ==== [PASSED] drm_test_connector_hdmi_init_type_invalid ====
[11:05:51] ============ [PASSED] drmm_connector_hdmi_init =============
[11:05:51] ============= drmm_connector_init (3 subtests) =============
[11:05:51] [PASSED] drm_test_drmm_connector_init
[11:05:51] [PASSED] drm_test_drmm_connector_init_null_ddc
[11:05:51] ========= drm_test_drmm_connector_init_type_valid =========
[11:05:51] [PASSED] Unknown
[11:05:51] [PASSED] VGA
[11:05:51] [PASSED] DVI-I
[11:05:51] [PASSED] DVI-D
[11:05:51] [PASSED] DVI-A
[11:05:51] [PASSED] Composite
[11:05:51] [PASSED] SVIDEO
[11:05:51] [PASSED] LVDS
[11:05:51] [PASSED] Component
[11:05:51] [PASSED] DIN
[11:05:51] [PASSED] DP
[11:05:51] [PASSED] HDMI-A
[11:05:51] [PASSED] HDMI-B
[11:05:51] [PASSED] TV
[11:05:51] [PASSED] eDP
[11:05:51] [PASSED] Virtual
[11:05:51] [PASSED] DSI
[11:05:51] [PASSED] DPI
[11:05:51] [PASSED] Writeback
[11:05:51] [PASSED] SPI
[11:05:51] [PASSED] USB
[11:05:51] ===== [PASSED] drm_test_drmm_connector_init_type_valid =====
[11:05:51] =============== [PASSED] drmm_connector_init ===============
[11:05:51] ========= drm_connector_dynamic_init (6 subtests) ==========
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_init
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_init_null_ddc
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_init_not_added
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_init_properties
[11:05:51] ===== drm_test_drm_connector_dynamic_init_type_valid ======
[11:05:51] [PASSED] Unknown
[11:05:51] [PASSED] VGA
[11:05:51] [PASSED] DVI-I
[11:05:51] [PASSED] DVI-D
[11:05:51] [PASSED] DVI-A
[11:05:51] [PASSED] Composite
[11:05:51] [PASSED] SVIDEO
[11:05:51] [PASSED] LVDS
[11:05:51] [PASSED] Component
[11:05:51] [PASSED] DIN
[11:05:51] [PASSED] DP
[11:05:51] [PASSED] HDMI-A
[11:05:51] [PASSED] HDMI-B
[11:05:51] [PASSED] TV
[11:05:51] [PASSED] eDP
[11:05:51] [PASSED] Virtual
[11:05:51] [PASSED] DSI
[11:05:51] [PASSED] DPI
[11:05:51] [PASSED] Writeback
[11:05:51] [PASSED] SPI
[11:05:51] [PASSED] USB
[11:05:51] = [PASSED] drm_test_drm_connector_dynamic_init_type_valid ==
[11:05:51] ======== drm_test_drm_connector_dynamic_init_name =========
[11:05:51] [PASSED] Unknown
[11:05:51] [PASSED] VGA
[11:05:51] [PASSED] DVI-I
[11:05:51] [PASSED] DVI-D
[11:05:51] [PASSED] DVI-A
[11:05:51] [PASSED] Composite
[11:05:51] [PASSED] SVIDEO
[11:05:51] [PASSED] LVDS
[11:05:51] [PASSED] Component
[11:05:51] [PASSED] DIN
[11:05:51] [PASSED] DP
[11:05:51] [PASSED] HDMI-A
[11:05:51] [PASSED] HDMI-B
[11:05:51] [PASSED] TV
[11:05:51] [PASSED] eDP
[11:05:51] [PASSED] Virtual
[11:05:51] [PASSED] DSI
[11:05:51] [PASSED] DPI
[11:05:51] [PASSED] Writeback
[11:05:51] [PASSED] SPI
[11:05:51] [PASSED] USB
[11:05:51] ==== [PASSED] drm_test_drm_connector_dynamic_init_name =====
[11:05:51] =========== [PASSED] drm_connector_dynamic_init ============
[11:05:51] ==== drm_connector_dynamic_register_early (4 subtests) =====
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_early_on_list
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_early_defer
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_early_no_init
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_early_no_mode_object
[11:05:51] ====== [PASSED] drm_connector_dynamic_register_early =======
[11:05:51] ======= drm_connector_dynamic_register (7 subtests) ========
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_on_list
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_no_defer
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_no_init
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_mode_object
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_sysfs
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_sysfs_name
[11:05:51] [PASSED] drm_test_drm_connector_dynamic_register_debugfs
[11:05:51] ========= [PASSED] drm_connector_dynamic_register ==========
[11:05:51] = drm_connector_attach_broadcast_rgb_property (2 subtests) =
[11:05:51] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property
[11:05:51] [PASSED] drm_test_drm_connector_attach_broadcast_rgb_property_hdmi_connector
[11:05:51] === [PASSED] drm_connector_attach_broadcast_rgb_property ===
[11:05:51] ========== drm_get_tv_mode_from_name (2 subtests) ==========
[11:05:51] ========== drm_test_get_tv_mode_from_name_valid ===========
[11:05:51] [PASSED] NTSC
[11:05:51] [PASSED] NTSC-443
[11:05:51] [PASSED] NTSC-J
[11:05:51] [PASSED] PAL
[11:05:51] [PASSED] PAL-M
[11:05:51] [PASSED] PAL-N
[11:05:51] [PASSED] SECAM
[11:05:51] [PASSED] Mono
[11:05:51] ====== [PASSED] drm_test_get_tv_mode_from_name_valid =======
[11:05:51] [PASSED] drm_test_get_tv_mode_from_name_truncated
[11:05:51] ============ [PASSED] drm_get_tv_mode_from_name ============
[11:05:51] = drm_test_connector_hdmi_compute_mode_clock (12 subtests) =
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_10bpc_vic_1
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_12bpc_vic_1
[11:05:51] [PASSED] drm_test_drm_hdmi_compute_mode_clock_rgb_double
[11:05:51] = drm_test_connector_hdmi_compute_mode_clock_yuv420_valid =
[11:05:51] [PASSED] VIC 96
[11:05:51] [PASSED] VIC 97
[11:05:51] [PASSED] VIC 101
[11:05:51] [PASSED] VIC 102
[11:05:51] [PASSED] VIC 106
[11:05:51] [PASSED] VIC 107
[11:05:51] === [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_valid ===
[11:05:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_10_bpc
[11:05:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv420_12_bpc
[11:05:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_8_bpc
[11:05:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_10_bpc
[11:05:51] [PASSED] drm_test_connector_hdmi_compute_mode_clock_yuv422_12_bpc
[11:05:51] === [PASSED] drm_test_connector_hdmi_compute_mode_clock ====
[11:05:51] == drm_hdmi_connector_get_broadcast_rgb_name (2 subtests) ==
[11:05:51] === drm_test_drm_hdmi_connector_get_broadcast_rgb_name ====
[11:05:51] [PASSED] Automatic
[11:05:51] [PASSED] Full
[11:05:51] [PASSED] Limited 16:235
[11:05:51] === [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name ===
[11:05:51] [PASSED] drm_test_drm_hdmi_connector_get_broadcast_rgb_name_invalid
[11:05:51] ==== [PASSED] drm_hdmi_connector_get_broadcast_rgb_name ====
[11:05:51] == drm_hdmi_connector_get_output_format_name (2 subtests) ==
[11:05:51] === drm_test_drm_hdmi_connector_get_output_format_name ====
[11:05:51] [PASSED] RGB
[11:05:51] [PASSED] YUV 4:2:0
[11:05:51] [PASSED] YUV 4:2:2
[11:05:51] [PASSED] YUV 4:4:4
[11:05:51] === [PASSED] drm_test_drm_hdmi_connector_get_output_format_name ===
[11:05:51] [PASSED] drm_test_drm_hdmi_connector_get_output_format_name_invalid
[11:05:51] ==== [PASSED] drm_hdmi_connector_get_output_format_name ====
[11:05:51] ============= drm_damage_helper (21 subtests) ==============
[11:05:51] [PASSED] drm_test_damage_iter_no_damage
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_fractional_src
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_src_moved
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_fractional_src_moved
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_not_visible
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_no_crtc
[11:05:51] [PASSED] drm_test_damage_iter_no_damage_no_fb
[11:05:51] [PASSED] drm_test_damage_iter_simple_damage
[11:05:51] [PASSED] drm_test_damage_iter_single_damage
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_intersect_src
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_outside_src
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_fractional_src
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_intersect_fractional_src
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_outside_fractional_src
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_src_moved
[11:05:51] [PASSED] drm_test_damage_iter_single_damage_fractional_src_moved
[11:05:51] [PASSED] drm_test_damage_iter_damage
[11:05:51] [PASSED] drm_test_damage_iter_damage_one_intersect
[11:05:51] [PASSED] drm_test_damage_iter_damage_one_outside
[11:05:51] [PASSED] drm_test_damage_iter_damage_src_moved
[11:05:51] [PASSED] drm_test_damage_iter_damage_not_visible
[11:05:51] ================ [PASSED] drm_damage_helper ================
[11:05:51] ============== drm_dp_mst_helper (3 subtests) ==============
[11:05:51] ============== drm_test_dp_mst_calc_pbn_mode ==============
[11:05:51] [PASSED] Clock 154000 BPP 30 DSC disabled
[11:05:51] [PASSED] Clock 234000 BPP 30 DSC disabled
[11:05:51] [PASSED] Clock 297000 BPP 24 DSC disabled
[11:05:51] [PASSED] Clock 332880 BPP 24 DSC enabled
[11:05:51] [PASSED] Clock 324540 BPP 24 DSC enabled
[11:05:51] ========== [PASSED] drm_test_dp_mst_calc_pbn_mode ==========
[11:05:51] ============== drm_test_dp_mst_calc_pbn_div ===============
[11:05:51] [PASSED] Link rate 2000000 lane count 4
[11:05:51] [PASSED] Link rate 2000000 lane count 2
[11:05:51] [PASSED] Link rate 2000000 lane count 1
[11:05:51] [PASSED] Link rate 1350000 lane count 4
[11:05:51] [PASSED] Link rate 1350000 lane count 2
[11:05:51] [PASSED] Link rate 1350000 lane count 1
[11:05:51] [PASSED] Link rate 1000000 lane count 4
[11:05:51] [PASSED] Link rate 1000000 lane count 2
[11:05:51] [PASSED] Link rate 1000000 lane count 1
[11:05:51] [PASSED] Link rate 810000 lane count 4
[11:05:51] [PASSED] Link rate 810000 lane count 2
[11:05:51] [PASSED] Link rate 810000 lane count 1
[11:05:51] [PASSED] Link rate 540000 lane count 4
[11:05:51] [PASSED] Link rate 540000 lane count 2
[11:05:51] [PASSED] Link rate 540000 lane count 1
[11:05:51] [PASSED] Link rate 270000 lane count 4
[11:05:51] [PASSED] Link rate 270000 lane count 2
[11:05:51] [PASSED] Link rate 270000 lane count 1
[11:05:51] [PASSED] Link rate 162000 lane count 4
[11:05:51] [PASSED] Link rate 162000 lane count 2
[11:05:51] [PASSED] Link rate 162000 lane count 1
[11:05:51] ========== [PASSED] drm_test_dp_mst_calc_pbn_div ===========
[11:05:51] ========= drm_test_dp_mst_sideband_msg_req_decode =========
[11:05:51] [PASSED] DP_ENUM_PATH_RESOURCES with port number
[11:05:51] [PASSED] DP_POWER_UP_PHY with port number
[11:05:51] [PASSED] DP_POWER_DOWN_PHY with port number
[11:05:51] [PASSED] DP_ALLOCATE_PAYLOAD with SDP stream sinks
[11:05:51] [PASSED] DP_ALLOCATE_PAYLOAD with port number
[11:05:51] [PASSED] DP_ALLOCATE_PAYLOAD with VCPI
[11:05:51] [PASSED] DP_ALLOCATE_PAYLOAD with PBN
[11:05:51] [PASSED] DP_QUERY_PAYLOAD with port number
[11:05:51] [PASSED] DP_QUERY_PAYLOAD with VCPI
[11:05:51] [PASSED] DP_REMOTE_DPCD_READ with port number
[11:05:51] [PASSED] DP_REMOTE_DPCD_READ with DPCD address
[11:05:51] [PASSED] DP_REMOTE_DPCD_READ with max number of bytes
[11:05:51] [PASSED] DP_REMOTE_DPCD_WRITE with port number
[11:05:51] [PASSED] DP_REMOTE_DPCD_WRITE with DPCD address
[11:05:51] [PASSED] DP_REMOTE_DPCD_WRITE with data array
[11:05:51] [PASSED] DP_REMOTE_I2C_READ with port number
[11:05:51] [PASSED] DP_REMOTE_I2C_READ with I2C device ID
[11:05:51] [PASSED] DP_REMOTE_I2C_READ with transactions array
[11:05:51] [PASSED] DP_REMOTE_I2C_WRITE with port number
[11:05:51] [PASSED] DP_REMOTE_I2C_WRITE with I2C device ID
[11:05:51] [PASSED] DP_REMOTE_I2C_WRITE with data array
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream ID
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with client ID
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream event
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with valid stream event
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with stream behavior
[11:05:51] [PASSED] DP_QUERY_STREAM_ENC_STATUS with a valid stream behavior
[11:05:51] ===== [PASSED] drm_test_dp_mst_sideband_msg_req_decode =====
[11:05:51] ================ [PASSED] drm_dp_mst_helper ================
[11:05:51] ================== drm_exec (7 subtests) ===================
[11:05:51] [PASSED] sanitycheck
[11:05:51] [PASSED] test_lock
[11:05:51] [PASSED] test_lock_unlock
[11:05:51] [PASSED] test_duplicates
[11:05:51] [PASSED] test_prepare
[11:05:51] [PASSED] test_prepare_array
[11:05:51] [PASSED] test_multiple_loops
[11:05:51] ==================== [PASSED] drm_exec =====================
[11:05:51] =========== drm_format_helper_test (17 subtests) ===========
[11:05:51] ============== drm_test_fb_xrgb8888_to_gray8 ==============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========== [PASSED] drm_test_fb_xrgb8888_to_gray8 ==========
[11:05:51] ============= drm_test_fb_xrgb8888_to_rgb332 ==============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb332 ==========
[11:05:51] ============= drm_test_fb_xrgb8888_to_rgb565 ==============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb565 ==========
[11:05:51] ============ drm_test_fb_xrgb8888_to_xrgb1555 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_xrgb1555 =========
[11:05:51] ============ drm_test_fb_xrgb8888_to_argb1555 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_argb1555 =========
[11:05:51] ============ drm_test_fb_xrgb8888_to_rgba5551 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_rgba5551 =========
[11:05:51] ============= drm_test_fb_xrgb8888_to_rgb888 ==============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========= [PASSED] drm_test_fb_xrgb8888_to_rgb888 ==========
[11:05:51] ============= drm_test_fb_xrgb8888_to_bgr888 ==============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========= [PASSED] drm_test_fb_xrgb8888_to_bgr888 ==========
[11:05:51] ============ drm_test_fb_xrgb8888_to_argb8888 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_argb8888 =========
[11:05:51] =========== drm_test_fb_xrgb8888_to_xrgb2101010 ===========
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======= [PASSED] drm_test_fb_xrgb8888_to_xrgb2101010 =======
[11:05:51] =========== drm_test_fb_xrgb8888_to_argb2101010 ===========
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======= [PASSED] drm_test_fb_xrgb8888_to_argb2101010 =======
[11:05:51] ============== drm_test_fb_xrgb8888_to_mono ===============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ========== [PASSED] drm_test_fb_xrgb8888_to_mono ===========
[11:05:51] ==================== drm_test_fb_swab =====================
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ================ [PASSED] drm_test_fb_swab =================
[11:05:51] ============ drm_test_fb_xrgb8888_to_xbgr8888 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_xbgr8888 =========
[11:05:51] ============ drm_test_fb_xrgb8888_to_abgr8888 =============
[11:05:51] [PASSED] single_pixel_source_buffer
[11:05:51] [PASSED] single_pixel_clip_rectangle
[11:05:51] [PASSED] well_known_colors
[11:05:51] [PASSED] destination_pitch
[11:05:51] ======== [PASSED] drm_test_fb_xrgb8888_to_abgr8888 =========
[11:05:51] ================= drm_test_fb_clip_offset =================
[11:05:51] [PASSED] pass through
[11:05:51] [PASSED] horizontal offset
[11:05:51] [PASSED] vertical offset
[11:05:51] [PASSED] horizontal and vertical offset
[11:05:51] [PASSED] horizontal offset (custom pitch)
[11:05:51] [PASSED] vertical offset (custom pitch)
[11:05:51] [PASSED] horizontal and vertical offset (custom pitch)
[11:05:51] ============= [PASSED] drm_test_fb_clip_offset =============
[11:05:51] =================== drm_test_fb_memcpy ====================
[11:05:51] [PASSED] single_pixel_source_buffer: XR24 little-endian (0x34325258)
[11:05:51] [PASSED] single_pixel_source_buffer: XRA8 little-endian (0x38415258)
[11:05:51] [PASSED] single_pixel_source_buffer: YU24 little-endian (0x34325559)
[11:05:51] [PASSED] single_pixel_clip_rectangle: XB24 little-endian (0x34324258)
[11:05:51] [PASSED] single_pixel_clip_rectangle: XRA8 little-endian (0x38415258)
[11:05:51] [PASSED] single_pixel_clip_rectangle: YU24 little-endian (0x34325559)
[11:05:51] [PASSED] well_known_colors: XB24 little-endian (0x34324258)
[11:05:51] [PASSED] well_known_colors: XRA8 little-endian (0x38415258)
[11:05:51] [PASSED] well_known_colors: YU24 little-endian (0x34325559)
[11:05:51] [PASSED] destination_pitch: XB24 little-endian (0x34324258)
[11:05:51] [PASSED] destination_pitch: XRA8 little-endian (0x38415258)
[11:05:51] [PASSED] destination_pitch: YU24 little-endian (0x34325559)
[11:05:51] =============== [PASSED] drm_test_fb_memcpy ================
[11:05:51] ============= [PASSED] drm_format_helper_test ==============
[11:05:51] ================= drm_format (18 subtests) =================
[11:05:51] [PASSED] drm_test_format_block_width_invalid
[11:05:51] [PASSED] drm_test_format_block_width_one_plane
[11:05:51] [PASSED] drm_test_format_block_width_two_plane
[11:05:51] [PASSED] drm_test_format_block_width_three_plane
[11:05:51] [PASSED] drm_test_format_block_width_tiled
[11:05:51] [PASSED] drm_test_format_block_height_invalid
[11:05:51] [PASSED] drm_test_format_block_height_one_plane
[11:05:51] [PASSED] drm_test_format_block_height_two_plane
[11:05:51] [PASSED] drm_test_format_block_height_three_plane
[11:05:51] [PASSED] drm_test_format_block_height_tiled
[11:05:51] [PASSED] drm_test_format_min_pitch_invalid
[11:05:51] [PASSED] drm_test_format_min_pitch_one_plane_8bpp
[11:05:51] [PASSED] drm_test_format_min_pitch_one_plane_16bpp
[11:05:51] [PASSED] drm_test_format_min_pitch_one_plane_24bpp
[11:05:51] [PASSED] drm_test_format_min_pitch_one_plane_32bpp
[11:05:51] [PASSED] drm_test_format_min_pitch_two_plane
[11:05:51] [PASSED] drm_test_format_min_pitch_three_plane_8bpp
[11:05:51] [PASSED] drm_test_format_min_pitch_tiled
[11:05:51] =================== [PASSED] drm_format ====================
[11:05:51] ============== drm_framebuffer (10 subtests) ===============
[11:05:51] ========== drm_test_framebuffer_check_src_coords ==========
[11:05:51] [PASSED] Success: source fits into fb
[11:05:51] [PASSED] Fail: overflowing fb with x-axis coordinate
[11:05:51] [PASSED] Fail: overflowing fb with y-axis coordinate
[11:05:51] [PASSED] Fail: overflowing fb with source width
[11:05:51] [PASSED] Fail: overflowing fb with source height
[11:05:51] ====== [PASSED] drm_test_framebuffer_check_src_coords ======
[11:05:51] [PASSED] drm_test_framebuffer_cleanup
[11:05:51] =============== drm_test_framebuffer_create ===============
[11:05:51] [PASSED] ABGR8888 normal sizes
[11:05:51] [PASSED] ABGR8888 max sizes
[11:05:51] [PASSED] ABGR8888 pitch greater than min required
[11:05:51] [PASSED] ABGR8888 pitch less than min required
[11:05:51] [PASSED] ABGR8888 Invalid width
[11:05:51] [PASSED] ABGR8888 Invalid buffer handle
[11:05:51] [PASSED] No pixel format
[11:05:51] [PASSED] ABGR8888 Width 0
[11:05:51] [PASSED] ABGR8888 Height 0
[11:05:51] [PASSED] ABGR8888 Out of bound height * pitch combination
[11:05:51] [PASSED] ABGR8888 Large buffer offset
[11:05:51] [PASSED] ABGR8888 Buffer offset for inexistent plane
[11:05:51] [PASSED] ABGR8888 Invalid flag
[11:05:51] [PASSED] ABGR8888 Set DRM_MODE_FB_MODIFIERS without modifiers
[11:05:51] [PASSED] ABGR8888 Valid buffer modifier
[11:05:51] [PASSED] ABGR8888 Invalid buffer modifier(DRM_FORMAT_MOD_SAMSUNG_64_32_TILE)
[11:05:51] [PASSED] ABGR8888 Extra pitches without DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] ABGR8888 Extra pitches with DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] NV12 Normal sizes
[11:05:51] [PASSED] NV12 Max sizes
[11:05:51] [PASSED] NV12 Invalid pitch
[11:05:51] [PASSED] NV12 Invalid modifier/missing DRM_MODE_FB_MODIFIERS flag
[11:05:51] [PASSED] NV12 different modifier per-plane
[11:05:51] [PASSED] NV12 with DRM_FORMAT_MOD_SAMSUNG_64_32_TILE
[11:05:51] [PASSED] NV12 Valid modifiers without DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] NV12 Modifier for inexistent plane
[11:05:51] [PASSED] NV12 Handle for inexistent plane
[11:05:51] [PASSED] NV12 Handle for inexistent plane without DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] YVU420 DRM_MODE_FB_MODIFIERS set without modifier
[11:05:51] [PASSED] YVU420 Normal sizes
[11:05:51] [PASSED] YVU420 Max sizes
[11:05:51] [PASSED] YVU420 Invalid pitch
[11:05:51] [PASSED] YVU420 Different pitches
[11:05:51] [PASSED] YVU420 Different buffer offsets/pitches
[11:05:51] [PASSED] YVU420 Modifier set just for plane 0, without DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] YVU420 Modifier set just for planes 0, 1, without DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] YVU420 Modifier set just for plane 0, 1, with DRM_MODE_FB_MODIFIERS
[11:05:51] [PASSED] YVU420 Valid modifier
[11:05:51] [PASSED] YVU420 Different modifiers per plane
[11:05:51] [PASSED] YVU420 Modifier for inexistent plane
[11:05:51] [PASSED] YUV420_10BIT Invalid modifier(DRM_FORMAT_MOD_LINEAR)
[11:05:51] [PASSED] X0L2 Normal sizes
[11:05:51] [PASSED] X0L2 Max sizes
[11:05:51] [PASSED] X0L2 Invalid pitch
[11:05:51] [PASSED] X0L2 Pitch greater than minimum required
[11:05:51] [PASSED] X0L2 Handle for inexistent plane
[11:05:51] [PASSED] X0L2 Offset for inexistent plane, without DRM_MODE_FB_MODIFIERS set
[11:05:51] [PASSED] X0L2 Modifier without DRM_MODE_FB_MODIFIERS set
[11:05:51] [PASSED] X0L2 Valid modifier
[11:05:51] [PASSED] X0L2 Modifier for inexistent plane
[11:05:51] =========== [PASSED] drm_test_framebuffer_create ===========
[11:05:51] [PASSED] drm_test_framebuffer_free
[11:05:51] [PASSED] drm_test_framebuffer_init
[11:05:51] [PASSED] drm_test_framebuffer_init_bad_format
[11:05:51] [PASSED] drm_test_framebuffer_init_dev_mismatch
[11:05:51] [PASSED] drm_test_framebuffer_lookup
[11:05:51] [PASSED] drm_test_framebuffer_lookup_inexistent
[11:05:51] [PASSED] drm_test_framebuffer_modifiers_not_supported
[11:05:51] ================= [PASSED] drm_framebuffer =================
[11:05:51] ================ drm_gem_shmem (8 subtests) ================
[11:05:51] [PASSED] drm_gem_shmem_test_obj_create
[11:05:51] [PASSED] drm_gem_shmem_test_obj_create_private
[11:05:51] [PASSED] drm_gem_shmem_test_pin_pages
[11:05:51] [PASSED] drm_gem_shmem_test_vmap
[11:05:51] [PASSED] drm_gem_shmem_test_get_sg_table
[11:05:51] [PASSED] drm_gem_shmem_test_get_pages_sgt
[11:05:51] [PASSED] drm_gem_shmem_test_madvise
[11:05:51] [PASSED] drm_gem_shmem_test_purge
[11:05:51] ================== [PASSED] drm_gem_shmem ==================
[11:05:51] === drm_atomic_helper_connector_hdmi_check (27 subtests) ===
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_auto_cea_mode_vic_1
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_full_cea_mode_vic_1
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_limited_cea_mode_vic_1
[11:05:51] ====== drm_test_check_broadcast_rgb_cea_mode_yuv420 =======
[11:05:51] [PASSED] Automatic
[11:05:51] [PASSED] Full
[11:05:51] [PASSED] Limited 16:235
[11:05:51] == [PASSED] drm_test_check_broadcast_rgb_cea_mode_yuv420 ===
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_changed
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_crtc_mode_not_changed
[11:05:51] [PASSED] drm_test_check_disable_connector
[11:05:51] [PASSED] drm_test_check_hdmi_funcs_reject_rate
[11:05:51] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_rgb
[11:05:51] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_yuv420
[11:05:51] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv422
[11:05:51] [PASSED] drm_test_check_max_tmds_rate_bpc_fallback_ignore_yuv420
[11:05:51] [PASSED] drm_test_check_driver_unsupported_fallback_yuv420
[11:05:51] [PASSED] drm_test_check_output_bpc_crtc_mode_changed
[11:05:51] [PASSED] drm_test_check_output_bpc_crtc_mode_not_changed
[11:05:51] [PASSED] drm_test_check_output_bpc_dvi
[11:05:51] [PASSED] drm_test_check_output_bpc_format_vic_1
[11:05:51] [PASSED] drm_test_check_output_bpc_format_display_8bpc_only
[11:05:51] [PASSED] drm_test_check_output_bpc_format_display_rgb_only
[11:05:51] [PASSED] drm_test_check_output_bpc_format_driver_8bpc_only
[11:05:51] [PASSED] drm_test_check_output_bpc_format_driver_rgb_only
[11:05:51] [PASSED] drm_test_check_tmds_char_rate_rgb_8bpc
[11:05:51] [PASSED] drm_test_check_tmds_char_rate_rgb_10bpc
[11:05:51] [PASSED] drm_test_check_tmds_char_rate_rgb_12bpc
[11:05:51] ===== [PASSED] drm_atomic_helper_connector_hdmi_check ======
[11:05:51] === drm_atomic_helper_connector_hdmi_reset (6 subtests) ====
[11:05:51] [PASSED] drm_test_check_broadcast_rgb_value
[11:05:51] [PASSED] drm_test_check_bpc_8_value
[11:05:51] [PASSED] drm_test_check_bpc_10_value
[11:05:51] [PASSED] drm_test_check_bpc_12_value
[11:05:51] [PASSED] drm_test_check_format_value
[11:05:51] [PASSED] drm_test_check_tmds_char_value
[11:05:51] ===== [PASSED] drm_atomic_helper_connector_hdmi_reset ======
[11:05:51] = drm_atomic_helper_connector_hdmi_mode_valid (4 subtests) =
[11:05:51] [PASSED] drm_test_check_mode_valid
[11:05:51] [PASSED] drm_test_check_mode_valid_reject
[11:05:51] [PASSED] drm_test_check_mode_valid_reject_rate
[11:05:51] [PASSED] drm_test_check_mode_valid_reject_max_clock
[11:05:51] === [PASSED] drm_atomic_helper_connector_hdmi_mode_valid ===
[11:05:51] = drm_atomic_helper_connector_hdmi_infoframes (5 subtests) =
[11:05:51] [PASSED] drm_test_check_infoframes
[11:05:51] [PASSED] drm_test_check_reject_avi_infoframe
[11:05:51] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_8
[11:05:51] [PASSED] drm_test_check_reject_hdr_infoframe_bpc_10
[11:05:51] [PASSED] drm_test_check_reject_audio_infoframe
[11:05:51] === [PASSED] drm_atomic_helper_connector_hdmi_infoframes ===
[11:05:51] ================= drm_managed (2 subtests) =================
[11:05:51] [PASSED] drm_test_managed_release_action
[11:05:51] [PASSED] drm_test_managed_run_action
[11:05:51] =================== [PASSED] drm_managed ===================
[11:05:51] =================== drm_mm (6 subtests) ====================
[11:05:51] [PASSED] drm_test_mm_init
[11:05:51] [PASSED] drm_test_mm_debug
[11:05:51] [PASSED] drm_test_mm_align32
[11:05:51] [PASSED] drm_test_mm_align64
[11:05:51] [PASSED] drm_test_mm_lowest
[11:05:51] [PASSED] drm_test_mm_highest
[11:05:51] ===================== [PASSED] drm_mm ======================
[11:05:51] ============= drm_modes_analog_tv (5 subtests) =============
[11:05:51] [PASSED] drm_test_modes_analog_tv_mono_576i
[11:05:51] [PASSED] drm_test_modes_analog_tv_ntsc_480i
[11:05:51] [PASSED] drm_test_modes_analog_tv_ntsc_480i_inlined
[11:05:51] [PASSED] drm_test_modes_analog_tv_pal_576i
[11:05:51] [PASSED] drm_test_modes_analog_tv_pal_576i_inlined
[11:05:51] =============== [PASSED] drm_modes_analog_tv ===============
[11:05:51] ============== drm_plane_helper (2 subtests) ===============
[11:05:51] =============== drm_test_check_plane_state ================
[11:05:51] [PASSED] clipping_simple
[11:05:51] [PASSED] clipping_rotate_reflect
[11:05:51] [PASSED] positioning_simple
[11:05:51] [PASSED] upscaling
[11:05:51] [PASSED] downscaling
[11:05:51] [PASSED] rounding1
[11:05:51] [PASSED] rounding2
[11:05:51] [PASSED] rounding3
[11:05:51] [PASSED] rounding4
[11:05:51] =========== [PASSED] drm_test_check_plane_state ============
[11:05:51] =========== drm_test_check_invalid_plane_state ============
[11:05:51] [PASSED] positioning_invalid
[11:05:51] [PASSED] upscaling_invalid
[11:05:51] [PASSED] downscaling_invalid
[11:05:51] ======= [PASSED] drm_test_check_invalid_plane_state ========
[11:05:51] ================ [PASSED] drm_plane_helper =================
[11:05:51] ====== drm_connector_helper_tv_get_modes (1 subtest) =======
[11:05:51] ====== drm_test_connector_helper_tv_get_modes_check =======
[11:05:51] [PASSED] None
[11:05:51] [PASSED] PAL
[11:05:51] [PASSED] NTSC
[11:05:51] [PASSED] Both, NTSC Default
[11:05:51] [PASSED] Both, PAL Default
[11:05:51] [PASSED] Both, NTSC Default, with PAL on command-line
[11:05:51] [PASSED] Both, PAL Default, with NTSC on command-line
[11:05:51] == [PASSED] drm_test_connector_helper_tv_get_modes_check ===
[11:05:51] ======== [PASSED] drm_connector_helper_tv_get_modes ========
[11:05:51] ================== drm_rect (9 subtests) ===================
[11:05:51] [PASSED] drm_test_rect_clip_scaled_div_by_zero
[11:05:51] [PASSED] drm_test_rect_clip_scaled_not_clipped
[11:05:51] [PASSED] drm_test_rect_clip_scaled_clipped
[11:05:51] [PASSED] drm_test_rect_clip_scaled_signed_vs_unsigned
[11:05:51] ================= drm_test_rect_intersect =================
[11:05:51] [PASSED] top-left x bottom-right: 2x2+1+1 x 2x2+0+0
[11:05:51] [PASSED] top-right x bottom-left: 2x2+0+0 x 2x2+1-1
[11:05:51] [PASSED] bottom-left x top-right: 2x2+1-1 x 2x2+0+0
[11:05:51] [PASSED] bottom-right x top-left: 2x2+0+0 x 2x2+1+1
[11:05:51] [PASSED] right x left: 2x1+0+0 x 3x1+1+0
[11:05:51] [PASSED] left x right: 3x1+1+0 x 2x1+0+0
[11:05:51] [PASSED] up x bottom: 1x2+0+0 x 1x3+0-1
[11:05:51] [PASSED] bottom x up: 1x3+0-1 x 1x2+0+0
[11:05:51] [PASSED] touching corner: 1x1+0+0 x 2x2+1+1
[11:05:51] [PASSED] touching side: 1x1+0+0 x 1x1+1+0
[11:05:51] [PASSED] equal rects: 2x2+0+0 x 2x2+0+0
[11:05:51] [PASSED] inside another: 2x2+0+0 x 1x1+1+1
[11:05:51] [PASSED] far away: 1x1+0+0 x 1x1+3+6
[11:05:51] [PASSED] points intersecting: 0x0+5+10 x 0x0+5+10
[11:05:51] [PASSED] points not intersecting: 0x0+0+0 x 0x0+5+10
[11:05:51] ============= [PASSED] drm_test_rect_intersect =============
[11:05:51] ================ drm_test_rect_calc_hscale ================
[11:05:51] [PASSED] normal use
[11:05:51] [PASSED] out of max range
[11:05:51] [PASSED] out of min range
[11:05:51] [PASSED] zero dst
[11:05:51] [PASSED] negative src
[11:05:51] [PASSED] negative dst
[11:05:51] ============ [PASSED] drm_test_rect_calc_hscale ============
[11:05:51] ================ drm_test_rect_calc_vscale ================
[11:05:51] [PASSED] normal use
[11:05:51] [PASSED] out of max range
[11:05:51] [PASSED] out of min range
[11:05:51] [PASSED] zero dst
[11:05:51] [PASSED] negative src
[11:05:51] [PASSED] negative dst
stty: 'standard input': Inappropriate ioctl for device
[11:05:51] ============ [PASSED] drm_test_rect_calc_vscale ============
[11:05:51] ================== drm_test_rect_rotate ===================
[11:05:51] [PASSED] reflect-x
[11:05:51] [PASSED] reflect-y
[11:05:51] [PASSED] rotate-0
[11:05:51] [PASSED] rotate-90
[11:05:51] [PASSED] rotate-180
[11:05:51] [PASSED] rotate-270
[11:05:51] ============== [PASSED] drm_test_rect_rotate ===============
[11:05:51] ================ drm_test_rect_rotate_inv =================
[11:05:51] [PASSED] reflect-x
[11:05:51] [PASSED] reflect-y
[11:05:51] [PASSED] rotate-0
[11:05:51] [PASSED] rotate-90
[11:05:51] [PASSED] rotate-180
[11:05:51] [PASSED] rotate-270
[11:05:51] ============ [PASSED] drm_test_rect_rotate_inv =============
[11:05:51] ==================== [PASSED] drm_rect =====================
[11:05:51] ============ drm_sysfb_modeset_test (1 subtest) ============
[11:05:51] ============ drm_test_sysfb_build_fourcc_list =============
[11:05:51] [PASSED] no native formats
[11:05:51] [PASSED] XRGB8888 as native format
[11:05:51] [PASSED] remove duplicates
[11:05:51] [PASSED] convert alpha formats
[11:05:51] [PASSED] random formats
[11:05:51] ======== [PASSED] drm_test_sysfb_build_fourcc_list =========
[11:05:51] ============= [PASSED] drm_sysfb_modeset_test ==============
[11:05:51] ================== drm_fixp (2 subtests) ===================
[11:05:51] [PASSED] drm_test_int2fixp
[11:05:51] [PASSED] drm_test_sm2fixp
[11:05:51] ==================== [PASSED] drm_fixp =====================
[11:05:51] ============================================================
[11:05:51] Testing complete. Ran 621 tests: passed: 621
[11:05:51] Elapsed time: 27.079s total, 1.648s configuring, 25.261s building, 0.137s running
+ /kernel/tools/testing/kunit/kunit.py run --kunitconfig /kernel/drivers/gpu/drm/ttm/tests/.kunitconfig
[11:05:52] Configuring KUnit Kernel ...
Regenerating .config ...
Populating config with:
$ make ARCH=um O=.kunit olddefconfig
[11:05:53] 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=48
[11:06:03] Starting KUnit Kernel (1/1)...
[11:06:03] ============================================================
Running tests with:
$ .kunit/linux kunit.enable=1 mem=1G console=tty kunit_shutdown=halt
[11:06:03] ================= ttm_device (5 subtests) ==================
[11:06:03] [PASSED] ttm_device_init_basic
[11:06:03] [PASSED] ttm_device_init_multiple
[11:06:03] [PASSED] ttm_device_fini_basic
[11:06:03] [PASSED] ttm_device_init_no_vma_man
[11:06:03] ================== ttm_device_init_pools ==================
[11:06:03] [PASSED] No DMA allocations, no DMA32 required
[11:06:03] [PASSED] DMA allocations, DMA32 required
[11:06:03] [PASSED] No DMA allocations, DMA32 required
[11:06:03] [PASSED] DMA allocations, no DMA32 required
[11:06:03] ============== [PASSED] ttm_device_init_pools ==============
[11:06:03] =================== [PASSED] ttm_device ====================
[11:06:03] ================== ttm_pool (8 subtests) ===================
[11:06:03] ================== ttm_pool_alloc_basic ===================
[11:06:03] [PASSED] One page
[11:06:03] [PASSED] More than one page
[11:06:03] [PASSED] Above the allocation limit
[11:06:03] [PASSED] One page, with coherent DMA mappings enabled
[11:06:03] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[11:06:03] ============== [PASSED] ttm_pool_alloc_basic ===============
[11:06:03] ============== ttm_pool_alloc_basic_dma_addr ==============
[11:06:03] [PASSED] One page
[11:06:03] [PASSED] More than one page
[11:06:03] [PASSED] Above the allocation limit
[11:06:03] [PASSED] One page, with coherent DMA mappings enabled
[11:06:03] [PASSED] Above the allocation limit, with coherent DMA mappings enabled
[11:06:03] ========== [PASSED] ttm_pool_alloc_basic_dma_addr ==========
[11:06:03] [PASSED] ttm_pool_alloc_order_caching_match
[11:06:03] [PASSED] ttm_pool_alloc_caching_mismatch
[11:06:03] [PASSED] ttm_pool_alloc_order_mismatch
[11:06:03] [PASSED] ttm_pool_free_dma_alloc
[11:06:03] [PASSED] ttm_pool_free_no_dma_alloc
[11:06:03] [PASSED] ttm_pool_fini_basic
[11:06:03] ==================== [PASSED] ttm_pool =====================
[11:06:03] ================ ttm_resource (8 subtests) =================
[11:06:03] ================= ttm_resource_init_basic =================
[11:06:03] [PASSED] Init resource in TTM_PL_SYSTEM
[11:06:03] [PASSED] Init resource in TTM_PL_VRAM
[11:06:03] [PASSED] Init resource in a private placement
[11:06:03] [PASSED] Init resource in TTM_PL_SYSTEM, set placement flags
[11:06:03] ============= [PASSED] ttm_resource_init_basic =============
[11:06:03] [PASSED] ttm_resource_init_pinned
[11:06:03] [PASSED] ttm_resource_fini_basic
[11:06:03] [PASSED] ttm_resource_manager_init_basic
[11:06:03] [PASSED] ttm_resource_manager_usage_basic
[11:06:03] [PASSED] ttm_resource_manager_set_used_basic
[11:06:03] [PASSED] ttm_sys_man_alloc_basic
[11:06:03] [PASSED] ttm_sys_man_free_basic
[11:06:03] ================== [PASSED] ttm_resource ===================
[11:06:03] =================== ttm_tt (15 subtests) ===================
[11:06:03] ==================== ttm_tt_init_basic ====================
[11:06:03] [PASSED] Page-aligned size
[11:06:03] [PASSED] Extra pages requested
[11:06:03] ================ [PASSED] ttm_tt_init_basic ================
[11:06:03] [PASSED] ttm_tt_init_misaligned
[11:06:03] [PASSED] ttm_tt_fini_basic
[11:06:03] [PASSED] ttm_tt_fini_sg
[11:06:03] [PASSED] ttm_tt_fini_shmem
[11:06:03] [PASSED] ttm_tt_create_basic
[11:06:03] [PASSED] ttm_tt_create_invalid_bo_type
[11:06:03] [PASSED] ttm_tt_create_ttm_exists
[11:06:03] [PASSED] ttm_tt_create_failed
[11:06:03] [PASSED] ttm_tt_destroy_basic
[11:06:03] [PASSED] ttm_tt_populate_null_ttm
[11:06:03] [PASSED] ttm_tt_populate_populated_ttm
[11:06:03] [PASSED] ttm_tt_unpopulate_basic
[11:06:03] [PASSED] ttm_tt_unpopulate_empty_ttm
[11:06:03] [PASSED] ttm_tt_swapin_basic
[11:06:03] ===================== [PASSED] ttm_tt ======================
[11:06:03] =================== ttm_bo (14 subtests) ===================
[11:06:03] =========== ttm_bo_reserve_optimistic_no_ticket ===========
[11:06:03] [PASSED] Cannot be interrupted and sleeps
[11:06:03] [PASSED] Cannot be interrupted, locks straight away
[11:06:03] [PASSED] Can be interrupted, sleeps
[11:06:03] ======= [PASSED] ttm_bo_reserve_optimistic_no_ticket =======
[11:06:03] [PASSED] ttm_bo_reserve_locked_no_sleep
[11:06:03] [PASSED] ttm_bo_reserve_no_wait_ticket
[11:06:03] [PASSED] ttm_bo_reserve_double_resv
[11:06:03] [PASSED] ttm_bo_reserve_interrupted
[11:06:03] [PASSED] ttm_bo_reserve_deadlock
[11:06:03] [PASSED] ttm_bo_unreserve_basic
[11:06:03] [PASSED] ttm_bo_unreserve_pinned
[11:06:03] [PASSED] ttm_bo_unreserve_bulk
[11:06:03] [PASSED] ttm_bo_fini_basic
[11:06:03] [PASSED] ttm_bo_fini_shared_resv
[11:06:03] [PASSED] ttm_bo_pin_basic
[11:06:03] [PASSED] ttm_bo_pin_unpin_resource
[11:06:03] [PASSED] ttm_bo_multiple_pin_one_unpin
[11:06:03] ===================== [PASSED] ttm_bo ======================
[11:06:03] ============== ttm_bo_validate (21 subtests) ===============
[11:06:03] ============== ttm_bo_init_reserved_sys_man ===============
[11:06:03] [PASSED] Buffer object for userspace
[11:06:03] [PASSED] Kernel buffer object
[11:06:03] [PASSED] Shared buffer object
[11:06:03] ========== [PASSED] ttm_bo_init_reserved_sys_man ===========
[11:06:03] ============== ttm_bo_init_reserved_mock_man ==============
[11:06:03] [PASSED] Buffer object for userspace
[11:06:03] [PASSED] Kernel buffer object
[11:06:03] [PASSED] Shared buffer object
[11:06:03] ========== [PASSED] ttm_bo_init_reserved_mock_man ==========
[11:06:03] [PASSED] ttm_bo_init_reserved_resv
[11:06:03] ================== ttm_bo_validate_basic ==================
[11:06:03] [PASSED] Buffer object for userspace
[11:06:03] [PASSED] Kernel buffer object
[11:06:03] [PASSED] Shared buffer object
[11:06:03] ============== [PASSED] ttm_bo_validate_basic ==============
[11:06:03] [PASSED] ttm_bo_validate_invalid_placement
[11:06:03] ============= ttm_bo_validate_same_placement ==============
[11:06:03] [PASSED] System manager
[11:06:03] [PASSED] VRAM manager
[11:06:03] ========= [PASSED] ttm_bo_validate_same_placement ==========
[11:06:03] [PASSED] ttm_bo_validate_failed_alloc
[11:06:03] [PASSED] ttm_bo_validate_pinned
[11:06:03] [PASSED] ttm_bo_validate_busy_placement
[11:06:03] ================ ttm_bo_validate_multihop =================
[11:06:03] [PASSED] Buffer object for userspace
[11:06:03] [PASSED] Kernel buffer object
[11:06:03] [PASSED] Shared buffer object
[11:06:03] ============ [PASSED] ttm_bo_validate_multihop =============
[11:06:03] ========== ttm_bo_validate_no_placement_signaled ==========
[11:06:03] [PASSED] Buffer object in system domain, no page vector
[11:06:03] [PASSED] Buffer object in system domain with an existing page vector
[11:06:03] ====== [PASSED] ttm_bo_validate_no_placement_signaled ======
[11:06:03] ======== ttm_bo_validate_no_placement_not_signaled ========
[11:06:03] [PASSED] Buffer object for userspace
[11:06:03] [PASSED] Kernel buffer object
[11:06:03] [PASSED] Shared buffer object
[11:06:03] ==== [PASSED] ttm_bo_validate_no_placement_not_signaled ====
[11:06:03] [PASSED] ttm_bo_validate_move_fence_signaled
[11:06:03] ========= ttm_bo_validate_move_fence_not_signaled =========
[11:06:03] [PASSED] Waits for GPU
[11:06:03] [PASSED] Tries to lock straight away
[11:06:03] ===== [PASSED] ttm_bo_validate_move_fence_not_signaled =====
[11:06:03] [PASSED] ttm_bo_validate_happy_evict
[11:06:03] [PASSED] ttm_bo_validate_all_pinned_evict
[11:06:03] [PASSED] ttm_bo_validate_allowed_only_evict
[11:06:03] [PASSED] ttm_bo_validate_deleted_evict
[11:06:03] [PASSED] ttm_bo_validate_busy_domain_evict
[11:06:03] [PASSED] ttm_bo_validate_evict_gutting
[11:06:03] [PASSED] ttm_bo_validate_recrusive_evict
stty: 'standard input': Inappropriate ioctl for device
[11:06:03] ================= [PASSED] ttm_bo_validate =================
[11:06:03] ============================================================
[11:06:03] Testing complete. Ran 101 tests: passed: 101
[11:06:03] Elapsed time: 11.488s total, 1.666s configuring, 9.606s building, 0.186s running
+ cleanup
++ stat -c %u:%g /kernel
+ chown -R 1003:1003 /kernel
^ permalink raw reply [flat|nested] 12+ messages in thread
* ✓ Xe.CI.BAT: success for Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
` (3 preceding siblings ...)
2026-02-10 11:06 ` ✓ CI.KUnit: success for Fix fs_reclaim deadlock caused by CCS save/restore (rev3) Patchwork
@ 2026-02-10 11:59 ` Patchwork
2026-02-10 14:29 ` ✗ Xe.CI.FULL: failure " Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2026-02-10 11:59 UTC (permalink / raw)
To: Satyanarayana K V P; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 880 bytes --]
== Series Details ==
Series: Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
URL : https://patchwork.freedesktop.org/series/160841/
State : success
== Summary ==
CI Bug Log - changes from xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061_BAT -> xe-pw-160841v3_BAT
====================================================
Summary
-------
**SUCCESS**
No regressions found.
Participating hosts (12 -> 12)
------------------------------
No changes in participating hosts
Changes
-------
No changes found
Build changes
-------------
* Linux: xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061 -> xe-pw-160841v3
IGT_8746: 8746
xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061: 4963fa04cc7c2c68030bef5889b6210052f59061
xe-pw-160841v3: 160841v3
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/index.html
[-- Attachment #2: Type: text/html, Size: 1428 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
@ 2026-02-10 12:09 ` Thomas Hellström
2026-02-11 13:32 ` Christian König
2026-02-11 0:20 ` Matthew Brost
1 sibling, 1 reply; 12+ messages in thread
From: Thomas Hellström @ 2026-02-10 12:09 UTC (permalink / raw)
To: Satyanarayana K V P, intel-xe
Cc: Matthew Brost, Michal Wajdeczko, Matthew Auld,
Christian König, dri-devel
On Tue, 2026-02-10 at 10:59 +0000, Satyanarayana K V P wrote:
> drm_suballoc_new() currently both allocates the SA object using
> kmalloc()
> and searches for a suitable hole in the sub-allocator for the
> requested
> size. If SA allocation is done by holding sub-allocator mutex, this
> design
> can lead to reclaim safety issues.
>
> By splitting the kmalloc() step outside of the critical section, we
> allow
> the memory allocation to use GFP_KERNEL (reclaim-safe) while ensuring
> that
> the initialization step that holds reclaim-tainted locks (sub-
> allocator
> mutex) operates in a reclaim-unsafe context with pre-allocated
> memory.
>
> This separation prevents potential deadlocks where memory reclaim
> could
> attempt to acquire locks that are already held during the sub-
> allocator
> operations.
>
> Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
> Suggested-by: Matthew Brost <matthew.brost@intel.com>
> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Matthew Auld <matthew.auld@intel.com>
> Cc: Christian König <christian.koenig@amd.com>
> Cc: dri-devel@lists.freedesktop.org
LGTM.
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>
> ---
> V2 -> V3:
> - Updated commit message (Matt, Thomas & Christian).
> - Removed timeout logic from drm_suballoc_init(). (Thomas &
> Christian).
>
> V1 -> V2:
> - Splitted drm_suballoc_new() into drm_suballoc_alloc() and
> drm_suballoc_init() (Thomas).
> ---
> drivers/gpu/drm/drm_suballoc.c | 110 ++++++++++++++++++++++++++-----
> --
> include/drm/drm_suballoc.h | 8 +++
> 2 files changed, 97 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_suballoc.c
> b/drivers/gpu/drm/drm_suballoc.c
> index 879ea33dbbc4..b97ffcd98d45 100644
> --- a/drivers/gpu/drm/drm_suballoc.c
> +++ b/drivers/gpu/drm/drm_suballoc.c
> @@ -123,7 +123,7 @@ static void drm_suballoc_remove_locked(struct
> drm_suballoc *sa)
> list_del_init(&sa->olist);
> list_del_init(&sa->flist);
> dma_fence_put(sa->fence);
> - kfree(sa);
> + drm_suballoc_release(sa);
> }
>
> static void drm_suballoc_try_free(struct drm_suballoc_manager
> *sa_manager)
> @@ -293,45 +293,74 @@ static bool drm_suballoc_next_hole(struct
> drm_suballoc_manager *sa_manager,
> }
>
> /**
> - * drm_suballoc_new() - Make a suballocation.
> + * drm_suballoc_alloc() - Allocate uninitialized suballoc object.
> + * @gfp: gfp flags used for memory allocation.
> + *
> + * Allocate memory for an uninitialized suballoc object. Intended
> usage is
> + * allocate memory for suballoc object outside of a reclaim tainted
> context
> + * and then be initialized at a later time in a reclaim tainted
> context.
> + *
> + * @drm_suballoc_release should be used to release the memory if
> returned
> + * suballoc object is in uninitialized state.
> + *
> + * Return: a new uninitialized suballoc object, or an ERR_PTR(-
> ENOMEM).
> + */
> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp)
> +{
> + struct drm_suballoc *sa;
> +
> + sa = kmalloc(sizeof(*sa), gfp);
> + if (!sa)
> + return ERR_PTR(-ENOMEM);
> +
> + return sa;
> +}
> +EXPORT_SYMBOL(drm_suballoc_alloc);
> +
> +/**
> + * drm_suballoc_release() - Release memory for suballocation.
> + * @sa: The struct drm_suballoc.
> + */
> +void drm_suballoc_release(struct drm_suballoc *sa)
> +{
> + kfree(sa);
> +}
> +EXPORT_SYMBOL(drm_suballoc_release);
> +
> +/**
> + * drm_suballoc_init() - Initialize a suballocation.
> * @sa_manager: pointer to the sa_manager
> + * @sa: The struct drm_suballoc.
> * @size: number of bytes we want to suballocate.
> - * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL
> but
> - * the argument is provided for suballocations from reclaim
> context or
> - * where the caller wants to avoid pipelining rather than wait
> for
> - * reclaim.
> * @intr: Whether to perform waits interruptible. This should
> typically
> * always be true, unless the caller needs to propagate a
> * non-interruptible context from above layers.
> * @align: Alignment. Must not exceed the default manager alignment.
> * If @align is zero, then the manager alignment is used.
> *
> - * Try to make a suballocation of size @size, which will be rounded
> - * up to the alignment specified in specified in
> drm_suballoc_manager_init().
> + * Try to make a suballocation on a pre-allocated suballoc object of
> size @size,
> + * which will be rounded up to the alignment specified in specified
> in
> + * drm_suballoc_manager_init().
> *
> - * Return: a new suballocated bo, or an ERR_PTR.
> + * Return: zero on success, errno on failure.
> */
> -struct drm_suballoc *
> -drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
> size,
> - gfp_t gfp, bool intr, size_t align)
> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
> + struct drm_suballoc *sa, size_t size,
> + bool intr, size_t align)
> {
> struct dma_fence *fences[DRM_SUBALLOC_MAX_QUEUES];
> unsigned int tries[DRM_SUBALLOC_MAX_QUEUES];
> unsigned int count;
> int i, r;
> - struct drm_suballoc *sa;
>
> if (WARN_ON_ONCE(align > sa_manager->align))
> - return ERR_PTR(-EINVAL);
> + return -EINVAL;
> if (WARN_ON_ONCE(size > sa_manager->size || !size))
> - return ERR_PTR(-EINVAL);
> + return -EINVAL;
>
> if (!align)
> align = sa_manager->align;
>
> - sa = kmalloc(sizeof(*sa), gfp);
> - if (!sa)
> - return ERR_PTR(-ENOMEM);
> sa->manager = sa_manager;
> sa->fence = NULL;
> INIT_LIST_HEAD(&sa->olist);
> @@ -348,7 +377,7 @@ drm_suballoc_new(struct drm_suballoc_manager
> *sa_manager, size_t size,
> if (drm_suballoc_try_alloc(sa_manager, sa,
> size, align)) {
> spin_unlock(&sa_manager->wq.lock);
> - return sa;
> + return 0;
> }
>
> /* see if we can skip over some allocations
> */
> @@ -385,8 +414,47 @@ drm_suballoc_new(struct drm_suballoc_manager
> *sa_manager, size_t size,
> } while (!r);
>
> spin_unlock(&sa_manager->wq.lock);
> - kfree(sa);
> - return ERR_PTR(r);
> + return r;
> +}
> +EXPORT_SYMBOL(drm_suballoc_init);
> +
> +/**
> + * drm_suballoc_new() - Make a suballocation.
> + * @sa_manager: pointer to the sa_manager
> + * @size: number of bytes we want to suballocate.
> + * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL
> but
> + * the argument is provided for suballocations from reclaim
> context or
> + * where the caller wants to avoid pipelining rather than wait
> for
> + * reclaim.
> + * @intr: Whether to perform waits interruptible. This should
> typically
> + * always be true, unless the caller needs to propagate a
> + * non-interruptible context from above layers.
> + * @align: Alignment. Must not exceed the default manager alignment.
> + * If @align is zero, then the manager alignment is used.
> + *
> + * Try to make a suballocation of size @size, which will be rounded
> + * up to the alignment specified in specified in
> drm_suballoc_manager_init().
> + *
> + * Return: a new suballocated bo, or an ERR_PTR.
> + */
> +struct drm_suballoc *
> +drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
> size,
> + gfp_t gfp, bool intr, size_t align)
> +{
> + struct drm_suballoc *sa;
> + int err;
> +
> + sa = drm_suballoc_alloc(gfp);
> + if (IS_ERR(sa))
> + return sa;
> +
> + err = drm_suballoc_init(sa_manager, sa, size, intr, align);
> + if (err) {
> + drm_suballoc_release(sa);
> + return ERR_PTR(err);
> + }
> +
> + return sa;
> }
> EXPORT_SYMBOL(drm_suballoc_new);
>
> diff --git a/include/drm/drm_suballoc.h b/include/drm/drm_suballoc.h
> index 7ba72a81a808..b8d1d5449fd8 100644
> --- a/include/drm/drm_suballoc.h
> +++ b/include/drm/drm_suballoc.h
> @@ -53,6 +53,14 @@ void drm_suballoc_manager_init(struct
> drm_suballoc_manager *sa_manager,
>
> void drm_suballoc_manager_fini(struct drm_suballoc_manager
> *sa_manager);
>
> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp);
> +
> +void drm_suballoc_release(struct drm_suballoc *sa);
> +
> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
> + struct drm_suballoc *sa, size_t size, bool
> intr,
> + size_t align);
> +
> struct drm_suballoc *
> drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
> size,
> gfp_t gfp, bool intr, size_t align);
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation
2026-02-10 10:59 ` [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation Satyanarayana K V P
@ 2026-02-10 13:02 ` Michal Wajdeczko
2026-02-10 14:01 ` Thomas Hellström
0 siblings, 1 reply; 12+ messages in thread
From: Michal Wajdeczko @ 2026-02-10 13:02 UTC (permalink / raw)
To: Satyanarayana K V P, intel-xe
Cc: Matthew Brost, Matthew Auld, Thomas Hellström
On 2/10/2026 11:59 AM, Satyanarayana K V P wrote:
> CCS save/restore batch buffers are attached during BO allocation and
> detached during BO teardown. The shrinker triggers xe_bo_move(), which is
> used for both allocation and deletion paths.
>
> When BO allocation and shrinking occur concurrently, a circular locking
> dependency involving fs_reclaim and swap_guard can occur, leading to a
> deadlock such as:
>
> ======================================================
> WARNING: possible circular locking dependency detected
> ------------------------------------------------------
>
> CPU0 CPU1
> ---- ----
> lock(fs_reclaim);
> lock(&sa_manager->swap_guard);
> lock(fs_reclaim);
> lock(&sa_manager->swap_guard);
>
> *** DEADLOCK ***
> =====================================================
>
> To avoid this, the BB pointer and SA are allocated using xe_bb_alloc()
> before taking lock and SA is initialized using xe_bb_init() preventing
> reclaim from being invoked in this context.
>
> Fixes: 864690cf4dd62 ("drm/xe/vf: Attach and detach CCS copy commands with BO")
> Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
> Cc: Matthew Brost <matthew.brost@intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Matthew Auld <matthew.auld@intel.com>
> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>
> ---
> V2 -> V3:
> - Created new functions xe_sa_bo_alloc(), xe_sa_bo_release() and
> xe_sa_bo_init(). (Thomas)
> - Created new functions xe_bb_alloc(), xe_bb_release and
> xe_bb_init(). (Thomas)
> - Updated guard() to scoped_guard() in xe_migrate_ccs_rw_copy(). (Thomas)
>
> V1 -> V2:
> - Used drm_suballoc_alloc() and drm_suballoc_init() for BB allocation
> (Thomas).
> ---
> drivers/gpu/drm/xe/xe_bb.c | 49 +++++++++++------
> drivers/gpu/drm/xe/xe_bb.h | 7 ++-
> drivers/gpu/drm/xe/xe_migrate.c | 96 ++++++++++++++++++---------------
> drivers/gpu/drm/xe/xe_sa.c | 40 ++++++++++++++
> drivers/gpu/drm/xe/xe_sa.h | 3 ++
> 5 files changed, 135 insertions(+), 60 deletions(-)
>
> diff --git a/drivers/gpu/drm/xe/xe_bb.c b/drivers/gpu/drm/xe/xe_bb.c
> index 8b678297aaa2..631ae564e719 100644
> --- a/drivers/gpu/drm/xe/xe_bb.c
> +++ b/drivers/gpu/drm/xe/xe_bb.c
> @@ -59,16 +59,43 @@ struct xe_bb *xe_bb_new(struct xe_gt *gt, u32 dwords, bool usm)
> return ERR_PTR(err);
> }
>
> -struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
> - enum xe_sriov_vf_ccs_rw_ctxs ctx_id)
shouldn't we add kernel-doc for all new/updated public functions?
> +struct xe_bb *xe_bb_alloc(struct xe_gt *gt)
> {
> struct xe_bb *bb = kmalloc(sizeof(*bb), GFP_KERNEL);
> struct xe_device *xe = gt_to_xe(gt);
> - struct xe_sa_manager *bb_pool;
> int err;
>
> if (!bb)
> return ERR_PTR(-ENOMEM);
> +
> + bb->bo = xe_sa_bo_alloc(GFP_KERNEL);
> + if (IS_ERR(bb->bo)) {
> + drm_err(&xe->drm, "Sub-allocator memory allocation failed with %ld\n",
nit: there is xe_err(xe, ...)
nit: we try to print errors using %pe, which here is even more appropriate
nit: maybe "Failed to allocate SA object for BO (%pe)\n" ?
nit: or maybe any error message should be in xe_sa_bo_alloc()
> + PTR_ERR(bb->bo));
> + err = PTR_ERR(bb->bo);
> + goto err;
> + }
> +
> + return bb;
> +
> +err:
> + kfree(bb);
> + return ERR_PTR(err);
> +}
> +
kernel-doc
> +void xe_bb_release(struct xe_bb *bb)
> +{
> + if (bb->bo)
do we need this?
in xe_bb_alloc() we guarantee that bb will either have valid .bo or no bb at all
> + xe_sa_bo_release(bb->bo);
> +
> + kfree(bb);
> +}
> +
kernel-doc
> +int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
as this is xe_bb function, it should take xe_bb as first param
> + struct xe_sa_manager *bb_pool, u32 dwords)
> +{
> + int err;
> +
> /*
> * We need to allocate space for the requested number of dwords &
> * one additional MI_BATCH_BUFFER_END dword. Since the whole SA
> @@ -76,22 +103,14 @@ struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
> * is not over written when the last chunk of SA is allocated for BB.
> * So, this extra DW acts as a guard here.
> */
> -
> - bb_pool = xe->sriov.vf.ccs.contexts[ctx_id].mem.ccs_bb_pool;
> - bb->bo = xe_sa_bo_new(bb_pool, 4 * (dwords + 1));
> -
> - if (IS_ERR(bb->bo)) {
> - err = PTR_ERR(bb->bo);
> - goto err;
> - }
> + err = xe_sa_bo_init(bb_pool, bb->bo, 4 * (dwords + 1));
> + if (err)
> + return err;
>
> bb->cs = xe_sa_bo_cpu_addr(bb->bo);
> bb->len = 0;
>
> - return bb;
> -err:
> - kfree(bb);
> - return ERR_PTR(err);
> + return 0;
> }
>
> static struct xe_sched_job *
> diff --git a/drivers/gpu/drm/xe/xe_bb.h b/drivers/gpu/drm/xe/xe_bb.h
> index 2a8adc9a6dee..3eb80925bfd1 100644
> --- a/drivers/gpu/drm/xe/xe_bb.h
> +++ b/drivers/gpu/drm/xe/xe_bb.h
> @@ -13,11 +13,14 @@ struct dma_fence;
> struct xe_gt;
> struct xe_exec_queue;
> struct xe_sched_job;
> +struct xe_sa_manager;
wrong order
> enum xe_sriov_vf_ccs_rw_ctxs;
>
> struct xe_bb *xe_bb_new(struct xe_gt *gt, u32 dwords, bool usm);
> -struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
> - enum xe_sriov_vf_ccs_rw_ctxs ctx_id);
> +struct xe_bb *xe_bb_alloc(struct xe_gt *gt);
> +void xe_bb_release(struct xe_bb *bb);
> +int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
> + struct xe_sa_manager *bb_pool, u32 dwords);
> struct xe_sched_job *xe_bb_create_job(struct xe_exec_queue *q,
> struct xe_bb *bb);
> struct xe_sched_job *xe_bb_create_migration_job(struct xe_exec_queue *q,
> diff --git a/drivers/gpu/drm/xe/xe_migrate.c b/drivers/gpu/drm/xe/xe_migrate.c
> index 078a9bc2821d..c858eaa70e3e 100644
> --- a/drivers/gpu/drm/xe/xe_migrate.c
> +++ b/drivers/gpu/drm/xe/xe_migrate.c
> @@ -1148,65 +1148,75 @@ int xe_migrate_ccs_rw_copy(struct xe_tile *tile, struct xe_exec_queue *q,
> size -= src_L0;
> }
>
> - bb_pool = ctx->mem.ccs_bb_pool;
> - guard(mutex) (xe_sa_bo_swap_guard(bb_pool));
> - xe_sa_bo_swap_shadow(bb_pool);
> -
> - bb = xe_bb_ccs_new(gt, batch_size, read_write);
> + bb = xe_bb_alloc(gt);
> if (IS_ERR(bb)) {
> - drm_err(&xe->drm, "BB allocation failed.\n");
> err = PTR_ERR(bb);
> return err;
nit: this could be
return PTR_ERR(bb);
> }
>
> - batch_size_allocated = batch_size;
> - size = xe_bo_size(src_bo);
> - batch_size = 0;
> + bb_pool = ctx->mem.ccs_bb_pool;
> + scoped_guard(mutex, xe_sa_bo_swap_guard(bb_pool)) {
> + xe_sa_bo_swap_shadow(bb_pool);
> +
> + err = xe_bb_init(gt, bb, bb_pool, batch_size);
> + if (err) {
> + drm_err(&xe->drm, "BB allocation failed.\n");
nit: there is xe_err() but since there is a gt maybe it should be xe_gt_err() ?
nit: or maybe move that message to xe_bb_init() ?
> + xe_bb_release(bb);
> + return err;
> + }
>
> - /*
> - * Emit PTE and copy commands here.
> - * The CCS copy command can only support limited size. If the size to be
> - * copied is more than the limit, divide copy into chunks. So, calculate
> - * sizes here again before copy command is emitted.
> - */
> - while (size) {
> - batch_size += 10; /* Flush + ggtt addr + 2 NOP */
> - u32 flush_flags = 0;
> - u64 ccs_ofs, ccs_size;
> - u32 ccs_pt;
> + batch_size_allocated = batch_size;
> + size = xe_bo_size(src_bo);
> + batch_size = 0;
>
> - u32 avail_pts = max_mem_transfer_per_pass(xe) / LEVEL0_PAGE_TABLE_ENCODE_SIZE;
> + /*
> + * Emit PTE and copy commands here.
> + * The CCS copy command can only support limited size. If the size to be
> + * copied is more than the limit, divide copy into chunks. So, calculate
> + * sizes here again before copy command is emitted.
> + */
>
> - src_L0 = xe_migrate_res_sizes(m, &src_it);
> + while (size) {
> + batch_size += 10; /* Flush + ggtt addr + 2 NOP */
> + u32 flush_flags = 0;
> + u64 ccs_ofs, ccs_size;
> + u32 ccs_pt;
>
> - batch_size += pte_update_size(m, false, src, &src_it, &src_L0,
> - &src_L0_ofs, &src_L0_pt, 0, 0,
> - avail_pts);
> + u32 avail_pts = max_mem_transfer_per_pass(xe) /
> + LEVEL0_PAGE_TABLE_ENCODE_SIZE;
>
> - ccs_size = xe_device_ccs_bytes(xe, src_L0);
> - batch_size += pte_update_size(m, 0, NULL, &ccs_it, &ccs_size, &ccs_ofs,
> - &ccs_pt, 0, avail_pts, avail_pts);
> - xe_assert(xe, IS_ALIGNED(ccs_it.start, PAGE_SIZE));
> - batch_size += EMIT_COPY_CCS_DW;
> + src_L0 = xe_migrate_res_sizes(m, &src_it);
> +
> + batch_size += pte_update_size(m, false, src, &src_it, &src_L0,
> + &src_L0_ofs, &src_L0_pt, 0, 0,
> + avail_pts);
> +
> + ccs_size = xe_device_ccs_bytes(xe, src_L0);
> + batch_size += pte_update_size(m, 0, NULL, &ccs_it, &ccs_size, &ccs_ofs,
> + &ccs_pt, 0, avail_pts, avail_pts);
> + xe_assert(xe, IS_ALIGNED(ccs_it.start, PAGE_SIZE));
> + batch_size += EMIT_COPY_CCS_DW;
>
> - emit_pte(m, bb, src_L0_pt, false, true, &src_it, src_L0, src);
> + emit_pte(m, bb, src_L0_pt, false, true, &src_it, src_L0, src);
>
> - emit_pte(m, bb, ccs_pt, false, false, &ccs_it, ccs_size, src);
> + emit_pte(m, bb, ccs_pt, false, false, &ccs_it, ccs_size, src);
>
> - bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
> - flush_flags = xe_migrate_ccs_copy(m, bb, src_L0_ofs, src_is_pltt,
> - src_L0_ofs, dst_is_pltt,
> - src_L0, ccs_ofs, true);
> - bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
> + bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
> + flush_flags = xe_migrate_ccs_copy(m, bb, src_L0_ofs, src_is_pltt,
> + src_L0_ofs, dst_is_pltt,
> + src_L0, ccs_ofs, true);
> + bb->len = emit_flush_invalidate(bb->cs, bb->len, flush_flags);
>
> - size -= src_L0;
> - }
> + size -= src_L0;
> + }
>
> - xe_assert(xe, (batch_size_allocated == bb->len));
> - src_bo->bb_ccs[read_write] = bb;
> + xe_assert(xe, (batch_size_allocated == bb->len));
> + src_bo->bb_ccs[read_write] = bb;
> +
> + xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
> + xe_sa_bo_sync_shadow(bb->bo);
> + }
>
> - xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
> - xe_sa_bo_sync_shadow(bb->bo);
> return 0;
> }
>
> diff --git a/drivers/gpu/drm/xe/xe_sa.c b/drivers/gpu/drm/xe/xe_sa.c
> index b738102575d4..59d0187b3e82 100644
> --- a/drivers/gpu/drm/xe/xe_sa.c
> +++ b/drivers/gpu/drm/xe/xe_sa.c
> @@ -175,6 +175,46 @@ struct drm_suballoc *__xe_sa_bo_new(struct xe_sa_manager *sa_manager, u32 size,
> return drm_suballoc_new(&sa_manager->base, size, gfp, true, 0);
> }
>
> +/**
> + * xe_sa_bo_alloc - Allocate uninitialized suballoc object.
> + * @gfp: gfp flags used for memory allocation.
> + *
> + * Allocate memory for an uninitialized suballoc object. Intended usage is
> + * allocate memory for suballoc object outside of a reclaim tainted context
> + * and then be initialized at a later time in a reclaim tainted context.
> + *
> + * Return: a new uninitialized suballoc object, or an ERR_PTR(-ENOMEM).
> + */
> +
extra \n
> +struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp)
> +{
> + return drm_suballoc_alloc(gfp);
> +}
> +
> +/**
> + * xe_sa_bo_release - Release memory for suballocation.
nit: add () to function name
* xe_sa_bo_release() - ...
> + * @sa: The struct drm_suballoc.
> + */
> +void xe_sa_bo_release(struct drm_suballoc *sa)
> +{
> + drm_suballoc_release(sa);
> +}
> +
> +/**
> + * xe_sa_bo_init - Initialize a suballocation.
ditto
> + * @sa_manager: pointer to the sa_manager
> + * @sa: The struct drm_suballoc.
> + * @size: number of bytes we want to suballocate.
> + *
> + * Try to make a suballocation on a pre-allocated suballoc object of size @size.
> + *
> + * Return: zero on success, errno on failure.
> + */
> +int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct drm_suballoc *sa, u32 size)
why size is u32 ? drm_suballoc_init() takes size_t
> +{
> + return drm_suballoc_init(&sa_manager->base, sa, size, true, 0);
> +}
> +
> /**
> * xe_sa_bo_flush_write() - Copy the data from the sub-allocation to the GPU memory.
> * @sa_bo: the &drm_suballoc to flush
> diff --git a/drivers/gpu/drm/xe/xe_sa.h b/drivers/gpu/drm/xe/xe_sa.h
> index 05e9a4e00e78..19d4b698a7d7 100644
> --- a/drivers/gpu/drm/xe/xe_sa.h
> +++ b/drivers/gpu/drm/xe/xe_sa.h
> @@ -38,6 +38,9 @@ static inline struct drm_suballoc *xe_sa_bo_new(struct xe_sa_manager *sa_manager
> return __xe_sa_bo_new(sa_manager, size, GFP_KERNEL);
> }
>
> +struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp);
> +void xe_sa_bo_release(struct drm_suballoc *sa);
> +int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct drm_suballoc *sa, u32 size);
> void xe_sa_bo_flush_write(struct drm_suballoc *sa_bo);
> void xe_sa_bo_sync_read(struct drm_suballoc *sa_bo);
> void xe_sa_bo_free(struct drm_suballoc *sa_bo, struct dma_fence *fence);
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation
2026-02-10 13:02 ` Michal Wajdeczko
@ 2026-02-10 14:01 ` Thomas Hellström
0 siblings, 0 replies; 12+ messages in thread
From: Thomas Hellström @ 2026-02-10 14:01 UTC (permalink / raw)
To: Michal Wajdeczko, Satyanarayana K V P, intel-xe
Cc: Matthew Brost, Matthew Auld
On Tue, 2026-02-10 at 14:02 +0100, Michal Wajdeczko wrote:
>
>
> On 2/10/2026 11:59 AM, Satyanarayana K V P wrote:
> > CCS save/restore batch buffers are attached during BO allocation
> > and
> > detached during BO teardown. The shrinker triggers xe_bo_move(),
> > which is
> > used for both allocation and deletion paths.
> >
> > When BO allocation and shrinking occur concurrently, a circular
> > locking
> > dependency involving fs_reclaim and swap_guard can occur, leading
> > to a
> > deadlock such as:
> >
> > ======================================================
> > WARNING: possible circular locking dependency detected
> > ------------------------------------------------------
> >
> > CPU0 CPU1
> > ---- ----
> > lock(fs_reclaim);
> > lock(&sa_manager->swap_guard);
> > lock(fs_reclaim);
> > lock(&sa_manager->swap_guard);
> >
> > *** DEADLOCK ***
> > =====================================================
> >
> > To avoid this, the BB pointer and SA are allocated using
> > xe_bb_alloc()
> > before taking lock and SA is initialized using xe_bb_init()
> > preventing
> > reclaim from being invoked in this context.
> >
> > Fixes: 864690cf4dd62 ("drm/xe/vf: Attach and detach CCS copy
> > commands with BO")
> > Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
> > Cc: Matthew Brost <matthew.brost@intel.com>
> > Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> > Cc: Matthew Auld <matthew.auld@intel.com>
> > Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> >
> > ---
> > V2 -> V3:
> > - Created new functions xe_sa_bo_alloc(), xe_sa_bo_release() and
> > xe_sa_bo_init(). (Thomas)
> > - Created new functions xe_bb_alloc(), xe_bb_release and
> > xe_bb_init(). (Thomas)
> > - Updated guard() to scoped_guard() in xe_migrate_ccs_rw_copy().
> > (Thomas)
> >
> > V1 -> V2:
> > - Used drm_suballoc_alloc() and drm_suballoc_init() for BB
> > allocation
> > (Thomas).
> > ---
> > drivers/gpu/drm/xe/xe_bb.c | 49 +++++++++++------
> > drivers/gpu/drm/xe/xe_bb.h | 7 ++-
> > drivers/gpu/drm/xe/xe_migrate.c | 96 ++++++++++++++++++-----------
> > ----
> > drivers/gpu/drm/xe/xe_sa.c | 40 ++++++++++++++
> > drivers/gpu/drm/xe/xe_sa.h | 3 ++
> > 5 files changed, 135 insertions(+), 60 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/xe/xe_bb.c
> > b/drivers/gpu/drm/xe/xe_bb.c
> > index 8b678297aaa2..631ae564e719 100644
> > --- a/drivers/gpu/drm/xe/xe_bb.c
> > +++ b/drivers/gpu/drm/xe/xe_bb.c
> > @@ -59,16 +59,43 @@ struct xe_bb *xe_bb_new(struct xe_gt *gt, u32
> > dwords, bool usm)
> > return ERR_PTR(err);
> > }
> >
> > -struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
> > - enum xe_sriov_vf_ccs_rw_ctxs ctx_id)
>
> shouldn't we add kernel-doc for all new/updated public functions?
+1.
>
> > +struct xe_bb *xe_bb_alloc(struct xe_gt *gt)
> > {
> > struct xe_bb *bb = kmalloc(sizeof(*bb), GFP_KERNEL);
> > struct xe_device *xe = gt_to_xe(gt);
> > - struct xe_sa_manager *bb_pool;
> > int err;
> >
> > if (!bb)
> > return ERR_PTR(-ENOMEM);
> > +
> > + bb->bo = xe_sa_bo_alloc(GFP_KERNEL);
> > + if (IS_ERR(bb->bo)) {
> > + drm_err(&xe->drm, "Sub-allocator memory allocation
> > failed with %ld\n",
>
> nit: there is xe_err(xe, ...)
>
> nit: we try to print errors using %pe, which here is even more
> appropriate
>
> nit: maybe "Failed to allocate SA object for BO (%pe)\n" ?
>
> nit: or maybe any error message should be in xe_sa_bo_alloc()
Typically we don't print errors other than in a debug mode, like
drm_dbg() (I don't think we have an xe_dbg, though). In particular with
memory allocation the core will spew out error messages anyway so I
think it's safe to remove that and just forward it to upper layers.
The rest with Michal's review comments addressed LGTM.
Thanks,
Thomas
>
> > + PTR_ERR(bb->bo));
> > + err = PTR_ERR(bb->bo);
> > + goto err;
> > + }
> > +
> > + return bb;
> > +
> > +err:
> > + kfree(bb);
> > + return ERR_PTR(err);
> > +}
> > +
>
> kernel-doc
>
> > +void xe_bb_release(struct xe_bb *bb)
> > +{
> > + if (bb->bo)
>
> do we need this?
> in xe_bb_alloc() we guarantee that bb will either have valid .bo or
> no bb at all
>
> > + xe_sa_bo_release(bb->bo);
> > +
> > + kfree(bb);
> > +}
> > +
>
> kernel-doc
>
> > +int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
>
> as this is xe_bb function, it should take xe_bb as first param
>
> > + struct xe_sa_manager *bb_pool, u32 dwords)
> > +{
> > + int err;
> > +
> > /*
> > * We need to allocate space for the requested number of
> > dwords &
> > * one additional MI_BATCH_BUFFER_END dword. Since the
> > whole SA
> > @@ -76,22 +103,14 @@ struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt,
> > u32 dwords,
> > * is not over written when the last chunk of SA is
> > allocated for BB.
> > * So, this extra DW acts as a guard here.
> > */
> > -
> > - bb_pool = xe-
> > >sriov.vf.ccs.contexts[ctx_id].mem.ccs_bb_pool;
> > - bb->bo = xe_sa_bo_new(bb_pool, 4 * (dwords + 1));
> > -
> > - if (IS_ERR(bb->bo)) {
> > - err = PTR_ERR(bb->bo);
> > - goto err;
> > - }
> > + err = xe_sa_bo_init(bb_pool, bb->bo, 4 * (dwords + 1));
> > + if (err)
> > + return err;
> >
> > bb->cs = xe_sa_bo_cpu_addr(bb->bo);
> > bb->len = 0;
> >
> > - return bb;
> > -err:
> > - kfree(bb);
> > - return ERR_PTR(err);
> > + return 0;
> > }
> >
> > static struct xe_sched_job *
> > diff --git a/drivers/gpu/drm/xe/xe_bb.h
> > b/drivers/gpu/drm/xe/xe_bb.h
> > index 2a8adc9a6dee..3eb80925bfd1 100644
> > --- a/drivers/gpu/drm/xe/xe_bb.h
> > +++ b/drivers/gpu/drm/xe/xe_bb.h
> > @@ -13,11 +13,14 @@ struct dma_fence;
> > struct xe_gt;
> > struct xe_exec_queue;
> > struct xe_sched_job;
> > +struct xe_sa_manager;
>
> wrong order
>
> > enum xe_sriov_vf_ccs_rw_ctxs;
> >
> > struct xe_bb *xe_bb_new(struct xe_gt *gt, u32 dwords, bool usm);
> > -struct xe_bb *xe_bb_ccs_new(struct xe_gt *gt, u32 dwords,
> > - enum xe_sriov_vf_ccs_rw_ctxs ctx_id);
> > +struct xe_bb *xe_bb_alloc(struct xe_gt *gt);
> > +void xe_bb_release(struct xe_bb *bb);
> > +int xe_bb_init(struct xe_gt *gt, struct xe_bb *bb,
> > + struct xe_sa_manager *bb_pool, u32 dwords);
> > struct xe_sched_job *xe_bb_create_job(struct xe_exec_queue *q,
> > struct xe_bb *bb);
> > struct xe_sched_job *xe_bb_create_migration_job(struct
> > xe_exec_queue *q,
> > diff --git a/drivers/gpu/drm/xe/xe_migrate.c
> > b/drivers/gpu/drm/xe/xe_migrate.c
> > index 078a9bc2821d..c858eaa70e3e 100644
> > --- a/drivers/gpu/drm/xe/xe_migrate.c
> > +++ b/drivers/gpu/drm/xe/xe_migrate.c
> > @@ -1148,65 +1148,75 @@ int xe_migrate_ccs_rw_copy(struct xe_tile
> > *tile, struct xe_exec_queue *q,
> > size -= src_L0;
> > }
> >
> > - bb_pool = ctx->mem.ccs_bb_pool;
> > - guard(mutex) (xe_sa_bo_swap_guard(bb_pool));
> > - xe_sa_bo_swap_shadow(bb_pool);
> > -
> > - bb = xe_bb_ccs_new(gt, batch_size, read_write);
> > + bb = xe_bb_alloc(gt);
> > if (IS_ERR(bb)) {
> > - drm_err(&xe->drm, "BB allocation failed.\n");
> > err = PTR_ERR(bb);
> > return err;
>
> nit: this could be
>
> return PTR_ERR(bb);
>
> > }
> >
> > - batch_size_allocated = batch_size;
> > - size = xe_bo_size(src_bo);
> > - batch_size = 0;
> > + bb_pool = ctx->mem.ccs_bb_pool;
> > + scoped_guard(mutex, xe_sa_bo_swap_guard(bb_pool)) {
> > + xe_sa_bo_swap_shadow(bb_pool);
> > +
> > + err = xe_bb_init(gt, bb, bb_pool, batch_size);
> > + if (err) {
> > + drm_err(&xe->drm, "BB allocation
> > failed.\n");
>
> nit: there is xe_err() but since there is a gt maybe it should be
> xe_gt_err() ?
>
> nit: or maybe move that message to xe_bb_init() ?
>
> > + xe_bb_release(bb);
> > + return err;
> > + }
> >
> > - /*
> > - * Emit PTE and copy commands here.
> > - * The CCS copy command can only support limited size. If
> > the size to be
> > - * copied is more than the limit, divide copy into chunks.
> > So, calculate
> > - * sizes here again before copy command is emitted.
> > - */
> > - while (size) {
> > - batch_size += 10; /* Flush + ggtt addr + 2 NOP */
> > - u32 flush_flags = 0;
> > - u64 ccs_ofs, ccs_size;
> > - u32 ccs_pt;
> > + batch_size_allocated = batch_size;
> > + size = xe_bo_size(src_bo);
> > + batch_size = 0;
> >
> > - u32 avail_pts = max_mem_transfer_per_pass(xe) /
> > LEVEL0_PAGE_TABLE_ENCODE_SIZE;
> > + /*
> > + * Emit PTE and copy commands here.
> > + * The CCS copy command can only support limited
> > size. If the size to be
> > + * copied is more than the limit, divide copy into
> > chunks. So, calculate
> > + * sizes here again before copy command is
> > emitted.
> > + */
> >
> > - src_L0 = xe_migrate_res_sizes(m, &src_it);
> > + while (size) {
> > + batch_size += 10; /* Flush + ggtt addr + 2
> > NOP */
> > + u32 flush_flags = 0;
> > + u64 ccs_ofs, ccs_size;
> > + u32 ccs_pt;
> >
> > - batch_size += pte_update_size(m, false, src,
> > &src_it, &src_L0,
> > - &src_L0_ofs,
> > &src_L0_pt, 0, 0,
> > - avail_pts);
> > + u32 avail_pts =
> > max_mem_transfer_per_pass(xe) /
> > + LEVEL0_PAGE_TABLE_ENCODE_S
> > IZE;
> >
> > - ccs_size = xe_device_ccs_bytes(xe, src_L0);
> > - batch_size += pte_update_size(m, 0, NULL, &ccs_it,
> > &ccs_size, &ccs_ofs,
> > - &ccs_pt, 0,
> > avail_pts, avail_pts);
> > - xe_assert(xe, IS_ALIGNED(ccs_it.start,
> > PAGE_SIZE));
> > - batch_size += EMIT_COPY_CCS_DW;
> > + src_L0 = xe_migrate_res_sizes(m, &src_it);
> > +
> > + batch_size += pte_update_size(m, false,
> > src, &src_it, &src_L0,
> > + &src_L0_ofs,
> > &src_L0_pt, 0, 0,
> > + avail_pts);
> > +
> > + ccs_size = xe_device_ccs_bytes(xe,
> > src_L0);
> > + batch_size += pte_update_size(m, 0, NULL,
> > &ccs_it, &ccs_size, &ccs_ofs,
> > + &ccs_pt, 0,
> > avail_pts, avail_pts);
> > + xe_assert(xe, IS_ALIGNED(ccs_it.start,
> > PAGE_SIZE));
> > + batch_size += EMIT_COPY_CCS_DW;
> >
> > - emit_pte(m, bb, src_L0_pt, false, true, &src_it,
> > src_L0, src);
> > + emit_pte(m, bb, src_L0_pt, false, true,
> > &src_it, src_L0, src);
> >
> > - emit_pte(m, bb, ccs_pt, false, false, &ccs_it,
> > ccs_size, src);
> > + emit_pte(m, bb, ccs_pt, false, false,
> > &ccs_it, ccs_size, src);
> >
> > - bb->len = emit_flush_invalidate(bb->cs, bb->len,
> > flush_flags);
> > - flush_flags = xe_migrate_ccs_copy(m, bb,
> > src_L0_ofs, src_is_pltt,
> > - src_L0_ofs,
> > dst_is_pltt,
> > - src_L0, ccs_ofs,
> > true);
> > - bb->len = emit_flush_invalidate(bb->cs, bb->len,
> > flush_flags);
> > + bb->len = emit_flush_invalidate(bb->cs,
> > bb->len, flush_flags);
> > + flush_flags = xe_migrate_ccs_copy(m, bb,
> > src_L0_ofs, src_is_pltt,
> > +
> > src_L0_ofs, dst_is_pltt,
> > + src_L0,
> > ccs_ofs, true);
> > + bb->len = emit_flush_invalidate(bb->cs,
> > bb->len, flush_flags);
> >
> > - size -= src_L0;
> > - }
> > + size -= src_L0;
> > + }
> >
> > - xe_assert(xe, (batch_size_allocated == bb->len));
> > - src_bo->bb_ccs[read_write] = bb;
> > + xe_assert(xe, (batch_size_allocated == bb->len));
> > + src_bo->bb_ccs[read_write] = bb;
> > +
> > + xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
> > + xe_sa_bo_sync_shadow(bb->bo);
> > + }
> >
> > - xe_sriov_vf_ccs_rw_update_bb_addr(ctx);
> > - xe_sa_bo_sync_shadow(bb->bo);
> > return 0;
> > }
> >
> > diff --git a/drivers/gpu/drm/xe/xe_sa.c
> > b/drivers/gpu/drm/xe/xe_sa.c
> > index b738102575d4..59d0187b3e82 100644
> > --- a/drivers/gpu/drm/xe/xe_sa.c
> > +++ b/drivers/gpu/drm/xe/xe_sa.c
> > @@ -175,6 +175,46 @@ struct drm_suballoc *__xe_sa_bo_new(struct
> > xe_sa_manager *sa_manager, u32 size,
> > return drm_suballoc_new(&sa_manager->base, size, gfp,
> > true, 0);
> > }
> >
> > +/**
> > + * xe_sa_bo_alloc - Allocate uninitialized suballoc object.
> > + * @gfp: gfp flags used for memory allocation.
> > + *
> > + * Allocate memory for an uninitialized suballoc object. Intended
> > usage is
> > + * allocate memory for suballoc object outside of a reclaim
> > tainted context
> > + * and then be initialized at a later time in a reclaim tainted
> > context.
> > + *
> > + * Return: a new uninitialized suballoc object, or an ERR_PTR(-
> > ENOMEM).
> > + */
> > +
>
> extra \n
>
> > +struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp)
> > +{
> > + return drm_suballoc_alloc(gfp);
> > +}
> > +
> > +/**
> > + * xe_sa_bo_release - Release memory for suballocation.
>
> nit: add () to function name
>
> * xe_sa_bo_release() - ...
>
>
> > + * @sa: The struct drm_suballoc.
> > + */
> > +void xe_sa_bo_release(struct drm_suballoc *sa)
> > +{
> > + drm_suballoc_release(sa);
> > +}
> > +
> > +/**
> > + * xe_sa_bo_init - Initialize a suballocation.
>
> ditto
>
> > + * @sa_manager: pointer to the sa_manager
> > + * @sa: The struct drm_suballoc.
> > + * @size: number of bytes we want to suballocate.
> > + *
> > + * Try to make a suballocation on a pre-allocated suballoc object
> > of size @size.
> > + *
> > + * Return: zero on success, errno on failure.
> > + */
> > +int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct
> > drm_suballoc *sa, u32 size)
>
> why size is u32 ? drm_suballoc_init() takes size_t
>
> > +{
> > + return drm_suballoc_init(&sa_manager->base, sa, size,
> > true, 0);
> > +}
> > +
> > /**
> > * xe_sa_bo_flush_write() - Copy the data from the sub-allocation
> > to the GPU memory.
> > * @sa_bo: the &drm_suballoc to flush
> > diff --git a/drivers/gpu/drm/xe/xe_sa.h
> > b/drivers/gpu/drm/xe/xe_sa.h
> > index 05e9a4e00e78..19d4b698a7d7 100644
> > --- a/drivers/gpu/drm/xe/xe_sa.h
> > +++ b/drivers/gpu/drm/xe/xe_sa.h
> > @@ -38,6 +38,9 @@ static inline struct drm_suballoc
> > *xe_sa_bo_new(struct xe_sa_manager *sa_manager
> > return __xe_sa_bo_new(sa_manager, size, GFP_KERNEL);
> > }
> >
> > +struct drm_suballoc *xe_sa_bo_alloc(gfp_t gfp);
> > +void xe_sa_bo_release(struct drm_suballoc *sa);
> > +int xe_sa_bo_init(struct xe_sa_manager *sa_manager, struct
> > drm_suballoc *sa, u32 size);
> > void xe_sa_bo_flush_write(struct drm_suballoc *sa_bo);
> > void xe_sa_bo_sync_read(struct drm_suballoc *sa_bo);
> > void xe_sa_bo_free(struct drm_suballoc *sa_bo, struct dma_fence
> > *fence);
^ permalink raw reply [flat|nested] 12+ messages in thread
* ✗ Xe.CI.FULL: failure for Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
` (4 preceding siblings ...)
2026-02-10 11:59 ` ✓ Xe.CI.BAT: " Patchwork
@ 2026-02-10 14:29 ` Patchwork
5 siblings, 0 replies; 12+ messages in thread
From: Patchwork @ 2026-02-10 14:29 UTC (permalink / raw)
To: Satyanarayana K V P; +Cc: intel-xe
[-- Attachment #1: Type: text/plain, Size: 31254 bytes --]
== Series Details ==
Series: Fix fs_reclaim deadlock caused by CCS save/restore (rev3)
URL : https://patchwork.freedesktop.org/series/160841/
State : failure
== Summary ==
CI Bug Log - changes from xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061_FULL -> xe-pw-160841v3_FULL
====================================================
Summary
-------
**FAILURE**
Serious unknown changes coming with xe-pw-160841v3_FULL absolutely need to be
verified manually.
If you think the reported changes have nothing to do with the changes
introduced in xe-pw-160841v3_FULL, 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 (2 -> 2)
------------------------------
No changes in participating hosts
Possible new issues
-------------------
Here are the unknown changes that may have been introduced in xe-pw-160841v3_FULL:
### IGT changes ###
#### Possible regressions ####
* igt@kms_cursor_legacy@flip-vs-cursor-varying-size:
- shard-lnl: [PASS][1] -> [FAIL][2]
[1]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-lnl-6/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
[2]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-lnl-7/igt@kms_cursor_legacy@flip-vs-cursor-varying-size.html
Known issues
------------
Here are the changes found in xe-pw-160841v3_FULL that come from known issues:
### IGT changes ###
#### Issues hit ####
* igt@kms_big_fb@y-tiled-8bpp-rotate-90:
- shard-bmg: NOTRUN -> [SKIP][3] ([Intel XE#1124]) +3 other tests skip
[3]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_big_fb@y-tiled-8bpp-rotate-90.html
* igt@kms_bw@connected-linear-tiling-2-displays-2560x1440p:
- shard-bmg: [PASS][4] -> [SKIP][5] ([Intel XE#2314] / [Intel XE#2894])
[4]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-10/igt@kms_bw@connected-linear-tiling-2-displays-2560x1440p.html
[5]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-6/igt@kms_bw@connected-linear-tiling-2-displays-2560x1440p.html
* igt@kms_bw@connected-linear-tiling-4-displays-3840x2160p:
- shard-bmg: NOTRUN -> [SKIP][6] ([Intel XE#2314] / [Intel XE#2894])
[6]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_bw@connected-linear-tiling-4-displays-3840x2160p.html
* igt@kms_ccs@bad-pixel-format-y-tiled-gen12-rc-ccs-cc:
- shard-bmg: NOTRUN -> [SKIP][7] ([Intel XE#2887]) +3 other tests skip
[7]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_ccs@bad-pixel-format-y-tiled-gen12-rc-ccs-cc.html
* igt@kms_ccs@crc-primary-suspend-4-tiled-lnl-ccs@pipe-a-dp-2:
- shard-bmg: NOTRUN -> [SKIP][8] ([Intel XE#2652] / [Intel XE#787]) +12 other tests skip
[8]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-3/igt@kms_ccs@crc-primary-suspend-4-tiled-lnl-ccs@pipe-a-dp-2.html
* igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode:
- shard-bmg: NOTRUN -> [SKIP][9] ([Intel XE#2252]) +2 other tests skip
[9]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_chamelium_hpd@dp-hpd-with-enabled-mode.html
* igt@kms_color_pipeline@plane-lut3d-green-only@pipe-c-hdmi-a-3:
- shard-bmg: NOTRUN -> [SKIP][10] ([Intel XE#6969]) +1 other test skip
[10]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_color_pipeline@plane-lut3d-green-only@pipe-c-hdmi-a-3.html
* igt@kms_color_pipeline@plane-lut3d-green-only@pipe-d-hdmi-a-3:
- shard-bmg: NOTRUN -> [SKIP][11] ([Intel XE#6969] / [Intel XE#7006])
[11]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_color_pipeline@plane-lut3d-green-only@pipe-d-hdmi-a-3.html
* igt@kms_content_protection@atomic-hdcp14@pipe-a-dp-2:
- shard-bmg: NOTRUN -> [FAIL][12] ([Intel XE#3304]) +1 other test fail
[12]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-9/igt@kms_content_protection@atomic-hdcp14@pipe-a-dp-2.html
* igt@kms_content_protection@uevent-hdcp14:
- shard-bmg: NOTRUN -> [FAIL][13] ([Intel XE#6707]) +1 other test fail
[13]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_content_protection@uevent-hdcp14.html
* igt@kms_cursor_crc@cursor-random-32x10:
- shard-bmg: NOTRUN -> [SKIP][14] ([Intel XE#2320])
[14]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_cursor_crc@cursor-random-32x10.html
* igt@kms_cursor_crc@cursor-random-512x512:
- shard-bmg: NOTRUN -> [SKIP][15] ([Intel XE#2321])
[15]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_cursor_crc@cursor-random-512x512.html
* igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size:
- shard-bmg: [PASS][16] -> [SKIP][17] ([Intel XE#2291]) +3 other tests skip
[16]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size.html
[17]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_cursor_legacy@cursorb-vs-flipa-atomic-transitions-varying-size.html
* igt@kms_cursor_legacy@flip-vs-cursor-legacy:
- shard-bmg: [PASS][18] -> [FAIL][19] ([Intel XE#5299])
[18]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html
[19]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_cursor_legacy@flip-vs-cursor-legacy.html
* igt@kms_dsc@dsc-with-output-formats:
- shard-bmg: NOTRUN -> [SKIP][20] ([Intel XE#2244])
[20]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_dsc@dsc-with-output-formats.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-bmg: NOTRUN -> [SKIP][21] ([Intel XE#4156])
[21]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_feature_discovery@display-2x:
- shard-bmg: [PASS][22] -> [SKIP][23] ([Intel XE#2373])
[22]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-10/igt@kms_feature_discovery@display-2x.html
[23]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-6/igt@kms_feature_discovery@display-2x.html
* igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible:
- shard-bmg: [PASS][24] -> [SKIP][25] ([Intel XE#2316]) +5 other tests skip
[24]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible.html
[25]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_flip@2x-flip-vs-absolute-wf_vblank-interruptible.html
* igt@kms_flip@flip-vs-suspend@c-hdmi-a3:
- shard-bmg: [PASS][26] -> [INCOMPLETE][27] ([Intel XE#2049] / [Intel XE#2597]) +1 other test incomplete
[26]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-7/igt@kms_flip@flip-vs-suspend@c-hdmi-a3.html
[27]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-7/igt@kms_flip@flip-vs-suspend@c-hdmi-a3.html
* igt@kms_frontbuffer_tracking@drrs-argb161616f-draw-blt:
- shard-bmg: NOTRUN -> [SKIP][28] ([Intel XE#7061])
[28]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_frontbuffer_tracking@drrs-argb161616f-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-onoff:
- shard-bmg: NOTRUN -> [SKIP][29] ([Intel XE#4141]) +5 other tests skip
[29]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-spr-indfb-onoff.html
* igt@kms_frontbuffer_tracking@fbcdrrs-2p-scndscrn-pri-indfb-draw-blt:
- shard-bmg: NOTRUN -> [SKIP][30] ([Intel XE#2311]) +6 other tests skip
[30]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_frontbuffer_tracking@fbcdrrs-2p-scndscrn-pri-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-1p-offscreen-pri-indfb-draw-mmap-wc:
- shard-bmg: NOTRUN -> [SKIP][31] ([Intel XE#2313]) +10 other tests skip
[31]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_frontbuffer_tracking@fbcpsr-1p-offscreen-pri-indfb-draw-mmap-wc.html
* igt@kms_hdr@static-swap:
- shard-bmg: [PASS][32] -> [SKIP][33] ([Intel XE#1503])
[32]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_hdr@static-swap.html
[33]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_hdr@static-swap.html
* igt@kms_panel_fitting@legacy:
- shard-bmg: NOTRUN -> [SKIP][34] ([Intel XE#2486])
[34]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_panel_fitting@legacy.html
* igt@kms_plane@pixel-format-4-tiled-dg2-rc-ccs-modifier-source-clamping@pipe-a-plane-3:
- shard-bmg: NOTRUN -> [SKIP][35] ([Intel XE#7130]) +7 other tests skip
[35]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_plane@pixel-format-4-tiled-dg2-rc-ccs-modifier-source-clamping@pipe-a-plane-3.html
* igt@kms_plane@pixel-format-4-tiled-dg2-rc-ccs-modifier-source-clamping@pipe-b-plane-5:
- shard-bmg: NOTRUN -> [SKIP][36] ([Intel XE#7111] / [Intel XE#7131]) +1 other test skip
[36]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_plane@pixel-format-4-tiled-dg2-rc-ccs-modifier-source-clamping@pipe-b-plane-5.html
* igt@kms_plane@pixel-format-yf-tiled-ccs-modifier-source-clamping:
- shard-bmg: NOTRUN -> [SKIP][37] ([Intel XE#7111] / [Intel XE#7130] / [Intel XE#7131]) +1 other test skip
[37]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_plane@pixel-format-yf-tiled-ccs-modifier-source-clamping.html
* igt@kms_plane@pixel-format-yf-tiled-ccs-modifier-source-clamping@pipe-a-plane-5:
- shard-bmg: NOTRUN -> [SKIP][38] ([Intel XE#7131]) +1 other test skip
[38]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_plane@pixel-format-yf-tiled-ccs-modifier-source-clamping@pipe-a-plane-5.html
* igt@kms_plane_lowres@tiling-y:
- shard-bmg: NOTRUN -> [SKIP][39] ([Intel XE#2393])
[39]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_plane_lowres@tiling-y.html
* igt@kms_plane_multiple@2x-tiling-y:
- shard-bmg: NOTRUN -> [SKIP][40] ([Intel XE#5021])
[40]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_plane_multiple@2x-tiling-y.html
* igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-75@pipe-b:
- shard-bmg: NOTRUN -> [SKIP][41] ([Intel XE#6886]) +4 other tests skip
[41]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_plane_scaling@planes-upscale-factor-0-25-downscale-factor-0-75@pipe-b.html
* igt@kms_pm_dc@dc6-psr:
- shard-bmg: NOTRUN -> [SKIP][42] ([Intel XE#2392])
[42]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_pm_dc@dc6-psr.html
* igt@kms_psr2_sf@psr2-overlay-plane-move-continuous-sf:
- shard-bmg: NOTRUN -> [SKIP][43] ([Intel XE#1406] / [Intel XE#1489]) +1 other test skip
[43]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_psr2_sf@psr2-overlay-plane-move-continuous-sf.html
* igt@kms_psr@fbc-psr2-cursor-plane-move:
- shard-bmg: NOTRUN -> [SKIP][44] ([Intel XE#1406] / [Intel XE#2234] / [Intel XE#2850]) +3 other tests skip
[44]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_psr@fbc-psr2-cursor-plane-move.html
* igt@kms_rotation_crc@primary-y-tiled-reflect-x-90:
- shard-bmg: NOTRUN -> [SKIP][45] ([Intel XE#3414] / [Intel XE#3904]) +1 other test skip
[45]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_rotation_crc@primary-y-tiled-reflect-x-90.html
* igt@kms_setmode@invalid-clone-single-crtc:
- shard-bmg: [PASS][46] -> [SKIP][47] ([Intel XE#1435])
[46]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_setmode@invalid-clone-single-crtc.html
[47]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_setmode@invalid-clone-single-crtc.html
* igt@kms_sharpness_filter@filter-basic:
- shard-bmg: NOTRUN -> [SKIP][48] ([Intel XE#6503]) +1 other test skip
[48]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_sharpness_filter@filter-basic.html
* igt@xe_eudebug@vma-ufence:
- shard-bmg: NOTRUN -> [SKIP][49] ([Intel XE#4837]) +2 other tests skip
[49]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_eudebug@vma-ufence.html
* igt@xe_eudebug_online@interrupt-other-debuggable:
- shard-bmg: NOTRUN -> [SKIP][50] ([Intel XE#4837] / [Intel XE#6665]) +1 other test skip
[50]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_eudebug_online@interrupt-other-debuggable.html
* igt@xe_evict@evict-beng-mixed-many-threads-small:
- shard-bmg: [PASS][51] -> [INCOMPLETE][52] ([Intel XE#6321])
[51]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@xe_evict@evict-beng-mixed-many-threads-small.html
[52]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@xe_evict@evict-beng-mixed-many-threads-small.html
* igt@xe_evict@evict-small-multi-queue-cm:
- shard-bmg: NOTRUN -> [SKIP][53] ([Intel XE#7140])
[53]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_evict@evict-small-multi-queue-cm.html
* igt@xe_exec_basic@multigpu-no-exec-bindexecqueue:
- shard-bmg: NOTRUN -> [SKIP][54] ([Intel XE#2322]) +4 other tests skip
[54]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@xe_exec_basic@multigpu-no-exec-bindexecqueue.html
* igt@xe_exec_fault_mode@once-multi-queue-userptr-invalidate-race-imm:
- shard-bmg: NOTRUN -> [SKIP][55] ([Intel XE#7136]) +1 other test skip
[55]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_exec_fault_mode@once-multi-queue-userptr-invalidate-race-imm.html
* igt@xe_exec_multi_queue@few-execs-preempt-mode-dyn-priority-smem:
- shard-bmg: NOTRUN -> [SKIP][56] ([Intel XE#6874]) +9 other tests skip
[56]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_exec_multi_queue@few-execs-preempt-mode-dyn-priority-smem.html
* igt@xe_exec_system_allocator@many-64k-mmap-huge-nomemset:
- shard-bmg: NOTRUN -> [SKIP][57] ([Intel XE#5007])
[57]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_exec_system_allocator@many-64k-mmap-huge-nomemset.html
* igt@xe_exec_system_allocator@threads-many-stride-mmap-new-huge:
- shard-bmg: NOTRUN -> [SKIP][58] ([Intel XE#4943]) +5 other tests skip
[58]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_exec_system_allocator@threads-many-stride-mmap-new-huge.html
* igt@xe_exec_threads@threads-multi-queue-mixed-shared-vm-userptr:
- shard-bmg: NOTRUN -> [SKIP][59] ([Intel XE#7138]) +2 other tests skip
[59]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@xe_exec_threads@threads-multi-queue-mixed-shared-vm-userptr.html
* igt@xe_multigpu_svm@mgpu-coherency-fail-basic:
- shard-bmg: NOTRUN -> [SKIP][60] ([Intel XE#6964])
[60]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@xe_multigpu_svm@mgpu-coherency-fail-basic.html
* igt@xe_query@multigpu-query-invalid-size:
- shard-bmg: NOTRUN -> [SKIP][61] ([Intel XE#944]) +1 other test skip
[61]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@xe_query@multigpu-query-invalid-size.html
#### Possible fixes ####
* igt@kms_bw@connected-linear-tiling-2-displays-2160x1440p:
- shard-bmg: [SKIP][62] ([Intel XE#2314] / [Intel XE#2894]) -> [PASS][63] +1 other test pass
[62]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_bw@connected-linear-tiling-2-displays-2160x1440p.html
[63]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-3/igt@kms_bw@connected-linear-tiling-2-displays-2160x1440p.html
* igt@kms_cursor_crc@cursor-suspend:
- shard-bmg: [DMESG-FAIL][64] ([Intel XE#6652] / [Intel XE#6819]) -> [PASS][65] +1 other test pass
[64]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_cursor_crc@cursor-suspend.html
[65]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-8/igt@kms_cursor_crc@cursor-suspend.html
* igt@kms_cursor_legacy@cursorb-vs-flipb-legacy:
- shard-bmg: [SKIP][66] ([Intel XE#2291]) -> [PASS][67] +3 other tests pass
[66]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_cursor_legacy@cursorb-vs-flipb-legacy.html
[67]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-3/igt@kms_cursor_legacy@cursorb-vs-flipb-legacy.html
* igt@kms_fbcon_fbt@fbc-suspend:
- shard-lnl: [FAIL][68] ([i915#4767]) -> [PASS][69]
[68]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-lnl-6/igt@kms_fbcon_fbt@fbc-suspend.html
[69]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-lnl-7/igt@kms_fbcon_fbt@fbc-suspend.html
* igt@kms_flip@2x-plain-flip-fb-recreate:
- shard-bmg: [SKIP][70] ([Intel XE#2316]) -> [PASS][71] +5 other tests pass
[70]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_flip@2x-plain-flip-fb-recreate.html
[71]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-9/igt@kms_flip@2x-plain-flip-fb-recreate.html
* igt@kms_setmode@clone-exclusive-crtc:
- shard-bmg: [SKIP][72] ([Intel XE#1435]) -> [PASS][73]
[72]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_setmode@clone-exclusive-crtc.html
[73]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-9/igt@kms_setmode@clone-exclusive-crtc.html
* igt@xe_evict@evict-mixed-many-threads-small:
- shard-bmg: [INCOMPLETE][74] ([Intel XE#6321]) -> [PASS][75]
[74]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-10/igt@xe_evict@evict-mixed-many-threads-small.html
[75]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-6/igt@xe_evict@evict-mixed-many-threads-small.html
* igt@xe_pmu@engine-activity-accuracy-90:
- shard-lnl: [FAIL][76] ([Intel XE#7072]) -> [PASS][77] +3 other tests pass
[76]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-lnl-5/igt@xe_pmu@engine-activity-accuracy-90.html
[77]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-lnl-2/igt@xe_pmu@engine-activity-accuracy-90.html
#### Warnings ####
* igt@kms_content_protection@atomic-hdcp14:
- shard-bmg: [SKIP][78] ([Intel XE#7194]) -> [FAIL][79] ([Intel XE#3304]) +1 other test fail
[78]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_content_protection@atomic-hdcp14.html
[79]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-9/igt@kms_content_protection@atomic-hdcp14.html
* igt@kms_content_protection@srm:
- shard-bmg: [FAIL][80] ([Intel XE#1178] / [Intel XE#3304]) -> [SKIP][81] ([Intel XE#2341])
[80]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_content_protection@srm.html
[81]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_content_protection@srm.html
* igt@kms_frontbuffer_tracking@drrs-2p-primscrn-indfb-pgflip-blt:
- shard-bmg: [SKIP][82] ([Intel XE#2312]) -> [SKIP][83] ([Intel XE#2311]) +16 other tests skip
[82]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_frontbuffer_tracking@drrs-2p-primscrn-indfb-pgflip-blt.html
[83]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-3/igt@kms_frontbuffer_tracking@drrs-2p-primscrn-indfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-spr-indfb-draw-blt:
- shard-bmg: [SKIP][84] ([Intel XE#2311]) -> [SKIP][85] ([Intel XE#2312]) +11 other tests skip
[84]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-3/igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-spr-indfb-draw-blt.html
[85]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_frontbuffer_tracking@drrs-2p-scndscrn-spr-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-pgflip-blt:
- shard-bmg: [SKIP][86] ([Intel XE#4141]) -> [SKIP][87] ([Intel XE#2312]) +2 other tests skip
[86]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-pgflip-blt.html
[87]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_frontbuffer_tracking@fbc-2p-primscrn-indfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-shrfb-pgflip-blt:
- shard-bmg: [SKIP][88] ([Intel XE#2312]) -> [SKIP][89] ([Intel XE#4141]) +7 other tests skip
[88]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-6/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-shrfb-pgflip-blt.html
[89]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-4/igt@kms_frontbuffer_tracking@fbc-2p-scndscrn-shrfb-pgflip-blt.html
* igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-draw-blt:
- shard-bmg: [SKIP][90] ([Intel XE#2313]) -> [SKIP][91] ([Intel XE#2312]) +9 other tests skip
[90]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-draw-blt.html
[91]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_frontbuffer_tracking@fbcpsr-2p-scndscrn-cur-indfb-draw-blt.html
* igt@kms_frontbuffer_tracking@psr-2p-scndscrn-shrfb-pgflip-blt:
- shard-bmg: [SKIP][92] ([Intel XE#2312]) -> [SKIP][93] ([Intel XE#2313]) +16 other tests skip
[92]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-5/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-shrfb-pgflip-blt.html
[93]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-9/igt@kms_frontbuffer_tracking@psr-2p-scndscrn-shrfb-pgflip-blt.html
* igt@kms_hdr@brightness-with-hdr:
- shard-bmg: [SKIP][94] ([Intel XE#3374] / [Intel XE#3544]) -> [SKIP][95] ([Intel XE#3544])
[94]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-1/igt@kms_hdr@brightness-with-hdr.html
[95]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-10/igt@kms_hdr@brightness-with-hdr.html
* igt@kms_tiled_display@basic-test-pattern:
- shard-bmg: [SKIP][96] ([Intel XE#2426]) -> [FAIL][97] ([Intel XE#1729])
[96]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-1/igt@kms_tiled_display@basic-test-pattern.html
[97]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-10/igt@kms_tiled_display@basic-test-pattern.html
* igt@kms_tiled_display@basic-test-pattern-with-chamelium:
- shard-bmg: [SKIP][98] ([Intel XE#2509]) -> [SKIP][99] ([Intel XE#2426])
[98]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-4/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
[99]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-5/igt@kms_tiled_display@basic-test-pattern-with-chamelium.html
* igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv:
- shard-bmg: [ABORT][100] ([Intel XE#5466] / [Intel XE#6652]) -> [ABORT][101] ([Intel XE#5466])
[100]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061/shard-bmg-6/igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv.html
[101]: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/shard-bmg-6/igt@xe_fault_injection@probe-fail-guc-xe_guc_ct_send_recv.html
[Intel XE#1124]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1124
[Intel XE#1178]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1178
[Intel XE#1406]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1406
[Intel XE#1435]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1435
[Intel XE#1489]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1489
[Intel XE#1503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1503
[Intel XE#1729]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/1729
[Intel XE#2049]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2049
[Intel XE#2234]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2234
[Intel XE#2244]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2244
[Intel XE#2252]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2252
[Intel XE#2291]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2291
[Intel XE#2311]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2311
[Intel XE#2312]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2312
[Intel XE#2313]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2313
[Intel XE#2314]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2314
[Intel XE#2316]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2316
[Intel XE#2320]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2320
[Intel XE#2321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2321
[Intel XE#2322]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2322
[Intel XE#2341]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2341
[Intel XE#2373]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2373
[Intel XE#2392]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2392
[Intel XE#2393]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2393
[Intel XE#2426]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2426
[Intel XE#2486]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2486
[Intel XE#2509]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2509
[Intel XE#2597]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2597
[Intel XE#2652]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2652
[Intel XE#2850]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2850
[Intel XE#2887]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2887
[Intel XE#2894]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/2894
[Intel XE#3304]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3304
[Intel XE#3374]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3374
[Intel XE#3414]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3414
[Intel XE#3544]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3544
[Intel XE#3904]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/3904
[Intel XE#4141]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4141
[Intel XE#4156]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4156
[Intel XE#4837]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4837
[Intel XE#4943]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/4943
[Intel XE#5007]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5007
[Intel XE#5021]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5021
[Intel XE#5299]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5299
[Intel XE#5466]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/5466
[Intel XE#6321]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6321
[Intel XE#6503]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6503
[Intel XE#6652]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6652
[Intel XE#6665]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6665
[Intel XE#6707]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6707
[Intel XE#6819]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6819
[Intel XE#6874]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6874
[Intel XE#6886]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6886
[Intel XE#6964]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6964
[Intel XE#6969]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/6969
[Intel XE#7006]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7006
[Intel XE#7061]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7061
[Intel XE#7072]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7072
[Intel XE#7111]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7111
[Intel XE#7130]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7130
[Intel XE#7131]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7131
[Intel XE#7136]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7136
[Intel XE#7138]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7138
[Intel XE#7140]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7140
[Intel XE#7194]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/7194
[Intel XE#787]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/787
[Intel XE#944]: https://gitlab.freedesktop.org/drm/xe/kernel/issues/944
[i915#4767]: https://gitlab.freedesktop.org/drm/i915/kernel/-/issues/4767
Build changes
-------------
* Linux: xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061 -> xe-pw-160841v3
IGT_8746: 8746
xe-4534-4963fa04cc7c2c68030bef5889b6210052f59061: 4963fa04cc7c2c68030bef5889b6210052f59061
xe-pw-160841v3: 160841v3
== Logs ==
For more details see: https://intel-gfx-ci.01.org/tree/intel-xe/xe-pw-160841v3/index.html
[-- Attachment #2: Type: text/html, Size: 35605 bytes --]
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
2026-02-10 12:09 ` Thomas Hellström
@ 2026-02-11 0:20 ` Matthew Brost
1 sibling, 0 replies; 12+ messages in thread
From: Matthew Brost @ 2026-02-11 0:20 UTC (permalink / raw)
To: Satyanarayana K V P
Cc: intel-xe, Thomas Hellström, Michal Wajdeczko, Matthew Auld,
Christian König, dri-devel
On Tue, Feb 10, 2026 at 10:59:31AM +0000, Satyanarayana K V P wrote:
> drm_suballoc_new() currently both allocates the SA object using kmalloc()
> and searches for a suitable hole in the sub-allocator for the requested
> size. If SA allocation is done by holding sub-allocator mutex, this design
> can lead to reclaim safety issues.
>
> By splitting the kmalloc() step outside of the critical section, we allow
> the memory allocation to use GFP_KERNEL (reclaim-safe) while ensuring that
> the initialization step that holds reclaim-tainted locks (sub-allocator
> mutex) operates in a reclaim-unsafe context with pre-allocated memory.
>
> This separation prevents potential deadlocks where memory reclaim could
> attempt to acquire locks that are already held during the sub-allocator
> operations.
>
> Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
> Suggested-by: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
> Cc: Matthew Auld <matthew.auld@intel.com>
> Cc: Christian König <christian.koenig@amd.com>
> Cc: dri-devel@lists.freedesktop.org
>
> ---
> V2 -> V3:
> - Updated commit message (Matt, Thomas & Christian).
> - Removed timeout logic from drm_suballoc_init(). (Thomas & Christian).
>
> V1 -> V2:
> - Splitted drm_suballoc_new() into drm_suballoc_alloc() and
> drm_suballoc_init() (Thomas).
> ---
> drivers/gpu/drm/drm_suballoc.c | 110 ++++++++++++++++++++++++++-------
> include/drm/drm_suballoc.h | 8 +++
> 2 files changed, 97 insertions(+), 21 deletions(-)
>
> diff --git a/drivers/gpu/drm/drm_suballoc.c b/drivers/gpu/drm/drm_suballoc.c
> index 879ea33dbbc4..b97ffcd98d45 100644
> --- a/drivers/gpu/drm/drm_suballoc.c
> +++ b/drivers/gpu/drm/drm_suballoc.c
> @@ -123,7 +123,7 @@ static void drm_suballoc_remove_locked(struct drm_suballoc *sa)
> list_del_init(&sa->olist);
> list_del_init(&sa->flist);
> dma_fence_put(sa->fence);
> - kfree(sa);
> + drm_suballoc_release(sa);
> }
>
> static void drm_suballoc_try_free(struct drm_suballoc_manager *sa_manager)
> @@ -293,45 +293,74 @@ static bool drm_suballoc_next_hole(struct drm_suballoc_manager *sa_manager,
> }
>
> /**
> - * drm_suballoc_new() - Make a suballocation.
> + * drm_suballoc_alloc() - Allocate uninitialized suballoc object.
> + * @gfp: gfp flags used for memory allocation.
> + *
> + * Allocate memory for an uninitialized suballoc object. Intended usage is
> + * allocate memory for suballoc object outside of a reclaim tainted context
> + * and then be initialized at a later time in a reclaim tainted context.
> + *
> + * @drm_suballoc_release should be used to release the memory if returned
> + * suballoc object is in uninitialized state.
> + *
> + * Return: a new uninitialized suballoc object, or an ERR_PTR(-ENOMEM).
> + */
> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp)
> +{
> + struct drm_suballoc *sa;
> +
> + sa = kmalloc(sizeof(*sa), gfp);
> + if (!sa)
> + return ERR_PTR(-ENOMEM);
> +
> + return sa;
> +}
> +EXPORT_SYMBOL(drm_suballoc_alloc);
> +
> +/**
> + * drm_suballoc_release() - Release memory for suballocation.
> + * @sa: The struct drm_suballoc.
> + */
> +void drm_suballoc_release(struct drm_suballoc *sa)
> +{
> + kfree(sa);
> +}
> +EXPORT_SYMBOL(drm_suballoc_release);
> +
> +/**
> + * drm_suballoc_init() - Initialize a suballocation.
> * @sa_manager: pointer to the sa_manager
> + * @sa: The struct drm_suballoc.
> * @size: number of bytes we want to suballocate.
> - * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL but
> - * the argument is provided for suballocations from reclaim context or
> - * where the caller wants to avoid pipelining rather than wait for
> - * reclaim.
> * @intr: Whether to perform waits interruptible. This should typically
> * always be true, unless the caller needs to propagate a
> * non-interruptible context from above layers.
> * @align: Alignment. Must not exceed the default manager alignment.
> * If @align is zero, then the manager alignment is used.
> *
> - * Try to make a suballocation of size @size, which will be rounded
> - * up to the alignment specified in specified in drm_suballoc_manager_init().
> + * Try to make a suballocation on a pre-allocated suballoc object of size @size,
> + * which will be rounded up to the alignment specified in specified in
> + * drm_suballoc_manager_init().
> *
> - * Return: a new suballocated bo, or an ERR_PTR.
> + * Return: zero on success, errno on failure.
> */
> -struct drm_suballoc *
> -drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
> - gfp_t gfp, bool intr, size_t align)
> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
> + struct drm_suballoc *sa, size_t size,
> + bool intr, size_t align)
> {
> struct dma_fence *fences[DRM_SUBALLOC_MAX_QUEUES];
> unsigned int tries[DRM_SUBALLOC_MAX_QUEUES];
> unsigned int count;
> int i, r;
> - struct drm_suballoc *sa;
>
> if (WARN_ON_ONCE(align > sa_manager->align))
> - return ERR_PTR(-EINVAL);
> + return -EINVAL;
> if (WARN_ON_ONCE(size > sa_manager->size || !size))
> - return ERR_PTR(-EINVAL);
> + return -EINVAL;
>
> if (!align)
> align = sa_manager->align;
>
> - sa = kmalloc(sizeof(*sa), gfp);
> - if (!sa)
> - return ERR_PTR(-ENOMEM);
> sa->manager = sa_manager;
> sa->fence = NULL;
> INIT_LIST_HEAD(&sa->olist);
> @@ -348,7 +377,7 @@ drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
> if (drm_suballoc_try_alloc(sa_manager, sa,
> size, align)) {
> spin_unlock(&sa_manager->wq.lock);
> - return sa;
> + return 0;
> }
>
> /* see if we can skip over some allocations */
> @@ -385,8 +414,47 @@ drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
> } while (!r);
>
> spin_unlock(&sa_manager->wq.lock);
> - kfree(sa);
> - return ERR_PTR(r);
> + return r;
> +}
> +EXPORT_SYMBOL(drm_suballoc_init);
> +
> +/**
> + * drm_suballoc_new() - Make a suballocation.
> + * @sa_manager: pointer to the sa_manager
> + * @size: number of bytes we want to suballocate.
> + * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL but
> + * the argument is provided for suballocations from reclaim context or
> + * where the caller wants to avoid pipelining rather than wait for
> + * reclaim.
> + * @intr: Whether to perform waits interruptible. This should typically
> + * always be true, unless the caller needs to propagate a
> + * non-interruptible context from above layers.
> + * @align: Alignment. Must not exceed the default manager alignment.
> + * If @align is zero, then the manager alignment is used.
> + *
> + * Try to make a suballocation of size @size, which will be rounded
> + * up to the alignment specified in specified in drm_suballoc_manager_init().
> + *
> + * Return: a new suballocated bo, or an ERR_PTR.
> + */
> +struct drm_suballoc *
> +drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
> + gfp_t gfp, bool intr, size_t align)
> +{
> + struct drm_suballoc *sa;
> + int err;
> +
> + sa = drm_suballoc_alloc(gfp);
> + if (IS_ERR(sa))
> + return sa;
> +
> + err = drm_suballoc_init(sa_manager, sa, size, intr, align);
> + if (err) {
> + drm_suballoc_release(sa);
> + return ERR_PTR(err);
> + }
> +
> + return sa;
> }
> EXPORT_SYMBOL(drm_suballoc_new);
>
> diff --git a/include/drm/drm_suballoc.h b/include/drm/drm_suballoc.h
> index 7ba72a81a808..b8d1d5449fd8 100644
> --- a/include/drm/drm_suballoc.h
> +++ b/include/drm/drm_suballoc.h
> @@ -53,6 +53,14 @@ void drm_suballoc_manager_init(struct drm_suballoc_manager *sa_manager,
>
> void drm_suballoc_manager_fini(struct drm_suballoc_manager *sa_manager);
>
> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp);
> +
> +void drm_suballoc_release(struct drm_suballoc *sa);
> +
> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
> + struct drm_suballoc *sa, size_t size, bool intr,
> + size_t align);
> +
> struct drm_suballoc *
> drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t size,
> gfp_t gfp, bool intr, size_t align);
> --
> 2.43.0
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers
2026-02-10 12:09 ` Thomas Hellström
@ 2026-02-11 13:32 ` Christian König
0 siblings, 0 replies; 12+ messages in thread
From: Christian König @ 2026-02-11 13:32 UTC (permalink / raw)
To: Thomas Hellström, Satyanarayana K V P, intel-xe
Cc: Matthew Brost, Michal Wajdeczko, Matthew Auld, dri-devel
On 2/10/26 13:09, Thomas Hellström wrote:
> On Tue, 2026-02-10 at 10:59 +0000, Satyanarayana K V P wrote:
>> drm_suballoc_new() currently both allocates the SA object using
>> kmalloc()
>> and searches for a suitable hole in the sub-allocator for the
>> requested
>> size. If SA allocation is done by holding sub-allocator mutex, this
>> design
>> can lead to reclaim safety issues.
>>
>> By splitting the kmalloc() step outside of the critical section, we
>> allow
>> the memory allocation to use GFP_KERNEL (reclaim-safe) while ensuring
>> that
>> the initialization step that holds reclaim-tainted locks (sub-
>> allocator
>> mutex) operates in a reclaim-unsafe context with pre-allocated
>> memory.
>>
>> This separation prevents potential deadlocks where memory reclaim
>> could
>> attempt to acquire locks that are already held during the sub-
>> allocator
>> operations.
>>
>> Signed-off-by: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
>> Suggested-by: Matthew Brost <matthew.brost@intel.com>
>> Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
>> Cc: Michal Wajdeczko <michal.wajdeczko@intel.com>
>> Cc: Matthew Auld <matthew.auld@intel.com>
>> Cc: Christian König <christian.koenig@amd.com>
>> Cc: dri-devel@lists.freedesktop.org
>
> LGTM.
> Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Where are patches 2 and 3 in that series?
Regards,
Christian.
>
>>
>> ---
>> V2 -> V3:
>> - Updated commit message (Matt, Thomas & Christian).
>> - Removed timeout logic from drm_suballoc_init(). (Thomas &
>> Christian).
>>
>> V1 -> V2:
>> - Splitted drm_suballoc_new() into drm_suballoc_alloc() and
>> drm_suballoc_init() (Thomas).
>> ---
>> drivers/gpu/drm/drm_suballoc.c | 110 ++++++++++++++++++++++++++-----
>> --
>> include/drm/drm_suballoc.h | 8 +++
>> 2 files changed, 97 insertions(+), 21 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/drm_suballoc.c
>> b/drivers/gpu/drm/drm_suballoc.c
>> index 879ea33dbbc4..b97ffcd98d45 100644
>> --- a/drivers/gpu/drm/drm_suballoc.c
>> +++ b/drivers/gpu/drm/drm_suballoc.c
>> @@ -123,7 +123,7 @@ static void drm_suballoc_remove_locked(struct
>> drm_suballoc *sa)
>> list_del_init(&sa->olist);
>> list_del_init(&sa->flist);
>> dma_fence_put(sa->fence);
>> - kfree(sa);
>> + drm_suballoc_release(sa);
>> }
>>
>> static void drm_suballoc_try_free(struct drm_suballoc_manager
>> *sa_manager)
>> @@ -293,45 +293,74 @@ static bool drm_suballoc_next_hole(struct
>> drm_suballoc_manager *sa_manager,
>> }
>>
>> /**
>> - * drm_suballoc_new() - Make a suballocation.
>> + * drm_suballoc_alloc() - Allocate uninitialized suballoc object.
>> + * @gfp: gfp flags used for memory allocation.
>> + *
>> + * Allocate memory for an uninitialized suballoc object. Intended
>> usage is
>> + * allocate memory for suballoc object outside of a reclaim tainted
>> context
>> + * and then be initialized at a later time in a reclaim tainted
>> context.
>> + *
>> + * @drm_suballoc_release should be used to release the memory if
>> returned
>> + * suballoc object is in uninitialized state.
>> + *
>> + * Return: a new uninitialized suballoc object, or an ERR_PTR(-
>> ENOMEM).
>> + */
>> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp)
>> +{
>> + struct drm_suballoc *sa;
>> +
>> + sa = kmalloc(sizeof(*sa), gfp);
>> + if (!sa)
>> + return ERR_PTR(-ENOMEM);
>> +
>> + return sa;
>> +}
>> +EXPORT_SYMBOL(drm_suballoc_alloc);
>> +
>> +/**
>> + * drm_suballoc_release() - Release memory for suballocation.
>> + * @sa: The struct drm_suballoc.
>> + */
>> +void drm_suballoc_release(struct drm_suballoc *sa)
>> +{
>> + kfree(sa);
>> +}
>> +EXPORT_SYMBOL(drm_suballoc_release);
>> +
>> +/**
>> + * drm_suballoc_init() - Initialize a suballocation.
>> * @sa_manager: pointer to the sa_manager
>> + * @sa: The struct drm_suballoc.
>> * @size: number of bytes we want to suballocate.
>> - * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL
>> but
>> - * the argument is provided for suballocations from reclaim
>> context or
>> - * where the caller wants to avoid pipelining rather than wait
>> for
>> - * reclaim.
>> * @intr: Whether to perform waits interruptible. This should
>> typically
>> * always be true, unless the caller needs to propagate a
>> * non-interruptible context from above layers.
>> * @align: Alignment. Must not exceed the default manager alignment.
>> * If @align is zero, then the manager alignment is used.
>> *
>> - * Try to make a suballocation of size @size, which will be rounded
>> - * up to the alignment specified in specified in
>> drm_suballoc_manager_init().
>> + * Try to make a suballocation on a pre-allocated suballoc object of
>> size @size,
>> + * which will be rounded up to the alignment specified in specified
>> in
>> + * drm_suballoc_manager_init().
>> *
>> - * Return: a new suballocated bo, or an ERR_PTR.
>> + * Return: zero on success, errno on failure.
>> */
>> -struct drm_suballoc *
>> -drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
>> size,
>> - gfp_t gfp, bool intr, size_t align)
>> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
>> + struct drm_suballoc *sa, size_t size,
>> + bool intr, size_t align)
>> {
>> struct dma_fence *fences[DRM_SUBALLOC_MAX_QUEUES];
>> unsigned int tries[DRM_SUBALLOC_MAX_QUEUES];
>> unsigned int count;
>> int i, r;
>> - struct drm_suballoc *sa;
>>
>> if (WARN_ON_ONCE(align > sa_manager->align))
>> - return ERR_PTR(-EINVAL);
>> + return -EINVAL;
>> if (WARN_ON_ONCE(size > sa_manager->size || !size))
>> - return ERR_PTR(-EINVAL);
>> + return -EINVAL;
>>
>> if (!align)
>> align = sa_manager->align;
>>
>> - sa = kmalloc(sizeof(*sa), gfp);
>> - if (!sa)
>> - return ERR_PTR(-ENOMEM);
>> sa->manager = sa_manager;
>> sa->fence = NULL;
>> INIT_LIST_HEAD(&sa->olist);
>> @@ -348,7 +377,7 @@ drm_suballoc_new(struct drm_suballoc_manager
>> *sa_manager, size_t size,
>> if (drm_suballoc_try_alloc(sa_manager, sa,
>> size, align)) {
>> spin_unlock(&sa_manager->wq.lock);
>> - return sa;
>> + return 0;
>> }
>>
>> /* see if we can skip over some allocations
>> */
>> @@ -385,8 +414,47 @@ drm_suballoc_new(struct drm_suballoc_manager
>> *sa_manager, size_t size,
>> } while (!r);
>>
>> spin_unlock(&sa_manager->wq.lock);
>> - kfree(sa);
>> - return ERR_PTR(r);
>> + return r;
>> +}
>> +EXPORT_SYMBOL(drm_suballoc_init);
>> +
>> +/**
>> + * drm_suballoc_new() - Make a suballocation.
>> + * @sa_manager: pointer to the sa_manager
>> + * @size: number of bytes we want to suballocate.
>> + * @gfp: gfp flags used for memory allocation. Typically GFP_KERNEL
>> but
>> + * the argument is provided for suballocations from reclaim
>> context or
>> + * where the caller wants to avoid pipelining rather than wait
>> for
>> + * reclaim.
>> + * @intr: Whether to perform waits interruptible. This should
>> typically
>> + * always be true, unless the caller needs to propagate a
>> + * non-interruptible context from above layers.
>> + * @align: Alignment. Must not exceed the default manager alignment.
>> + * If @align is zero, then the manager alignment is used.
>> + *
>> + * Try to make a suballocation of size @size, which will be rounded
>> + * up to the alignment specified in specified in
>> drm_suballoc_manager_init().
>> + *
>> + * Return: a new suballocated bo, or an ERR_PTR.
>> + */
>> +struct drm_suballoc *
>> +drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
>> size,
>> + gfp_t gfp, bool intr, size_t align)
>> +{
>> + struct drm_suballoc *sa;
>> + int err;
>> +
>> + sa = drm_suballoc_alloc(gfp);
>> + if (IS_ERR(sa))
>> + return sa;
>> +
>> + err = drm_suballoc_init(sa_manager, sa, size, intr, align);
>> + if (err) {
>> + drm_suballoc_release(sa);
>> + return ERR_PTR(err);
>> + }
>> +
>> + return sa;
>> }
>> EXPORT_SYMBOL(drm_suballoc_new);
>>
>> diff --git a/include/drm/drm_suballoc.h b/include/drm/drm_suballoc.h
>> index 7ba72a81a808..b8d1d5449fd8 100644
>> --- a/include/drm/drm_suballoc.h
>> +++ b/include/drm/drm_suballoc.h
>> @@ -53,6 +53,14 @@ void drm_suballoc_manager_init(struct
>> drm_suballoc_manager *sa_manager,
>>
>> void drm_suballoc_manager_fini(struct drm_suballoc_manager
>> *sa_manager);
>>
>> +struct drm_suballoc *drm_suballoc_alloc(gfp_t gfp);
>> +
>> +void drm_suballoc_release(struct drm_suballoc *sa);
>> +
>> +int drm_suballoc_init(struct drm_suballoc_manager *sa_manager,
>> + struct drm_suballoc *sa, size_t size, bool
>> intr,
>> + size_t align);
>> +
>> struct drm_suballoc *
>> drm_suballoc_new(struct drm_suballoc_manager *sa_manager, size_t
>> size,
>> gfp_t gfp, bool intr, size_t align);
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2026-02-11 13:32 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-10 10:59 [PATCH v3 0/3] Fix fs_reclaim deadlock caused by CCS save/restore Satyanarayana K V P
2026-02-10 10:59 ` [PATCH v3 1/3] drm/sa: Split drm_suballoc_new() into SA alloc and init helpers Satyanarayana K V P
2026-02-10 12:09 ` Thomas Hellström
2026-02-11 13:32 ` Christian König
2026-02-11 0:20 ` Matthew Brost
2026-02-10 10:59 ` [PATCH v3 2/3] drm/xe/vf: Fix fs_reclaim warning with CCS save/restore BB allocation Satyanarayana K V P
2026-02-10 13:02 ` Michal Wajdeczko
2026-02-10 14:01 ` Thomas Hellström
2026-02-10 10:59 ` [PATCH v3 3/3] drm/xe/sa: Add lockdep annotations for SA manager swap_guard Satyanarayana K V P
2026-02-10 11:06 ` ✓ CI.KUnit: success for Fix fs_reclaim deadlock caused by CCS save/restore (rev3) Patchwork
2026-02-10 11:59 ` ✓ Xe.CI.BAT: " Patchwork
2026-02-10 14:29 ` ✗ Xe.CI.FULL: failure " Patchwork
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox