[igt-dev] [PATCH i-g-t] In Existing kms_plane_multiple we are testing Multiple plane position on one pipe at a time, we do not have a testcase where all the pipes are enabled at the same time and checking for the plane positions.

[Nidhi Gupta]

Adding a subtest with all the pipes enabled and performing a test to
check the multiple plane positions with all the pipes at the same time. Clearing
of the resources are done after performing plane position test on all pipes.
---
 tests/kms_plane_multiple.c | 71 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 71 insertions(+)

diff --git a/tests/kms_plane_multiple.c b/tests/kms_plane_multiple.c
index 0df1053..f178c3a 100644
--- a/tests/kms_plane_multiple.c
+++ b/tests/kms_plane_multiple.c
@@ -506,6 +506,52 @@ test_plane_position_with_output(data_t *data, enum pipe pipe,
 }
 
 static void
+test_multiple_plane_position_with_output(data_t *data, enum pipe pipe,
+				igt_output_t *output, int n_planes,
+				uint64_t tiling)
+{
+	color_t blue  = { 0.0f, 0.0f, 1.0f };
+	igt_crc_t crc;
+	int i;
+	int iterations = opt.iterations < 1 ? 1 : opt.iterations;
+	bool loop_forever;
+	char info[256];
+
+	if (opt.iterations == LOOP_FOREVER) {
+		loop_forever = true;
+		sprintf(info, "forever");
+	} else {
+		loop_forever = false;
+		sprintf(info, "for %d %s",
+			iterations, iterations > 1 ? "iterations" : "iteration");
+	}
+
+	igt_info("Testing connector %s using pipe %s with %d planes %s with seed %d\n",
+		 igt_output_name(output), kmstest_pipe_name(pipe), n_planes,
+		 info, opt.seed);
+
+	test_init(data, pipe, n_planes);
+
+	test_grab_crc(data, output, pipe, &blue, tiling);
+
+	i = 0;
+	while (i < iterations || loop_forever) {
+		prepare_planes(data, pipe, &blue, tiling, n_planes, output);
+
+		igt_display_commit2(&data->display, COMMIT_ATOMIC);
+
+		igt_pipe_crc_get_current(data->display.drm_fd, data->pipe_crc, &crc);
+
+		igt_assert_crc_equal(&data->ref_crc, &crc);
+
+		i++;
+	}
+
+	//test_fini(data, output, n_planes);
+}
+
+
+static void
 test_plane_position(data_t *data, enum pipe pipe, uint64_t tiling)
 {
 	igt_output_t *output;
@@ -528,6 +574,25 @@ test_plane_position(data_t *data, enum pipe pipe, uint64_t tiling)
 }
 
 static void
+test_multiple_plane_position(data_t *data, enum pipe pipe, uint64_t tiling)
+{
+	igt_output_t *output;
+	int n_planes = data->display.pipes[pipe].n_planes;
+
+	output = igt_get_single_output_for_pipe(&data->display, pipe);
+	igt_require(output);
+
+	if (!opt.user_seed)
+		opt.seed = time(NULL);
+
+	srand(opt.seed);
+
+	test_multiple_plane_position_with_output(data, pipe, output,
+					n_planes, tiling);
+}
+
+
+static void
 run_tests_for_pipe(data_t *data, enum pipe pipe)
 {
 	igt_fixture {
@@ -612,6 +677,12 @@ igt_main_args("", long_options, help_str, opt_handler, NULL)
 			run_tests_for_pipe(&data, pipe);
 	}
 
+	igt_subtest_f("atomic-maximum-pipes-tiling-none") {
+	for_each_pipe_static(pipe)
+		test_multiple_plane_position(&data, pipe, LOCAL_DRM_FORMAT_MOD_NONE);
+	}
+
+
 	igt_fixture {
 		igt_display_fini(&data.display);
 	}
-- 
2.7.4