@@ -39,13 +39,12 @@ gen7_render_flush(struct intel_batchbuffer *batch, uint32_t batch_end)
void
gen7_render_context_flush(struct intel_batchbuffer *batch, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, batch->ctx,
- batch_end, 0);
- igt_assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo, batch->ctx,
+ batch_end, 0),
+ 0);
}
uint32_t
@@ -124,13 +124,12 @@ static void
gen4_render_flush(struct intel_batchbuffer *batch,
drm_intel_context *context, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, context,
- batch_end, 0);
- assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo, context,
+ batch_end, 0),
+ 0);
}
static uint32_t
@@ -62,13 +62,12 @@ static void
gen6_render_flush(struct intel_batchbuffer *batch,
drm_intel_context *context, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, context,
- batch_end, 0);
- igt_assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo,
+ context, batch_end, 0),
+ 0);
}
static uint32_t
@@ -36,13 +36,12 @@ static void
gen7_render_flush(struct intel_batchbuffer *batch,
drm_intel_context *context, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, context,
- batch_end, 0);
- igt_assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo, context,
+ batch_end, 0),
+ 0);
}
static uint32_t
@@ -132,13 +132,12 @@ static void
gen6_render_flush(struct intel_batchbuffer *batch,
drm_intel_context *context, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, context,
- batch_end, 0);
- igt_assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo, context,
+ batch_end, 0),
+ 0);
}
/* Mostly copy+paste from gen6, except height, width, pitch moved */
@@ -182,13 +182,12 @@ static void
gen6_render_flush(struct intel_batchbuffer *batch,
drm_intel_context *context, uint32_t batch_end)
{
- int ret;
-
- ret = drm_intel_bo_subdata(batch->bo, 0, 4096, batch->buffer);
- if (ret == 0)
- ret = drm_intel_gem_bo_context_exec(batch->bo, context,
- batch_end, 0);
- assert(ret == 0);
+ igt_assert_eq(drm_intel_bo_subdata(batch->bo,
+ 0, 4096, batch->buffer),
+ 0);
+ igt_assert_eq(drm_intel_gem_bo_context_exec(batch->bo,
+ context, batch_end, 0),
+ 0);
}
/* Mostly copy+paste from gen6, except height, width, pitch moved */
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> --- lib/gpu_cmds.c | 13 ++++++------- lib/rendercopy_gen4.c | 13 ++++++------- lib/rendercopy_gen6.c | 13 ++++++------- lib/rendercopy_gen7.c | 13 ++++++------- lib/rendercopy_gen8.c | 13 ++++++------- lib/rendercopy_gen9.c | 13 ++++++------- 6 files changed, 36 insertions(+), 42 deletions(-)