[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