[PATCH i-g-t v2] tests/kms_setmode: increase MAX_CRTCS to 6

Harry Wentland harry.wentland at amd.com
Tue May 30 20:51:11 UTC 2017


AMD GPUs can have 6 CRTCs.

This requires us to allocate the combinations on the heap.

v2:
  Of course We should free the new allocation. Thanks, Alex.

Signed-off-by: Harry Wentland <harry.wentland at amd.com>
---
 tests/kms_setmode.c | 28 ++++++++++++++++++----------
 1 file changed, 18 insertions(+), 10 deletions(-)

diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
index 430568a1c24e..62905ec27e80 100644
--- a/tests/kms_setmode.c
+++ b/tests/kms_setmode.c
@@ -35,11 +35,13 @@
 #include "intel_bufmgr.h"
 
 #define MAX_CONNECTORS  10
-#define MAX_CRTCS       3
+#define MAX_CRTCS       6
 
 /* max combinations with repetitions */
+/* MAX_CONNECTORS ^ MAX_CRTCS */
+/* TODO should really be MAX_CONNECTORS ^ MAX_CONNECTORS ??? */
 #define MAX_COMBINATION_COUNT   \
-	(MAX_CONNECTORS * MAX_CONNECTORS * MAX_CONNECTORS)
+	(MAX_CONNECTORS * MAX_CONNECTORS * MAX_CONNECTORS * MAX_CONNECTORS * MAX_CONNECTORS * MAX_CONNECTORS)
 #define MAX_COMBINATION_ELEMS   MAX_CRTCS
 
 static int drm_fd;
@@ -743,22 +745,25 @@ static void get_combinations(int n, int k, bool allow_repetitions,
 static void test_combinations(const struct test_config *tconf,
 			      int connector_count)
 {
-	struct combination_set connector_combs;
-	struct combination_set crtc_combs;
+	struct combination_set *connector_combs;
+	struct combination_set *crtc_combs;
 	struct connector_config *cconfs;
 	int i;
 
 	if (connector_count > 2 && (tconf->flags & TEST_STEALING))
 		return;
 
+	connector_combs = malloc(sizeof(*connector_combs));
+	crtc_combs = malloc(sizeof(*crtc_combs));
+
 	get_combinations(tconf->resources->count_connectors, connector_count,
-			 false, &connector_combs);
+			 false, connector_combs);
 	get_combinations(tconf->resources->count_crtcs, connector_count,
-			 true, &crtc_combs);
+			 true, crtc_combs);
 
 	igt_info("Testing: %s %d connector combinations\n", tconf->name,
 		 connector_count);
-	for (i = 0; i < connector_combs.count; i++) {
+	for (i = 0; i < connector_combs->count; i++) {
 		int *connector_idxs;
 		int ret;
 		int j;
@@ -766,14 +771,14 @@ static void test_combinations(const struct test_config *tconf,
 		cconfs = malloc(sizeof(*cconfs) * connector_count);
 		igt_assert(cconfs);
 
-		connector_idxs = &connector_combs.items[i].elems[0];
+		connector_idxs = &connector_combs->items[i].elems[0];
 		ret = get_connectors(tconf->resources, connector_idxs,
 				     connector_count, cconfs);
 		if (ret < 0)
 			goto free_cconfs;
 
-		for (j = 0; j < crtc_combs.count; j++) {
-			int *crtc_idxs = &crtc_combs.items[j].elems[0];
+		for (j = 0; j < crtc_combs->count; j++) {
+			int *crtc_idxs = &crtc_combs->items[j].elems[0];
 			ret = assign_crtc_to_connectors(tconf, crtc_idxs,
 							connector_count,
 						        cconfs);
@@ -787,6 +792,9 @@ static void test_combinations(const struct test_config *tconf,
 free_cconfs:
 		free(cconfs);
 	}
+
+	free(connector_combs);
+	free(crtc_combs);
 }
 
 static void run_test(const struct test_config *tconf)
-- 
2.11.0



More information about the amd-gfx mailing list