From d47221b094ef1344ddd349fffd5fc92be12a9b6c Mon Sep 17 00:00:00 2001 From: Chris Wilson Date: Sun, 17 Apr 2011 10:36:26 +0100 Subject: intel: Beware the unsigned promotion when checking for batch overflows Reported-by: Modestas Vainius References: https://bugs.freedesktop.org/show_bug.cgi?id=36319 Signed-off-by: Chris Wilson (cherry picked from commit a51cd83d25f2f9f2107219d5671194f931601244) Signed-off-by: Owain G. Ainsworth --- src/intel_batchbuffer.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/intel_batchbuffer.h b/src/intel_batchbuffer.h index 86e65444..3022f254 100644 --- a/src/intel_batchbuffer.h +++ b/src/intel_batchbuffer.h @@ -50,14 +50,14 @@ static inline int intel_vertex_space(intel_screen_private *intel) } static inline void -intel_batch_require_space(ScrnInfoPtr scrn, intel_screen_private *intel, unsigned int sz) +intel_batch_require_space(ScrnInfoPtr scrn, intel_screen_private *intel, int sz) { assert(sz < intel->batch_bo->size - 8); if (intel_batch_space(intel) < sz) intel_batch_submit(scrn); } -static inline void intel_batch_start_atomic(ScrnInfoPtr scrn, unsigned int sz) +static inline void intel_batch_start_atomic(ScrnInfoPtr scrn, int sz) { intel_screen_private *intel = intel_get_screen_private(scrn); -- cgit v1.2.3