From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by gabe.freedesktop.org (Postfix) with ESMTPS id A189010E157 for ; Tue, 20 Sep 2022 08:29:03 +0000 (UTC) From: Jeevan B To: igt-dev@lists.freedesktop.org Date: Tue, 20 Sep 2022 13:53:46 +0530 Message-Id: <20220920082346.7661-1-jeevan.b@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Subject: [igt-dev] [PATCH i-g-t v4] kms_flip_event_leak: Create dynamic subtests List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: petri.latvala@intel.com Errors-To: igt-dev-bounces@lists.freedesktop.org Sender: "igt-dev" List-ID: Converting the existing subtests to dynamic subtests. v2: Add igt_dispaly_require_output and igt_display_reset and update the test name. (Bhanu) v3: Rename subtest. (Petri) Signed-off-by: Jeevan B Reviewed-by: Bhanuprakash Modem --- tests/kms_flip_event_leak.c | 29 ++++++++++++++++++----------- 1 file changed, 18 insertions(+), 11 deletions(-) diff --git a/tests/kms_flip_event_leak.c b/tests/kms_flip_event_leak.c index ac201293..9815f1e2 100644 --- a/tests/kms_flip_event_leak.c +++ b/tests/kms_flip_event_leak.c @@ -48,6 +48,7 @@ static void test(data_t *data, enum pipe pipe, igt_output_t *output) struct igt_fb fb[2]; int fd, ret; + igt_display_reset(&data->display); /* select the pipe we want to use */ igt_output_set_pipe(output, pipe); @@ -89,24 +90,30 @@ static void test(data_t *data, enum pipe pipe, igt_output_t *output) igt_remove_fb(data->drm_fd, &fb[0]); } -igt_simple_main +igt_main { data_t data = {}; igt_output_t *output; - int valid_tests = 0; enum pipe pipe; - data.drm_fd = drm_open_driver_master(DRIVER_ANY); - kmstest_set_vt_graphics_mode(); + igt_fixture { + data.drm_fd = drm_open_driver_master(DRIVER_ANY); + kmstest_set_vt_graphics_mode(); - igt_display_require(&data.display, data.drm_fd); - - for_each_pipe_with_valid_output(&data.display, pipe, output) { - test(&data, pipe, output); - valid_tests++; + igt_display_require(&data.display, data.drm_fd); + igt_display_require_output(&data.display); } - igt_require_f(valid_tests, "no valid crtc/connector combinations found\n"); - igt_display_fini(&data.display); + igt_subtest_with_dynamic("basic") { + for_each_pipe_with_valid_output(&data.display, pipe, output) { + igt_dynamic_f("pipe-%s-%s", kmstest_pipe_name(pipe), igt_output_name(output)) { + test(&data, pipe, output); + } + } + } + + igt_fixture { + igt_display_fini(&data.display); + } } -- 2.36.0