Re: [PATCH i-g-t v9 2/3] lib/xe: Add failable variant of xe_vm_bind_lr_sync()

["Hellstrom, Thomas" <thomas.hellstrom@intel.com>]

On Mon, 2026-03-30 at 08:01 +0000, Sobin Thomas wrote:
> Add __xe_vm_bind_lr_sync helper function which returns standard error
> codes instead of asserting on failure. This allows calling function
> to handle VM bind failures explicitly while preserving the existing
> xe_vm_bind_lr_sync() wrapper for tests. This enables callers that
> expect bind / overcommit failures.
> 
> v7: Introduced xe_vm_bind_lr_sync_failable (Thomas)
> v8: Modified xe_vm_bind_lr_sync_failable and xe_vm_bind_lr_sync to
> call
>     __xe_vm_bind_lr_sync
> v9: Removed redundant typecast and removed
> xe_vm_bind_lr_sync_failable
> 
> Signed-off-by: Sobin Thomas <sobin.thomas@intel.com>

Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>



> ---
>  lib/xe/xe_ioctl.c | 35 +++++++++++++++++++++++++----------
>  lib/xe/xe_ioctl.h |  2 ++
>  2 files changed, 27 insertions(+), 10 deletions(-)
> 
> diff --git a/lib/xe/xe_ioctl.c b/lib/xe/xe_ioctl.c
> index ea3f2fcaa..daa725d0c 100644
> --- a/lib/xe/xe_ioctl.c
> +++ b/lib/xe/xe_ioctl.c
> @@ -822,23 +822,38 @@ void xe_vm_madvise(int fd, uint32_t vm,
> uint64_t addr, uint64_t range,
>  }
>  
>  #define	BIND_SYNC_VAL	0x686868
> -void xe_vm_bind_lr_sync(int fd, uint32_t vm, uint32_t bo, uint64_t
> offset,
> -			uint64_t addr, uint64_t size, uint32_t
> flags)
> +int __xe_vm_bind_lr_sync(int fd, uint32_t vm, uint32_t bo, uint64_t
> offset,
> +			 uint64_t addr, uint64_t size, uint32_t
> flags)
>  {
> -	volatile uint64_t *sync_addr = malloc(sizeof(*sync_addr));
> +	uint64_t *sync_addr = malloc(sizeof(*sync_addr));
>  	struct drm_xe_sync sync = {
>  		.flags = DRM_XE_SYNC_FLAG_SIGNAL,
>  		.type = DRM_XE_SYNC_TYPE_USER_FENCE,
> -		.addr = to_user_pointer((uint64_t *)sync_addr),
> +		.addr = to_user_pointer(sync_addr),
>  		.timeline_value = BIND_SYNC_VAL,
>  	};
> -
> -	igt_assert(!!sync_addr);
> -	xe_vm_bind_async_flags(fd, vm, 0, bo, 0, addr, size, &sync,
> 1, flags);
> -	if (*sync_addr != BIND_SYNC_VAL)
> -		xe_wait_ufence(fd, (uint64_t *)sync_addr,
> BIND_SYNC_VAL, 0, NSEC_PER_SEC * 10);
> +	int ret = 0;
> +
> +	if (!sync_addr)
> +		return -ENOMEM;
> +	WRITE_ONCE(*sync_addr, 0);
> +	ret = __xe_vm_bind(fd, vm, 0, bo, offset, addr, size,
> DRM_XE_VM_BIND_OP_MAP, flags,
> +			   &sync, 1, 0,  DEFAULT_PAT_INDEX, 0);
> +	if (ret)
> +		goto out;
> +
> +	if (READ_ONCE(*sync_addr) != BIND_SYNC_VAL)
> +		xe_wait_ufence(fd, sync_addr, BIND_SYNC_VAL, 0,
> NSEC_PER_SEC * 10);
>  	/* Only free if the wait succeeds */
> -	free((void *)sync_addr);
> +out:
> +	free(sync_addr);
> +	return ret;
> +}
> +
> +void xe_vm_bind_lr_sync(int fd, uint32_t vm, uint32_t bo, uint64_t
> offset,
> +			uint64_t addr, uint64_t size, uint32_t
> flags)
> +{
> +	igt_assert_eq(__xe_vm_bind_lr_sync(fd, vm, bo, offset, addr,
> size, flags), 0);
>  }
>  
>  void xe_vm_unbind_lr_sync(int fd, uint32_t vm, uint64_t offset,
> diff --git a/lib/xe/xe_ioctl.h b/lib/xe/xe_ioctl.h
> index b62d259fd..2e4bd3b50 100644
> --- a/lib/xe/xe_ioctl.h
> +++ b/lib/xe/xe_ioctl.h
> @@ -117,6 +117,8 @@ struct drm_xe_mem_range_attr
>  void xe_vm_bind_lr_sync(int fd, uint32_t vm, uint32_t bo,
>  			uint64_t offset, uint64_t addr,
>  			uint64_t size, uint32_t flags);
> +int __xe_vm_bind_lr_sync(int fd, uint32_t vm, uint32_t bo, uint64_t
> offset,
> +			 uint64_t addr, uint64_t size, uint32_t
> flags);
>  void xe_vm_unbind_lr_sync(int fd, uint32_t vm, uint64_t offset,
>  			  uint64_t addr, uint64_t size);
>  #endif /* XE_IOCTL_H */