diff options
| author | Chris Wilson <chris@chris-wilson.co.uk> | 2011-04-17 10:36:26 +0100 |
|---|---|---|
| committer | Chris Wilson <chris@chris-wilson.co.uk> | 2011-04-17 10:43:35 +0100 |
| commit | a51cd83d25f2f9f2107219d5671194f931601244 (patch) | |
| tree | 70d4d8786c311ed43889d91b63d8cef98262b228 /src | |
| parent | 030aa3d13643a0acb7126fa1fa6c5a8eac134639 (diff) | |
intel: Beware the unsigned promotion when checking for batch overflows
Reported-by: Modestas Vainius <geromanas@mailas.com>
References: https://bugs.freedesktop.org/show_bug.cgi?id=36319
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Diffstat (limited to 'src')
| -rw-r--r-- | src/intel_batchbuffer.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/intel_batchbuffer.h b/src/intel_batchbuffer.h index 605932a4..2403a387 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); |