diff options
| author | Michal Wajdeczko <michal.wajdeczko@intel.com> | 2017-12-04 13:02:55 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2017-12-15 12:59:21 +0000 |
| commit | c0be3310715e2f744b892c51f09e62273bcc8e57 (patch) | |
| tree | 70ed44b9f66ae97158814322b82ea950acade986 /tests/kms_frontbuffer_tracking.c | |
| parent | 417b29e2fbc97f99455a9e4bd22270bab0a4628f (diff) | |
tests/kms_frontbuffer_tracking: Correctly handle debugfs errors
In commit 3f6ae7b19 ("igt/kms_frontbuffer_tracking: Keep the debugfs
dir around") we introduced custom variant of __igt_debugfs_read function
that fires assert when debugfs returns an error. Replace that assert
with proper error handling to allow use of errors like -ENODEV.
v2: allow only -ENODEV (Chris)
Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'tests/kms_frontbuffer_tracking.c')
| -rw-r--r-- | tests/kms_frontbuffer_tracking.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tests/kms_frontbuffer_tracking.c b/tests/kms_frontbuffer_tracking.c index a068c8af..c6586c41 100644 --- a/tests/kms_frontbuffer_tracking.c +++ b/tests/kms_frontbuffer_tracking.c @@ -783,7 +783,10 @@ static bool set_mode_for_params(struct modeset_params *params) static void __debugfs_read(const char *param, char *buf, int len) { len = igt_sysfs_read(drm.debugfs, param, buf, len - 1); - igt_assert(len > 0); + if (len < 0) { + igt_assert_eq(len, -ENODEV); + len = 0; + } buf[len] = '\0'; } |