Igt-dev Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Sujaritha Sundaresan <sujaritha.sundaresan@intel.com>
To: igt-dev@lists.freedesktop.org
Cc: kamil.konieczny@linux.intel.com, anshuman.gupta@intel.com,
	katarzyna.piecielska@intel.com,
	Sujaritha Sundaresan <sujaritha.sundaresan@intel.com>,
	"Katarzyna Piecielska Katarzyna . piecielska @ intel . com"
	<Katarzyna.piecielska@intel.com>
Subject: [PATCH i-g-t, v4] tests/intel/xe_pm: Add tests for suspend without display
Date: Thu, 22 Aug 2024 14:50:58 +0530	[thread overview]
Message-ID: <20240822092058.26784-1-sujaritha.sundaresan@intel.com> (raw)

Add tests to validate basic execution suspend/resume cycle
without display module to rule out display related issues
from the suspend/resume stack.

v2: Add normal reload cycle after running test (Anshuman)

v3: Rebase

v4: Change commit name/header (Kamil)

Signed-off-by: Sujaritha Sundaresan <sujaritha.sundaresan@intel.com>
Acked-by: Katarzyna Piecielska Katarzyna.piecielska@intel.com<Katarzyna.piecielska@intel.com>
Reviewed-by: Anshuman Gupta <anshuman.gupta@intel.com>
---
 tests/intel/xe_pm.c | 34 ++++++++++++++++++++++++++++++++++
 1 file changed, 34 insertions(+)

diff --git a/tests/intel/xe_pm.c b/tests/intel/xe_pm.c
index eee89428c..1623a56c7 100644
--- a/tests/intel/xe_pm.c
+++ b/tests/intel/xe_pm.c
@@ -17,6 +17,7 @@
 
 #include "igt.h"
 #include "lib/igt_device.h"
+#include "lib/igt_kmod.h"
 #include "lib/igt_pm.h"
 #include "lib/igt_sysfs.h"
 #include "lib/igt_syncobj.h"
@@ -229,6 +230,10 @@ static void close_fw_handle(int sig)
  * Description: suspend/autoresume on %arg[1] state and exec after RPM
  * Functionality: pm - %arg[1]
  *
+ * SUBTEST: %s-without-display
+ * Description: suspend/autoresume on %arg[1] state without display
+ * Functionality: pm - %arg[1]
+ *
  * arg[1]:
  *
  * @s2idle:	s2idle
@@ -681,6 +686,7 @@ igt_main
 	struct drm_xe_engine_class_instance *hwe;
 	device_t device;
 	uint32_t d3cold_allowed;
+	const char *opts;
 	int sysfs_fd;
 
 	const struct s_state {
@@ -757,6 +763,34 @@ igt_main
 					  NO_RPM, 0);
 		}
 
+		igt_subtest_f("%s-without-display", s->name) {
+
+			if (!drmModeGetResources(device.fd_xe))
+				return;
+
+			xe_for_each_engine(device.fd_xe, hwe) {
+
+				igt_debug("Reload w/o display\n");
+
+				igt_kmsg(KMSG_INFO "Unloading Xe\n");
+				igt_assert_eq(igt_xe_driver_unload(), 0);
+
+				igt_kmsg(KMSG_INFO "Re-loading Xe without display\n");
+				igt_assert_eq(igt_xe_driver_load("enable_display=0"), 0);
+
+				test_exec(device, hwe, 1, 2, s->state,
+					  NO_RPM, 0);
+
+				igt_debug("Reload as normal\n");
+
+				igt_kmsg(KMSG_INFO "Unloading Xe\n");
+				igt_assert_eq(igt_xe_driver_unload(), 0);
+
+				igt_kmsg(KMSG_INFO "Re-loading Xe\n");
+				igt_assert_eq(igt_xe_driver_load(opts), 0);
+			}
+		}
+
 		for (const struct vm_op *op = vm_op; op->name; op++) {
 			igt_subtest_f("%s-vm-bind-%s", s->name, op->name) {
 				xe_for_each_engine(device.fd_xe, hwe)
-- 
2.34.1


             reply	other threads:[~2024-08-22  8:57 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-08-22  9:20 Sujaritha Sundaresan [this message]
2024-08-22 10:07 ` ✓ CI.xeBAT: success for tests/intel/xe_pm: Add tests for suspend without display Patchwork
2024-08-22 10:09 ` ✓ Fi.CI.BAT: " Patchwork
2024-08-22 14:36 ` ✗ CI.xeFULL: failure " Patchwork
2024-08-22 16:52 ` [PATCH i-g-t, v4] " Kamil Konieczny
2024-08-22 17:45   ` Gustavo Sousa
2024-08-22 21:01     ` Lucas De Marchi
2024-08-26 11:12       ` Sundaresan, Sujaritha
2024-08-26 13:57         ` Lucas De Marchi
2024-08-27  8:10           ` Sundaresan, Sujaritha
2024-08-26 14:36       ` Gustavo Sousa
2024-08-27  8:11         ` Sundaresan, Sujaritha
2024-08-27  9:29           ` Gupta, Anshuman
2024-08-27 12:48             ` Gustavo Sousa
     [not found]               ` <CY5PR11MB6211B873C18594896BD21FA295942@CY5PR11MB6211.namprd11.prod.outlook.com>
2024-08-28  8:46                 ` Sundaresan, Sujaritha
2024-09-11  5:19                   ` Sundaresan, Sujaritha
2024-09-11 15:10                     ` Jani Nikula
2024-09-13  8:29                       ` Sundaresan, Sujaritha
2024-09-23  9:30                         ` Jani Nikula
2024-08-23  6:20 ` ✗ Fi.CI.IGT: failure for " Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240822092058.26784-1-sujaritha.sundaresan@intel.com \
    --to=sujaritha.sundaresan@intel.com \
    --cc=anshuman.gupta@intel.com \
    --cc=igt-dev@lists.freedesktop.org \
    --cc=kamil.konieczny@linux.intel.com \
    --cc=katarzyna.piecielska@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox