From: Chris Wilson Date: Fri, 10 Jan 2014 21:13:56 +0000 (+0000) Subject: tests/gem_pipe_control_store_loop: Delete bogus assertion X-Git-Tag: intel-gpu-tools-1.6~193 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=45d63644d28c34202a97bd812f6c46c853e97d28;p=platform%2Fupstream%2Fintel-gpu-tools.git tests/gem_pipe_control_store_loop: Delete bogus assertion target_bo->offset was just being used to verify that the batch submission worked and fortunately was not being relied upon for any subsequent conditions. However, address 0 is valid and so the assertion itself was bogus as it is possible (almost assured with full-ppgtt) for the target_bo to be located at address 0. Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72984 Signed-off-by: Chris Wilson --- diff --git a/tests/gem_pipe_control_store_loop.c b/tests/gem_pipe_control_store_loop.c index cef7160..f4c03ba 100644 --- a/tests/gem_pipe_control_store_loop.c +++ b/tests/gem_pipe_control_store_loop.c @@ -94,9 +94,7 @@ store_pipe_control_loop(bool preuse_buffer) ADVANCE_BATCH(); intel_batchbuffer_flush(batch); - igt_assert(target_bo->offset != 0); - } else - igt_assert(target_bo->offset == 0); + } /* gem_storedw_batches_loop.c is a bit overenthusiastic with * creating new batchbuffers - with buffer reuse disabled, the