All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test
@ 2022-03-08 14:22 Bhanuprakash Modem
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests Bhanuprakash Modem
                   ` (4 more replies)
  0 siblings, 5 replies; 10+ messages in thread
From: Bhanuprakash Modem @ 2022-03-08 14:22 UTC (permalink / raw)
  To: igt-dev

This series will:
* Convert to dynamic subtests.
* Revert the restriction of execution to 2 pipes, since we
  are using dynamic subtests.
* Sort connector modes only if they don't fit into link BW.

Bhanuprakash Modem (3):
  tests/kms_setmode: Use dynamic subtests
  Revert "tests/kms_setmode: Restrict the test execution to two pipes"
  tests/kms_setmode: Sort modes only if they dont fit in link BW

 tests/kms_setmode.c | 201 +++++++++++++++++++++-----------------------
 1 file changed, 94 insertions(+), 107 deletions(-)

--
2.35.1

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests
  2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
@ 2022-03-08 14:22 ` Bhanuprakash Modem
  2022-03-15 13:37   ` Rodrigo Siqueira Jordao
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 2/3] Revert "tests/kms_setmode: Restrict the test execution to two pipes" Bhanuprakash Modem
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 10+ messages in thread
From: Bhanuprakash Modem @ 2022-03-08 14:22 UTC (permalink / raw)
  To: igt-dev

Add support to use dynamic subtests.

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
---
 tests/kms_setmode.c | 37 ++++++++++++++++++++++++++++++++++---
 1 file changed, 34 insertions(+), 3 deletions(-)

diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
index 18d81ee4f3..a47d47e5ab 100644
--- a/tests/kms_setmode.c
+++ b/tests/kms_setmode.c
@@ -653,6 +653,26 @@ static void test_crtc_config(const struct test_config *tconf,
 	return;
 }
 
+static int get_test_name_str(struct crtc_config *crtc, char *buf,
+				size_t buf_size)
+{
+	int pos;
+	int i;
+
+	pos = snprintf(buf, buf_size, "pipe-%s-", kmstest_pipe_name(crtc->pipe_id));
+
+	for (i = 0; i < crtc->connector_count; i++) {
+		drmModeConnector *connector = crtc->cconfs[i].connector;
+
+		pos += snprintf(&buf[pos], buf_size - pos,
+			"%s%s-%d", i ? "-" : "",
+			kmstest_connector_type_str(connector->connector_type),
+			connector->connector_type_id);
+	}
+
+	return pos;
+}
+
 static void test_one_combination(const struct test_config *tconf,
 				 struct connector_config *cconfs,
 				 int connector_count)
@@ -664,8 +684,19 @@ static void test_one_combination(const struct test_config *tconf,
 	setup_crtcs(tconf, cconfs, connector_count, crtcs,
 		    &crtc_count, &config_valid);
 
-	if (config_valid == !(tconf->flags & TEST_INVALID))
-		test_crtc_config(tconf, crtcs, crtc_count);
+	if (config_valid == !(tconf->flags & TEST_INVALID)) {
+		int i, pos = 0;
+		char test_name[256];
+
+		for (i = 0; i < crtc_count; i++) {
+			if (i > 0)
+				pos += snprintf(&test_name[pos], ARRAY_SIZE(test_name) - pos, "-");
+			pos += get_test_name_str(&crtcs[i], &test_name[pos], ARRAY_SIZE(test_name) - pos);
+		}
+
+		igt_dynamic_f("%s", test_name)
+			test_crtc_config(tconf, crtcs, crtc_count);
+	}
 
 	cleanup_crtcs(crtcs, crtc_count);
 }
@@ -939,7 +970,7 @@ igt_main_args("det:", NULL, help_str, opt_handler, NULL)
 
 	for (i = 0; i < ARRAY_SIZE(tests); i++) {
 		igt_describe("Tests the mode by iterating through all valid/invalid crtc/connector combinations");
-		igt_subtest(tests[i].name) {
+		igt_subtest_with_dynamic(tests[i].name) {
 			struct test_config tconf = {
 				.flags		= tests[i].flags,
 				.name		= tests[i].name,
-- 
2.35.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [igt-dev] [v2 i-g-t 2/3] Revert "tests/kms_setmode: Restrict the test execution to two pipes"
  2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests Bhanuprakash Modem
@ 2022-03-08 14:22 ` Bhanuprakash Modem
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 3/3] tests/kms_setmode: Sort modes only if they dont fit in link BW Bhanuprakash Modem
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 10+ messages in thread
From: Bhanuprakash Modem @ 2022-03-08 14:22 UTC (permalink / raw)
  To: igt-dev

Initially we restricted the execution to two pipes, since it'll
take lot of time to exectute single subtest. As we are using
dynamic subtests it is ok to run subtests on all pipes.

This reverts commit 8c8499c29dd2aa189c3d687e057ba4df326b1732.

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
---
 tests/kms_setmode.c | 35 +++++++++--------------------------
 1 file changed, 9 insertions(+), 26 deletions(-)

diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
index a47d47e5ab..a47070e50c 100644
--- a/tests/kms_setmode.c
+++ b/tests/kms_setmode.c
@@ -37,14 +37,10 @@
 /* max combinations with repetitions */
 #define MAX_COMBINATION_ELEMS   MAX_CRTCS
 
-/* restricted pipe count */
-#define CRTC_RESTRICT_CNT 2
-
 static int drm_fd;
 static drmModeRes *drm_resources;
 static int filter_test_id;
 static bool dry_run;
-static bool all_pipes = false;
 
 static char str_buf[MAX_CRTCS][1024];
 static const char *crtc_strs[MAX_CRTCS];
@@ -832,14 +828,6 @@ static void get_combinations(int n, int k, bool allow_repetitions,
 	iterate_combinations(n, k, allow_repetitions, 0, 0, &comb, set);
 }
 
-static int get_limit_crtc_count(int count_crtcs, bool all_pipe)
-{
-	if ((count_crtcs <= CRTC_RESTRICT_CNT) || all_pipe)
-		return count_crtcs;
-	else
-		return CRTC_RESTRICT_CNT;
-}
-
 static void test_combinations(const struct test_config *tconf,
 			      int connector_count)
 {
@@ -847,7 +835,6 @@ static void test_combinations(const struct test_config *tconf,
 	struct combination_set crtc_combs;
 	struct connector_config *cconfs;
 	int i;
-	int crtc_count = get_limit_crtc_count(tconf->resources->count_crtcs, all_pipes);
 
 	if (connector_count > 2 && (tconf->flags & TEST_STEALING))
 		return;
@@ -855,19 +842,20 @@ static void test_combinations(const struct test_config *tconf,
 	igt_assert(tconf->resources);
 
 	connector_combs.capacity = pow(tconf->resources->count_connectors,
-				       crtc_count + 1);
-	crtc_combs.capacity = pow(crtc_count,
-				  crtc_count + 1);
+				       tconf->resources->count_crtcs + 1);
+	crtc_combs.capacity = pow(tconf->resources->count_crtcs,
+				  tconf->resources->count_crtcs + 1);
+
 	connector_combs.items = malloc(connector_combs.capacity * sizeof(struct combination));
 	crtc_combs.items = malloc(crtc_combs.capacity * sizeof(struct combination));
 
 	get_combinations(tconf->resources->count_connectors, connector_count,
 			 false, &connector_combs);
-	get_combinations(crtc_count, connector_count, true, &crtc_combs);
+	get_combinations(tconf->resources->count_crtcs, connector_count,
+			 true, &crtc_combs);
 
 	igt_info("Testing: %s %d connector combinations\n", tconf->name,
 		 connector_count);
-
 	for (i = 0; i < connector_combs.count; i++) {
 		int *connector_idxs;
 		int ret;
@@ -905,10 +893,9 @@ free_cconfs:
 static void run_test(const struct test_config *tconf)
 {
 	int connector_num;
-	int crtc_count = get_limit_crtc_count(tconf->resources->count_crtcs, all_pipes);
 
 	connector_num = tconf->flags & TEST_CLONE ? 2 : 1;
-	for (; connector_num <= crtc_count; connector_num++)
+	for (; connector_num <= tconf->resources->count_crtcs; connector_num++)
 		test_combinations(tconf, connector_num);
 }
 
@@ -918,9 +905,6 @@ static int opt_handler(int opt, int opt_index, void *data)
 	case 'd':
 		dry_run = true;
 		break;
-	case 'e':
-		all_pipes = true;
-		break;
 	case 't':
 		filter_test_id = atoi(optarg);
 		break;
@@ -933,10 +917,9 @@ static int opt_handler(int opt, int opt_index, void *data)
 
 const char *help_str =
 	"  -d\t\tDon't run any test, only print what would be done. (still needs DRM access)\n"
-	"  -t <test id>\tRun only the test with this id\n"
-	"  -e \t\tRun on all pipes. (Default it will Run only two pipes)\n";
+	"  -t <test id>\tRun only the test with this id.";
 
-igt_main_args("det:", NULL, help_str, opt_handler, NULL)
+igt_main_args("dt:", NULL, help_str, opt_handler, NULL)
 {
 	const struct {
 		enum test_flags flags;
-- 
2.35.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [igt-dev] [v2 i-g-t 3/3] tests/kms_setmode: Sort modes only if they dont fit in link BW
  2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests Bhanuprakash Modem
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 2/3] Revert "tests/kms_setmode: Restrict the test execution to two pipes" Bhanuprakash Modem
@ 2022-03-08 14:22 ` Bhanuprakash Modem
  2022-03-08 19:26 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_setmode: Fix stealing test (rev3) Patchwork
  2022-03-09  0:55 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Bhanuprakash Modem @ 2022-03-08 14:22 UTC (permalink / raw)
  To: igt-dev

This patch reverts sorting logic and uses it only when modes
tried dont fit into link BW. This avoids failure due to
setting unsupported modes on connector.

Test affected (fail -> pass after change) :
  * igt@kms_setmode@invalid-clone-single-[crtc|crtc-stealing]
  * igt@kms_setmode@basic

V2: Fix typo in commit message (Swati)

Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
---
 tests/kms_setmode.c | 135 ++++++++++++++++++--------------------------
 1 file changed, 54 insertions(+), 81 deletions(-)

diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
index a47070e50c..d5cb45d438 100644
--- a/tests/kms_setmode.c
+++ b/tests/kms_setmode.c
@@ -42,9 +42,6 @@ static drmModeRes *drm_resources;
 static int filter_test_id;
 static bool dry_run;
 
-static char str_buf[MAX_CRTCS][1024];
-static const char *crtc_strs[MAX_CRTCS];
-
 const drmModeModeInfo mode_640_480 = {
 	.name		= "640x480",
 	.vrefresh	= 60,
@@ -536,87 +533,18 @@ static int sort_drm_modes(const void *a, const void *b)
 {
 	const drmModeModeInfo *mode1 = a, *mode2 = b;
 
-	return (mode1->clock < mode2->clock) - (mode2->clock < mode1->clock);
-}
-
-static
-int __test_crtc_config(struct crtc_config *crtcs, int crtc_count,
-		       const struct test_config *tconf, bool *config_failed,
-		       int base)
-{
-	struct crtc_config *crtc = NULL;
-	int ret = 0;
-
-	crtc = &crtcs[base];
-
-	/* Sort the modes in descending order by clock freq. */
-	qsort(crtc->cconfs->connector->modes,
-		crtc->cconfs->connector->count_modes,
-		sizeof(drmModeModeInfo),
-		sort_drm_modes);
-
-	for (int i = 0; i < crtc->cconfs->connector->count_modes; i++) {
-		uint32_t *ids;
-
-		crtc->mode = crtc->cconfs->connector->modes[i];
-
-		get_crtc_config_str(crtc, str_buf[base], sizeof(str_buf[base]));
-		crtc_strs[base] = &str_buf[base][0];
-		igt_info("    %s\n", crtc_strs[base]);
-
-		create_fb_for_crtc(crtc, &crtc->fb_info);
-		paint_fb(&crtc->fb_info, tconf->name, crtc_strs, crtc_count, base);
-
-		ids = get_connector_ids(crtc);
-		if (tconf->flags & TEST_STEALING)
-			ret = test_stealing(drm_fd, crtc, ids);
-		else
-			ret = drmModeSetCrtc(drm_fd, crtc->crtc_id,
-					     crtc->fb_info.fb_id, 0, 0, ids,
-					     crtc->connector_count, &crtc->mode);
-
-		free(ids);
-
-		/* crtcs[base].modes[i] don't fit, try next mode. */
-		if (ret < 0 && errno == ENOSPC)
-			continue;
-
-		if (ret < 0) {
-			igt_assert_eq(errno, EINVAL);
-			*config_failed = true;
-
-			return ret;
-		}
-
-		/* Try all crtcs recursively. */
-		if (base + 1 < crtc_count)
-			ret = __test_crtc_config(crtcs, crtc_count, tconf, config_failed, base + 1);
-
-		/*
-		 * With crtcs[base].modes[i], None of the crtc[base+1] modes fits
-		 * into the link BW.
-		 *
-		 * Lets try with crtcs[base].modes[i+1]
-		 */
-		if (ret < 0 && errno == ENOSPC)
-			continue;
-
-		/*
-		 * ret == 0, (or) ret < 0 && errno == EINVAL
-		 * No need to try other modes of crtcs[base].
-		 */
-		return ret;
-	}
-
-	/* When all crtcs[base].modes are tried & failed to fit into link BW. */
-	return ret;
+	return (mode2->clock < mode1->clock) - (mode1->clock < mode2->clock);
 }
 
 static void test_crtc_config(const struct test_config *tconf,
 			     struct crtc_config *crtcs, int crtc_count)
 {
+	char str_buf[MAX_CRTCS][1024];
+	const char *crtc_strs[MAX_CRTCS];
+	struct crtc_config *crtc;
 	static int test_id;
 	bool config_failed = false;
+	bool retry = false;
 	int ret = 0;
 	int i;
 
@@ -627,6 +555,21 @@ static void test_crtc_config(const struct test_config *tconf,
 
 	igt_info("  Test id#%d CRTC count %d\n", test_id, crtc_count);
 
+retry:
+	if (retry) {
+		kmstest_unset_all_crtcs(drm_fd, tconf->resources);
+
+		for (i = 0; i < crtc_count; i++) {
+			/* Sort the modes in asending order by clock freq. */
+			qsort(crtcs[i].cconfs->connector->modes,
+			      crtcs[i].cconfs->connector->count_modes,
+			      sizeof(drmModeModeInfo),
+			      sort_drm_modes);
+
+			crtcs[i].mode = crtcs[i].cconfs->connector->modes[0];
+		}
+	}
+
 	for (i = 0; i < crtc_count; i++) {
 		get_crtc_config_str(&crtcs[i], str_buf[i], sizeof(str_buf[i]));
 		crtc_strs[i] = &str_buf[i][0];
@@ -638,9 +581,39 @@ static void test_crtc_config(const struct test_config *tconf,
 		return;
 	}
 
-	ret = __test_crtc_config(crtcs, crtc_count, tconf, &config_failed, 0);
-	igt_skip_on_f((ret < 0 && errno == ENOSPC),
-			"No suitable mode(s) found to fit into the link BW\n");
+	for (i = 0; i < crtc_count; i++) {
+		uint32_t *ids;
+
+		crtc = &crtcs[i];
+
+		igt_info("    %s\n", crtc_strs[i]);
+
+		create_fb_for_crtc(crtc, &crtc->fb_info);
+		paint_fb(&crtc->fb_info, tconf->name, crtc_strs, crtc_count, i);
+
+		ids = get_connector_ids(crtc);
+		if (tconf->flags & TEST_STEALING)
+			ret = test_stealing(drm_fd, crtc, ids);
+		else
+			ret = drmModeSetCrtc(drm_fd, crtc->crtc_id,
+					     crtc->fb_info.fb_id, 0, 0, ids,
+					     crtc->connector_count, &crtc->mode);
+
+		free(ids);
+
+		if (ret < 0) {
+			if (errno == ENOSPC) {
+				igt_skip_on_f(retry, "No suitable mode(s) found to fit into the link BW.\n");
+
+				retry = true;
+				goto retry;
+			}
+
+			igt_assert_eq(errno, EINVAL);
+			config_failed = true;
+		}
+	}
+
 	igt_assert(config_failed == !!(tconf->flags & TEST_INVALID));
 
 	if (ret == 0 && tconf->flags & TEST_TIMINGS)
@@ -712,7 +685,7 @@ static int assign_crtc_to_connectors(const struct test_config *tconf,
 		    crtc_idx_mask & ~(1 << crtc_idx))
 			return -1;
 
-		if ((tconf->flags & TEST_EXCLUSIVE_CRTC_CLONE) &&
+		if ((tconf->flags & (TEST_EXCLUSIVE_CRTC_CLONE | TEST_TIMINGS)) &&
 		    crtc_idx_mask & (1 << crtc_idx))
 			return -1;
 
-- 
2.35.1

^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_setmode: Fix stealing test (rev3)
  2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
                   ` (2 preceding siblings ...)
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 3/3] tests/kms_setmode: Sort modes only if they dont fit in link BW Bhanuprakash Modem
@ 2022-03-08 19:26 ` Patchwork
  2022-03-09  0:55 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-03-08 19:26 UTC (permalink / raw)
  To: Bhanuprakash Modem; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 15644 bytes --]

== Series Details ==

Series: tests/kms_setmode: Fix stealing test (rev3)
URL   : https://patchwork.freedesktop.org/series/100893/
State : success

== Summary ==

CI Bug Log - changes from CI_DRM_11336 -> IGTPW_6754
====================================================

Summary
-------

  **SUCCESS**

  No regressions found.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/index.html

Participating hosts (47 -> 42)
------------------------------

  Additional (1): fi-pnv-d510 
  Missing    (6): shard-tglu fi-bsw-cyan fi-ctg-p8600 fi-kbl-8809g shard-rkl shard-dg1 

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_6754:

### IGT changes ###

#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * igt@gem_busy@busy@all:
    - {bat-dg2-9}:        [PASS][1] -> [DMESG-WARN][2]
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-dg2-9/igt@gem_busy@busy@all.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-dg2-9/igt@gem_busy@busy@all.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - {bat-rpls-2}:       [PASS][3] -> [SKIP][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-rpls-2/igt@kms_setmode@basic-clone-single-crtc.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-rpls-2/igt@kms_setmode@basic-clone-single-crtc.html
    - {fi-jsl-1}:         [PASS][5] -> [SKIP][6]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-jsl-1/igt@kms_setmode@basic-clone-single-crtc.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-jsl-1/igt@kms_setmode@basic-clone-single-crtc.html
    - {bat-jsl-1}:        [PASS][7] -> [SKIP][8]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-jsl-1/igt@kms_setmode@basic-clone-single-crtc.html
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-jsl-1/igt@kms_setmode@basic-clone-single-crtc.html
    - {fi-ehl-2}:         [PASS][9] -> [SKIP][10]
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-ehl-2/igt@kms_setmode@basic-clone-single-crtc.html
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-ehl-2/igt@kms_setmode@basic-clone-single-crtc.html
    - {bat-jsl-2}:        [PASS][11] -> [SKIP][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-jsl-2/igt@kms_setmode@basic-clone-single-crtc.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-jsl-2/igt@kms_setmode@basic-clone-single-crtc.html

  
New tests
---------

  New tests have been introduced between CI_DRM_11336 and IGTPW_6754:

### New IGT tests (2) ###

  * igt@kms_setmode@basic-clone-single-crtc@pipe-a-vga-1-hdmi-a-1:
    - Statuses : 1 pass(s)
    - Exec time: [0.08] s

  * igt@kms_setmode@basic-clone-single-crtc@pipe-b-vga-1-hdmi-a-1:
    - Statuses : 1 pass(s)
    - Exec time: [0.08] s

  

Known issues
------------

  Here are the changes found in IGTPW_6754 that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@amdgpu/amd_cs_nop@sync-fork-compute0:
    - fi-snb-2600:        NOTRUN -> [SKIP][13] ([fdo#109271]) +17 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-snb-2600/igt@amdgpu/amd_cs_nop@sync-fork-compute0.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-skl-6600u:       [PASS][14] -> [INCOMPLETE][15] ([i915#4547])
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@gem_huc_copy@huc-copy:
    - fi-pnv-d510:        NOTRUN -> [SKIP][16] ([fdo#109271]) +58 similar issues
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-pnv-d510/igt@gem_huc_copy@huc-copy.html

  * igt@i915_selftest@live@gt_lrc:
    - fi-bsw-n3050:       [PASS][17] -> [DMESG-FAIL][18] ([i915#1436] / [i915#2373])
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bsw-n3050/igt@i915_selftest@live@gt_lrc.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - fi-skl-6700k2:      [PASS][19] -> [SKIP][20] ([fdo#109271])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-skl-6700k2/igt@kms_setmode@basic-clone-single-crtc.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-skl-6700k2/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-blb-e6850:       [PASS][21] -> [SKIP][22] ([fdo#109271])
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-blb-e6850/igt@kms_setmode@basic-clone-single-crtc.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-blb-e6850/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-kbl-x1275:       [PASS][23] -> [SKIP][24] ([fdo#109271])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-kbl-x1275/igt@kms_setmode@basic-clone-single-crtc.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-kbl-x1275/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-bsw-kefka:       [PASS][25] -> [SKIP][26] ([fdo#109271])
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bsw-kefka/igt@kms_setmode@basic-clone-single-crtc.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bsw-kefka/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-bwr-2160:        [PASS][27] -> [SKIP][28] ([fdo#109271])
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bwr-2160/igt@kms_setmode@basic-clone-single-crtc.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bwr-2160/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-bsw-nick:        [PASS][29] -> [SKIP][30] ([fdo#109271])
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bsw-nick/igt@kms_setmode@basic-clone-single-crtc.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bsw-nick/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-kbl-guc:         [PASS][31] -> [SKIP][32] ([fdo#109271])
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-kbl-guc/igt@kms_setmode@basic-clone-single-crtc.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-kbl-guc/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-rkl-guc:         [PASS][33] -> [SKIP][34] ([i915#3555] / [i915#4098])
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-rkl-guc/igt@kms_setmode@basic-clone-single-crtc.html
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-rkl-guc/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-ilk-650:         [PASS][35] -> [SKIP][36] ([fdo#109271])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-ilk-650/igt@kms_setmode@basic-clone-single-crtc.html
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-ilk-650/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-elk-e7500:       [PASS][37] -> [SKIP][38] ([fdo#109271])
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-elk-e7500/igt@kms_setmode@basic-clone-single-crtc.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-elk-e7500/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-glk-dsi:         [PASS][39] -> [SKIP][40] ([fdo#109271])
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-glk-dsi/igt@kms_setmode@basic-clone-single-crtc.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-glk-dsi/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-snb-2520m:       [PASS][41] -> [SKIP][42] ([fdo#109271])
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-snb-2520m/igt@kms_setmode@basic-clone-single-crtc.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-snb-2520m/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-kbl-soraka:      [PASS][43] -> [SKIP][44] ([fdo#109271])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-kbl-soraka/igt@kms_setmode@basic-clone-single-crtc.html
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-kbl-soraka/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-cfl-8109u:       [PASS][45] -> [SKIP][46] ([fdo#109271])
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-cfl-8109u/igt@kms_setmode@basic-clone-single-crtc.html
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-cfl-8109u/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-kbl-7500u:       [PASS][47] -> [SKIP][48] ([fdo#109271])
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-kbl-7500u/igt@kms_setmode@basic-clone-single-crtc.html
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-kbl-7500u/igt@kms_setmode@basic-clone-single-crtc.html
    - bat-dg1-6:          [PASS][49] -> [SKIP][50] ([i915#3555])
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-dg1-6/igt@kms_setmode@basic-clone-single-crtc.html
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-dg1-6/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-bxt-dsi:         [PASS][51] -> [SKIP][52] ([fdo#109271])
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bxt-dsi/igt@kms_setmode@basic-clone-single-crtc.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bxt-dsi/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-hsw-4770:        [PASS][53] -> [SKIP][54] ([fdo#109271])
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-hsw-4770/igt@kms_setmode@basic-clone-single-crtc.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-hsw-4770/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-cfl-guc:         [PASS][55] -> [SKIP][56] ([fdo#109271])
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-cfl-guc/igt@kms_setmode@basic-clone-single-crtc.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-cfl-guc/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-kbl-7567u:       [PASS][57] -> [SKIP][58] ([fdo#109271])
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-kbl-7567u/igt@kms_setmode@basic-clone-single-crtc.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-kbl-7567u/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-skl-guc:         [PASS][59] -> [SKIP][60] ([fdo#109271])
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-skl-guc/igt@kms_setmode@basic-clone-single-crtc.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-skl-guc/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-bsw-n3050:       [PASS][61] -> [SKIP][62] ([fdo#109271])
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-bsw-n3050/igt@kms_setmode@basic-clone-single-crtc.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bsw-n3050/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-ivb-3770:        [PASS][63] -> [SKIP][64] ([fdo#109271])
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-ivb-3770/igt@kms_setmode@basic-clone-single-crtc.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-ivb-3770/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-cfl-8700k:       [PASS][65] -> [SKIP][66] ([fdo#109271])
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-cfl-8700k/igt@kms_setmode@basic-clone-single-crtc.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-cfl-8700k/igt@kms_setmode@basic-clone-single-crtc.html
    - bat-dg1-5:          [PASS][67] -> [SKIP][68] ([i915#3555])
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html
    - fi-glk-j4005:       [PASS][69] -> [SKIP][70] ([fdo#109271])
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-glk-j4005/igt@kms_setmode@basic-clone-single-crtc.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-glk-j4005/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@runner@aborted:
    - fi-bsw-n3050:       NOTRUN -> [FAIL][71] ([fdo#109271] / [i915#1436] / [i915#3428] / [i915#4312])
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-bsw-n3050/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@i915_module_load@reload:
    - fi-tgl-1115g4:      [DMESG-WARN][72] ([i915#4002]) -> [PASS][73] +1 similar issue
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-tgl-1115g4/igt@i915_module_load@reload.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-tgl-1115g4/igt@i915_module_load@reload.html

  * igt@i915_selftest@live@hangcheck:
    - fi-snb-2600:        [INCOMPLETE][74] ([i915#3921]) -> [PASS][75]
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-snb-2600/igt@i915_selftest@live@hangcheck.html

  * igt@i915_selftest@live@requests:
    - {bat-rpls-2}:       [INCOMPLETE][76] -> [PASS][77]
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-rpls-2/igt@i915_selftest@live@requests.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-rpls-2/igt@i915_selftest@live@requests.html

  * igt@kms_busy@basic@flip:
    - {bat-adlp-6}:       [DMESG-WARN][78] ([i915#3576]) -> [PASS][79]
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/bat-adlp-6/igt@kms_busy@basic@flip.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/bat-adlp-6/igt@kms_busy@basic@flip.html

  
#### Warnings ####

  * igt@kms_setmode@basic-clone-single-crtc:
    - fi-tgl-1115g4:      [DMESG-WARN][80] ([i915#4002]) -> [SKIP][81] ([i915#3555])
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/fi-tgl-1115g4/igt@kms_setmode@basic-clone-single-crtc.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/fi-tgl-1115g4/igt@kms_setmode@basic-clone-single-crtc.html

  
  {name}: This element is suppressed. This means it is ignored when computing
          the status of the difference (SUCCESS, WARNING, or FAILURE).

  [fdo#109271]: https://bugs.freedesktop.org/show_bug.cgi?id=109271
  [i915#1436]: https://gitlab.freedesktop.org/drm/intel/issues/1436
  [i915#2373]: https://gitlab.freedesktop.org/drm/intel/issues/2373
  [i915#3428]: https://gitlab.freedesktop.org/drm/intel/issues/3428
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3576]: https://gitlab.freedesktop.org/drm/intel/issues/3576
  [i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
  [i915#4002]: https://gitlab.freedesktop.org/drm/intel/issues/4002
  [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4391]: https://gitlab.freedesktop.org/drm/intel/issues/4391
  [i915#4547]: https://gitlab.freedesktop.org/drm/intel/issues/4547


Build changes
-------------

  * CI: CI-20190529 -> None
  * IGT: IGT_6368 -> IGTPW_6754

  CI-20190529: 20190529
  CI_DRM_11336: 9c5e35853b19af1d1c8a1379722b60e7bb7d6753 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_6754: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/index.html
  IGT_6368: 60e5ffca027b38398c279fba0f5a1b7517aa6061 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/index.html

[-- Attachment #2: Type: text/html, Size: 18909 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* [igt-dev] ✗ Fi.CI.IGT: failure for tests/kms_setmode: Fix stealing test (rev3)
  2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
                   ` (3 preceding siblings ...)
  2022-03-08 19:26 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_setmode: Fix stealing test (rev3) Patchwork
@ 2022-03-09  0:55 ` Patchwork
  4 siblings, 0 replies; 10+ messages in thread
From: Patchwork @ 2022-03-09  0:55 UTC (permalink / raw)
  To: Bhanuprakash Modem; +Cc: igt-dev

[-- Attachment #1: Type: text/plain, Size: 30262 bytes --]

== Series Details ==

Series: tests/kms_setmode: Fix stealing test (rev3)
URL   : https://patchwork.freedesktop.org/series/100893/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_11336_full -> IGTPW_6754_full
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_6754_full absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_6754_full, please notify your bug team to allow them
  to document this new failure mode, which will reduce false positives in CI.

  External URL: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/index.html

Participating hosts (13 -> 8)
------------------------------

  Missing    (5): pig-kbl-iris pig-glk-j5005 pig-skl-6260u shard-rkl shard-dg1 

Possible new issues
-------------------

  Here are the unknown changes that may have been introduced in IGTPW_6754_full:

### IGT changes ###

#### Possible regressions ####

  * {igt@kms_setmode@basic@pipe-a-vga-1} (NEW):
    - shard-snb:          NOTRUN -> [FAIL][1] +1 similar issue
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb6/igt@kms_setmode@basic@pipe-a-vga-1.html

  * igt@kms_setmode@invalid-clone-exclusive-crtc:
    - shard-tglb:         [PASS][2] -> [SKIP][3] +3 similar issues
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-tglb6/igt@kms_setmode@invalid-clone-exclusive-crtc.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb5/igt@kms_setmode@invalid-clone-exclusive-crtc.html

  * igt@kms_setmode@invalid-clone-single-crtc-stealing:
    - shard-iclb:         [PASS][4] -> [SKIP][5] +4 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-iclb8/igt@kms_setmode@invalid-clone-single-crtc-stealing.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@kms_setmode@invalid-clone-single-crtc-stealing.html

  
#### Suppressed ####

  The following results come from untrusted machines, tests, or statuses.
  They do not affect the overall result.

  * {igt@kms_plane_scaling@scaler-with-pixel-format-unity-scaling@pipe-b-edp-1-scaler-with-pixel-format}:
    - shard-iclb:         NOTRUN -> [INCOMPLETE][6]
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb2/igt@kms_plane_scaling@scaler-with-pixel-format-unity-scaling@pipe-b-edp-1-scaler-with-pixel-format.html

  * igt@kms_setmode@invalid-clone-single-crtc:
    - {shard-tglu}:       NOTRUN -> [SKIP][7]
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglu-8/igt@kms_setmode@invalid-clone-single-crtc.html

  * igt@kms_setmode@invalid-clone-single-crtc-stealing:
    - {shard-tglu}:       [PASS][8] -> [SKIP][9] +2 similar issues
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-tglu-5/igt@kms_setmode@invalid-clone-single-crtc-stealing.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglu-5/igt@kms_setmode@invalid-clone-single-crtc-stealing.html

  
New tests
---------

  New tests have been introduced between CI_DRM_11336_full and IGTPW_6754_full:

### New IGT tests (34) ###

  * igt@kms_setmode@basic@pipe-a-dp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.18, 2.25] s

  * igt@kms_setmode@basic@pipe-a-edp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.21, 2.85] s

  * igt@kms_setmode@basic@pipe-a-hdmi-a-1:
    - Statuses : 2 pass(s)
    - Exec time: [1.05, 2.12] s

  * igt@kms_setmode@basic@pipe-a-hdmi-a-1-pipe-b-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.36] s

  * igt@kms_setmode@basic@pipe-a-hdmi-a-1-pipe-c-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.38] s

  * igt@kms_setmode@basic@pipe-a-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.13] s

  * igt@kms_setmode@basic@pipe-a-vga-1:
    - Statuses : 1 fail(s)
    - Exec time: [2.12] s

  * igt@kms_setmode@basic@pipe-b-dp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.19, 2.25] s

  * igt@kms_setmode@basic@pipe-b-edp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.22, 2.91] s

  * igt@kms_setmode@basic@pipe-b-hdmi-a-1:
    - Statuses : 2 pass(s)
    - Exec time: [1.05, 2.12] s

  * igt@kms_setmode@basic@pipe-b-hdmi-a-1-pipe-a-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.38] s

  * igt@kms_setmode@basic@pipe-b-hdmi-a-1-pipe-c-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.38] s

  * igt@kms_setmode@basic@pipe-b-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.12] s

  * igt@kms_setmode@basic@pipe-b-vga-1:
    - Statuses : 1 fail(s)
    - Exec time: [2.11] s

  * igt@kms_setmode@basic@pipe-c-dp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.18, 2.25] s

  * igt@kms_setmode@basic@pipe-c-edp-1:
    - Statuses : 2 pass(s)
    - Exec time: [2.17, 2.89] s

  * igt@kms_setmode@basic@pipe-c-hdmi-a-1:
    - Statuses : 2 pass(s)
    - Exec time: [1.05, 2.12] s

  * igt@kms_setmode@basic@pipe-c-hdmi-a-1-pipe-a-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.37] s

  * igt@kms_setmode@basic@pipe-c-hdmi-a-1-pipe-b-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.35] s

  * igt@kms_setmode@basic@pipe-c-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [2.13] s

  * igt@kms_setmode@basic@pipe-d-edp-1:
    - Statuses : 1 pass(s)
    - Exec time: [2.17] s

  * igt@kms_setmode@basic@pipe-d-hdmi-a-1:
    - Statuses : 1 pass(s)
    - Exec time: [1.05] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-a-hdmi-a-1-pipe-b-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.28] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-a-hdmi-a-1-pipe-c-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.28] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-b-hdmi-a-1-pipe-a-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.28] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-b-hdmi-a-1-pipe-c-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.41] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-c-hdmi-a-1-pipe-a-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.27] s

  * igt@kms_setmode@clone-exclusive-crtc@pipe-c-hdmi-a-1-pipe-b-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.28] s

  * igt@kms_setmode@invalid-clone-single-crtc-stealing@pipe-a-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.33] s

  * igt@kms_setmode@invalid-clone-single-crtc-stealing@pipe-b-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.33] s

  * igt@kms_setmode@invalid-clone-single-crtc-stealing@pipe-c-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.32] s

  * igt@kms_setmode@invalid-clone-single-crtc@pipe-a-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.04] s

  * igt@kms_setmode@invalid-clone-single-crtc@pipe-b-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.05] s

  * igt@kms_setmode@invalid-clone-single-crtc@pipe-c-hdmi-a-1-hdmi-a-2:
    - Statuses : 1 pass(s)
    - Exec time: [0.05] s

  

Known issues
------------

  Here are the changes found in IGTPW_6754_full that come from known issues:

### IGT changes ###

#### Issues hit ####

  * igt@feature_discovery@display-4x:
    - shard-tglb:         NOTRUN -> [SKIP][10] ([i915#1839]) +1 similar issue
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb5/igt@feature_discovery@display-4x.html
    - shard-iclb:         NOTRUN -> [SKIP][11] ([i915#1839]) +1 similar issue
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@feature_discovery@display-4x.html

  * igt@gem_create@create-massive:
    - shard-iclb:         NOTRUN -> [DMESG-WARN][12] ([i915#4991])
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@gem_create@create-massive.html
    - shard-snb:          NOTRUN -> [DMESG-WARN][13] ([i915#4991])
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb7/igt@gem_create@create-massive.html
    - shard-kbl:          NOTRUN -> [DMESG-WARN][14] ([i915#4991])
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl7/igt@gem_create@create-massive.html
    - shard-tglb:         NOTRUN -> [DMESG-WARN][15] ([i915#4991])
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb7/igt@gem_create@create-massive.html
    - shard-apl:          NOTRUN -> [DMESG-WARN][16] ([i915#4991])
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl7/igt@gem_create@create-massive.html

  * igt@gem_ctx_isolation@preservation-s3@bcs0:
    - shard-apl:          [PASS][17] -> [DMESG-WARN][18] ([i915#180]) +1 similar issue
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-apl3/igt@gem_ctx_isolation@preservation-s3@bcs0.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl4/igt@gem_ctx_isolation@preservation-s3@bcs0.html

  * igt@gem_ctx_persistence@engines-mixed:
    - shard-snb:          NOTRUN -> [SKIP][19] ([fdo#109271] / [i915#1099])
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb6/igt@gem_ctx_persistence@engines-mixed.html

  * igt@gem_ctx_persistence@many-contexts:
    - shard-tglb:         [PASS][20] -> [FAIL][21] ([i915#2410])
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-tglb7/igt@gem_ctx_persistence@many-contexts.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb7/igt@gem_ctx_persistence@many-contexts.html

  * igt@gem_exec_balancer@parallel-contexts:
    - shard-iclb:         NOTRUN -> [SKIP][22] ([i915#4525])
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb8/igt@gem_exec_balancer@parallel-contexts.html

  * igt@gem_exec_balancer@parallel-keep-in-fence:
    - shard-tglb:         NOTRUN -> [DMESG-WARN][23] ([i915#5076])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@gem_exec_balancer@parallel-keep-in-fence.html
    - shard-kbl:          NOTRUN -> [DMESG-WARN][24] ([i915#5076]) +1 similar issue
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl1/igt@gem_exec_balancer@parallel-keep-in-fence.html
    - shard-iclb:         NOTRUN -> [DMESG-WARN][25] ([i915#5076])
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb2/igt@gem_exec_balancer@parallel-keep-in-fence.html

  * igt@gem_exec_fair@basic-none-share@rcs0:
    - shard-iclb:         [PASS][26] -> [FAIL][27] ([i915#2842])
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-iclb7/igt@gem_exec_fair@basic-none-share@rcs0.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@gem_exec_fair@basic-none-share@rcs0.html

  * igt@gem_exec_fair@basic-pace-share@rcs0:
    - shard-tglb:         [PASS][28] -> [FAIL][29] ([i915#2842])
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-tglb2/igt@gem_exec_fair@basic-pace-share@rcs0.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb3/igt@gem_exec_fair@basic-pace-share@rcs0.html

  * igt@gem_exec_fair@basic-pace-solo@rcs0:
    - shard-iclb:         NOTRUN -> [FAIL][30] ([i915#2842])
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb5/igt@gem_exec_fair@basic-pace-solo@rcs0.html
    - shard-glk:          [PASS][31] -> [FAIL][32] ([i915#2842]) +1 similar issue
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-glk4/igt@gem_exec_fair@basic-pace-solo@rcs0.html
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk2/igt@gem_exec_fair@basic-pace-solo@rcs0.html
    - shard-tglb:         NOTRUN -> [FAIL][33] ([i915#2842])
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb8/igt@gem_exec_fair@basic-pace-solo@rcs0.html

  * igt@gem_exec_fair@basic-throttle@rcs0:
    - shard-iclb:         NOTRUN -> [FAIL][34] ([i915#2849])
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@gem_exec_fair@basic-throttle@rcs0.html

  * igt@gem_exec_params@no-bsd:
    - shard-tglb:         NOTRUN -> [SKIP][35] ([fdo#109283])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@gem_exec_params@no-bsd.html
    - shard-iclb:         NOTRUN -> [SKIP][36] ([fdo#109283])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb2/igt@gem_exec_params@no-bsd.html

  * igt@gem_lmem_swapping@parallel-random-verify:
    - shard-iclb:         NOTRUN -> [SKIP][37] ([i915#4613]) +2 similar issues
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@gem_lmem_swapping@parallel-random-verify.html

  * igt@gem_lmem_swapping@random:
    - shard-apl:          NOTRUN -> [SKIP][38] ([fdo#109271] / [i915#4613]) +3 similar issues
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl3/igt@gem_lmem_swapping@random.html
    - shard-tglb:         NOTRUN -> [SKIP][39] ([i915#4613]) +3 similar issues
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb7/igt@gem_lmem_swapping@random.html
    - shard-glk:          NOTRUN -> [SKIP][40] ([fdo#109271] / [i915#4613])
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk1/igt@gem_lmem_swapping@random.html

  * igt@gem_lmem_swapping@smem-oom:
    - shard-kbl:          NOTRUN -> [SKIP][41] ([fdo#109271] / [i915#4613]) +4 similar issues
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl4/igt@gem_lmem_swapping@smem-oom.html

  * igt@gem_pwrite@basic-exhaustion:
    - shard-snb:          NOTRUN -> [WARN][42] ([i915#2658])
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb7/igt@gem_pwrite@basic-exhaustion.html
    - shard-kbl:          NOTRUN -> [WARN][43] ([i915#2658])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl3/igt@gem_pwrite@basic-exhaustion.html

  * igt@gem_pxp@reject-modify-context-protection-off-1:
    - shard-iclb:         NOTRUN -> [SKIP][44] ([i915#4270]) +4 similar issues
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@gem_pxp@reject-modify-context-protection-off-1.html

  * igt@gem_pxp@verify-pxp-stale-ctx-execution:
    - shard-tglb:         NOTRUN -> [SKIP][45] ([i915#4270]) +3 similar issues
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb2/igt@gem_pxp@verify-pxp-stale-ctx-execution.html

  * igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs:
    - shard-iclb:         NOTRUN -> [SKIP][46] ([i915#768]) +2 similar issues
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@gem_render_copy@y-tiled-ccs-to-y-tiled-mc-ccs.html

  * igt@gem_userptr_blits@dmabuf-sync:
    - shard-apl:          NOTRUN -> [SKIP][47] ([fdo#109271] / [i915#3323])
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl4/igt@gem_userptr_blits@dmabuf-sync.html

  * igt@gem_userptr_blits@dmabuf-unsync:
    - shard-tglb:         NOTRUN -> [SKIP][48] ([i915#3297]) +1 similar issue
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb3/igt@gem_userptr_blits@dmabuf-unsync.html

  * igt@gem_userptr_blits@unsync-unmap:
    - shard-iclb:         NOTRUN -> [SKIP][49] ([i915#3297]) +1 similar issue
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb1/igt@gem_userptr_blits@unsync-unmap.html

  * igt@gem_userptr_blits@vma-merge:
    - shard-snb:          NOTRUN -> [FAIL][50] ([i915#2724])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb6/igt@gem_userptr_blits@vma-merge.html
    - shard-apl:          NOTRUN -> [FAIL][51] ([i915#3318])
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl7/igt@gem_userptr_blits@vma-merge.html
    - shard-iclb:         NOTRUN -> [FAIL][52] ([i915#3318])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb1/igt@gem_userptr_blits@vma-merge.html
    - shard-glk:          NOTRUN -> [FAIL][53] ([i915#3318])
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk9/igt@gem_userptr_blits@vma-merge.html
    - shard-kbl:          NOTRUN -> [FAIL][54] ([i915#3318])
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl7/igt@gem_userptr_blits@vma-merge.html
    - shard-tglb:         NOTRUN -> [FAIL][55] ([i915#3318])
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@gem_userptr_blits@vma-merge.html

  * igt@gen7_exec_parse@basic-offset:
    - shard-apl:          NOTRUN -> [SKIP][56] ([fdo#109271]) +140 similar issues
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl8/igt@gen7_exec_parse@basic-offset.html

  * igt@gen7_exec_parse@batch-without-end:
    - shard-tglb:         NOTRUN -> [SKIP][57] ([fdo#109289]) +1 similar issue
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb8/igt@gen7_exec_parse@batch-without-end.html

  * igt@gen9_exec_parse@allowed-all:
    - shard-iclb:         NOTRUN -> [SKIP][58] ([i915#2856]) +2 similar issues
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@gen9_exec_parse@allowed-all.html

  * igt@gen9_exec_parse@shadow-peek:
    - shard-tglb:         NOTRUN -> [SKIP][59] ([i915#2527] / [i915#2856]) +3 similar issues
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb5/igt@gen9_exec_parse@shadow-peek.html

  * igt@i915_pm_dc@dc9-dpms:
    - shard-iclb:         [PASS][60] -> [SKIP][61] ([i915#4281])
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-iclb7/igt@i915_pm_dc@dc9-dpms.html
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@i915_pm_dc@dc9-dpms.html

  * igt@i915_pm_rc6_residency@rc6-idle:
    - shard-tglb:         NOTRUN -> [WARN][62] ([i915#2681] / [i915#2684])
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb1/igt@i915_pm_rc6_residency@rc6-idle.html
    - shard-iclb:         NOTRUN -> [WARN][63] ([i915#1804] / [i915#2684])
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@i915_pm_rc6_residency@rc6-idle.html

  * igt@i915_pm_rpm@modeset-non-lpsp-stress:
    - shard-tglb:         NOTRUN -> [SKIP][64] ([fdo#111644] / [i915#1397] / [i915#2411]) +1 similar issue
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb2/igt@i915_pm_rpm@modeset-non-lpsp-stress.html

  * igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait:
    - shard-iclb:         NOTRUN -> [SKIP][65] ([fdo#110892]) +1 similar issue
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb8/igt@i915_pm_rpm@modeset-non-lpsp-stress-no-wait.html

  * igt@i915_pm_rpm@modeset-pc8-residency-stress:
    - shard-tglb:         NOTRUN -> [SKIP][66] ([fdo#109506] / [i915#2411]) +1 similar issue
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@i915_pm_rpm@modeset-pc8-residency-stress.html

  * igt@kms_atomic_transition@plane-all-modeset-transition-fencing:
    - shard-iclb:         NOTRUN -> [SKIP][67] ([i915#1769])
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html
    - shard-tglb:         NOTRUN -> [SKIP][68] ([i915#1769]) +1 similar issue
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb3/igt@kms_atomic_transition@plane-all-modeset-transition-fencing.html

  * igt@kms_big_fb@linear-64bpp-rotate-90:
    - shard-iclb:         NOTRUN -> [SKIP][69] ([fdo#110725] / [fdo#111614]) +2 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@kms_big_fb@linear-64bpp-rotate-90.html

  * igt@kms_big_fb@linear-8bpp-rotate-270:
    - shard-tglb:         NOTRUN -> [SKIP][70] ([fdo#111614]) +4 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb1/igt@kms_big_fb@linear-8bpp-rotate-270.html

  * igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip:
    - shard-kbl:          NOTRUN -> [SKIP][71] ([fdo#109271] / [i915#3777]) +3 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl6/igt@kms_big_fb@x-tiled-max-hw-stride-64bpp-rotate-180-hflip-async-flip.html

  * igt@kms_big_fb@y-tiled-32bpp-rotate-0:
    - shard-glk:          [PASS][72] -> [DMESG-WARN][73] ([i915#118])
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-glk9/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk8/igt@kms_big_fb@y-tiled-32bpp-rotate-0.html

  * igt@kms_big_fb@yf-tiled-8bpp-rotate-270:
    - shard-iclb:         NOTRUN -> [SKIP][74] ([fdo#110723]) +3 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb2/igt@kms_big_fb@yf-tiled-8bpp-rotate-270.html

  * igt@kms_big_fb@yf-tiled-addfb-size-overflow:
    - shard-tglb:         NOTRUN -> [SKIP][75] ([fdo#111615]) +6 similar issues
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb1/igt@kms_big_fb@yf-tiled-addfb-size-overflow.html

  * igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip:
    - shard-apl:          NOTRUN -> [SKIP][76] ([fdo#109271] / [i915#3777]) +2 similar issues
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl1/igt@kms_big_fb@yf-tiled-max-hw-stride-32bpp-rotate-0-hflip.html

  * igt@kms_big_joiner@basic:
    - shard-tglb:         NOTRUN -> [SKIP][77] ([i915#2705])
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb3/igt@kms_big_joiner@basic.html
    - shard-iclb:         NOTRUN -> [SKIP][78] ([i915#2705])
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@kms_big_joiner@basic.html

  * igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc:
    - shard-kbl:          NOTRUN -> [SKIP][79] ([fdo#109271] / [i915#3886]) +10 similar issues
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl1/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html
    - shard-glk:          NOTRUN -> [SKIP][80] ([fdo#109271] / [i915#3886]) +3 similar issues
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk8/igt@kms_ccs@pipe-a-crc-primary-rotation-180-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_ccs:
    - shard-snb:          NOTRUN -> [SKIP][81] ([fdo#109271]) +257 similar issues
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb7/igt@kms_ccs@pipe-b-bad-pixel-format-y_tiled_ccs.html

  * igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_ccs:
    - shard-tglb:         NOTRUN -> [SKIP][82] ([i915#3689]) +8 similar issues
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb5/igt@kms_ccs@pipe-b-bad-rotation-90-y_tiled_ccs.html

  * igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs:
    - shard-tglb:         NOTRUN -> [SKIP][83] ([i915#3689] / [i915#3886]) +5 similar issues
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb2/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_mc_ccs.html

  * igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc:
    - shard-apl:          NOTRUN -> [SKIP][84] ([fdo#109271] / [i915#3886]) +8 similar issues
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl8/igt@kms_ccs@pipe-b-crc-sprite-planes-basic-y_tiled_gen12_rc_ccs_cc.html

  * igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_mc_ccs:
    - shard-iclb:         NOTRUN -> [SKIP][85] ([fdo#109278] / [i915#3886]) +8 similar issues
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@kms_ccs@pipe-b-random-ccs-data-y_tiled_gen12_mc_ccs.html

  * igt@kms_ccs@pipe-c-random-ccs-data-yf_tiled_ccs:
    - shard-tglb:         NOTRUN -> [SKIP][86] ([fdo#111615] / [i915#3689]) +4 similar issues
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb7/igt@kms_ccs@pipe-c-random-ccs-data-yf_tiled_ccs.html

  * igt@kms_chamelium@hdmi-crc-fast:
    - shard-iclb:         NOTRUN -> [SKIP][87] ([fdo#109284] / [fdo#111827]) +13 similar issues
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@kms_chamelium@hdmi-crc-fast.html

  * igt@kms_chamelium@hdmi-hpd-storm:
    - shard-kbl:          NOTRUN -> [SKIP][88] ([fdo#109271] / [fdo#111827]) +24 similar issues
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl3/igt@kms_chamelium@hdmi-hpd-storm.html

  * igt@kms_chamelium@vga-hpd:
    - shard-tglb:         NOTRUN -> [SKIP][89] ([fdo#109284] / [fdo#111827]) +16 similar issues
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb8/igt@kms_chamelium@vga-hpd.html

  * igt@kms_color@pipe-d-ctm-0-25:
    - shard-iclb:         NOTRUN -> [SKIP][90] ([fdo#109278] / [i915#1149]) +2 similar issues
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@kms_color@pipe-d-ctm-0-25.html

  * igt@kms_color_chamelium@pipe-a-ctm-green-to-red:
    - shard-glk:          NOTRUN -> [SKIP][91] ([fdo#109271] / [fdo#111827]) +7 similar issues
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk5/igt@kms_color_chamelium@pipe-a-ctm-green-to-red.html

  * igt@kms_color_chamelium@pipe-c-ctm-0-25:
    - shard-apl:          NOTRUN -> [SKIP][92] ([fdo#109271] / [fdo#111827]) +11 similar issues
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl8/igt@kms_color_chamelium@pipe-c-ctm-0-25.html

  * igt@kms_color_chamelium@pipe-d-ctm-0-5:
    - shard-snb:          NOTRUN -> [SKIP][93] ([fdo#109271] / [fdo#111827]) +14 similar issues
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-snb4/igt@kms_color_chamelium@pipe-d-ctm-0-5.html

  * igt@kms_color_chamelium@pipe-d-gamma:
    - shard-iclb:         NOTRUN -> [SKIP][94] ([fdo#109278] / [fdo#109284] / [fdo#111827]) +1 similar issue
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb5/igt@kms_color_chamelium@pipe-d-gamma.html

  * igt@kms_content_protection@dp-mst-type-0:
    - shard-tglb:         NOTRUN -> [SKIP][95] ([i915#3116] / [i915#3299])
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb1/igt@kms_content_protection@dp-mst-type-0.html
    - shard-iclb:         NOTRUN -> [SKIP][96] ([i915#3116])
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb6/igt@kms_content_protection@dp-mst-type-0.html

  * igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen:
    - shard-iclb:         NOTRUN -> [SKIP][97] ([fdo#109278] / [fdo#109279]) +1 similar issue
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@kms_cursor_crc@pipe-a-cursor-512x512-offscreen.html

  * igt@kms_cursor_crc@pipe-b-cursor-32x32-sliding:
    - shard-tglb:         NOTRUN -> [SKIP][98] ([i915#3319]) +2 similar issues
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb5/igt@kms_cursor_crc@pipe-b-cursor-32x32-sliding.html

  * igt@kms_cursor_crc@pipe-b-cursor-max-size-rapid-movement:
    - shard-tglb:         NOTRUN -> [SKIP][99] ([i915#3359]) +4 similar issues
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@kms_cursor_crc@pipe-b-cursor-max-size-rapid-movement.html

  * igt@kms_cursor_crc@pipe-d-cursor-256x85-random:
    - shard-glk:          NOTRUN -> [SKIP][100] ([fdo#109271]) +71 similar issues
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-glk8/igt@kms_cursor_crc@pipe-d-cursor-256x85-random.html

  * igt@kms_cursor_crc@pipe-d-cursor-512x170-offscreen:
    - shard-tglb:         NOTRUN -> [SKIP][101] ([fdo#109279] / [i915#3359]) +4 similar issues
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@kms_cursor_crc@pipe-d-cursor-512x170-offscreen.html

  * igt@kms_cursor_crc@pipe-d-cursor-suspend:
    - shard-kbl:          NOTRUN -> [SKIP][102] ([fdo#109271]) +246 similar issues
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl4/igt@kms_cursor_crc@pipe-d-cursor-suspend.html

  * igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size:
    - shard-iclb:         NOTRUN -> [SKIP][103] ([fdo#109274] / [fdo#109278]) +5 similar issues
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb3/igt@kms_cursor_legacy@cursorb-vs-flipb-atomic-transitions-varying-size.html

  * igt@kms_cursor_legacy@pipe-d-torture-bo:
    - shard-kbl:          NOTRUN -> [SKIP][104] ([fdo#109271] / [i915#533]) +1 similar issue
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl6/igt@kms_cursor_legacy@pipe-d-torture-bo.html
    - shard-apl:          NOTRUN -> [SKIP][105] ([fdo#109271] / [i915#533]) +1 similar issue
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-apl1/igt@kms_cursor_legacy@pipe-d-torture-bo.html

  * igt@kms_cursor_legacy@short-busy-flip-before-cursor-toggle:
    - shard-tglb:         NOTRUN -> [SKIP][106] ([i915#4103])
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb7/igt@kms_cursor_legacy@short-busy-flip-before-cursor-toggle.html

  * igt@kms_flip@2x-absolute-wf_vblank:
    - shard-tglb:         NOTRUN -> [SKIP][107] ([fdo#109274] / [fdo#111825] / [i915#3966])
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@kms_flip@2x-absolute-wf_vblank.html

  * igt@kms_flip@2x-blocking-absolute-wf_vblank-interruptible:
    - shard-tglb:         NOTRUN -> [SKIP][108] ([fdo#109274] / [fdo#111825]) +9 similar issues
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-tglb6/igt@kms_flip@2x-blocking-absolute-wf_vblank-interruptible.html

  * igt@kms_flip@2x-plain-flip-fb-recreate-interruptible:
    - shard-iclb:         NOTRUN -> [SKIP][109] ([fdo#109274]) +4 similar issues
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-iclb4/igt@kms_flip@2x-plain-flip-fb-recreate-interruptible.html

  * igt@kms_flip@flip-vs-suspend-interruptible@a-dp1:
    - shard-kbl:          [PASS][110] -> [DMESG-WARN][111] ([i915#180]) +3 similar issues
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11336/shard-kbl7/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/shard-kbl6/igt@kms_flip@flip-vs-suspend-interruptible@a-dp1.html

  * igt@kms_flip@flip-vs-s

== Logs ==

For more details see: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_6754/index.html

[-- Attachment #2: Type: text/html, Size: 34516 bytes --]

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests
  2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests Bhanuprakash Modem
@ 2022-03-15 13:37   ` Rodrigo Siqueira Jordao
  2022-03-15 13:52     ` Petri Latvala
  2022-03-15 13:56     ` Modem, Bhanuprakash
  0 siblings, 2 replies; 10+ messages in thread
From: Rodrigo Siqueira Jordao @ 2022-03-15 13:37 UTC (permalink / raw)
  To: Bhanuprakash Modem, Swati Sharma, Mark Yacoub, Pillai, Aurabindo,
	Nicholas Choi
  Cc: igt-dev, Hayden Goodfellow



On 2022-03-08 09:22, Bhanuprakash Modem wrote:
> Add support to use dynamic subtests.
> 
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
> ---
>   tests/kms_setmode.c | 37 ++++++++++++++++++++++++++++++++++---
>   1 file changed, 34 insertions(+), 3 deletions(-)
> 
> diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
> index 18d81ee4f3..a47d47e5ab 100644
> --- a/tests/kms_setmode.c
> +++ b/tests/kms_setmode.c
> @@ -653,6 +653,26 @@ static void test_crtc_config(const struct test_config *tconf,
>   	return;
>   }
>   
> +static int get_test_name_str(struct crtc_config *crtc, char *buf,
> +				size_t buf_size)
> +{
> +	int pos;
> +	int i;
> +
> +	pos = snprintf(buf, buf_size, "pipe-%s-", kmstest_pipe_name(crtc->pipe_id));
> +
> +	for (i = 0; i < crtc->connector_count; i++) {
> +		drmModeConnector *connector = crtc->cconfs[i].connector;
> +
> +		pos += snprintf(&buf[pos], buf_size - pos,
> +			"%s%s-%d", i ? "-" : "",
> +			kmstest_connector_type_str(connector->connector_type),
> +			connector->connector_type_id);
> +	}
> +
> +	return pos;
> +}
> +
>   static void test_one_combination(const struct test_config *tconf,
>   				 struct connector_config *cconfs,
>   				 int connector_count)
> @@ -664,8 +684,19 @@ static void test_one_combination(const struct test_config *tconf,
>   	setup_crtcs(tconf, cconfs, connector_count, crtcs,
>   		    &crtc_count, &config_valid);
>   
> -	if (config_valid == !(tconf->flags & TEST_INVALID))
> -		test_crtc_config(tconf, crtcs, crtc_count);
> +	if (config_valid == !(tconf->flags & TEST_INVALID)) {
> +		int i, pos = 0;
> +		char test_name[256];
> +
> +		for (i = 0; i < crtc_count; i++) {
> +			if (i > 0)
> +				pos += snprintf(&test_name[pos], ARRAY_SIZE(test_name) - pos, "-");
> +			pos += get_test_name_str(&crtcs[i], &test_name[pos], ARRAY_SIZE(test_name) - pos);
> +		}
> +
> +		igt_dynamic_f("%s", test_name)
> +			test_crtc_config(tconf, crtcs, crtc_count);
> +	}
>   
>   	cleanup_crtcs(crtcs, crtc_count);
>   }
> @@ -939,7 +970,7 @@ igt_main_args("det:", NULL, help_str, opt_handler, NULL)
>   
>   	for (i = 0; i < ARRAY_SIZE(tests); i++) {
>   		igt_describe("Tests the mode by iterating through all valid/invalid crtc/connector combinations");
> -		igt_subtest(tests[i].name) {
> +		igt_subtest_with_dynamic(tests[i].name) {

Hi Bhanuprakash,

The above change causes a regression in AMD CI. Previously, all the 
below tests were passing, but now they are skipping.

basic-clone-single-crtc
clone-exclusive-crtc
invalid-clone-exclusive-crtc
invalid-clone-single-crtc
invalid-clone-single-crtc-stealing
basic-clone-single-crtc

If we use igt_subtest instead of igt_subtest_with_dynamic, everything 
gets back to normal. Can we reintroduce igt_subtest? Or is there a 
better way to fix this issue?

Thanks
Siqueira


>   			struct test_config tconf = {
>   				.flags		= tests[i].flags,
>   				.name		= tests[i].name,

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests
  2022-03-15 13:37   ` Rodrigo Siqueira Jordao
@ 2022-03-15 13:52     ` Petri Latvala
  2022-03-16 12:34       ` Rodrigo Siqueira Jordao
  2022-03-15 13:56     ` Modem, Bhanuprakash
  1 sibling, 1 reply; 10+ messages in thread
From: Petri Latvala @ 2022-03-15 13:52 UTC (permalink / raw)
  To: Rodrigo Siqueira Jordao; +Cc: Hayden Goodfellow, igt-dev

On Tue, Mar 15, 2022 at 09:37:31AM -0400, Rodrigo Siqueira Jordao wrote:
> 
> 
> On 2022-03-08 09:22, Bhanuprakash Modem wrote:
> > Add support to use dynamic subtests.
> > 
> > Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
> > Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
> > ---
> >   tests/kms_setmode.c | 37 ++++++++++++++++++++++++++++++++++---
> >   1 file changed, 34 insertions(+), 3 deletions(-)
> > 
> > diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
> > index 18d81ee4f3..a47d47e5ab 100644
> > --- a/tests/kms_setmode.c
> > +++ b/tests/kms_setmode.c
> > @@ -653,6 +653,26 @@ static void test_crtc_config(const struct test_config *tconf,
> >   	return;
> >   }
> > +static int get_test_name_str(struct crtc_config *crtc, char *buf,
> > +				size_t buf_size)
> > +{
> > +	int pos;
> > +	int i;
> > +
> > +	pos = snprintf(buf, buf_size, "pipe-%s-", kmstest_pipe_name(crtc->pipe_id));
> > +
> > +	for (i = 0; i < crtc->connector_count; i++) {
> > +		drmModeConnector *connector = crtc->cconfs[i].connector;
> > +
> > +		pos += snprintf(&buf[pos], buf_size - pos,
> > +			"%s%s-%d", i ? "-" : "",
> > +			kmstest_connector_type_str(connector->connector_type),
> > +			connector->connector_type_id);
> > +	}
> > +
> > +	return pos;
> > +}
> > +
> >   static void test_one_combination(const struct test_config *tconf,
> >   				 struct connector_config *cconfs,
> >   				 int connector_count)
> > @@ -664,8 +684,19 @@ static void test_one_combination(const struct test_config *tconf,
> >   	setup_crtcs(tconf, cconfs, connector_count, crtcs,
> >   		    &crtc_count, &config_valid);
> > -	if (config_valid == !(tconf->flags & TEST_INVALID))
> > -		test_crtc_config(tconf, crtcs, crtc_count);
> > +	if (config_valid == !(tconf->flags & TEST_INVALID)) {
> > +		int i, pos = 0;
> > +		char test_name[256];
> > +
> > +		for (i = 0; i < crtc_count; i++) {
> > +			if (i > 0)
> > +				pos += snprintf(&test_name[pos], ARRAY_SIZE(test_name) - pos, "-");
> > +			pos += get_test_name_str(&crtcs[i], &test_name[pos], ARRAY_SIZE(test_name) - pos);
> > +		}
> > +
> > +		igt_dynamic_f("%s", test_name)
> > +			test_crtc_config(tconf, crtcs, crtc_count);
> > +	}
> >   	cleanup_crtcs(crtcs, crtc_count);
> >   }
> > @@ -939,7 +970,7 @@ igt_main_args("det:", NULL, help_str, opt_handler, NULL)
> >   	for (i = 0; i < ARRAY_SIZE(tests); i++) {
> >   		igt_describe("Tests the mode by iterating through all valid/invalid crtc/connector combinations");
> > -		igt_subtest(tests[i].name) {
> > +		igt_subtest_with_dynamic(tests[i].name) {
> 
> Hi Bhanuprakash,
> 
> The above change causes a regression in AMD CI. Previously, all the below
> tests were passing, but now they are skipping.
> 
> basic-clone-single-crtc
> clone-exclusive-crtc
> invalid-clone-exclusive-crtc
> invalid-clone-single-crtc
> invalid-clone-single-crtc-stealing
> basic-clone-single-crtc
> 
> If we use igt_subtest instead of igt_subtest_with_dynamic, everything gets
> back to normal. Can we reintroduce igt_subtest? Or is there a better way to
> fix this issue?

What is the skip message? If it's "no dynamic subtests executed", then
you don't have any valid configs. Previously that would just silently
give a pass even when the test did absolutely nothing.


-- 
Petri Latvala

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests
  2022-03-15 13:37   ` Rodrigo Siqueira Jordao
  2022-03-15 13:52     ` Petri Latvala
@ 2022-03-15 13:56     ` Modem, Bhanuprakash
  1 sibling, 0 replies; 10+ messages in thread
From: Modem, Bhanuprakash @ 2022-03-15 13:56 UTC (permalink / raw)
  To: Rodrigo Siqueira Jordao, Swati Sharma, Mark Yacoub, Pillai,
	Aurabindo, Nicholas Choi
  Cc: igt-dev, Hayden Goodfellow

On Tue-15-03-2022 07:07 pm, Rodrigo Siqueira Jordao wrote:
> 
> 
> On 2022-03-08 09:22, Bhanuprakash Modem wrote:
>> Add support to use dynamic subtests.
>>
>> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
>> Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
>> ---
>>   tests/kms_setmode.c | 37 ++++++++++++++++++++++++++++++++++---
>>   1 file changed, 34 insertions(+), 3 deletions(-)
>>
>> diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
>> index 18d81ee4f3..a47d47e5ab 100644
>> --- a/tests/kms_setmode.c
>> +++ b/tests/kms_setmode.c
>> @@ -653,6 +653,26 @@ static void test_crtc_config(const struct 
>> test_config *tconf,
>>       return;
>>   }
>> +static int get_test_name_str(struct crtc_config *crtc, char *buf,
>> +                size_t buf_size)
>> +{
>> +    int pos;
>> +    int i;
>> +
>> +    pos = snprintf(buf, buf_size, "pipe-%s-", 
>> kmstest_pipe_name(crtc->pipe_id));
>> +
>> +    for (i = 0; i < crtc->connector_count; i++) {
>> +        drmModeConnector *connector = crtc->cconfs[i].connector;
>> +
>> +        pos += snprintf(&buf[pos], buf_size - pos,
>> +            "%s%s-%d", i ? "-" : "",
>> +            kmstest_connector_type_str(connector->connector_type),
>> +            connector->connector_type_id);
>> +    }
>> +
>> +    return pos;
>> +}
>> +
>>   static void test_one_combination(const struct test_config *tconf,
>>                    struct connector_config *cconfs,
>>                    int connector_count)
>> @@ -664,8 +684,19 @@ static void test_one_combination(const struct 
>> test_config *tconf,
>>       setup_crtcs(tconf, cconfs, connector_count, crtcs,
>>               &crtc_count, &config_valid);
>> -    if (config_valid == !(tconf->flags & TEST_INVALID))
>> -        test_crtc_config(tconf, crtcs, crtc_count);
>> +    if (config_valid == !(tconf->flags & TEST_INVALID)) {
>> +        int i, pos = 0;
>> +        char test_name[256];
>> +
>> +        for (i = 0; i < crtc_count; i++) {
>> +            if (i > 0)
>> +                pos += snprintf(&test_name[pos], 
>> ARRAY_SIZE(test_name) - pos, "-");
>> +            pos += get_test_name_str(&crtcs[i], &test_name[pos], 
>> ARRAY_SIZE(test_name) - pos);
>> +        }
>> +
>> +        igt_dynamic_f("%s", test_name)
>> +            test_crtc_config(tconf, crtcs, crtc_count);
>> +    }
>>       cleanup_crtcs(crtcs, crtc_count);
>>   }
>> @@ -939,7 +970,7 @@ igt_main_args("det:", NULL, help_str, opt_handler, 
>> NULL)
>>       for (i = 0; i < ARRAY_SIZE(tests); i++) {
>>           igt_describe("Tests the mode by iterating through all 
>> valid/invalid crtc/connector combinations");
>> -        igt_subtest(tests[i].name) {
>> +        igt_subtest_with_dynamic(tests[i].name) {
> 
> Hi Bhanuprakash,
> 
> The above change causes a regression in AMD CI. Previously, all the 
> below tests were passing, but now they are skipping.
> 
> basic-clone-single-crtc
> clone-exclusive-crtc
> invalid-clone-exclusive-crtc
> invalid-clone-single-crtc
> invalid-clone-single-crtc-stealing
> basic-clone-single-crtc
> 
> If we use igt_subtest instead of igt_subtest_with_dynamic, everything 
> gets back to normal. Can we reintroduce igt_subtest? Or is there a 
> better way to fix this issue?

Hi Siqueira,

It is not a regression. On single display config these SKIPs are 
expected due to the introduction of dynamic subtests. In other wards, we 
need at least 2 displays to run these tests. Earlier on single display 
setup, these tests were just throwing SUCCESS without actually 
triggering the test.

Also, due to the long CI execution time on multidisplay setups, we were 
restricted the execution to 2 pipes only. In patch [2/3] in this series 
we reverted that limitation too. Hence we introduced dynamic subtests.

- Bhanu

> 
> Thanks
> Siqueira
> 
> 
>>               struct test_config tconf = {
>>                   .flags        = tests[i].flags,
>>                   .name        = tests[i].name,
> 

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests
  2022-03-15 13:52     ` Petri Latvala
@ 2022-03-16 12:34       ` Rodrigo Siqueira Jordao
  0 siblings, 0 replies; 10+ messages in thread
From: Rodrigo Siqueira Jordao @ 2022-03-16 12:34 UTC (permalink / raw)
  To: Petri Latvala, Bhanuprakash Modem; +Cc: igt-dev, Hayden Goodfellow



On 2022-03-15 09:52, Petri Latvala wrote:
> On Tue, Mar 15, 2022 at 09:37:31AM -0400, Rodrigo Siqueira Jordao wrote:
>>
>>
>> On 2022-03-08 09:22, Bhanuprakash Modem wrote:
>>> Add support to use dynamic subtests.
>>>
>>> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
>>> Reviewed-by: Swati Sharma <swati2.sharma@intel.com>
>>> ---
>>>    tests/kms_setmode.c | 37 ++++++++++++++++++++++++++++++++++---
>>>    1 file changed, 34 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/tests/kms_setmode.c b/tests/kms_setmode.c
>>> index 18d81ee4f3..a47d47e5ab 100644
>>> --- a/tests/kms_setmode.c
>>> +++ b/tests/kms_setmode.c
>>> @@ -653,6 +653,26 @@ static void test_crtc_config(const struct test_config *tconf,
>>>    	return;
>>>    }
>>> +static int get_test_name_str(struct crtc_config *crtc, char *buf,
>>> +				size_t buf_size)
>>> +{
>>> +	int pos;
>>> +	int i;
>>> +
>>> +	pos = snprintf(buf, buf_size, "pipe-%s-", kmstest_pipe_name(crtc->pipe_id));
>>> +
>>> +	for (i = 0; i < crtc->connector_count; i++) {
>>> +		drmModeConnector *connector = crtc->cconfs[i].connector;
>>> +
>>> +		pos += snprintf(&buf[pos], buf_size - pos,
>>> +			"%s%s-%d", i ? "-" : "",
>>> +			kmstest_connector_type_str(connector->connector_type),
>>> +			connector->connector_type_id);
>>> +	}
>>> +
>>> +	return pos;
>>> +}
>>> +
>>>    static void test_one_combination(const struct test_config *tconf,
>>>    				 struct connector_config *cconfs,
>>>    				 int connector_count)
>>> @@ -664,8 +684,19 @@ static void test_one_combination(const struct test_config *tconf,
>>>    	setup_crtcs(tconf, cconfs, connector_count, crtcs,
>>>    		    &crtc_count, &config_valid);
>>> -	if (config_valid == !(tconf->flags & TEST_INVALID))
>>> -		test_crtc_config(tconf, crtcs, crtc_count);
>>> +	if (config_valid == !(tconf->flags & TEST_INVALID)) {
>>> +		int i, pos = 0;
>>> +		char test_name[256];
>>> +
>>> +		for (i = 0; i < crtc_count; i++) {
>>> +			if (i > 0)
>>> +				pos += snprintf(&test_name[pos], ARRAY_SIZE(test_name) - pos, "-");
>>> +			pos += get_test_name_str(&crtcs[i], &test_name[pos], ARRAY_SIZE(test_name) - pos);
>>> +		}
>>> +
>>> +		igt_dynamic_f("%s", test_name)
>>> +			test_crtc_config(tconf, crtcs, crtc_count);
>>> +	}
>>>    	cleanup_crtcs(crtcs, crtc_count);
>>>    }
>>> @@ -939,7 +970,7 @@ igt_main_args("det:", NULL, help_str, opt_handler, NULL)
>>>    	for (i = 0; i < ARRAY_SIZE(tests); i++) {
>>>    		igt_describe("Tests the mode by iterating through all valid/invalid crtc/connector combinations");
>>> -		igt_subtest(tests[i].name) {
>>> +		igt_subtest_with_dynamic(tests[i].name) {
>>
>> Hi Bhanuprakash,
>>
>> The above change causes a regression in AMD CI. Previously, all the below
>> tests were passing, but now they are skipping.
>>
>> basic-clone-single-crtc
>> clone-exclusive-crtc
>> invalid-clone-exclusive-crtc
>> invalid-clone-single-crtc
>> invalid-clone-single-crtc-stealing
>> basic-clone-single-crtc
>>
>> If we use igt_subtest instead of igt_subtest_with_dynamic, everything gets
>> back to normal. Can we reintroduce igt_subtest? Or is there a better way to
>> fix this issue?
> 
> What is the skip message? If it's "no dynamic subtests executed", then
> you don't have any valid configs. Previously that would just silently
> give a pass even when the test did absolutely nothing.
> 
> 

Hi Petri/Bhanuprakash

Thanks for the explanation. I checked with other ASICs on our side, and 
it looks like we have different results per device, which makes sense 
based on your explanation. We already updated our baseline.

Thanks again
Siqueira

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2022-03-16 12:34 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-03-08 14:22 [igt-dev] [v2 i-g-t 0/3] tests/kms_setmode: Fix stealing test Bhanuprakash Modem
2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 1/3] tests/kms_setmode: Use dynamic subtests Bhanuprakash Modem
2022-03-15 13:37   ` Rodrigo Siqueira Jordao
2022-03-15 13:52     ` Petri Latvala
2022-03-16 12:34       ` Rodrigo Siqueira Jordao
2022-03-15 13:56     ` Modem, Bhanuprakash
2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 2/3] Revert "tests/kms_setmode: Restrict the test execution to two pipes" Bhanuprakash Modem
2022-03-08 14:22 ` [igt-dev] [v2 i-g-t 3/3] tests/kms_setmode: Sort modes only if they dont fit in link BW Bhanuprakash Modem
2022-03-08 19:26 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/kms_setmode: Fix stealing test (rev3) Patchwork
2022-03-09  0:55 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.