From: Kenneth Graunke Date: Wed, 30 Aug 2017 07:54:40 +0000 (-0700) Subject: i965: Delete a batch size assertion that isn't very useful. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d1245211417c092bd1c12339559caced98d7560f;p=mesa.git 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 --- 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);