On Tuesday, May 16, 2017 3:45:35 PM PDT Jason Ekstrand wrote: > The width and height of the copy don't have to be aligned to the block > size if they specify the right or bottom edges of the image. (See also > the comment and asserts right above). We need to round them up when we > do the division in order to get it 100% right. > > Cc: "17.0 17.1" <mesa-sta...@lists.freedesktop.org> > --- > src/mesa/drivers/dri/i965/intel_blit.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/src/mesa/drivers/dri/i965/intel_blit.c > b/src/mesa/drivers/dri/i965/intel_blit.c > index 568ed54..2925fc2 100644 > --- a/src/mesa/drivers/dri/i965/intel_blit.c > +++ b/src/mesa/drivers/dri/i965/intel_blit.c > @@ -411,8 +411,8 @@ intel_miptree_copy(struct brw_context *brw, > > src_x /= (int)bw; > src_y /= (int)bh; > - src_width /= (int)bw; > - src_height /= (int)bh; > + src_width = DIV_ROUND_UP(src_width, (int)bw); > + src_height = DIV_ROUND_UP(src_height, (int)bh); > } > src_x += src_image_x; > src_y += src_image_y; >
Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev