diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2014-01-10 21:13:56 +0000 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2014-01-10 21:17:43 +0000 |
| commit | 45d63644d28c34202a97bd812f6c46c853e97d28 (patch) | |
| tree | b14df0124f4528fcb634497b79578fbbe64e3b16 | |
| parent | 84af2b9be8792fb288d4ce75821a27706e5351c3 (diff) | |
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 <chris@chris-wilson.co.uk>
| -rw-r--r-- | tests/gem_pipe_control_store_loop.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/tests/gem_pipe_control_store_loop.c b/tests/gem_pipe_control_store_loop.c index cef7160f..f4c03bad 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 |