diff options
author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-09-25 14:36:59 +0200 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2013-09-25 14:37:07 +0200 |
commit | f3c54d0cb4744af9d58b5be45f574b625bbc8231 (patch) | |
tree | 9c4bd1af155ee0b1b51382a617109aa2e5f119b2 /tests/gem_reloc_vs_gpu.c | |
parent | 88ad6d7be8ab5741b7019a535952145c96577c9c (diff) |
tests: use igt_assert/igt_require more
With the new _f variants we can replace almost all of them.
Also remove a ton of checks for argc != 1, they're a bit useless ...
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Diffstat (limited to 'tests/gem_reloc_vs_gpu.c')
-rw-r--r-- | tests/gem_reloc_vs_gpu.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/gem_reloc_vs_gpu.c b/tests/gem_reloc_vs_gpu.c index cab726ee..14d11042 100644 --- a/tests/gem_reloc_vs_gpu.c +++ b/tests/gem_reloc_vs_gpu.c @@ -256,10 +256,8 @@ static void do_test(int fd, bool faulting_reloc) /* Only check at the end to avoid unnecessary synchronous behaviour. */ for (i = 0; i < NUM_TARGET_BOS; i++) { drm_intel_bo_get_subdata(pc_target_bo[i], 0, 4, &test); - if (test != 0xdeadbeef) { - fprintf(stderr, "mismatch in buffer %i: 0x%08x instead of 0xdeadbeef\n", i, test); - igt_fail(1); - } + igt_assert_f(test == 0xdeadbeef, + "mismatch in buffer %i: 0x%08x instead of 0xdeadbeef\n", i, test); drm_intel_bo_unreference(pc_target_bo[i]); } |