public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t] lib/pm: Fail gracefully if CONFIG_PM is not built into the kernel
@ 2018-09-05  9:07 Chris Wilson
  2018-09-05 12:14 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Chris Wilson @ 2018-09-05  9:07 UTC (permalink / raw)
  To: igt-dev; +Cc: Tvrtko Ursulin

We report the failure from igt_setup_runtime_pm() so that the caller can
decide how they may proceed, so replace the hard assert with the error
propagation.

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Imre Deak <imre.deak@intel.com>
---
 lib/igt_pm.c | 15 ++++++++++-----
 1 file changed, 10 insertions(+), 5 deletions(-)

diff --git a/lib/igt_pm.c b/lib/igt_pm.c
index 615463a08..49027238b 100644
--- a/lib/igt_pm.c
+++ b/lib/igt_pm.c
@@ -511,12 +511,16 @@ bool igt_setup_runtime_pm(void)
 
 	igt_pm_enable_audio_runtime_pm();
 
-	/* Our implementation uses autosuspend. Try to set it to 0ms so the test
-	 * suite goes faster and we have a higher probability of triggering race
-	 * conditions. */
+	/*
+	 * Our implementation uses autosuspend. Try to set it to 0ms so the
+	 * test suite goes faster and we have a higher probability of
+	 * triggering race conditions.
+	 */
 	fd = open(POWER_DIR "/autosuspend_delay_ms", O_RDWR);
-	igt_assert_f(fd >= 0,
-		     "Can't open " POWER_DIR "/autosuspend_delay_ms\n");
+	if (fd < 0) {
+		igt_pm_audio_restore_runtime_pm();
+		return false;
+	}
 
 	/*
 	 * Save previous values to be able to  install exit handler to restore
@@ -531,6 +535,7 @@ bool igt_setup_runtime_pm(void)
 	 */
 	if (size <= 0) {
 		close(fd);
+		igt_pm_audio_restore_runtime_pm();
 		return false;
 	}
 
-- 
2.19.0.rc1

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

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

end of thread, other threads:[~2018-09-06  9:02 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-09-05  9:07 [igt-dev] [PATCH i-g-t] lib/pm: Fail gracefully if CONFIG_PM is not built into the kernel Chris Wilson
2018-09-05 12:14 ` [igt-dev] ✓ Fi.CI.BAT: success for " Patchwork
2018-09-05 15:01 ` [igt-dev] [PATCH i-g-t] " Tvrtko Ursulin
2018-09-05 15:13   ` Chris Wilson
2018-09-06  8:51     ` Tvrtko Ursulin
2018-09-06  9:02       ` Chris Wilson
2018-09-05 15:39 ` [igt-dev] ✓ Fi.CI.IGT: success for " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox