[PATCH i-g-t] tests/syncobj_wait: Update invalid-wait-zero-handles test

Kamil Konieczny kamil.konieczny at linux.intel.com
Thu Oct 31 10:22:40 UTC 2024


Hi Krzysztof,
On 2024-10-30 at 15:48:22 +0100, Krzysztof Niemiec wrote:
> The behavior of DRM_IOCTL_SYNCOBJ_TIMELINE_WAIT has been changed in [1]
> to accept empty lists of syncobj handles, but the
> invalid-wait-zero-handles test has not been updated to mirror this
> change. Fix the assert to check against 0 instead of -EINVAL, and rename
> the test to wait-zero-handles as this is no longer an invalid scenario.
> 
> This change mirrors [2], but for the syncobj_wait test.
> 
> [1] https://patchwork.freedesktop.org/patch/554042/
> [2] https://patchwork.freedesktop.org/patch/618483/
> 
> Signed-off-by: Krzysztof Niemiec <krzysztof.niemiec at intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/syncobj_wait.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/tests/syncobj_wait.c b/tests/syncobj_wait.c
> index be7e67163..e5f251205 100644
> --- a/tests/syncobj_wait.c
> +++ b/tests/syncobj_wait.c
> @@ -80,8 +80,6 @@
>   *
>   * SUBTEST: invalid-wait-illegal-handle
>   *
> - * SUBTEST: invalid-wait-zero-handles
> - *
>   * SUBTEST: multi-wait-all-for-submit-signaled
>   *
>   * SUBTEST: multi-wait-all-for-submit-submitted
> @@ -177,6 +175,8 @@
>   * SUBTEST: wait-for-submit-delayed-submit
>   *
>   * SUBTEST: wait-for-submit-snapshot
> + *
> + * SUBTEST: wait-zero-handles
>   */
>  
>  IGT_TEST_DESCRIPTION("Tests for the drm sync object wait API");
> @@ -297,7 +297,7 @@ static void
>  test_wait_zero_handles(int fd)
>  {
>  	struct drm_syncobj_wait wait = { 0 };
> -	igt_assert_eq(__syncobj_wait(fd, &wait), -EINVAL);
> +	igt_assert_eq(__syncobj_wait(fd, &wait), 0);
>  }
>  
>  static void
> @@ -918,7 +918,7 @@ igt_main
>  	igt_subtest("invalid-wait-bad-flags")
>  		test_wait_bad_flags(fd);
>  
> -	igt_subtest("invalid-wait-zero-handles")
> +	igt_subtest("wait-zero-handles")
>  		test_wait_zero_handles(fd);
>  
>  	igt_subtest("invalid-wait-illegal-handle")
> -- 
> 2.45.2
> 


More information about the igt-dev mailing list