All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names
@ 2022-07-13 15:52 Rodrigo Vivi
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement Rodrigo Vivi
                   ` (6 more replies)
  0 siblings, 7 replies; 13+ messages in thread
From: Rodrigo Vivi @ 2022-07-13 15:52 UTC (permalink / raw)
  To: igt-dev; +Cc: Rodrigo Vivi

Mixing the names with the state names can cause confusion.
Let's stick with the official names.

Cc: Riana Tauro <riana.tauro@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 lib/igt_aux.c             | 8 ++++----
 lib/igt_aux.h             | 6 +++---
 tests/i915/i915_suspend.c | 2 +-
 3 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 35c67f10..5129d9f0 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -712,9 +712,9 @@ static const char *suspend_test_name[] = {
 };
 
 static const char *mem_sleep_name[] = {
-	[MEM_SLEEP_FREEZE] = "s2idle",
-	[MEM_SLEEP_STANDBY] = "shallow",
-	[MEM_SLEEP_MEM] = "deep"
+	[MEM_SLEEP_S2IDLE] = "s2idle",
+	[MEM_SLEEP_SHALLOW] = "shallow",
+	[MEM_SLEEP_DEEP] = "deep"
 };
 
 static enum igt_suspend_test get_suspend_test(int power_dir)
@@ -993,7 +993,7 @@ int igt_get_memsleep_state(void)
 		return MEM_SLEEP_NONE;
 	}
 
-	for (mem_sleep = MEM_SLEEP_FREEZE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
+	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
 		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
 			break;
 	}
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index af663d2c..9d5b7bd2 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -196,9 +196,9 @@ enum igt_suspend_test {
  */
 enum igt_mem_sleep {
 	MEM_SLEEP_NONE,
-	MEM_SLEEP_FREEZE,
-	MEM_SLEEP_STANDBY,
-	MEM_SLEEP_MEM,
+	MEM_SLEEP_S2IDLE,
+	MEM_SLEEP_SHALLOW,
+	MEM_SLEEP_DEEP,
 
 	/*<private>*/
 	MEM_SLEEP_NUM,
diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
index 23e451d3..f992d447 100644
--- a/tests/i915/i915_suspend.c
+++ b/tests/i915/i915_suspend.c
@@ -209,7 +209,7 @@ test_suspend_without_i915(int state)
 	igt_kmsg(KMSG_INFO "Unloading i915\n");
 	igt_assert_eq(igt_i915_driver_unload(),0);
 
-	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_FREEZE && state > SUSPEND_STATE_FREEZE,
+	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE && state > SUSPEND_STATE_FREEZE,
 		      "Platform default mem_sleep state is s2idle\n");
 	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
 
-- 
2.35.3

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

* [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement.
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
@ 2022-07-13 15:52 ` Rodrigo Vivi
  2022-07-18  6:24   ` Anshuman Gupta
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state Rodrigo Vivi
                   ` (5 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Rodrigo Vivi @ 2022-07-13 15:52 UTC (permalink / raw)
  To: igt-dev; +Cc: Rodrigo Vivi

1. We are not using DISK
2. DISK has no dependency on mem_sleep.

Let's be clear on what exactly we are trying to do.

Cc: Riana Tauro <riana.tauro@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 tests/i915/i915_suspend.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
index f992d447..222a8e4a 100644
--- a/tests/i915/i915_suspend.c
+++ b/tests/i915/i915_suspend.c
@@ -209,8 +209,9 @@ test_suspend_without_i915(int state)
 	igt_kmsg(KMSG_INFO "Unloading i915\n");
 	igt_assert_eq(igt_i915_driver_unload(),0);
 
-	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE && state > SUSPEND_STATE_FREEZE,
-		      "Platform default mem_sleep state is s2idle\n");
+	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE &&
+		      state == SUSPEND_STATE_MEM,
+		      "S3 not possible: platform default mem_sleep state is s2idle\n");
 	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
 
 	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
-- 
2.35.3

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

* [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state.
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement Rodrigo Vivi
@ 2022-07-13 15:52 ` Rodrigo Vivi
  2022-07-18  6:47   ` Anshuman Gupta
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3 Rodrigo Vivi
                   ` (4 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Rodrigo Vivi @ 2022-07-13 15:52 UTC (permalink / raw)
  To: igt-dev; +Cc: Rodrigo Vivi

Testing both Suspend-to-Idle and Suspend-to-RAM is critical.
So far our test suite are relying on the system's default
for the "suspend-to-mem", what reduces our ability to cover
multiple scenarios or requires manual intervention.
This also brings confusions in some debugging scenarios.

For the Suspend-to-Idle it is easy. The FREEZE state already
enforces it. However, there's only one way to enforce the
Suspend-to-RAM (aka S3) which is using MEM state with mem_sleep
at "deep".

However, let's not break the whole world overnight. Let's
first introduce the ability to force the Suspend-to-RAM.
Then convert the tests as needed.

So, the SUSPEND_STATE_MEM will continue to exist and do
exactly what is expected when you set "mem" to /sys/power/state,
which is respect whatever is in the /sys/power/mem_sleep.

Let's introduce a new SUSPEND_STATE_S3 state aiming to force
the Suspend-to-RAM, by changing both /sys/power/state and
/sys/power/mem_sleep.

Cc: Riana Tauro <riana.tauro@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 lib/igt_aux.c | 118 ++++++++++++++++++++++++++++++--------------------
 lib/igt_aux.h |  19 +++++---
 2 files changed, 84 insertions(+), 53 deletions(-)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 5129d9f0..69ae9b58 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -698,7 +698,8 @@ static int autoresume_delay;
 static const char *suspend_state_name[] = {
 	[SUSPEND_STATE_FREEZE] = "freeze",
 	[SUSPEND_STATE_STANDBY] = "standby",
-	[SUSPEND_STATE_MEM] = "mem",
+	[SUSPEND_STATE_S3] = "mem", /* Forces Suspend-to-Ram (S3) */
+	[SUSPEND_STATE_MEM] = "mem", /* Respects system default */
 	[SUSPEND_STATE_DISK] = "disk",
 };
 
@@ -840,6 +841,63 @@ static uint32_t get_supported_suspend_states(int power_dir)
 	return state_mask;
 }
 
+/**
+ * igt_get_memsleep_state
+ *
+ * Reads the value of /sys/power/mem_sleep and
+ * returns the current suspend state associated with 'mem'.
+ *
+ * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
+ */
+int igt_get_memsleep_state(void)
+{
+	char *mem_sleep_states;
+	char *mem_sleep_state;
+	enum igt_mem_sleep mem_sleep;
+	int power_dir;
+
+	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
+
+	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
+		return MEM_SLEEP_NONE;
+
+	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
+	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
+	     mem_sleep_state = strtok(NULL, " ")) {
+		if (mem_sleep_state[0] == '[') {
+			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
+			mem_sleep_state++;
+			break;
+		}
+	}
+
+	if (!mem_sleep_state) {
+		free(mem_sleep_states);
+		return MEM_SLEEP_NONE;
+	}
+
+	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
+		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
+			break;
+	}
+
+	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
+
+	free(mem_sleep_states);
+	close(power_dir);
+	return mem_sleep;
+}
+
+static void set_mem_sleep(int power_dir, enum igt_mem_sleep sleep)
+{
+	igt_assert(sleep < MEM_SLEEP_NUM);
+
+	igt_assert_eq(faccessat(power_dir, "mem_sleep", W_OK, 0), 0);
+
+	igt_assert(igt_sysfs_set(power_dir, "mem_sleep",
+				 mem_sleep_name[sleep]));
+}
+
 /**
  * igt_system_suspend_autoresume:
  * @state: an #igt_suspend_state, the target suspend state
@@ -866,6 +924,7 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 {
 	int power_dir;
 	enum igt_suspend_test orig_test;
+	enum igt_mem_sleep orig_mem_sleep = MEM_SLEEP_NONE;
 
 	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
 	igt_require(get_supported_suspend_states(power_dir) & (1 << state));
@@ -877,6 +936,14 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 		      "Suspend to disk requires swap space.\n");
 
 	orig_test = get_suspend_test(power_dir);
+
+	if (state == SUSPEND_STATE_S3) {
+		orig_mem_sleep = igt_get_memsleep_state();
+		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
+		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
+			      "S3 not possible in this system.\n");
+	}
+
 	set_suspend_test(power_dir, test);
 
 	if (test == SUSPEND_TEST_NONE)
@@ -884,6 +951,9 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 	else
 		suspend_via_sysfs(power_dir, state);
 
+	if (orig_mem_sleep)
+		set_mem_sleep(power_dir, orig_mem_sleep);
+
 	set_suspend_test(power_dir, orig_test);
 	close(power_dir);
 }
@@ -958,52 +1028,6 @@ int igt_get_autoresume_delay(enum igt_suspend_state state)
 	return delay;
 }
 
-/**
- * igt_get_memsleep_state
- *
- * Reads the value of /sys/power/mem_sleep and
- * returns the current suspend state associated with 'mem'.
- *
- * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
- */
-int igt_get_memsleep_state(void)
-{
-	char *mem_sleep_states;
-	char *mem_sleep_state;
-	enum igt_mem_sleep mem_sleep;
-	int power_dir;
-
-	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
-
-	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
-		return MEM_SLEEP_NONE;
-
-	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
-	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
-	     mem_sleep_state = strtok(NULL, " ")) {
-		if (mem_sleep_state[0] == '[') {
-			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
-			mem_sleep_state++;
-			break;
-		}
-	}
-
-	if (!mem_sleep_state) {
-		free(mem_sleep_states);
-		return MEM_SLEEP_NONE;
-	}
-
-	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
-		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
-			break;
-	}
-
-	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
-
-	free(mem_sleep_states);
-	close(power_dir);
-	return mem_sleep;
-}
 /**
  * igt_drop_root:
  *
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index 9d5b7bd2..b1e48b0f 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -135,13 +135,19 @@ bool igt_aub_dump_enabled(void);
 
 /**
  *  igt_suspend_state:
- *  @SUSPEND_STATE_FREEZE: suspend-to-idle target state, aka S0ix or freeze,
+ *  @SUSPEND_STATE_FREEZE: Suspend-To-Idle target state, aka S0ix or freeze,
  *			   first non-hibernation state
- *  @SUSPEND_STATE_STANDBY: standby target state, aka S1, second
+ *  @SUSPEND_STATE_STANDBY: "Power-On Suspend" target state, aka S1, second
  *			    non-hibernation state
- *  @SUSPEND_STATE_MEM: suspend-to-mem target state aka S3, third
- *			non-hibernation state
- *  @SUSPEND_STATE_DISK: suspend-to-disk target state, aka S4 or hibernation
+ *  @SUSPEND_STATE_S3: Suspend-To-RAM: It enforces a "deep" state to mem_sleep,
+ *		       what forces the system to go to the third
+ *		       non-hibernation state, aka S3.
+ *  @SUSPEND_STATE_MEM: A memory sleep (non-hibernation) target state,
+ *			respecting the system's mem_sleep default:
+ *				s2idle: Suspend-To-Idle target state
+ *				shallow: "Power-On Suspend"
+ *				deep: Suspend-To-RAM
+ *  @SUSPEND_STATE_DISK: Suspend-To-Disk target state, aka S4 or hibernation
  *
  *  Target suspend states used with igt_system_suspend_autoresume().
  *  See /sys/power/state for the available states on a given machine.
@@ -149,7 +155,8 @@ bool igt_aub_dump_enabled(void);
 enum igt_suspend_state {
 	SUSPEND_STATE_FREEZE,
 	SUSPEND_STATE_STANDBY,
-	SUSPEND_STATE_MEM,
+	SUSPEND_STATE_S3, /* Forces Suspend-to-Ram (S3) */
+	SUSPEND_STATE_MEM, /* Respects system default */
 	SUSPEND_STATE_DISK,
 
 	/*< private >*/
-- 
2.35.3

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

* [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3.
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement Rodrigo Vivi
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state Rodrigo Vivi
@ 2022-07-13 15:52 ` Rodrigo Vivi
  2022-07-18  7:09   ` Anshuman Gupta
  2022-07-13 18:58 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names Patchwork
                   ` (3 subsequent siblings)
  6 siblings, 1 reply; 13+ messages in thread
From: Rodrigo Vivi @ 2022-07-13 15:52 UTC (permalink / raw)
  To: igt-dev; +Cc: Rodrigo Vivi

Now that we have the ability to enforce the S3, let's use it
and avoid the extra check in the test.

Cc: Riana Tauro <riana.tauro@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
---
 lib/igt_aux.c             | 14 +++-----------
 lib/igt_aux.h             |  2 --
 tests/i915/i915_suspend.c |  5 +----
 3 files changed, 4 insertions(+), 17 deletions(-)

diff --git a/lib/igt_aux.c b/lib/igt_aux.c
index 69ae9b58..268bad7c 100644
--- a/lib/igt_aux.c
+++ b/lib/igt_aux.c
@@ -841,15 +841,7 @@ static uint32_t get_supported_suspend_states(int power_dir)
 	return state_mask;
 }
 
-/**
- * igt_get_memsleep_state
- *
- * Reads the value of /sys/power/mem_sleep and
- * returns the current suspend state associated with 'mem'.
- *
- * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
- */
-int igt_get_memsleep_state(void)
+static int get_mem_sleep(void)
 {
 	char *mem_sleep_states;
 	char *mem_sleep_state;
@@ -938,9 +930,9 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 	orig_test = get_suspend_test(power_dir);
 
 	if (state == SUSPEND_STATE_S3) {
-		orig_mem_sleep = igt_get_memsleep_state();
+		orig_mem_sleep = get_mem_sleep();
 		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
-		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
+		igt_skip_on_f(get_mem_sleep() != MEM_SLEEP_DEEP,
 			      "S3 not possible in this system.\n");
 	}
 
diff --git a/lib/igt_aux.h b/lib/igt_aux.h
index b1e48b0f..89437a90 100644
--- a/lib/igt_aux.h
+++ b/lib/igt_aux.h
@@ -216,8 +216,6 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
 void igt_set_autoresume_delay(int delay_secs);
 int igt_get_autoresume_delay(enum igt_suspend_state state);
 
-int igt_get_memsleep_state(void);
-
 /* dropping priviledges */
 void igt_drop_root(void);
 
diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
index 222a8e4a..94935595 100644
--- a/tests/i915/i915_suspend.c
+++ b/tests/i915/i915_suspend.c
@@ -209,9 +209,6 @@ test_suspend_without_i915(int state)
 	igt_kmsg(KMSG_INFO "Unloading i915\n");
 	igt_assert_eq(igt_i915_driver_unload(),0);
 
-	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE &&
-		      state == SUSPEND_STATE_MEM,
-		      "S3 not possible: platform default mem_sleep state is s2idle\n");
 	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
 
 	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
@@ -228,7 +225,7 @@ igt_main
 
 	igt_describe("Validate S3 without i915 module");
 	igt_subtest("basic-s3-without-i915")
-		test_suspend_without_i915(SUSPEND_STATE_MEM);
+		test_suspend_without_i915(SUSPEND_STATE_S3);
 
 	igt_fixture
 		fd = drm_open_driver(DRIVER_INTEL);
-- 
2.35.3

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

* [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
                   ` (2 preceding siblings ...)
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3 Rodrigo Vivi
@ 2022-07-13 18:58 ` Patchwork
  2022-07-18  5:54 ` [igt-dev] [PATCH i-g-t 1/4] " Anshuman Gupta
                   ` (2 subsequent siblings)
  6 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-07-13 18:58 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

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

== Series Details ==

Series: series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names
URL   : https://patchwork.freedesktop.org/series/106312/
State : failure

== Summary ==

CI Bug Log - changes from IGT_6581 -> IGTPW_7509
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_7509 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_7509, 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_7509/index.html

Participating hosts (33 -> 39)
------------------------------

  Additional (7): fi-kbl-soraka bat-adlm-1 bat-dg2-9 bat-adlp-6 bat-adlp-4 bat-rpls-2 bat-jsl-1 
  Missing    (1): fi-pnv-d510 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-rkl-11600:       NOTRUN -> [SKIP][1] +3 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-rkl-11600/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-adl-ddr5:        [PASS][2] -> [SKIP][3] +4 similar issues
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
    - bat-adlp-4:         NOTRUN -> [SKIP][4]
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-rkl-guc:         [PASS][5] -> [SKIP][6] +3 similar issues
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-dp-2:
    - fi-icl-u2:          NOTRUN -> [SKIP][7] +3 similar issues
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-icl-u2/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-dp-2.html

  
#### Suppressed ####

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

  * igt@gem_exec_suspend@basic-s0@smem:
    - {bat-adlm-1}:       NOTRUN -> [DMESG-WARN][8]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlm-1/igt@gem_exec_suspend@basic-s0@smem.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - {bat-jsl-1}:        NOTRUN -> [SKIP][9] +3 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-jsl-1/igt@gem_exec_suspend@basic-s3@smem.html
    - {fi-ehl-2}:         NOTRUN -> [SKIP][10] +3 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-ehl-2/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_suspend@basic-s3-without-i915:
    - {fi-tgl-dsi}:       [SKIP][11] ([i915#5903]) -> [INCOMPLETE][12]
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-tgl-dsi/igt@i915_suspend@basic-s3-without-i915.html
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-tgl-dsi/igt@i915_suspend@basic-s3-without-i915.html

  * igt@kms_pipe_crc_basic@suspend-read-crc:
    - {bat-adlm-1}:       NOTRUN -> [SKIP][13] +1 similar issue
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlm-1/igt@kms_pipe_crc_basic@suspend-read-crc.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1:
    - {fi-jsl-1}:         [PASS][14] -> [SKIP][15] +3 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1:
    - {bat-adlp-6}:       NOTRUN -> [SKIP][16] +4 similar issues
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_gttfill@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][17] ([fdo#109271]) +8 similar issues
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-glk-j4005:       [PASS][18] -> [SKIP][19] ([fdo#109271]) +3 similar issues
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bxt-dsi:         [PASS][20] -> [SKIP][21] ([fdo#109271]) +3 similar issues
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-bxt-dsi/igt@gem_exec_suspend@basic-s3@smem.html
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bxt-dsi/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6700k2:      [PASS][22] -> [SKIP][23] ([fdo#109271]) +3 similar issues
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-7567u:       [PASS][24] -> [SKIP][25] ([fdo#109271])
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-8809g:       [PASS][26] -> [SKIP][27] ([fdo#109271])
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-kbl-8809g/igt@gem_exec_suspend@basic-s3@smem.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-8809g/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6600u:       [PASS][28] -> [SKIP][29] ([fdo#109271]) +3 similar issues
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bsw-nick:        [PASS][30] -> [SKIP][31] ([fdo#109271])
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@gem_huc_copy@huc-copy:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][32] ([fdo#109271] / [i915#2190])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html

  * igt@gem_lmem_swapping@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][33] ([fdo#109271] / [i915#4613]) +3 similar issues
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html

  * igt@gem_lmem_swapping@parallel-random-engines:
    - bat-adlp-4:         NOTRUN -> [SKIP][34] ([i915#4613]) +3 similar issues
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@gem_lmem_swapping@parallel-random-engines.html

  * igt@gem_tiled_pread_basic:
    - bat-adlp-4:         NOTRUN -> [SKIP][35] ([i915#3282])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@gem_tiled_pread_basic.html

  * igt@i915_selftest@live@gem_contexts:
    - fi-bdw-5557u:       [PASS][36] -> [INCOMPLETE][37] ([i915#5502])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-bdw-5557u/igt@i915_selftest@live@gem_contexts.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bdw-5557u/igt@i915_selftest@live@gem_contexts.html

  * igt@i915_selftest@live@gt_pm:
    - fi-kbl-soraka:      NOTRUN -> [DMESG-FAIL][38] ([i915#1886])
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html

  * igt@i915_selftest@live@hangcheck:
    - fi-snb-2600:        [PASS][39] -> [INCOMPLETE][40] ([i915#3921])
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-snb-2600/igt@i915_selftest@live@hangcheck.html
    - fi-adl-ddr5:        [PASS][41] -> [DMESG-WARN][42] ([i915#5591])
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-adl-ddr5/igt@i915_selftest@live@hangcheck.html
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-adl-ddr5/igt@i915_selftest@live@hangcheck.html

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-bsw-n3050:       NOTRUN -> [SKIP][43] ([fdo#109271] / [fdo#111827])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bsw-n3050/igt@kms_chamelium@common-hpd-after-suspend.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][44] ([fdo#111827])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-rkl-11600/igt@kms_chamelium@common-hpd-after-suspend.html
    - fi-icl-u2:          NOTRUN -> [SKIP][45] ([fdo#111827])
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-icl-u2/igt@kms_chamelium@common-hpd-after-suspend.html

  * igt@kms_chamelium@dp-crc-fast:
    - bat-adlp-4:         NOTRUN -> [SKIP][46] ([fdo#111827]) +8 similar issues
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@kms_chamelium@dp-crc-fast.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][47] ([fdo#109271] / [fdo#111827]) +7 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor:
    - bat-adlp-4:         NOTRUN -> [SKIP][48] ([i915#4103])
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html

  * igt@kms_force_connector_basic@force-load-detect:
    - bat-adlp-4:         NOTRUN -> [SKIP][49] ([i915#4093]) +3 similar issues
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_pipe_crc_basic@suspend-read-crc:
    - bat-adlp-4:         NOTRUN -> [SKIP][50] ([i915#3546])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@kms_pipe_crc_basic@suspend-read-crc.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-cfl-8700k:       [PASS][51] -> [SKIP][52] ([fdo#109271]) +3 similar issues
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
    - fi-elk-e7500:       [PASS][53] -> [SKIP][54] ([fdo#109271]) +2 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1:
    - fi-ilk-650:         [PASS][55] -> [SKIP][56] ([fdo#109271]) +2 similar issues
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2:
    - fi-cfl-8109u:       [PASS][57] -> [SKIP][58] ([fdo#109271]) +3 similar issues
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
    - fi-skl-guc:         [PASS][59] -> [SKIP][60] ([fdo#109271]) +3 similar issues
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1:
    - fi-glk-dsi:         [PASS][61] -> [SKIP][62] ([fdo#109271]) +3 similar issues
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1:
    - fi-bsw-kefka:       [PASS][63] -> [SKIP][64] ([fdo#109271]) +2 similar issues
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1:
    - fi-cfl-guc:         [PASS][65] -> [SKIP][66] ([fdo#109271]) +3 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1:
    - fi-snb-2520m:       [PASS][67] -> [SKIP][68] ([fdo#109271]) +2 similar issues
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-hdmi-a-2:
    - fi-bsw-n3050:       NOTRUN -> [SKIP][69] ([fdo#109271]) +1 similar issue
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bsw-n3050/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-hdmi-a-2.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1:
    - fi-hsw-4770:        [PASS][70] -> [SKIP][71] ([fdo#109271]) +3 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-ivb-3770:        [PASS][72] -> [SKIP][73] ([fdo#109271]) +3 similar issues
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-hsw-g3258:       [PASS][74] -> [SKIP][75] ([fdo#109271]) +3 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-hsw-g3258/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-hsw-g3258/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - bat-adlp-4:         NOTRUN -> [SKIP][76] ([i915#3555] / [i915#4579])
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-userptr:
    - bat-adlp-4:         NOTRUN -> [SKIP][77] ([fdo#109295] / [i915#3301] / [i915#3708])
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@prime_vgem@basic-userptr.html

  * igt@prime_vgem@basic-write:
    - bat-adlp-4:         NOTRUN -> [SKIP][78] ([fdo#109295] / [i915#3291] / [i915#3708]) +2 similar issues
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/bat-adlp-4/igt@prime_vgem@basic-write.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s0@smem:
    - {fi-ehl-2}:         [DMESG-WARN][79] ([i915#5122]) -> [PASS][80]
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html

  * igt@i915_selftest@live@execlists:
    - fi-bsw-n3050:       [INCOMPLETE][81] ([i915#2940]) -> [PASS][82]
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-bsw-n3050/igt@i915_selftest@live@execlists.html
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-bsw-n3050/igt@i915_selftest@live@execlists.html

  * igt@i915_selftest@live@hangcheck:
    - fi-icl-u2:          [INCOMPLETE][83] ([i915#4890]) -> [PASS][84]
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-icl-u2/igt@i915_selftest@live@hangcheck.html
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-icl-u2/igt@i915_selftest@live@hangcheck.html

  * igt@i915_suspend@basic-s3-without-i915:
    - fi-rkl-11600:       [INCOMPLETE][85] ([i915#5982]) -> [PASS][86]
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-ehl-2}:         [SKIP][87] ([i915#5903]) -> [PASS][88]
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-jsl-1}:         [SKIP][89] ([i915#5903]) -> [PASS][90]
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/IGT_6581/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.html
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.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
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1849]: https://gitlab.freedesktop.org/drm/intel/issues/1849
  [i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#2940]: https://gitlab.freedesktop.org/drm/intel/issues/2940
  [i915#3003]: https://gitlab.freedesktop.org/drm/intel/issues/3003
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
  [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3595]: https://gitlab.freedesktop.org/drm/intel/issues/3595
  [i915#3637]: https://gitlab.freedesktop.org/drm/intel/issues/3637
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#3921]: https://gitlab.freedesktop.org/drm/intel/issues/3921
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
  [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
  [i915#4093]: https://gitlab.freedesktop.org/drm/intel/issues/4093
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
  [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
  [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
  [i915#4258]: https://gitlab.freedesktop.org/drm/intel/issues/4258
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4873]: https://gitlab.freedesktop.org/drm/intel/issues/4873
  [i915#4890]: https://gitlab.freedesktop.org/drm/intel/issues/4890
  [i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122
  [i915#5174]: https://gitlab.freedesktop.org/drm/intel/issues/5174
  [i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
  [i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
  [i915#5502]: https://gitlab.freedesktop.org/drm/intel/issues/5502
  [i915#5591]: https://gitlab.freedesktop.org/drm/intel/issues/5591
  [i915#5763]: https://gitlab.freedesktop.org/drm/intel/issues/5763
  [i915#5903]: https://gitlab.freedesktop.org/drm/intel/issues/5903
  [i915#5982]: https://gitlab.freedesktop.org/drm/intel/issues/5982
  [i915#5998]: https://gitlab.freedesktop.org/drm/intel/issues/5998
  [i915#6367]: https://gitlab.freedesktop.org/drm/intel/issues/6367


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6581 -> IGTPW_7509

  CI-20190529: 20190529
  CI_DRM_11877: e55cefc370de5a38ee848aa96082d9d9f4cacdb9 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_7509: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7509/index.html
  IGT_6581: 3c3a4cc75cfa7f6f34f2803ba68b0e8c5d1956e5 @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

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

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

* Re: [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
                   ` (3 preceding siblings ...)
  2022-07-13 18:58 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names Patchwork
@ 2022-07-18  5:54 ` Anshuman Gupta
  2022-07-19 13:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev2) Patchwork
  2022-07-21 10:26 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev3) Patchwork
  6 siblings, 0 replies; 13+ messages in thread
From: Anshuman Gupta @ 2022-07-18  5:54 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

On 2022-07-13 at 11:52:30 -0400, Rodrigo Vivi wrote:
> Mixing the names with the state names can cause confusion.
> Let's stick with the official names.
> 
> Cc: Riana Tauro <riana.tauro@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  lib/igt_aux.c             | 8 ++++----
>  lib/igt_aux.h             | 6 +++---
>  tests/i915/i915_suspend.c | 2 +-
>  3 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/lib/igt_aux.c b/lib/igt_aux.c
> index 35c67f10..5129d9f0 100644
> --- a/lib/igt_aux.c
> +++ b/lib/igt_aux.c
> @@ -712,9 +712,9 @@ static const char *suspend_test_name[] = {
>  };
>  
>  static const char *mem_sleep_name[] = {
> -	[MEM_SLEEP_FREEZE] = "s2idle",
> -	[MEM_SLEEP_STANDBY] = "shallow",
> -	[MEM_SLEEP_MEM] = "deep"
> +	[MEM_SLEEP_S2IDLE] = "s2idle",
> +	[MEM_SLEEP_SHALLOW] = "shallow",
> +	[MEM_SLEEP_DEEP] = "deep"
>  };
>  
>  static enum igt_suspend_test get_suspend_test(int power_dir)
> @@ -993,7 +993,7 @@ int igt_get_memsleep_state(void)
>  		return MEM_SLEEP_NONE;
>  	}
>  
> -	for (mem_sleep = MEM_SLEEP_FREEZE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
> +	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
>  		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
>  			break;
>  	}
> diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> index af663d2c..9d5b7bd2 100644
> --- a/lib/igt_aux.h
> +++ b/lib/igt_aux.h
> @@ -196,9 +196,9 @@ enum igt_suspend_test {
>   */
>  enum igt_mem_sleep {
>  	MEM_SLEEP_NONE,
> -	MEM_SLEEP_FREEZE,
> -	MEM_SLEEP_STANDBY,
> -	MEM_SLEEP_MEM,
> +	MEM_SLEEP_S2IDLE,
> +	MEM_SLEEP_SHALLOW,
> +	MEM_SLEEP_DEEP,
>  
>  	/*<private>*/
>  	MEM_SLEEP_NUM,
> diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
> index 23e451d3..f992d447 100644
> --- a/tests/i915/i915_suspend.c
> +++ b/tests/i915/i915_suspend.c
> @@ -209,7 +209,7 @@ test_suspend_without_i915(int state)
>  	igt_kmsg(KMSG_INFO "Unloading i915\n");
>  	igt_assert_eq(igt_i915_driver_unload(),0);
>  
> -	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_FREEZE && state > SUSPEND_STATE_FREEZE,
> +	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE && state > SUSPEND_STATE_FREEZE,
>  		      "Platform default mem_sleep state is s2idle\n");
>  	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
Looks good to me.
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
>  
> -- 
> 2.35.3
> 

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

* Re: [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement.
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement Rodrigo Vivi
@ 2022-07-18  6:24   ` Anshuman Gupta
  0 siblings, 0 replies; 13+ messages in thread
From: Anshuman Gupta @ 2022-07-18  6:24 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

On 2022-07-13 at 11:52:31 -0400, Rodrigo Vivi wrote:
> 1. We are not using DISK
> 2. DISK has no dependency on mem_sleep.
> 
> Let's be clear on what exactly we are trying to do.
> 
> Cc: Riana Tauro <riana.tauro@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  tests/i915/i915_suspend.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
> 
> diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
> index f992d447..222a8e4a 100644
> --- a/tests/i915/i915_suspend.c
> +++ b/tests/i915/i915_suspend.c
> @@ -209,8 +209,9 @@ test_suspend_without_i915(int state)
>  	igt_kmsg(KMSG_INFO "Unloading i915\n");
>  	igt_assert_eq(igt_i915_driver_unload(),0);
>  
> -	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE && state > SUSPEND_STATE_FREEZE,
> -		      "Platform default mem_sleep state is s2idle\n");
> +	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE &&
> +		      state == SUSPEND_STATE_MEM,
STANDBY state has the dependency on mem_sleep, so "state > SUSPEND_STATE_FREEZE" should be the correct condition,
but as follow on patch removes above hunk.
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> +		      "S3 not possible: platform default mem_sleep state is s2idle\n");
>  	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
>  
>  	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
> -- 
> 2.35.3
> 

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

* Re: [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state.
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state Rodrigo Vivi
@ 2022-07-18  6:47   ` Anshuman Gupta
  2022-07-18 21:42     ` Rodrigo Vivi
  0 siblings, 1 reply; 13+ messages in thread
From: Anshuman Gupta @ 2022-07-18  6:47 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

On 2022-07-13 at 11:52:32 -0400, Rodrigo Vivi wrote:
> Testing both Suspend-to-Idle and Suspend-to-RAM is critical.
> So far our test suite are relying on the system's default
> for the "suspend-to-mem", what reduces our ability to cover
> multiple scenarios or requires manual intervention.
> This also brings confusions in some debugging scenarios.
> 
> For the Suspend-to-Idle it is easy. The FREEZE state already
> enforces it. However, there's only one way to enforce the
> Suspend-to-RAM (aka S3) which is using MEM state with mem_sleep
> at "deep".
> 
> However, let's not break the whole world overnight. Let's
> first introduce the ability to force the Suspend-to-RAM.
> Then convert the tests as needed.
> 
> So, the SUSPEND_STATE_MEM will continue to exist and do
> exactly what is expected when you set "mem" to /sys/power/state,
> which is respect whatever is in the /sys/power/mem_sleep.
> 
> Let's introduce a new SUSPEND_STATE_S3 state aiming to force
> the Suspend-to-RAM, by changing both /sys/power/state and
> /sys/power/mem_sleep.
> 
> Cc: Riana Tauro <riana.tauro@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
>  lib/igt_aux.c | 118 ++++++++++++++++++++++++++++++--------------------
>  lib/igt_aux.h |  19 +++++---
>  2 files changed, 84 insertions(+), 53 deletions(-)
> 
> diff --git a/lib/igt_aux.c b/lib/igt_aux.c
> index 5129d9f0..69ae9b58 100644
> --- a/lib/igt_aux.c
> +++ b/lib/igt_aux.c
> @@ -698,7 +698,8 @@ static int autoresume_delay;
>  static const char *suspend_state_name[] = {
>  	[SUSPEND_STATE_FREEZE] = "freeze",
>  	[SUSPEND_STATE_STANDBY] = "standby",
> -	[SUSPEND_STATE_MEM] = "mem",
> +	[SUSPEND_STATE_S3] = "mem", /* Forces Suspend-to-Ram (S3) */
> +	[SUSPEND_STATE_MEM] = "mem", /* Respects system default */
>  	[SUSPEND_STATE_DISK] = "disk",
>  };
>  
> @@ -840,6 +841,63 @@ static uint32_t get_supported_suspend_states(int power_dir)
>  	return state_mask;
>  }
>  
> +/**
> + * igt_get_memsleep_state
> + *
> + * Reads the value of /sys/power/mem_sleep and
> + * returns the current suspend state associated with 'mem'.
> + *
> + * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
> + */
> +int igt_get_memsleep_state(void)
> +{
> +	char *mem_sleep_states;
> +	char *mem_sleep_state;
> +	enum igt_mem_sleep mem_sleep;
> +	int power_dir;
> +
> +	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> +
> +	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> +		return MEM_SLEEP_NONE;
> +
> +	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
> +	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
> +	     mem_sleep_state = strtok(NULL, " ")) {
> +		if (mem_sleep_state[0] == '[') {
> +			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> +			mem_sleep_state++;
> +			break;
> +		}
> +	}
> +
> +	if (!mem_sleep_state) {
> +		free(mem_sleep_states);
> +		return MEM_SLEEP_NONE;
> +	}
> +
> +	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
> +		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
> +			break;
> +	}
> +
> +	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
> +
> +	free(mem_sleep_states);
> +	close(power_dir);
> +	return mem_sleep;
> +}
> +
> +static void set_mem_sleep(int power_dir, enum igt_mem_sleep sleep)
> +{
> +	igt_assert(sleep < MEM_SLEEP_NUM);
> +
> +	igt_assert_eq(faccessat(power_dir, "mem_sleep", W_OK, 0), 0);
> +
> +	igt_assert(igt_sysfs_set(power_dir, "mem_sleep",
> +				 mem_sleep_name[sleep]));
> +}
> +
>  /**
>   * igt_system_suspend_autoresume:
>   * @state: an #igt_suspend_state, the target suspend state
> @@ -866,6 +924,7 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
>  {
>  	int power_dir;
>  	enum igt_suspend_test orig_test;
> +	enum igt_mem_sleep orig_mem_sleep = MEM_SLEEP_NONE;
>  
>  	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
>  	igt_require(get_supported_suspend_states(power_dir) & (1 << state));
> @@ -877,6 +936,14 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
>  		      "Suspend to disk requires swap space.\n");
>  
>  	orig_test = get_suspend_test(power_dir);
> +
> +	if (state == SUSPEND_STATE_S3) {
> +		orig_mem_sleep = igt_get_memsleep_state();
> +		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
> +		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
> +			      "S3 not possible in this system.\n");
		This should be reordered, skip first and then set_mem_sleep().
		set_mem_sleep() may assert if "deep" is not supported by mem_sleep.
		With that addressed.
		Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>

> +	}
> +
>  	set_suspend_test(power_dir, test);
>  
>  	if (test == SUSPEND_TEST_NONE)
> @@ -884,6 +951,9 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
>  	else
>  		suspend_via_sysfs(power_dir, state);
>  
> +	if (orig_mem_sleep)
> +		set_mem_sleep(power_dir, orig_mem_sleep);
> +
>  	set_suspend_test(power_dir, orig_test);
>  	close(power_dir);
>  }
> @@ -958,52 +1028,6 @@ int igt_get_autoresume_delay(enum igt_suspend_state state)
>  	return delay;
>  }
>  
> -/**
> - * igt_get_memsleep_state
> - *
> - * Reads the value of /sys/power/mem_sleep and
> - * returns the current suspend state associated with 'mem'.
> - *
> - * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
> - */
> -int igt_get_memsleep_state(void)
> -{
> -	char *mem_sleep_states;
> -	char *mem_sleep_state;
> -	enum igt_mem_sleep mem_sleep;
> -	int power_dir;
> -
> -	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> -
> -	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> -		return MEM_SLEEP_NONE;
> -
> -	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
> -	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
> -	     mem_sleep_state = strtok(NULL, " ")) {
> -		if (mem_sleep_state[0] == '[') {
> -			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> -			mem_sleep_state++;
> -			break;
> -		}
> -	}
> -
> -	if (!mem_sleep_state) {
> -		free(mem_sleep_states);
> -		return MEM_SLEEP_NONE;
> -	}
> -
> -	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
> -		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
> -			break;
> -	}
> -
> -	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
> -
> -	free(mem_sleep_states);
> -	close(power_dir);
> -	return mem_sleep;
> -}
>  /**
>   * igt_drop_root:
>   *
> diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> index 9d5b7bd2..b1e48b0f 100644
> --- a/lib/igt_aux.h
> +++ b/lib/igt_aux.h
> @@ -135,13 +135,19 @@ bool igt_aub_dump_enabled(void);
>  
>  /**
>   *  igt_suspend_state:
> - *  @SUSPEND_STATE_FREEZE: suspend-to-idle target state, aka S0ix or freeze,
> + *  @SUSPEND_STATE_FREEZE: Suspend-To-Idle target state, aka S0ix or freeze,
>   *			   first non-hibernation state
> - *  @SUSPEND_STATE_STANDBY: standby target state, aka S1, second
> + *  @SUSPEND_STATE_STANDBY: "Power-On Suspend" target state, aka S1, second
>   *			    non-hibernation state
> - *  @SUSPEND_STATE_MEM: suspend-to-mem target state aka S3, third
> - *			non-hibernation state
> - *  @SUSPEND_STATE_DISK: suspend-to-disk target state, aka S4 or hibernation
> + *  @SUSPEND_STATE_S3: Suspend-To-RAM: It enforces a "deep" state to mem_sleep,
> + *		       what forces the system to go to the third
> + *		       non-hibernation state, aka S3.
> + *  @SUSPEND_STATE_MEM: A memory sleep (non-hibernation) target state,
> + *			respecting the system's mem_sleep default:
> + *				s2idle: Suspend-To-Idle target state
> + *				shallow: "Power-On Suspend"
> + *				deep: Suspend-To-RAM
> + *  @SUSPEND_STATE_DISK: Suspend-To-Disk target state, aka S4 or hibernation
>   *
>   *  Target suspend states used with igt_system_suspend_autoresume().
>   *  See /sys/power/state for the available states on a given machine.
> @@ -149,7 +155,8 @@ bool igt_aub_dump_enabled(void);
>  enum igt_suspend_state {
>  	SUSPEND_STATE_FREEZE,
>  	SUSPEND_STATE_STANDBY,
> -	SUSPEND_STATE_MEM,
> +	SUSPEND_STATE_S3, /* Forces Suspend-to-Ram (S3) */
> +	SUSPEND_STATE_MEM, /* Respects system default */
>  	SUSPEND_STATE_DISK,
>  
>  	/*< private >*/
> -- 
> 2.35.3
> 

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

* Re: [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3.
  2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3 Rodrigo Vivi
@ 2022-07-18  7:09   ` Anshuman Gupta
  0 siblings, 0 replies; 13+ messages in thread
From: Anshuman Gupta @ 2022-07-18  7:09 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

On 2022-07-13 at 11:52:33 -0400, Rodrigo Vivi wrote:
> Now that we have the ability to enforce the S3, let's use it
> and avoid the extra check in the test.
> 
> Cc: Riana Tauro <riana.tauro@intel.com>
> Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
LGTM.
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> ---
>  lib/igt_aux.c             | 14 +++-----------
>  lib/igt_aux.h             |  2 --
>  tests/i915/i915_suspend.c |  5 +----
>  3 files changed, 4 insertions(+), 17 deletions(-)
> 
> diff --git a/lib/igt_aux.c b/lib/igt_aux.c
> index 69ae9b58..268bad7c 100644
> --- a/lib/igt_aux.c
> +++ b/lib/igt_aux.c
> @@ -841,15 +841,7 @@ static uint32_t get_supported_suspend_states(int power_dir)
>  	return state_mask;
>  }
>  
> -/**
> - * igt_get_memsleep_state
> - *
> - * Reads the value of /sys/power/mem_sleep and
> - * returns the current suspend state associated with 'mem'.
> - *
> - * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
> - */
> -int igt_get_memsleep_state(void)
> +static int get_mem_sleep(void)
>  {
>  	char *mem_sleep_states;
>  	char *mem_sleep_state;
> @@ -938,9 +930,9 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
>  	orig_test = get_suspend_test(power_dir);
>  
>  	if (state == SUSPEND_STATE_S3) {
> -		orig_mem_sleep = igt_get_memsleep_state();
> +		orig_mem_sleep = get_mem_sleep();
>  		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
> -		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
> +		igt_skip_on_f(get_mem_sleep() != MEM_SLEEP_DEEP,
>  			      "S3 not possible in this system.\n");
>  	}
>  
> diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> index b1e48b0f..89437a90 100644
> --- a/lib/igt_aux.h
> +++ b/lib/igt_aux.h
> @@ -216,8 +216,6 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
>  void igt_set_autoresume_delay(int delay_secs);
>  int igt_get_autoresume_delay(enum igt_suspend_state state);
>  
> -int igt_get_memsleep_state(void);
> -
>  /* dropping priviledges */
>  void igt_drop_root(void);
>  
> diff --git a/tests/i915/i915_suspend.c b/tests/i915/i915_suspend.c
> index 222a8e4a..94935595 100644
> --- a/tests/i915/i915_suspend.c
> +++ b/tests/i915/i915_suspend.c
> @@ -209,9 +209,6 @@ test_suspend_without_i915(int state)
>  	igt_kmsg(KMSG_INFO "Unloading i915\n");
>  	igt_assert_eq(igt_i915_driver_unload(),0);
>  
> -	igt_skip_on_f(igt_get_memsleep_state() == MEM_SLEEP_S2IDLE &&
> -		      state == SUSPEND_STATE_MEM,
> -		      "S3 not possible: platform default mem_sleep state is s2idle\n");
>  	igt_system_suspend_autoresume(state, SUSPEND_TEST_NONE);
>  
>  	igt_kmsg(KMSG_INFO "Re-loading i915 \n");
> @@ -228,7 +225,7 @@ igt_main
>  
>  	igt_describe("Validate S3 without i915 module");
>  	igt_subtest("basic-s3-without-i915")
> -		test_suspend_without_i915(SUSPEND_STATE_MEM);
> +		test_suspend_without_i915(SUSPEND_STATE_S3);
>  
>  	igt_fixture
>  		fd = drm_open_driver(DRIVER_INTEL);
> -- 
> 2.35.3
> 

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

* Re: [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state.
  2022-07-18  6:47   ` Anshuman Gupta
@ 2022-07-18 21:42     ` Rodrigo Vivi
  2022-07-19  5:20       ` Gupta, Anshuman
  0 siblings, 1 reply; 13+ messages in thread
From: Rodrigo Vivi @ 2022-07-18 21:42 UTC (permalink / raw)
  To: Anshuman Gupta; +Cc: igt-dev

On Mon, Jul 18, 2022 at 12:17:49PM +0530, Anshuman Gupta wrote:
> On 2022-07-13 at 11:52:32 -0400, Rodrigo Vivi wrote:
> > Testing both Suspend-to-Idle and Suspend-to-RAM is critical.
> > So far our test suite are relying on the system's default
> > for the "suspend-to-mem", what reduces our ability to cover
> > multiple scenarios or requires manual intervention.
> > This also brings confusions in some debugging scenarios.
> >
> > For the Suspend-to-Idle it is easy. The FREEZE state already
> > enforces it. However, there's only one way to enforce the
> > Suspend-to-RAM (aka S3) which is using MEM state with mem_sleep
> > at "deep".
> >
> > However, let's not break the whole world overnight. Let's
> > first introduce the ability to force the Suspend-to-RAM.
> > Then convert the tests as needed.
> >
> > So, the SUSPEND_STATE_MEM will continue to exist and do
> > exactly what is expected when you set "mem" to /sys/power/state,
> > which is respect whatever is in the /sys/power/mem_sleep.
> >
> > Let's introduce a new SUSPEND_STATE_S3 state aiming to force
> > the Suspend-to-RAM, by changing both /sys/power/state and
> > /sys/power/mem_sleep.
> >
> > Cc: Riana Tauro <riana.tauro@intel.com>
> > Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > ---
> >  lib/igt_aux.c | 118 ++++++++++++++++++++++++++++++--------------------
> >  lib/igt_aux.h |  19 +++++---
> >  2 files changed, 84 insertions(+), 53 deletions(-)
> >
> > diff --git a/lib/igt_aux.c b/lib/igt_aux.c
> > index 5129d9f0..69ae9b58 100644
> > --- a/lib/igt_aux.c
> > +++ b/lib/igt_aux.c
> > @@ -698,7 +698,8 @@ static int autoresume_delay;
> >  static const char *suspend_state_name[] = {
> >  	[SUSPEND_STATE_FREEZE] = "freeze",
> >  	[SUSPEND_STATE_STANDBY] = "standby",
> > -	[SUSPEND_STATE_MEM] = "mem",
> > +	[SUSPEND_STATE_S3] = "mem", /* Forces Suspend-to-Ram (S3) */
> > +	[SUSPEND_STATE_MEM] = "mem", /* Respects system default */
> >  	[SUSPEND_STATE_DISK] = "disk",
> >  };
> >
> > @@ -840,6 +841,63 @@ static uint32_t get_supported_suspend_states(int power_dir)
> >  	return state_mask;
> >  }
> >
> > +/**
> > + * igt_get_memsleep_state
> > + *
> > + * Reads the value of /sys/power/mem_sleep and
> > + * returns the current suspend state associated with 'mem'.
> > + *
> > + * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
> > + */
> > +int igt_get_memsleep_state(void)
> > +{
> > +	char *mem_sleep_states;
> > +	char *mem_sleep_state;
> > +	enum igt_mem_sleep mem_sleep;
> > +	int power_dir;
> > +
> > +	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> > +
> > +	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> > +		return MEM_SLEEP_NONE;
> > +
> > +	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
> > +	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
> > +	     mem_sleep_state = strtok(NULL, " ")) {
> > +		if (mem_sleep_state[0] == '[') {
> > +			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> > +			mem_sleep_state++;
> > +			break;
> > +		}
> > +	}
> > +
> > +	if (!mem_sleep_state) {
> > +		free(mem_sleep_states);
> > +		return MEM_SLEEP_NONE;
> > +	}
> > +
> > +	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
> > +		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
> > +			break;
> > +	}
> > +
> > +	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
> > +
> > +	free(mem_sleep_states);
> > +	close(power_dir);
> > +	return mem_sleep;
> > +}
> > +
> > +static void set_mem_sleep(int power_dir, enum igt_mem_sleep sleep)
> > +{
> > +	igt_assert(sleep < MEM_SLEEP_NUM);
> > +
> > +	igt_assert_eq(faccessat(power_dir, "mem_sleep", W_OK, 0), 0);
> > +
> > +	igt_assert(igt_sysfs_set(power_dir, "mem_sleep",
> > +				 mem_sleep_name[sleep]));
> > +}
> > +
> >  /**
> >   * igt_system_suspend_autoresume:
> >   * @state: an #igt_suspend_state, the target suspend state
> > @@ -866,6 +924,7 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
> >  {
> >  	int power_dir;
> >  	enum igt_suspend_test orig_test;
> > +	enum igt_mem_sleep orig_mem_sleep = MEM_SLEEP_NONE;
> >
> >  	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> >  	igt_require(get_supported_suspend_states(power_dir) & (1 << state));
> > @@ -877,6 +936,14 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
> >  		      "Suspend to disk requires swap space.\n");
> >
> >  	orig_test = get_suspend_test(power_dir);
> > +
> > +	if (state == SUSPEND_STATE_S3) {
> > +		orig_mem_sleep = igt_get_memsleep_state();
> > +		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
> > +		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
> > +			      "S3 not possible in this system.\n");
> 		This should be reordered, skip first and then set_mem_sleep().

If we change the order the purpose is defeated.
We need to change to the desired mode and then check if it sticks.
If we check it before it will skip without putting us to the desired mode.

> 		set_mem_sleep() may assert if "deep" is not supported by mem_sleep.

it kind of does, no?!
igt_assert(igt_sysfs_set(power_dir, "mem_sleep", mem_sleep_name[sleep]));

will fail with Invalid argument if the mode is not supported.
I don't believe that extra checks are needed.

Unless I'm missing something.

Thanks for all the reviews.

> 		With that addressed.
> 		Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
>
> > +	}
> > +
> >  	set_suspend_test(power_dir, test);
> >
> >  	if (test == SUSPEND_TEST_NONE)
> > @@ -884,6 +951,9 @@ void igt_system_suspend_autoresume(enum igt_suspend_state state,
> >  	else
> >  		suspend_via_sysfs(power_dir, state);
> >
> > +	if (orig_mem_sleep)
> > +		set_mem_sleep(power_dir, orig_mem_sleep);
> > +
> >  	set_suspend_test(power_dir, orig_test);
> >  	close(power_dir);
> >  }
> > @@ -958,52 +1028,6 @@ int igt_get_autoresume_delay(enum igt_suspend_state state)
> >  	return delay;
> >  }
> >
> > -/**
> > - * igt_get_memsleep_state
> > - *
> > - * Reads the value of /sys/power/mem_sleep and
> > - * returns the current suspend state associated with 'mem'.
> > - *
> > - * Returns : an #igt_mem_sleep state, current suspend state associated with 'mem'.
> > - */
> > -int igt_get_memsleep_state(void)
> > -{
> > -	char *mem_sleep_states;
> > -	char *mem_sleep_state;
> > -	enum igt_mem_sleep mem_sleep;
> > -	int power_dir;
> > -
> > -	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> > -
> > -	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> > -		return MEM_SLEEP_NONE;
> > -
> > -	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir, "mem_sleep")));
> > -	for (mem_sleep_state = strtok(mem_sleep_states, " "); mem_sleep_state;
> > -	     mem_sleep_state = strtok(NULL, " ")) {
> > -		if (mem_sleep_state[0] == '[') {
> > -			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> > -			mem_sleep_state++;
> > -			break;
> > -		}
> > -	}
> > -
> > -	if (!mem_sleep_state) {
> > -		free(mem_sleep_states);
> > -		return MEM_SLEEP_NONE;
> > -	}
> > -
> > -	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep < MEM_SLEEP_NUM; mem_sleep++) {
> > -		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state) == 0)
> > -			break;
> > -	}
> > -
> > -	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep state\n");
> > -
> > -	free(mem_sleep_states);
> > -	close(power_dir);
> > -	return mem_sleep;
> > -}
> >  /**
> >   * igt_drop_root:
> >   *
> > diff --git a/lib/igt_aux.h b/lib/igt_aux.h
> > index 9d5b7bd2..b1e48b0f 100644
> > --- a/lib/igt_aux.h
> > +++ b/lib/igt_aux.h
> > @@ -135,13 +135,19 @@ bool igt_aub_dump_enabled(void);
> >
> >  /**
> >   *  igt_suspend_state:
> > - *  @SUSPEND_STATE_FREEZE: suspend-to-idle target state, aka S0ix or freeze,
> > + *  @SUSPEND_STATE_FREEZE: Suspend-To-Idle target state, aka S0ix or freeze,
> >   *			   first non-hibernation state
> > - *  @SUSPEND_STATE_STANDBY: standby target state, aka S1, second
> > + *  @SUSPEND_STATE_STANDBY: "Power-On Suspend" target state, aka S1, second
> >   *			    non-hibernation state
> > - *  @SUSPEND_STATE_MEM: suspend-to-mem target state aka S3, third
> > - *			non-hibernation state
> > - *  @SUSPEND_STATE_DISK: suspend-to-disk target state, aka S4 or hibernation
> > + *  @SUSPEND_STATE_S3: Suspend-To-RAM: It enforces a "deep" state to mem_sleep,
> > + *		       what forces the system to go to the third
> > + *		       non-hibernation state, aka S3.
> > + *  @SUSPEND_STATE_MEM: A memory sleep (non-hibernation) target state,
> > + *			respecting the system's mem_sleep default:
> > + *				s2idle: Suspend-To-Idle target state
> > + *				shallow: "Power-On Suspend"
> > + *				deep: Suspend-To-RAM
> > + *  @SUSPEND_STATE_DISK: Suspend-To-Disk target state, aka S4 or hibernation
> >   *
> >   *  Target suspend states used with igt_system_suspend_autoresume().
> >   *  See /sys/power/state for the available states on a given machine.
> > @@ -149,7 +155,8 @@ bool igt_aub_dump_enabled(void);
> >  enum igt_suspend_state {
> >  	SUSPEND_STATE_FREEZE,
> >  	SUSPEND_STATE_STANDBY,
> > -	SUSPEND_STATE_MEM,
> > +	SUSPEND_STATE_S3, /* Forces Suspend-to-Ram (S3) */
> > +	SUSPEND_STATE_MEM, /* Respects system default */
> >  	SUSPEND_STATE_DISK,
> >
> >  	/*< private >*/
> > --
> > 2.35.3
> >

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

* Re: [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state.
  2022-07-18 21:42     ` Rodrigo Vivi
@ 2022-07-19  5:20       ` Gupta, Anshuman
  0 siblings, 0 replies; 13+ messages in thread
From: Gupta, Anshuman @ 2022-07-19  5:20 UTC (permalink / raw)
  To: Vivi, Rodrigo; +Cc: igt-dev



> -----Original Message-----
> From: Vivi, Rodrigo <rodrigo.vivi@intel.com>
> Sent: Tuesday, July 19, 2022 3:12 AM
> To: Gupta, Anshuman <anshuman.gupta@intel.com>
> Cc: igt-dev@lists.freedesktop.org; Tauro, Riana <riana.tauro@intel.com>
> Subject: Re: [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3
> suspend state.
> 
> On Mon, Jul 18, 2022 at 12:17:49PM +0530, Anshuman Gupta wrote:
> > On 2022-07-13 at 11:52:32 -0400, Rodrigo Vivi wrote:
> > > Testing both Suspend-to-Idle and Suspend-to-RAM is critical.
> > > So far our test suite are relying on the system's default for the
> > > "suspend-to-mem", what reduces our ability to cover multiple
> > > scenarios or requires manual intervention.
> > > This also brings confusions in some debugging scenarios.
> > >
> > > For the Suspend-to-Idle it is easy. The FREEZE state already
> > > enforces it. However, there's only one way to enforce the
> > > Suspend-to-RAM (aka S3) which is using MEM state with mem_sleep at
> > > "deep".
> > >
> > > However, let's not break the whole world overnight. Let's first
> > > introduce the ability to force the Suspend-to-RAM.
> > > Then convert the tests as needed.
> > >
> > > So, the SUSPEND_STATE_MEM will continue to exist and do exactly what
> > > is expected when you set "mem" to /sys/power/state, which is respect
> > > whatever is in the /sys/power/mem_sleep.
> > >
> > > Let's introduce a new SUSPEND_STATE_S3 state aiming to force the
> > > Suspend-to-RAM, by changing both /sys/power/state and
> > > /sys/power/mem_sleep.
> > >
> > > Cc: Riana Tauro <riana.tauro@intel.com>
> > > Cc: Anshuman Gupta <anshuman.gupta@intel.com>
> > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> > > ---
> > >  lib/igt_aux.c | 118
> > > ++++++++++++++++++++++++++++++--------------------
> > >  lib/igt_aux.h |  19 +++++---
> > >  2 files changed, 84 insertions(+), 53 deletions(-)
> > >
> > > diff --git a/lib/igt_aux.c b/lib/igt_aux.c index 5129d9f0..69ae9b58
> > > 100644
> > > --- a/lib/igt_aux.c
> > > +++ b/lib/igt_aux.c
> > > @@ -698,7 +698,8 @@ static int autoresume_delay;  static const char
> > > *suspend_state_name[] = {
> > >  	[SUSPEND_STATE_FREEZE] = "freeze",
> > >  	[SUSPEND_STATE_STANDBY] = "standby",
> > > -	[SUSPEND_STATE_MEM] = "mem",
> > > +	[SUSPEND_STATE_S3] = "mem", /* Forces Suspend-to-Ram (S3) */
> > > +	[SUSPEND_STATE_MEM] = "mem", /* Respects system default */
> > >  	[SUSPEND_STATE_DISK] = "disk",
> > >  };
> > >
> > > @@ -840,6 +841,63 @@ static uint32_t get_supported_suspend_states(int
> power_dir)
> > >  	return state_mask;
> > >  }
> > >
> > > +/**
> > > + * igt_get_memsleep_state
> > > + *
> > > + * Reads the value of /sys/power/mem_sleep and
> > > + * returns the current suspend state associated with 'mem'.
> > > + *
> > > + * Returns : an #igt_mem_sleep state, current suspend state associated
> with 'mem'.
> > > + */
> > > +int igt_get_memsleep_state(void)
> > > +{
> > > +	char *mem_sleep_states;
> > > +	char *mem_sleep_state;
> > > +	enum igt_mem_sleep mem_sleep;
> > > +	int power_dir;
> > > +
> > > +	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> > > +
> > > +	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> > > +		return MEM_SLEEP_NONE;
> > > +
> > > +	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir,
> "mem_sleep")));
> > > +	for (mem_sleep_state = strtok(mem_sleep_states, " ");
> mem_sleep_state;
> > > +	     mem_sleep_state = strtok(NULL, " ")) {
> > > +		if (mem_sleep_state[0] == '[') {
> > > +			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> > > +			mem_sleep_state++;
> > > +			break;
> > > +		}
> > > +	}
> > > +
> > > +	if (!mem_sleep_state) {
> > > +		free(mem_sleep_states);
> > > +		return MEM_SLEEP_NONE;
> > > +	}
> > > +
> > > +	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep <
> MEM_SLEEP_NUM; mem_sleep++) {
> > > +		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state)
> == 0)
> > > +			break;
> > > +	}
> > > +
> > > +	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep
> > > +state\n");
> > > +
> > > +	free(mem_sleep_states);
> > > +	close(power_dir);
> > > +	return mem_sleep;
> > > +}
> > > +
> > > +static void set_mem_sleep(int power_dir, enum igt_mem_sleep sleep)
> > > +{
> > > +	igt_assert(sleep < MEM_SLEEP_NUM);
> > > +
> > > +	igt_assert_eq(faccessat(power_dir, "mem_sleep", W_OK, 0), 0);
> > > +
> > > +	igt_assert(igt_sysfs_set(power_dir, "mem_sleep",
> > > +				 mem_sleep_name[sleep]));
> > > +}
> > > +
> > >  /**
> > >   * igt_system_suspend_autoresume:
> > >   * @state: an #igt_suspend_state, the target suspend state @@
> > > -866,6 +924,7 @@ void igt_system_suspend_autoresume(enum
> > > igt_suspend_state state,  {
> > >  	int power_dir;
> > >  	enum igt_suspend_test orig_test;
> > > +	enum igt_mem_sleep orig_mem_sleep = MEM_SLEEP_NONE;
> > >
> > >  	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> > >  	igt_require(get_supported_suspend_states(power_dir) & (1 <<
> > > state)); @@ -877,6 +936,14 @@ void
> igt_system_suspend_autoresume(enum igt_suspend_state state,
> > >  		      "Suspend to disk requires swap space.\n");
> > >
> > >  	orig_test = get_suspend_test(power_dir);
> > > +
> > > +	if (state == SUSPEND_STATE_S3) {
> > > +		orig_mem_sleep = igt_get_memsleep_state();
> > > +		set_mem_sleep(power_dir, MEM_SLEEP_DEEP);
> > > +		igt_skip_on_f(igt_get_memsleep_state() != MEM_SLEEP_DEEP,
> > > +			      "S3 not possible in this system.\n");
> > 		This should be reordered, skip first and then set_mem_sleep().
> 
> If we change the order the purpose is defeated.
> We need to change to the desired mode and then check if it sticks.
> If we check it before it will skip without putting us to the desired mode.
> 
> > 		set_mem_sleep() may assert if "deep" is not supported by
> mem_sleep.
> 
> it kind of does, no?!
> igt_assert(igt_sysfs_set(power_dir, "mem_sleep", mem_sleep_name[sleep]));
Sorry to create confusion , yes the approach is correct.
You can use my RB.
Thanks,
Anshuman Gupta. 
> 
> will fail with Invalid argument if the mode is not supported.
> I don't believe that extra checks are needed.
> 
> Unless I'm missing something.
> 
> Thanks for all the reviews.
> 
> > 		With that addressed.
> > 		Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
> >
> > > +	}
> > > +
> > >  	set_suspend_test(power_dir, test);
> > >
> > >  	if (test == SUSPEND_TEST_NONE)
> > > @@ -884,6 +951,9 @@ void igt_system_suspend_autoresume(enum
> igt_suspend_state state,
> > >  	else
> > >  		suspend_via_sysfs(power_dir, state);
> > >
> > > +	if (orig_mem_sleep)
> > > +		set_mem_sleep(power_dir, orig_mem_sleep);
> > > +
> > >  	set_suspend_test(power_dir, orig_test);
> > >  	close(power_dir);
> > >  }
> > > @@ -958,52 +1028,6 @@ int igt_get_autoresume_delay(enum
> igt_suspend_state state)
> > >  	return delay;
> > >  }
> > >
> > > -/**
> > > - * igt_get_memsleep_state
> > > - *
> > > - * Reads the value of /sys/power/mem_sleep and
> > > - * returns the current suspend state associated with 'mem'.
> > > - *
> > > - * Returns : an #igt_mem_sleep state, current suspend state associated with
> 'mem'.
> > > - */
> > > -int igt_get_memsleep_state(void)
> > > -{
> > > -	char *mem_sleep_states;
> > > -	char *mem_sleep_state;
> > > -	enum igt_mem_sleep mem_sleep;
> > > -	int power_dir;
> > > -
> > > -	igt_require((power_dir = open("/sys/power", O_RDONLY)) >= 0);
> > > -
> > > -	if (faccessat(power_dir, "mem_sleep", R_OK, 0))
> > > -		return MEM_SLEEP_NONE;
> > > -
> > > -	igt_assert((mem_sleep_states = igt_sysfs_get(power_dir,
> "mem_sleep")));
> > > -	for (mem_sleep_state = strtok(mem_sleep_states, " ");
> mem_sleep_state;
> > > -	     mem_sleep_state = strtok(NULL, " ")) {
> > > -		if (mem_sleep_state[0] == '[') {
> > > -			mem_sleep_state[strlen(mem_sleep_state) - 1] = '\0';
> > > -			mem_sleep_state++;
> > > -			break;
> > > -		}
> > > -	}
> > > -
> > > -	if (!mem_sleep_state) {
> > > -		free(mem_sleep_states);
> > > -		return MEM_SLEEP_NONE;
> > > -	}
> > > -
> > > -	for (mem_sleep = MEM_SLEEP_S2IDLE; mem_sleep <
> MEM_SLEEP_NUM; mem_sleep++) {
> > > -		if (strcmp(mem_sleep_name[mem_sleep], mem_sleep_state)
> == 0)
> > > -			break;
> > > -	}
> > > -
> > > -	igt_assert_f(mem_sleep < MEM_SLEEP_NUM, "Invalid mem_sleep
> state\n");
> > > -
> > > -	free(mem_sleep_states);
> > > -	close(power_dir);
> > > -	return mem_sleep;
> > > -}
> > >  /**
> > >   * igt_drop_root:
> > >   *
> > > diff --git a/lib/igt_aux.h b/lib/igt_aux.h index 9d5b7bd2..b1e48b0f
> > > 100644
> > > --- a/lib/igt_aux.h
> > > +++ b/lib/igt_aux.h
> > > @@ -135,13 +135,19 @@ bool igt_aub_dump_enabled(void);
> > >
> > >  /**
> > >   *  igt_suspend_state:
> > > - *  @SUSPEND_STATE_FREEZE: suspend-to-idle target state, aka S0ix
> > > or freeze,
> > > + *  @SUSPEND_STATE_FREEZE: Suspend-To-Idle target state, aka S0ix
> > > + or freeze,
> > >   *			   first non-hibernation state
> > > - *  @SUSPEND_STATE_STANDBY: standby target state, aka S1, second
> > > + *  @SUSPEND_STATE_STANDBY: "Power-On Suspend" target state, aka
> > > + S1, second
> > >   *			    non-hibernation state
> > > - *  @SUSPEND_STATE_MEM: suspend-to-mem target state aka S3, third
> > > - *			non-hibernation state
> > > - *  @SUSPEND_STATE_DISK: suspend-to-disk target state, aka S4 or
> > > hibernation
> > > + *  @SUSPEND_STATE_S3: Suspend-To-RAM: It enforces a "deep" state to
> mem_sleep,
> > > + *		       what forces the system to go to the third
> > > + *		       non-hibernation state, aka S3.
> > > + *  @SUSPEND_STATE_MEM: A memory sleep (non-hibernation) target
> state,
> > > + *			respecting the system's mem_sleep default:
> > > + *				s2idle: Suspend-To-Idle target state
> > > + *				shallow: "Power-On Suspend"
> > > + *				deep: Suspend-To-RAM
> > > + *  @SUSPEND_STATE_DISK: Suspend-To-Disk target state, aka S4 or
> > > + hibernation
> > >   *
> > >   *  Target suspend states used with igt_system_suspend_autoresume().
> > >   *  See /sys/power/state for the available states on a given machine.
> > > @@ -149,7 +155,8 @@ bool igt_aub_dump_enabled(void);  enum
> > > igt_suspend_state {
> > >  	SUSPEND_STATE_FREEZE,
> > >  	SUSPEND_STATE_STANDBY,
> > > -	SUSPEND_STATE_MEM,
> > > +	SUSPEND_STATE_S3, /* Forces Suspend-to-Ram (S3) */
> > > +	SUSPEND_STATE_MEM, /* Respects system default */
> > >  	SUSPEND_STATE_DISK,
> > >
> > >  	/*< private >*/
> > > --
> > > 2.35.3
> > >

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

* [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev2)
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
                   ` (4 preceding siblings ...)
  2022-07-18  5:54 ` [igt-dev] [PATCH i-g-t 1/4] " Anshuman Gupta
@ 2022-07-19 13:17 ` Patchwork
  2022-07-21 10:26 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev3) Patchwork
  6 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-07-19 13:17 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

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

== Series Details ==

Series: series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev2)
URL   : https://patchwork.freedesktop.org/series/106312/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_11921 -> IGTPW_7540
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_7540 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_7540, 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_7540/index.html

Participating hosts (32 -> 36)
------------------------------

  Additional (5): bat-dg1-5 bat-dg2-8 bat-adlp-4 bat-adln-1 bat-rpls-2 
  Missing    (1): bat-jsl-3 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-adl-ddr5:        [PASS][1] -> [SKIP][2] +4 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
    - bat-adlp-4:         NOTRUN -> [SKIP][3]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-rkl-guc:         [PASS][4] -> [SKIP][5] +3 similar issues
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-dp-2:
    - fi-icl-u2:          [PASS][6] -> [SKIP][7] +3 similar issues
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-icl-u2/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-dp-2.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-icl-u2/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-dp-2.html

  
#### Suppressed ####

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

  * igt@gem_exec_suspend@basic-s3@lmem0:
    - {bat-dg2-8}:        NOTRUN -> [SKIP][8] +1 similar issue
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg2-8/igt@gem_exec_suspend@basic-s3@lmem0.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - {fi-ehl-2}:         NOTRUN -> [SKIP][9] +3 similar issues
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-ehl-2/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_suspend@basic-s3-without-i915:
    - {fi-tgl-dsi}:       [SKIP][10] ([i915#5903]) -> [INCOMPLETE][11]
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-tgl-dsi/igt@i915_suspend@basic-s3-without-i915.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-tgl-dsi/igt@i915_suspend@basic-s3-without-i915.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1:
    - {fi-jsl-1}:         [PASS][12] -> [SKIP][13] +3 similar issues
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1:
    - {bat-adlp-6}:       [PASS][14] -> [SKIP][15] +4 similar issues
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1.html
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@fbdev@nullptr:
    - bat-dg1-5:          NOTRUN -> [SKIP][16] ([i915#2582]) +4 similar issues
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@fbdev@nullptr.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-glk-j4005:       [PASS][17] -> [SKIP][18] ([fdo#109271]) +3 similar issues
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-snb-2600:        [PASS][19] -> [SKIP][20] ([fdo#109271]) +2 similar issues
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-snb-2600/igt@gem_exec_suspend@basic-s3@smem.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-snb-2600/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6700k2:      [PASS][21] -> [SKIP][22] ([fdo#109271]) +3 similar issues
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-7567u:       [PASS][23] -> [SKIP][24] ([fdo#109271])
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6600u:       [PASS][25] -> [SKIP][26] ([fdo#109271]) +3 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bdw-5557u:       [PASS][27] -> [SKIP][28] ([fdo#109271]) +3 similar issues
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bsw-nick:        [PASS][29] -> [SKIP][30] ([fdo#109271])
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@gem_lmem_swapping@parallel-random-engines:
    - bat-adlp-4:         NOTRUN -> [SKIP][31] ([i915#4613]) +3 similar issues
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@gem_lmem_swapping@parallel-random-engines.html

  * igt@gem_mmap@basic:
    - bat-dg1-5:          NOTRUN -> [SKIP][32] ([i915#4083])
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@gem_mmap@basic.html

  * igt@gem_tiled_blits@basic:
    - bat-dg1-5:          NOTRUN -> [SKIP][33] ([i915#4077]) +2 similar issues
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@gem_tiled_blits@basic.html

  * igt@gem_tiled_pread_basic:
    - bat-dg1-5:          NOTRUN -> [SKIP][34] ([i915#4079]) +1 similar issue
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@gem_tiled_pread_basic.html
    - bat-adlp-4:         NOTRUN -> [SKIP][35] ([i915#3282])
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@gem_tiled_pread_basic.html

  * igt@i915_pm_backlight@basic-brightness:
    - bat-dg1-5:          NOTRUN -> [SKIP][36] ([i915#1155])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@i915_pm_backlight@basic-brightness.html

  * igt@i915_selftest@live@late_gt_pm:
    - fi-bsw-n3050:       [PASS][37] -> [DMESG-FAIL][38] ([i915#3428] / [i915#6217])
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-bsw-n3050/igt@i915_selftest@live@late_gt_pm.html
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bsw-n3050/igt@i915_selftest@live@late_gt_pm.html

  * igt@i915_selftest@live@requests:
    - fi-pnv-d510:        [PASS][39] -> [DMESG-FAIL][40] ([i915#4528])
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-pnv-d510/igt@i915_selftest@live@requests.html
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-pnv-d510/igt@i915_selftest@live@requests.html

  * igt@i915_suspend@basic-s2idle-without-i915:
    - bat-dg1-5:          NOTRUN -> [INCOMPLETE][41] ([i915#6011])
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@i915_suspend@basic-s2idle-without-i915.html

  * igt@kms_addfb_basic@basic-x-tiled-legacy:
    - bat-dg1-5:          NOTRUN -> [SKIP][42] ([i915#4212]) +7 similar issues
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_addfb_basic@basic-x-tiled-legacy.html

  * igt@kms_addfb_basic@basic-y-tiled-legacy:
    - bat-dg1-5:          NOTRUN -> [SKIP][43] ([i915#4215])
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_addfb_basic@basic-y-tiled-legacy.html

  * igt@kms_busy@basic:
    - bat-dg1-5:          NOTRUN -> [SKIP][44] ([i915#1845] / [i915#4303])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_busy@basic.html

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-hsw-g3258:       NOTRUN -> [SKIP][45] ([fdo#109271] / [fdo#111827])
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-hsw-g3258/igt@kms_chamelium@common-hpd-after-suspend.html

  * igt@kms_chamelium@dp-crc-fast:
    - bat-adlp-4:         NOTRUN -> [SKIP][46] ([fdo#111827]) +8 similar issues
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@kms_chamelium@dp-crc-fast.html
    - bat-dg1-5:          NOTRUN -> [SKIP][47] ([fdo#111827]) +7 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_chamelium@dp-crc-fast.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor:
    - bat-adlp-4:         NOTRUN -> [SKIP][48] ([i915#4103])
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html

  * igt@kms_force_connector_basic@force-load-detect:
    - bat-adlp-4:         NOTRUN -> [SKIP][49] ([i915#4093]) +3 similar issues
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@kms_force_connector_basic@force-load-detect.html
    - bat-dg1-5:          NOTRUN -> [SKIP][50] ([fdo#109285])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_pipe_crc_basic@nonblocking-crc:
    - bat-dg1-5:          NOTRUN -> [SKIP][51] ([i915#4078]) +13 similar issues
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_pipe_crc_basic@nonblocking-crc.html

  * igt@kms_pipe_crc_basic@suspend-read-crc:
    - bat-adlp-4:         NOTRUN -> [SKIP][52] ([i915#3546])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@kms_pipe_crc_basic@suspend-read-crc.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-elk-e7500:       [PASS][53] -> [SKIP][54] ([fdo#109271]) +2 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1:
    - fi-ilk-650:         [PASS][55] -> [SKIP][56] ([fdo#109271]) +2 similar issues
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2:
    - fi-cfl-8109u:       [PASS][57] -> [SKIP][58] ([fdo#109271]) +3 similar issues
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
    - fi-skl-guc:         [PASS][59] -> [SKIP][60] ([fdo#109271]) +3 similar issues
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1:
    - fi-glk-dsi:         [PASS][61] -> [SKIP][62] ([fdo#109271]) +3 similar issues
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1:
    - fi-bsw-kefka:       [PASS][63] -> [SKIP][64] ([fdo#109271]) +2 similar issues
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1:
    - fi-cfl-guc:         [PASS][65] -> [SKIP][66] ([fdo#109271]) +3 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1:
    - fi-snb-2520m:       [PASS][67] -> [SKIP][68] ([fdo#109271]) +2 similar issues
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1:
    - fi-hsw-4770:        [PASS][69] -> [SKIP][70] ([fdo#109271]) +3 similar issues
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-ivb-3770:        [PASS][71] -> [SKIP][72] ([fdo#109271]) +3 similar issues
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-hsw-g3258:       NOTRUN -> [SKIP][73] ([fdo#109271]) +3 similar issues
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-hsw-g3258/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html

  * igt@kms_psr@primary_page_flip:
    - bat-dg1-5:          NOTRUN -> [SKIP][74] ([i915#1072] / [i915#4078]) +3 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_psr@primary_page_flip.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - bat-dg1-5:          NOTRUN -> [SKIP][75] ([i915#3555])
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@kms_setmode@basic-clone-single-crtc.html
    - bat-adlp-4:         NOTRUN -> [SKIP][76] ([i915#3555] / [i915#4579])
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-fence-flip:
    - bat-dg1-5:          NOTRUN -> [SKIP][77] ([i915#1845] / [i915#3708])
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@prime_vgem@basic-fence-flip.html

  * igt@prime_vgem@basic-fence-read:
    - bat-dg1-5:          NOTRUN -> [SKIP][78] ([i915#3708]) +2 similar issues
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@prime_vgem@basic-fence-read.html

  * igt@prime_vgem@basic-gtt:
    - bat-dg1-5:          NOTRUN -> [SKIP][79] ([i915#3708] / [i915#4077]) +1 similar issue
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@prime_vgem@basic-gtt.html

  * igt@prime_vgem@basic-userptr:
    - bat-adlp-4:         NOTRUN -> [SKIP][80] ([fdo#109295] / [i915#3301] / [i915#3708])
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@prime_vgem@basic-userptr.html
    - bat-dg1-5:          NOTRUN -> [SKIP][81] ([i915#3708] / [i915#4873])
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@prime_vgem@basic-userptr.html

  * igt@prime_vgem@basic-write:
    - bat-adlp-4:         NOTRUN -> [SKIP][82] ([fdo#109295] / [i915#3291] / [i915#3708]) +2 similar issues
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-4/igt@prime_vgem@basic-write.html

  * igt@runner@aborted:
    - fi-bsw-n3050:       NOTRUN -> [FAIL][83] ([fdo#109271] / [i915#4312])
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bsw-n3050/igt@runner@aborted.html
    - bat-dg1-5:          NOTRUN -> [FAIL][84] ([i915#4312] / [i915#5257])
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-dg1-5/igt@runner@aborted.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s0@smem:
    - {fi-ehl-2}:         [DMESG-WARN][85] ([i915#5122]) -> [PASS][86]
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html

  * igt@i915_pm_rpm@module-reload:
    - fi-bsw-kefka:       [DMESG-WARN][87] ([i915#1982]) -> [PASS][88]
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-bsw-kefka/igt@i915_pm_rpm@module-reload.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-bsw-kefka/igt@i915_pm_rpm@module-reload.html

  * igt@i915_selftest@live@hangcheck:
    - fi-hsw-g3258:       [INCOMPLETE][89] ([i915#4785]) -> [PASS][90]
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html

  * igt@i915_suspend@basic-s3-without-i915:
    - {bat-adlp-6}:       [SKIP][91] ([i915#5903]) -> [PASS][92]
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/bat-adlp-6/igt@i915_suspend@basic-s3-without-i915.html
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/bat-adlp-6/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-ehl-2}:         [SKIP][93] ([i915#5903]) -> [PASS][94]
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
    - fi-icl-u2:          [SKIP][95] ([i915#5903]) -> [PASS][96]
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-icl-u2/igt@i915_suspend@basic-s3-without-i915.html
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-icl-u2/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-jsl-1}:         [SKIP][97] ([i915#5903]) -> [PASS][98]
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11921/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.html
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.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
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1155]: https://gitlab.freedesktop.org/drm/intel/issues/1155
  [i915#1845]: https://gitlab.freedesktop.org/drm/intel/issues/1845
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#2582]: https://gitlab.freedesktop.org/drm/intel/issues/2582
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
  [i915#3428]: https://gitlab.freedesktop.org/drm/intel/issues/3428
  [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3595]: https://gitlab.freedesktop.org/drm/intel/issues/3595
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4077]: https://gitlab.freedesktop.org/drm/intel/issues/4077
  [i915#4078]: https://gitlab.freedesktop.org/drm/intel/issues/4078
  [i915#4079]: https://gitlab.freedesktop.org/drm/intel/issues/4079
  [i915#4083]: https://gitlab.freedesktop.org/drm/intel/issues/4083
  [i915#4093]: https://gitlab.freedesktop.org/drm/intel/issues/4093
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4212]: https://gitlab.freedesktop.org/drm/intel/issues/4212
  [i915#4213]: https://gitlab.freedesktop.org/drm/intel/issues/4213
  [i915#4215]: https://gitlab.freedesktop.org/drm/intel/issues/4215
  [i915#4303]: https://gitlab.freedesktop.org/drm/intel/issues/4303
  [i915#4312]: https://gitlab.freedesktop.org/drm/intel/issues/4312
  [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
  [i915#4579]: https://gitlab.freedesktop.org/drm/intel/issues/4579
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
  [i915#4873]: https://gitlab.freedesktop.org/drm/intel/issues/4873
  [i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122
  [i915#5174]: https://gitlab.freedesktop.org/drm/intel/issues/5174
  [i915#5190]: https://gitlab.freedesktop.org/drm/intel/issues/5190
  [i915#5257]: https://gitlab.freedesktop.org/drm/intel/issues/5257
  [i915#5274]: https://gitlab.freedesktop.org/drm/intel/issues/5274
  [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
  [i915#5763]: https://gitlab.freedesktop.org/drm/intel/issues/5763
  [i915#5903]: https://gitlab.freedesktop.org/drm/intel/issues/5903
  [i915#5950]: https://gitlab.freedesktop.org/drm/intel/issues/5950
  [i915#6011]: https://gitlab.freedesktop.org/drm/intel/issues/6011
  [i915#6217]: https://gitlab.freedesktop.org/drm/intel/issues/6217
  [i915#6297]: https://gitlab.freedesktop.org/drm/intel/issues/6297


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6592 -> IGTPW_7540

  CI-20190529: 20190529
  CI_DRM_11921: a65867059d4429f7a13965ec292d7bf222d5cc16 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_7540: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7540/index.html
  IGT_6592: d7c0659613199a5dcf535ed3add68ed1991ead7e @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

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

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

* [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev3)
  2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
                   ` (5 preceding siblings ...)
  2022-07-19 13:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev2) Patchwork
@ 2022-07-21 10:26 ` Patchwork
  6 siblings, 0 replies; 13+ messages in thread
From: Patchwork @ 2022-07-21 10:26 UTC (permalink / raw)
  To: Rodrigo Vivi; +Cc: igt-dev

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

== Series Details ==

Series: series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev3)
URL   : https://patchwork.freedesktop.org/series/106312/
State : failure

== Summary ==

CI Bug Log - changes from CI_DRM_11932 -> IGTPW_7549
====================================================

Summary
-------

  **FAILURE**

  Serious unknown changes coming with IGTPW_7549 absolutely need to be
  verified manually.
  
  If you think the reported changes have nothing to do with the changes
  introduced in IGTPW_7549, 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_7549/index.html

Participating hosts (45 -> 42)
------------------------------

  Additional (2): fi-kbl-soraka fi-rkl-11600 
  Missing    (5): fi-tgl-dsi fi-hsw-4200u fi-ctg-p8600 bat-jsl-3 fi-bdw-samus 

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

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

### IGT changes ###

#### Possible regressions ####

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-adl-ddr5:        [PASS][1] -> [SKIP][2] +4 similar issues
   [1]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
   [2]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-adl-ddr5/igt@gem_exec_suspend@basic-s3@smem.html
    - bat-adlp-4:         [PASS][3] -> [SKIP][4]
   [3]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlp-4/igt@gem_exec_suspend@basic-s3@smem.html
   [4]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlp-4/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_selftest@live@perf:
    - fi-kbl-soraka:      NOTRUN -> [INCOMPLETE][5]
   [5]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@i915_selftest@live@perf.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-rkl-guc:         [PASS][6] -> [SKIP][7] +3 similar issues
   [6]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [7]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  
#### Warnings ####

  * igt@i915_suspend@basic-s3-without-i915:
    - fi-kbl-x1275:       [SKIP][8] ([fdo#109271]) -> [FAIL][9]
   [8]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-kbl-x1275/igt@i915_suspend@basic-s3-without-i915.html
   [9]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-x1275/igt@i915_suspend@basic-s3-without-i915.html

  
#### Suppressed ####

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

  * igt@gem_exec_suspend@basic-s3@lmem0:
    - {bat-dg2-9}:        [DMESG-WARN][10] ([i915#5763]) -> [SKIP][11] +4 similar issues
   [10]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-dg2-9/igt@gem_exec_suspend@basic-s3@lmem0.html
   [11]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-dg2-9/igt@gem_exec_suspend@basic-s3@lmem0.html
    - {bat-dg2-8}:        NOTRUN -> [SKIP][12] +1 similar issue
   [12]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-dg2-8/igt@gem_exec_suspend@basic-s3@lmem0.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - {bat-jsl-1}:        [PASS][13] -> [SKIP][14] +3 similar issues
   [13]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-jsl-1/igt@gem_exec_suspend@basic-s3@smem.html
   [14]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-jsl-1/igt@gem_exec_suspend@basic-s3@smem.html
    - {fi-ehl-2}:         NOTRUN -> [SKIP][15] +3 similar issues
   [15]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-ehl-2/igt@gem_exec_suspend@basic-s3@smem.html
    - {bat-adlm-1}:       [PASS][16] -> [SKIP][17]
   [16]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlm-1/igt@gem_exec_suspend@basic-s3@smem.html
   [17]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlm-1/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@i915_selftest@live@requests:
    - {bat-rpls-1}:       NOTRUN -> [INCOMPLETE][18]
   [18]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-rpls-1/igt@i915_selftest@live@requests.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-3:
    - {bat-dg2-9}:        [DMESG-WARN][19] ([i915#1982] / [i915#5763]) -> [SKIP][20]
   [19]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-dg2-9/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-3.html
   [20]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-dg2-9/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-dp-3.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1:
    - {fi-jsl-1}:         [PASS][21] -> [SKIP][22] +3 similar issues
   [21]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html
   [22]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-jsl-1/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1:
    - {bat-adlp-6}:       [PASS][23] -> [SKIP][24] +4 similar issues
   [23]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1.html
   [24]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlp-6/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-d-edp-1.html

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

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

### IGT changes ###

#### Issues hit ####

  * igt@gem_exec_gttfill@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][25] ([fdo#109271]) +8 similar issues
   [25]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@gem_exec_gttfill@basic.html

  * igt@gem_exec_suspend@basic-s3@smem:
    - fi-glk-j4005:       [PASS][26] -> [SKIP][27] ([fdo#109271]) +3 similar issues
   [26]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
   [27]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-glk-j4005/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bxt-dsi:         [PASS][28] -> [SKIP][29] ([fdo#109271]) +3 similar issues
   [28]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bxt-dsi/igt@gem_exec_suspend@basic-s3@smem.html
   [29]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bxt-dsi/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-x1275:       [PASS][30] -> [SKIP][31] ([fdo#109271]) +3 similar issues
   [30]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-kbl-x1275/igt@gem_exec_suspend@basic-s3@smem.html
   [31]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-x1275/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6700k2:      [PASS][32] -> [SKIP][33] ([fdo#109271]) +3 similar issues
   [32]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
   [33]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-skl-6700k2/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-7567u:       [PASS][34] -> [SKIP][35] ([fdo#109271])
   [34]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
   [35]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-7567u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-kbl-8809g:       [PASS][36] -> [SKIP][37] ([fdo#109271])
   [36]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-kbl-8809g/igt@gem_exec_suspend@basic-s3@smem.html
   [37]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-8809g/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-skl-6600u:       [PASS][38] -> [SKIP][39] ([fdo#109271]) +3 similar issues
   [38]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
   [39]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-skl-6600u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bdw-5557u:       [PASS][40] -> [SKIP][41] ([fdo#109271]) +3 similar issues
   [40]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
   [41]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bdw-5557u/igt@gem_exec_suspend@basic-s3@smem.html
    - fi-bsw-nick:        [PASS][42] -> [SKIP][43] ([fdo#109271])
   [42]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html
   [43]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bsw-nick/igt@gem_exec_suspend@basic-s3@smem.html

  * igt@gem_huc_copy@huc-copy:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][44] ([fdo#109271] / [i915#2190])
   [44]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@gem_huc_copy@huc-copy.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][45] ([i915#2190])
   [45]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@gem_huc_copy@huc-copy.html

  * igt@gem_lmem_swapping@basic:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][46] ([fdo#109271] / [i915#4613]) +3 similar issues
   [46]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@gem_lmem_swapping@basic.html
    - fi-rkl-11600:       NOTRUN -> [SKIP][47] ([i915#4613]) +3 similar issues
   [47]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@gem_lmem_swapping@basic.html

  * igt@gem_tiled_pread_basic:
    - fi-rkl-11600:       NOTRUN -> [SKIP][48] ([i915#3282])
   [48]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@gem_tiled_pread_basic.html

  * igt@i915_pm_backlight@basic-brightness:
    - fi-rkl-11600:       NOTRUN -> [SKIP][49] ([i915#3012])
   [49]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@i915_pm_backlight@basic-brightness.html

  * igt@i915_pm_rpm@module-reload:
    - fi-cfl-8109u:       [PASS][50] -> [DMESG-FAIL][51] ([i915#62])
   [50]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html
   [51]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8109u/igt@i915_pm_rpm@module-reload.html

  * igt@i915_selftest@live@gt_pm:
    - fi-kbl-soraka:      NOTRUN -> [DMESG-FAIL][52] ([i915#1886])
   [52]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@i915_selftest@live@gt_pm.html

  * igt@i915_selftest@live@late_gt_pm:
    - fi-cfl-8109u:       [PASS][53] -> [DMESG-WARN][54] ([i915#5904]) +29 similar issues
   [53]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8109u/igt@i915_selftest@live@late_gt_pm.html
   [54]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8109u/igt@i915_selftest@live@late_gt_pm.html

  * igt@i915_suspend@basic-s2idle-without-i915:
    - fi-cfl-8109u:       [PASS][55] -> [DMESG-WARN][56] ([i915#5904] / [i915#62])
   [55]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8109u/igt@i915_suspend@basic-s2idle-without-i915.html
   [56]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8109u/igt@i915_suspend@basic-s2idle-without-i915.html

  * igt@i915_suspend@basic-s3-without-i915:
    - fi-rkl-11600:       NOTRUN -> [INCOMPLETE][57] ([i915#5982])
   [57]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@i915_suspend@basic-s3-without-i915.html

  * igt@kms_chamelium@common-hpd-after-suspend:
    - fi-hsw-g3258:       NOTRUN -> [SKIP][58] ([fdo#109271] / [fdo#111827])
   [58]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-hsw-g3258/igt@kms_chamelium@common-hpd-after-suspend.html

  * igt@kms_chamelium@hdmi-edid-read:
    - fi-rkl-11600:       NOTRUN -> [SKIP][59] ([fdo#111827]) +7 similar issues
   [59]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@kms_chamelium@hdmi-edid-read.html

  * igt@kms_chamelium@hdmi-hpd-fast:
    - fi-kbl-soraka:      NOTRUN -> [SKIP][60] ([fdo#109271] / [fdo#111827]) +7 similar issues
   [60]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-soraka/igt@kms_chamelium@hdmi-hpd-fast.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor:
    - fi-rkl-11600:       NOTRUN -> [SKIP][61] ([i915#4103])
   [61]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@kms_cursor_legacy@basic-busy-flip-before-cursor.html

  * igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size:
    - fi-bsw-kefka:       [PASS][62] -> [FAIL][63] ([i915#6298])
   [62]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bsw-kefka/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size.html
   [63]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bsw-kefka/igt@kms_cursor_legacy@basic-busy-flip-before-cursor@atomic-transitions-varying-size.html

  * igt@kms_force_connector_basic@force-load-detect:
    - fi-rkl-11600:       NOTRUN -> [SKIP][64] ([fdo#109285] / [i915#4098])
   [64]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@kms_force_connector_basic@force-load-detect.html

  * igt@kms_frontbuffer_tracking@basic:
    - fi-cfl-8109u:       [PASS][65] -> [DMESG-WARN][66] ([i915#62]) +12 similar issues
   [65]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8109u/igt@kms_frontbuffer_tracking@basic.html
   [66]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8109u/igt@kms_frontbuffer_tracking@basic.html

  * igt@kms_pipe_crc_basic@suspend-read-crc:
    - bat-adlp-4:         NOTRUN -> [SKIP][67] ([i915#3546])
   [67]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlp-4/igt@kms_pipe_crc_basic@suspend-read-crc.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1:
    - fi-cfl-8700k:       [PASS][68] -> [SKIP][69] ([fdo#109271]) +3 similar issues
   [68]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [69]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8700k/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
    - fi-elk-e7500:       [PASS][70] -> [SKIP][71] ([fdo#109271]) +2 similar issues
   [70]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html
   [71]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-elk-e7500/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1:
    - fi-ilk-650:         [PASS][72] -> [SKIP][73] ([fdo#109271]) +2 similar issues
   [72]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html
   [73]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-ilk-650/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-a-vga-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2:
    - fi-cfl-8109u:       [PASS][74] -> [SKIP][75] ([fdo#109271]) +3 similar issues
   [74]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [75]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-8109u/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
    - fi-skl-guc:         [PASS][76] -> [SKIP][77] ([fdo#109271]) +3 similar issues
   [76]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html
   [77]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-skl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dp-2.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1:
    - fi-glk-dsi:         [PASS][78] -> [SKIP][79] ([fdo#109271]) +3 similar issues
   [78]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html
   [79]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-glk-dsi/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-dsi-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1:
    - fi-bsw-kefka:       [PASS][80] -> [SKIP][81] ([fdo#109271]) +2 similar issues
   [80]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html
   [81]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bsw-kefka/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-edp-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1:
    - fi-cfl-guc:         [PASS][82] -> [SKIP][83] ([fdo#109271]) +3 similar issues
   [82]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html
   [83]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-cfl-guc/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-hdmi-a-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1:
    - fi-snb-2520m:       [PASS][84] -> [SKIP][85] ([fdo#109271]) +2 similar issues
   [84]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html
   [85]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-snb-2520m/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-lvds-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-vga-1:
    - fi-blb-e6850:       NOTRUN -> [SKIP][86] ([fdo#109271]) +3 similar issues
   [86]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-blb-e6850/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-b-vga-1.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-hdmi-a-2:
    - fi-bsw-n3050:       [PASS][87] -> [SKIP][88] ([fdo#109271]) +1 similar issue
   [87]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-bsw-n3050/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-hdmi-a-2.html
   [88]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-bsw-n3050/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-hdmi-a-2.html

  * igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1:
    - fi-hsw-4770:        [PASS][89] -> [SKIP][90] ([fdo#109271]) +3 similar issues
   [89]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [90]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-hsw-4770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-ivb-3770:        [PASS][91] -> [SKIP][92] ([fdo#109271]) +3 similar issues
   [91]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
   [92]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-ivb-3770/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html
    - fi-hsw-g3258:       NOTRUN -> [SKIP][93] ([fdo#109271]) +3 similar issues
   [93]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-hsw-g3258/igt@kms_pipe_crc_basic@suspend-read-crc@pipe-c-vga-1.html

  * igt@kms_psr@primary_page_flip:
    - fi-rkl-11600:       NOTRUN -> [SKIP][94] ([i915#1072]) +3 similar issues
   [94]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@kms_psr@primary_page_flip.html

  * igt@kms_setmode@basic-clone-single-crtc:
    - fi-rkl-11600:       NOTRUN -> [SKIP][95] ([i915#3555] / [i915#4098])
   [95]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@kms_setmode@basic-clone-single-crtc.html

  * igt@prime_vgem@basic-read:
    - fi-rkl-11600:       NOTRUN -> [SKIP][96] ([fdo#109295] / [i915#3291] / [i915#3708]) +2 similar issues
   [96]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@prime_vgem@basic-read.html

  * igt@prime_vgem@basic-userptr:
    - fi-rkl-11600:       NOTRUN -> [SKIP][97] ([fdo#109295] / [i915#3301] / [i915#3708])
   [97]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-rkl-11600/igt@prime_vgem@basic-userptr.html

  
#### Possible fixes ####

  * igt@gem_exec_suspend@basic-s0@smem:
    - {fi-ehl-2}:         [DMESG-WARN][98] ([i915#5122]) -> [PASS][99]
   [98]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html
   [99]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-ehl-2/igt@gem_exec_suspend@basic-s0@smem.html

  * igt@i915_selftest@live@gt_heartbeat:
    - fi-kbl-7567u:       [DMESG-FAIL][100] ([i915#5334]) -> [PASS][101]
   [100]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-kbl-7567u/igt@i915_selftest@live@gt_heartbeat.html
   [101]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-kbl-7567u/igt@i915_selftest@live@gt_heartbeat.html
    - {bat-rpls-1}:       [INCOMPLETE][102] -> [PASS][103]
   [102]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-rpls-1/igt@i915_selftest@live@gt_heartbeat.html
   [103]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-rpls-1/igt@i915_selftest@live@gt_heartbeat.html

  * igt@i915_selftest@live@guc:
    - {bat-dg2-9}:        [DMESG-WARN][104] ([i915#5763]) -> [PASS][105] +5 similar issues
   [104]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-dg2-9/igt@i915_selftest@live@guc.html
   [105]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-dg2-9/igt@i915_selftest@live@guc.html

  * igt@i915_selftest@live@hangcheck:
    - fi-hsw-g3258:       [INCOMPLETE][106] ([i915#3303] / [i915#4785]) -> [PASS][107]
   [106]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html
   [107]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-hsw-g3258/igt@i915_selftest@live@hangcheck.html

  * igt@i915_selftest@live@requests:
    - fi-blb-e6850:       [DMESG-FAIL][108] ([i915#4528]) -> [PASS][109]
   [108]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-blb-e6850/igt@i915_selftest@live@requests.html
   [109]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-blb-e6850/igt@i915_selftest@live@requests.html

  * igt@i915_selftest@live@uncore:
    - {bat-adln-1}:       [DMESG-FAIL][110] ([i915#6297]) -> [PASS][111]
   [110]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adln-1/igt@i915_selftest@live@uncore.html
   [111]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adln-1/igt@i915_selftest@live@uncore.html

  * igt@i915_suspend@basic-s3-without-i915:
    - {bat-adlp-6}:       [SKIP][112] ([i915#5903]) -> [PASS][113]
   [112]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlp-6/igt@i915_suspend@basic-s3-without-i915.html
   [113]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlp-6/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-ehl-2}:         [SKIP][114] ([i915#5903]) -> [PASS][115]
   [114]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
   [115]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-ehl-2/igt@i915_suspend@basic-s3-without-i915.html
    - {bat-adlm-1}:       [SKIP][116] ([i915#5903]) -> [PASS][117]
   [116]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlm-1/igt@i915_suspend@basic-s3-without-i915.html
   [117]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlm-1/igt@i915_suspend@basic-s3-without-i915.html
    - bat-adlp-4:         [SKIP][118] ([i915#5903]) -> [PASS][119]
   [118]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-adlp-4/igt@i915_suspend@basic-s3-without-i915.html
   [119]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-adlp-4/igt@i915_suspend@basic-s3-without-i915.html
    - {bat-jsl-1}:        [SKIP][120] ([i915#5903]) -> [PASS][121]
   [120]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/bat-jsl-1/igt@i915_suspend@basic-s3-without-i915.html
   [121]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/bat-jsl-1/igt@i915_suspend@basic-s3-without-i915.html
    - {fi-jsl-1}:         [SKIP][122] ([i915#5903]) -> [PASS][123]
   [122]: https://intel-gfx-ci.01.org/tree/drm-tip/CI_DRM_11932/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.html
   [123]: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/fi-jsl-1/igt@i915_suspend@basic-s3-without-i915.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
  [fdo#109285]: https://bugs.freedesktop.org/show_bug.cgi?id=109285
  [fdo#109295]: https://bugs.freedesktop.org/show_bug.cgi?id=109295
  [fdo#111827]: https://bugs.freedesktop.org/show_bug.cgi?id=111827
  [i915#1072]: https://gitlab.freedesktop.org/drm/intel/issues/1072
  [i915#1886]: https://gitlab.freedesktop.org/drm/intel/issues/1886
  [i915#1982]: https://gitlab.freedesktop.org/drm/intel/issues/1982
  [i915#2190]: https://gitlab.freedesktop.org/drm/intel/issues/2190
  [i915#3012]: https://gitlab.freedesktop.org/drm/intel/issues/3012
  [i915#3282]: https://gitlab.freedesktop.org/drm/intel/issues/3282
  [i915#3291]: https://gitlab.freedesktop.org/drm/intel/issues/3291
  [i915#3301]: https://gitlab.freedesktop.org/drm/intel/issues/3301
  [i915#3303]: https://gitlab.freedesktop.org/drm/intel/issues/3303
  [i915#3546]: https://gitlab.freedesktop.org/drm/intel/issues/3546
  [i915#3555]: https://gitlab.freedesktop.org/drm/intel/issues/3555
  [i915#3708]: https://gitlab.freedesktop.org/drm/intel/issues/3708
  [i915#4098]: https://gitlab.freedesktop.org/drm/intel/issues/4098
  [i915#4103]: https://gitlab.freedesktop.org/drm/intel/issues/4103
  [i915#4528]: https://gitlab.freedesktop.org/drm/intel/issues/4528
  [i915#4613]: https://gitlab.freedesktop.org/drm/intel/issues/4613
  [i915#4785]: https://gitlab.freedesktop.org/drm/intel/issues/4785
  [i915#5122]: https://gitlab.freedesktop.org/drm/intel/issues/5122
  [i915#5334]: https://gitlab.freedesktop.org/drm/intel/issues/5334
  [i915#5354]: https://gitlab.freedesktop.org/drm/intel/issues/5354
  [i915#5763]: https://gitlab.freedesktop.org/drm/intel/issues/5763
  [i915#5903]: https://gitlab.freedesktop.org/drm/intel/issues/5903
  [i915#5904]: https://gitlab.freedesktop.org/drm/intel/issues/5904
  [i915#5982]: https://gitlab.freedesktop.org/drm/intel/issues/5982
  [i915#62]: https://gitlab.freedesktop.org/drm/intel/issues/62
  [i915#6297]: https://gitlab.freedesktop.org/drm/intel/issues/6297
  [i915#6298]: https://gitlab.freedesktop.org/drm/intel/issues/6298


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

  * CI: CI-20190529 -> None
  * IGT: IGT_6594 -> IGTPW_7549

  CI-20190529: 20190529
  CI_DRM_11932: 7324d18fbc95e41f719b0e3bd8b8987d834d7905 @ git://anongit.freedesktop.org/gfx-ci/linux
  IGTPW_7549: https://intel-gfx-ci.01.org/tree/drm-tip/IGTPW_7549/index.html
  IGT_6594: 326629f105459f9bd201456a0454759628e6a43d @ https://gitlab.freedesktop.org/drm/igt-gpu-tools.git

== Logs ==

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

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

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

end of thread, other threads:[~2022-07-21 10:26 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-13 15:52 [igt-dev] [PATCH i-g-t 1/4] lib/igt_aux: Let's use the official mem_sleep names Rodrigo Vivi
2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 2/4] tests/i915/i915_suspend: Be more specific on the S3 mem_sleep requirement Rodrigo Vivi
2022-07-18  6:24   ` Anshuman Gupta
2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 3/4] lib/igt_aux: Introduce the ability to force S3 suspend state Rodrigo Vivi
2022-07-18  6:47   ` Anshuman Gupta
2022-07-18 21:42     ` Rodrigo Vivi
2022-07-19  5:20       ` Gupta, Anshuman
2022-07-13 15:52 ` [igt-dev] [PATCH i-g-t 4/4] test/i915/i915_suspend: Use SUSPEND_STATE_S3 to enforce S3 Rodrigo Vivi
2022-07-18  7:09   ` Anshuman Gupta
2022-07-13 18:58 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names Patchwork
2022-07-18  5:54 ` [igt-dev] [PATCH i-g-t 1/4] " Anshuman Gupta
2022-07-19 13:17 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev2) Patchwork
2022-07-21 10:26 ` [igt-dev] ✗ Fi.CI.BAT: failure for series starting with [i-g-t,1/4] lib/igt_aux: Let's use the official mem_sleep names (rev3) 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.