From 45d63644d28c34202a97bd812f6c46c853e97d28 Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Fri, 10 Jan 2014 21:13:56 +0000 Subject: [PATCH] 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 --- tests/gem_pipe_control_store_loop.c | 4 +--- 1 file changed, 1 insertion(+), 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