Commit 6028f596 authored by Matthew Brost's avatar Matthew Brost
Browse files

drm/xe: Covert return of -EBUSY to -ENOMEM in VM bind IOCTL 



xe_vma_userptr_pin_pages can return -EBUSY but -EBUSY has special
meaning in VM bind IOCTLs that user fence is pending that is attached to
the VMA. Convert -EBUSY to -ENOMEM in this case as -EBUSY in practice
means we are low or out of memory.

Signed-off-by: default avatarMatthew Brost <matthew.brost@intel.com>
Reviewed-by: default avatarTejas Upadhyay <tejas.upadhyay@intel.com>
Link: https://patch.msgid.link/20251122012502.382587-2-matthew.brost@intel.com
parent 9fb1f125

drivers/gpu/drm/xe/xe_vm.c

+10 −1
Original line number Diff line number Diff line
@@ -2455,8 +2455,17 @@ static struct xe_vma *new_vma(struct xe_vm *vm, struct drm_gpuva_op_map *op, 
		if (IS_ERR(vma))

			return vma;



		if (xe_vma_is_userptr(vma))

		if (xe_vma_is_userptr(vma)) {

			err = xe_vma_userptr_pin_pages(to_userptr_vma(vma));

			/*

			 * -EBUSY has dedicated meaning that a user fence

			 * attached to the VMA is busy, in practice

			 * xe_vma_userptr_pin_pages can only fail with -EBUSY if

			 * we are low on memory so convert this to -ENOMEM.

			 */

			if (err == -EBUSY)

				err = -ENOMEM;

		}

	}

	if (err) {

		prep_vma_destroy(vm, vma, false);