[PATCH i-g-t v8 4/4] tests/kms_async_flips: Remove allow_fail flag from async flip tests

[Borah, Chaitanya Kumar]

> -----Original Message-----
> From: Reddy Guddati, Santhosh <santhosh.reddy.guddati at intel.com>
> Sent: Thursday, March 27, 2025 2:39 PM
> To: igt-dev at lists.freedesktop.org
> Cc: B S, Karthik <karthik.b.s at intel.com>; Murthy, Arun R
> <arun.r.murthy at intel.com>; Borah, Chaitanya Kumar
> <chaitanya.kumar.borah at intel.com>; Reddy Guddati, Santhosh
> <santhosh.reddy.guddati at intel.com>
> Subject: [PATCH i-g-t v8 4/4] tests/kms_async_flips: Remove allow_fail flag
> from async flip tests
> 
> Remove allow_fail flag as the async supported format modifiers are now
> known to the user space. The test now executes for all supported format
> modifiers.
> 
> V2: Remove check for frame 1, as no longer needed now as valid
>     modifiers are being in use. (Chaitanya)
> 

LGTM

Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah at intel.com>

> Signed-off-by: Santhosh Reddy Guddati <santhosh.reddy.guddati at intel.com>
> ---
>  tests/kms_async_flips.c | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/tests/kms_async_flips.c b/tests/kms_async_flips.c index
> c3a875ced..233f54f0e 100644
> --- a/tests/kms_async_flips.c
> +++ b/tests/kms_async_flips.c
> @@ -123,7 +123,6 @@ typedef struct {
>  	enum pipe pipe;
>  	bool alternate_sync_async;
>  	bool suspend_resume;
> -	bool allow_fail;
>  	struct buf_ops *bops;
>  	bool atomic_path;
>  	unsigned int plane_format;
> @@ -346,11 +345,7 @@ static void test_async_flip(data_t *data)
>  		}
> 
>  		ret = perform_flip(data, frame, flags);
> -
> -		if (frame == 1 && data->allow_fail)
> -			igt_skip_on(ret == -EINVAL);
> -		else
> -			igt_assert_eq(ret, 0);
> +		igt_assert_eq(ret, 0);
> 
>  		wait_flip_event(data);
> 
> @@ -724,7 +719,6 @@ static void run_test(data_t *data, void (*test)(data_t
> *))
>  			continue;
> 
>  		test_init(data);
> -		data->allow_fail = false;
>  		data->modifier = default_modifier(data);
>  		igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(data-
> >pipe), data->output->name) {
>  			/*
> --
> 2.25.1