diff options
| author | Matt Roper <matthew.d.roper@intel.com> | 2015-03-05 15:01:00 -0800 |
|---|---|---|
| committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-03-06 18:06:10 +0100 |
| commit | 07be8fec156ba8405f2aafa3a37fe29d06893f05 (patch) | |
| tree | 0591bf1aca05c2c8f5f89bb0400d948e8a2d0fe6 /tests/prime_nv_pcopy.c | |
| parent | 14a3d88a0967718124fc3b6c111a877203da025f (diff) | |
igt.cocci: Replace igt_assert() with igt_assert_CMP() where possible
The integer comparison macros give us better error output by including
the actual values that failed the comparison.
Signed-off-by: Matt Roper <matthew.d.roper@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@intel.com>
Diffstat (limited to 'tests/prime_nv_pcopy.c')
| -rw-r--r-- | tests/prime_nv_pcopy.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/tests/prime_nv_pcopy.c b/tests/prime_nv_pcopy.c index 218f4ba0..e02835ee 100644 --- a/tests/prime_nv_pcopy.c +++ b/tests/prime_nv_pcopy.c @@ -411,7 +411,7 @@ static void perform_copy(struct nouveau_bo *nvbo, const rect *dst, ret = nouveau_pushbuf_kick(push, push->channel); while (!ret && *query < query_counter) { usleep(1000); } - igt_assert(ret == 0); + igt_assert_eq(ret, 0); } static void check1_macro(uint32_t *p, uint32_t w, uint32_t h) @@ -561,7 +561,7 @@ static void test1_micro(void) igt_assert(drm_intel_gem_bo_map_gtt(test_intel_bo) == 0); drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd); - igt_assert(prime_fd >= 0); + igt_assert_lte(0, prime_fd); noop_intel(test_intel_bo); nv_bo_alloc(&bo_intel, &intel, w, h, tile_intel_y, prime_fd, 0); @@ -695,7 +695,7 @@ static void test3_base(int tile_src, int tile_dst) igt_assert(test_intel_bo); drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd); - igt_assert(prime_fd >= 0); + igt_assert_lte(0, prime_fd); nv_bo_alloc(&bo_intel, &intel, 2048 * cpp, 768, tile_dst, prime_fd, 0); nv_bo_alloc(&bo_nvidia, &nvidia, 300 * cpp, 300, tile_src, -1, NOUVEAU_BO_VRAM); @@ -783,7 +783,7 @@ static void test_semaphore(void) igt_assert(test_intel_bo); drm_intel_bo_gem_export_to_prime(test_intel_bo, &prime_fd); - igt_assert(prime_fd >= 0); + igt_assert_lte(0, prime_fd); igt_assert(nouveau_bo_prime_handle_ref(ndev, prime_fd, &sema_bo) == 0); close(prime_fd); |