tests/gem_pipe_control_store_loop: Delete bogus assertion
authorChris Wilson <chris@chris-wilson.co.uk>
Fri, 10 Jan 2014 21:13:56 +0000 (21:13 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Fri, 10 Jan 2014 21:17:43 +0000 (21:17 +0000)
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 <chris@chris-wilson.co.uk>
tests/gem_pipe_control_store_loop.c

index cef7160..f4c03ba 100644 (file)
@@ -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