From d1245211417c092bd1c12339559caced98d7560f Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Wed, 30 Aug 2017 00:54:40 -0700 Subject: [PATCH] i965: Delete a batch size assertion that isn't very useful. This assertion prevents you from doing intel_batchbuffer_require_space with a size so huge it won't fit in the batchbuffer. This doesn't seem like a common mistake, and I've never seen the assert to be useful. Soon, I hope to have batches grow, at which point this won't make sense. Reviewed-by: Matt Turner Reviewed-by: Chris Wilson --- src/mesa/drivers/dri/i965/intel_batchbuffer.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/mesa/drivers/dri/i965/intel_batchbuffer.c b/src/mesa/drivers/dri/i965/intel_batchbuffer.c index 515b595bc92..cd118f6c6fc 100644 --- a/src/mesa/drivers/dri/i965/intel_batchbuffer.c +++ b/src/mesa/drivers/dri/i965/intel_batchbuffer.c @@ -238,9 +238,6 @@ intel_batchbuffer_require_space(struct brw_context *brw, GLuint sz, intel_batchbuffer_flush(brw); } -#ifdef DEBUG - assert(sz < BATCH_SZ - BATCH_RESERVED); -#endif if (intel_batchbuffer_space(&brw->batch) < sz) intel_batchbuffer_flush(brw); -- 2.30.2