V2 is Acked-by: Edward O'Callaghan <funfunc...@folklore1984.net>
On 11/14/2016 09:32 PM, Eduardo Lima Mitev wrote:
> This option is currently ignored when packing compressed textures.
>
> Fixes CTS test (on gen8+):
> * GL45-CTS.gtf32.GL3Tests.packed_pixels.packed_pixels_pixelstore
> ---
> src/mesa/drivers/common/meta.c | 9 ++++++---
> 1 file changed, 6 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/drivers/common/meta.c b/src/mesa/drivers/common/meta.c
> index 5ab1e6c..9437357 100644
> --- a/src/mesa/drivers/common/meta.c
> +++ b/src/mesa/drivers/common/meta.c
> @@ -3255,9 +3255,12 @@ _mesa_meta_GetTexSubImage(struct gl_context *ctx,
> packing.SkipRows = 0;
> dst = _mesa_image_address3d(&packing, pixels, width, height,
> format, type, slice, 0, 0);
> - }
> - else {
> - dst = pixels;
> + } else {
> + /* Account for PACK_SKIP_IMAGES option, if supported */
> + GLint imageStride =
> + _mesa_image_image_stride(&ctx->Pack,width,height,
> + format, type);
> + dst = (GLubyte *) pixels + imageStride * ctx->Pack.SkipImages;
> }
> result = decompress_texture_image(ctx, texImage, slice,
> xoffset, yoffset, width, height,
> 
signature.asc
Description: OpenPGP digital signature
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
