Re: [PATCH i-g-t 1/2] tests/xe_spin_batch: Correct xe_vm_bind_async usage

[Matthew Brost <matthew.brost@intel.com>]

On Fri, Feb 23, 2024 at 03:04:06PM -0800, Jonathan Cavitt wrote:
> The third parameter of xe_vm_bind_async is the exec_queue id, not the gt
> id.  Set it to 0 in the preempter, as there should not be an exec_queue
> at the time of execution.
> 
> Signed-off-by: Jonathan Cavitt <jonathan.cavitt@intel.com>
> Suggested-by: Matthew Brost <matthew.brost@intel.com>

Reviewed-by: Matthew Brost <matthew.brost@intel.com>

> ---
>  tests/intel/xe_spin_batch.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/intel/xe_spin_batch.c b/tests/intel/xe_spin_batch.c
> index c183063507..faf569baf8 100644
> --- a/tests/intel/xe_spin_batch.c
> +++ b/tests/intel/xe_spin_batch.c
> @@ -198,7 +198,7 @@ static void preempter(int fd, struct drm_xe_engine_class_instance *hwe)
>  			  vram_if_possible(fd, hwe->gt_id),
>  			  DRM_XE_GEM_CREATE_FLAG_NEEDS_VISIBLE_VRAM);
>  
> -	xe_vm_bind_async(fd, vm, hwe->gt_id, bo, 0, addr, bo_size, &sync, 1);
> +	xe_vm_bind_async(fd, vm, 0, bo, 0, addr, bo_size, &sync, 1);
>  	data = xe_bo_map(fd, bo, bo_size);
>  	store_dword_batch(data, addr, value);
>  
> -- 
> 2.25.1
>