All of lore.kernel.org
 help / color / mirror / Atom feed
* [igt-dev] [i-g-t v2] tests/i915/i915_pm_dc: Remove PC8+ dependency for DGFX DC9
@ 2022-07-12 16:47 Anusha Srivatsa
  2022-07-12 17:49 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Anusha Srivatsa @ 2022-07-12 16:47 UTC (permalink / raw)
  To: igt-dev; +Cc: Rodrigo Vivi

For DGFX, the DC9 flow and dependency is different from igfx.
DC9 does not depend on BIOS/PCU to execute save
and restore or any other sequence when
going to PC9/PC10

DC9 actually depends on D3 and PCODE simply remove power
if SW allowed

v2: Get the info about platform being discrete or not from
i915_gpu_info debugfs (Anshuman)

v2: Update commit message and subject to reflect that the changes
Cc: Imre Deak <imre.deak@intel.com>
Cc: Anshuman Gupta <anshuman.gupta@intel.com>
Cc: Anusha Srivatsa <anusha.srivatsa@intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Signed-off-by: Anusha Srivatsa <anusha.srivatsa@intel.com>(v2)
---
 tests/i915/i915_pm_dc.c | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/tests/i915/i915_pm_dc.c b/tests/i915/i915_pm_dc.c
index a780f2cb..f664a2a2 100644
--- a/tests/i915/i915_pm_dc.c
+++ b/tests/i915/i915_pm_dc.c
@@ -422,6 +422,15 @@ static void check_dc9(data_t *data, int dc_target, int prev_dc)
 			data->debugfs_dump = igt_sysfs_get(data->debugfs_fd, RPM_STATUS));
 }
 
+static bool check_is_dgfx(data_t *data)
+{
+	char buf[4096];
+
+	igt_debugfs_simple_read(data->debugfs_fd, "i915_gpu_info",
+				buf, sizeof(buf));
+	return strstr(buf, "is_dgfx: yes");
+}
+
 static void setup_dc9_dpms(data_t *data, int dc_target)
 {
 	int prev_dc, sysfs_fd;
@@ -532,8 +541,9 @@ int main(int argc, char *argv[])
 
 	igt_describe("This test validates display engine entry to DC9 state");
 	igt_subtest("dc9-dpms") {
-		igt_require_f(igt_pm_pc8_plus_residencies_enabled(data.msr_fd),
-				"PC8+ residencies not supported\n");
+		if (!(check_is_dgfx(&data)))
+			igt_require_f(igt_pm_pc8_plus_residencies_enabled(data.msr_fd),
+				      "PC8+ residencies not supported\n");
 		test_dc9_dpms(&data);
 	}
 
-- 
2.25.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread
end of thread, other threads:[~2022-08-01 18:12 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-07-12 16:47 [igt-dev] [i-g-t v2] tests/i915/i915_pm_dc: Remove PC8+ dependency for DGFX DC9 Anusha Srivatsa
2022-07-12 17:49 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2022-07-12 21:25 ` [igt-dev] ✗ Fi.CI.IGT: failure " Patchwork
2022-07-14 19:28 ` [igt-dev] [i-g-t v2] " Rodrigo Vivi
2022-08-01 18:12   ` Rodrigo Vivi
2022-07-20  0:01 ` [igt-dev] ✗ Fi.CI.BAT: failure for tests/i915/i915_pm_dc: Remove PC8+ dependency for DGFX DC9 (rev2) 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.