diff options
author | Chris Wilson <chris@chris-wilson.co.uk> | 2018-10-03 20:43:49 +0100 |
---|---|---|
committer | Arkadiusz Hiler <arkadiusz.hiler@intel.com> | 2018-10-04 15:44:09 +0300 |
commit | 519003cace0c32aa226005dc7409544e26c243c2 (patch) | |
tree | ebbc2d82b3c547d33477c05417602ecb95d431cc /tests/kms_flip.c | |
parent | 596f48dcd59fd2f8c16671514f3e69d4a2891374 (diff) |
igt: Check drmModeGetResources()
If KMS is not supported on the device, drmModeGetResources() will return
NULL, often this is an indication that we should not attempt to run the
test. Although it would be preferred to use something like
igt_require_display() as the canonical check and assert that
drmModeGetResources() did not hit an error, it is not always practical
as the tests do not utilize the common igt_display abstraction.
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Diffstat (limited to 'tests/kms_flip.c')
-rw-r--r-- | tests/kms_flip.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/kms_flip.c b/tests/kms_flip.c index 44a82053..f28272dd 100644 --- a/tests/kms_flip.c +++ b/tests/kms_flip.c @@ -1355,7 +1355,7 @@ static int run_test(int duration, int flags) igt_require(igt_setup_runtime_pm()); resources = drmModeGetResources(drm_fd); - igt_assert(resources); + igt_require(resources); /* Count output configurations to scale test runtime. */ for (i = 0; i < resources->count_connectors; i++) { @@ -1412,7 +1412,7 @@ static int run_pair(int duration, int flags) igt_require((flags & TEST_HANG) == 0 || !is_wedged(drm_fd)); resources = drmModeGetResources(drm_fd); - igt_assert(resources); + igt_require(resources); /* Find a pair of connected displays */ for (i = 0; i < resources->count_connectors; i++) { |