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

Santhosh Reddy Guddati santhosh.reddy.guddati at intel.com
Thu Mar 27 09:08:51 UTC 2025


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)

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



More information about the igt-dev mailing list