diff options
author | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-10-24 14:27:32 -0700 |
---|---|---|
committer | Jason Ekstrand <jason.ekstrand@intel.com> | 2016-10-27 14:43:24 -0700 |
commit | 174f4900b294f939c85cfa94f5d8401ce73e5522 (patch) | |
tree | 75a2c55dacaa065069583879ca5b02f55073474b /src | |
parent | 6da814960138c6ccc829c4652d1a781e15465523 (diff) |
i965/blit: Remove a bogus assertion
This assertion, while valid for linear buffers, doesn't work properly for
tiled memory. It used to work most of the time because the offset provided
was always to the left-hand edge of the image. However, if you use a byte
offset to get to the inside of the image, the height * stride calculation
may actually end up being too large.
Signed-off-by: Jason Ekstrand <jason@jlekstrand.net>
Reviewed-by: Topi Pohjolainen <topi.pohjolainen@intel.com>
Reviewed-by: Anuj Phogat <anuj.phogat@gmail.com>
Diffstat (limited to 'src')
-rw-r--r-- | src/mesa/drivers/dri/i965/intel_blit.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/src/mesa/drivers/dri/i965/intel_blit.c b/src/mesa/drivers/dri/i965/intel_blit.c index b7a9cc951cb..bc97e665c9f 100644 --- a/src/mesa/drivers/dri/i965/intel_blit.c +++ b/src/mesa/drivers/dri/i965/intel_blit.c @@ -585,10 +585,6 @@ intelEmitCopyBlit(struct brw_context *brw, assert(dst_x < dst_x2); assert(dst_y < dst_y2); - assert(src_offset + (src_y + h - 1) * abs(src_pitch) + - (w * cpp) <= src_buffer->size); - assert(dst_offset + (dst_y + h - 1) * abs(dst_pitch) + - (w * cpp) <= dst_buffer->size); BEGIN_BATCH_BLT_TILED(length, dst_y_tiled, src_y_tiled); OUT_BATCH(CMD | (length - 2)); |