[igt-dev] [PATCH i-g-t 1/2] lib/psr: Skip PSR tests if sink not reliable

José Roberto de Souza jose.souza at intel.com
Wed Oct 23 21:50:30 UTC 2019


Right now if sink reported any PSR error or if it fails to
acknowledge the PSR wakeup it sets a flag and do not attempt to
enable PSR anymore. That is the safest approach to avoid repetitive
glitches and allowed us to have PSR enabled by default.

But from time to time even good PSR panels have a PSR error, causing
tests to fail. And for now we are not yet to the point were we could
try to recover from PSR errors, so lets check for this information in
debugfs and skip PSR tests because of sink errors and eliminate this
noise from CI runs.

Cc: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
Cc: Rodrigo Vivi <rodrigo.vivi at intel.com>
Cc: Matt Roper <matthew.d.roper at intel.com>
Cc: Ap Kamal <kamal.ap at intel.com>
Signed-off-by: José Roberto de Souza <jose.souza at intel.com>
---
 lib/igt_psr.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lib/igt_psr.c b/lib/igt_psr.c
index b92ea73f..0e79dcd9 100644
--- a/lib/igt_psr.c
+++ b/lib/igt_psr.c
@@ -33,6 +33,8 @@ static bool psr_active_check(int debugfs_fd, enum psr_mode mode)
 	igt_debugfs_simple_read(debugfs_fd, "i915_edp_psr_status", buf,
 				sizeof(buf));
 
+	igt_skip_on(strstr(buf, "PSR sink not reliable: yes"));
+
 	return strstr(buf, state);
 }
 
-- 
2.23.0



More information about the igt-dev mailing list