Re: [PATCH 1/2] tests/intel/xe_eudebug_online: Set timeout for debugger VM fd sync

[Maciej Patelczyk <maciej.patelczyk@intel.com>]

On 27/02/2026 12:41, Jan Maslak wrote:

> The vm_open_trigger() function opens a debugger VM fd via
> DRM_XE_EUDEBUG_IOCTL_VM_OPEN leaving timeout_ns at zero. With a zero
> timeout, fsync() on the fd may return -ETIMEDOUT immediately instead of
> waiting for GPU cache invalidation to complete.
>
> Set timeout_ns to DRM_XE_EUDEBUG_VM_SYNC_MAX_TIMEOUT_NSECS to allow
> up to 10 seconds for cache invalidation to complete.
>
> Signed-off-by: Jan Maslak <jan.maslak@intel.com>
> ---
>   tests/intel/xe_eudebug_online.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/tests/intel/xe_eudebug_online.c b/tests/intel/xe_eudebug_online.c
> index 059f918a6..6009da4d6 100644
> --- a/tests/intel/xe_eudebug_online.c
> +++ b/tests/intel/xe_eudebug_online.c
> @@ -904,6 +904,7 @@ static void vm_open_trigger(struct xe_eudebug_debugger *d,
>   	struct drm_xe_eudebug_vm_open vo = {
>   		.client_handle = vm->client_handle,
>   		.vm_handle = vm->vm_handle,
> +		.timeout_ns = DRM_XE_EUDEBUG_VM_SYNC_MAX_TIMEOUT_NSECS,
>   	};
>   	int fd;
>   

Oh, good catch!

Reviewed-by: Maciej Patelczyk <maciej.patelczyk@intel.com>