summaryrefslogtreecommitdiff
path: root/tests/i915/kms_psr2_su.c
diff options
context:
space:
mode:
authorJosé Roberto de Souza <jose.souza@intel.com>2021-10-04 16:32:45 -0700
committerJosé Roberto de Souza <jose.souza@intel.com>2021-10-06 10:48:24 -0700
commitec1111356de468d175cbb1f55b2294bce115d07f (patch)
tree6519e7450145817fc0335ee55eb307f941be6101 /tests/i915/kms_psr2_su.c
parenta35b89cd2b88c33205777635d405ecb641aa26de (diff)
tests/kms_psr2_su: Skip test when necessary
The whole test needs to skip in display 13 platforms as PSR2_SU_STATUS is not being updated anymore by HW. It was not reliable in older display versions and now it is 100% dead. For display 12 and newer platforms now we have PSR2 selective fetch enabled default, so software is doing all the tracking of the areas that needs updates but we still can't do that for frontbuffer rendering so we are doing full frame fetches for now what will cause this test to fail, so also skipping it for the frontbuffer subtest. Cc: Jeevan B <jeevan.b@intel.com> Cc: Swati Sharma <swati2.sharma@intel.com> Cc: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> Reviewed-by: Gwan-gyeong Mun <gwan-gyeong.mun@intel.com> Signed-off-by: José Roberto de Souza <jose.souza@intel.com>
Diffstat (limited to 'tests/i915/kms_psr2_su.c')
-rw-r--r--tests/i915/kms_psr2_su.c18
1 files changed, 18 insertions, 0 deletions
diff --git a/tests/i915/kms_psr2_su.c b/tests/i915/kms_psr2_su.c
index 318deefd..3018bcc6 100644
--- a/tests/i915/kms_psr2_su.c
+++ b/tests/i915/kms_psr2_su.c
@@ -251,6 +251,9 @@ igt_main
data.debugfs_fd, PSR_MODE_2),
"Sink does not support PSR2\n");
+ igt_require_f(intel_display_ver(intel_get_drm_devid(data.drm_fd)) < 13,
+ "Registers used by this test do not work on display 13\n");
+
display_init(&data);
/* Test if PSR2 can be enabled */
@@ -280,6 +283,21 @@ igt_main
for (data.op = PAGE_FLIP; data.op < LAST; data.op++) {
igt_describe("Test that selective update works when screen changes");
igt_subtest_f("%s", op_str(data.op)) {
+
+ if (data.op == FRONTBUFFER &&
+ intel_display_ver(intel_get_drm_devid(data.drm_fd)) >= 12) {
+ /*
+ * FIXME: Display 12+ platforms now have PSR2
+ * selective fetch enabled by default but we
+ * still can't properly handle frontbuffer
+ * rendering, so right it does full frame
+ * fetches at every frontbuffer rendering.
+ * So it is expected that this test will fail
+ * in display 12+ platform fow now.
+ */
+ igt_skip("PSR2 selective fetch is doing full frame fetches for frontbuffer rendering\n");
+ }
+
prepare(&data);
run(&data);
cleanup(&data);