Commit 52a36e7e authored by Matthew Auld's avatar Matthew Auld
Browse files

drm/xe: allow non-contig VRAM kernel BO 



If the kernel bo doesn't care about vmap(), either directly or
indirectly with save/restore then we don't need to force contig for such
buffers.

Signed-off-by: default avatarMatthew Auld <matthew.auld@intel.com>
Cc: Satyanarayana K V P <satyanarayana.k.v.p@intel.com>
Cc: Thomas Hellström <thomas.hellstrom@linux.intel.com>
Cc: Matthew Brost <matthew.brost@intel.com>
Reviewed-by: default avatarSatyanarayana K V P <satyanarayana.k.v.p@intel.com>
Link: https://lore.kernel.org/r/20250403102440.266113-15-matthew.auld@intel.com
parent 8e8e9c26

drivers/gpu/drm/xe/xe_bo.c

+8 −1
Original line number Diff line number Diff line
@@ -191,11 +191,18 @@ static void try_add_system(struct xe_device *xe, struct xe_bo *bo, 


static bool force_contiguous(u32 bo_flags)

{

	if (bo_flags & XE_BO_FLAG_STOLEN)

		return true; /* users expect this */

	else if (bo_flags & XE_BO_FLAG_PINNED &&

		 !(bo_flags & XE_BO_FLAG_PINNED_LATE_RESTORE))

		return true; /* needs vmap */



	/*

	 * For eviction / restore on suspend / resume objects pinned in VRAM

	 * must be contiguous, also only contiguous BOs support xe_bo_vmap.

	 */

	return bo_flags & (XE_BO_FLAG_PINNED | XE_BO_FLAG_GGTT);

	return bo_flags & XE_BO_FLAG_NEEDS_CPU_ACCESS &&

	       bo_flags & XE_BO_FLAG_PINNED;

}



static void add_vram(struct xe_device *xe, struct xe_bo *bo,