[i-g-t] tests/intel/xe_dma_buf_sync.c: Ensure spinner completion

Kamil Konieczny kamil.konieczny at linux.intel.com
Fri Jun 27 16:38:54 UTC 2025


On 2025-06-27 at 03:00:02 +0800, Jagmeet Randhawa wrote:

One small nit, please remove ".c" from subject, so it will
read:

[i-g-t] tests/intel/xe_dma_buf_sync: Ensure spinner completion

This could be done at merge time, no need for resend.

Regards,
Kamil

> Add syncobj_wait() to verify that the spinner operation on fd[0]
> has completely finished before proceeding to clean up.
> 
> Signed-off-by: Jagmeet Randhawa <jagmeet.randhawa at intel.com>
> Cc: Brian Nguyen <brian3.nguyen at intel.com>
> Suggested-by: Matthew Brost <matthew.brost at intel.com>
> ---
>  tests/intel/xe_dma_buf_sync.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/tests/intel/xe_dma_buf_sync.c b/tests/intel/xe_dma_buf_sync.c
> index 3e8d89e95..1c0e7c670 100644
> --- a/tests/intel/xe_dma_buf_sync.c
> +++ b/tests/intel/xe_dma_buf_sync.c
> @@ -203,6 +203,8 @@ test_export_dma_buf(struct drm_xe_engine_class_instance *hwe0,
>  		xe_spin_end(&data[i]->spin);
>  		igt_assert(syncobj_wait(fd[1], &sync[1].handle, 1, INT64_MAX,
>  					0, NULL));
> +		igt_assert(syncobj_wait(fd[0], &syncobj_signal, 1, INT64_MAX,
> +					0, NULL));
>  		igt_assert_eq(data[i]->data, 0xc0ffee);
>  
>  		/* Clean up */


More information about the igt-dev mailing list