<p dir="ltr"><br>
On Mar 11, 2016 12:33 PM, "Alejandro Piñeiro" <<a href="mailto:apinheiro@igalia.com">apinheiro@igalia.com</a>> wrote:<br>
><br>
> On 11/03/16 20:15, Anuj Phogat wrote:<br>
> > yoffset is also applicable to 1d array textures.<br>
> ><br>
> > Signed-off-by: Anuj Phogat <<a href="mailto:anuj.phogat@gmail.com">anuj.phogat@gmail.com</a>><br>
> > ---<br>
> > I don't know if it fixes any test, but it looked incorrect to me.<br>
><br>
> No one fixed doing a piglit all.py run (also no regression). Didn't test<br>
> with a deqp run.</p>
<p dir="ltr">There are very few tests for glGetTexImage.  Not hitting one doesn't mean much.</p>
<p dir="ltr">> In any case, I also agree that the change seems to make sense.<br>
><br>
> ><br>
> >  src/mesa/main/texgetimage.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/src/mesa/main/texgetimage.c b/src/mesa/main/texgetimage.c<br>
> > index 06bc8f1..dc21551 100644<br>
> > --- a/src/mesa/main/texgetimage.c<br>
> > +++ b/src/mesa/main/texgetimage.c<br>
> > @@ -1046,7 +1046,7 @@ dimensions_error_check(struct gl_context *ctx,<br>
> >                          "%s(xoffset = %d)", caller, xoffset);<br>
> >              return true;<br>
> >           }<br>
> > -         if (target != GL_TEXTURE_1D && target != GL_TEXTURE_1D_ARRAY) {<br>
> > +         if (target != GL_TEXTURE_1D) {<br>
> >              if (yoffset % bh != 0) {<br>
> >                 _mesa_error(ctx, GL_INVALID_VALUE,<br>
> >                             "%s(yoffset = %d)", caller, yoffset);</p>
<p dir="ltr">I don't think this is correct.  The check is for compressed textures to ensure that the texture coordinates are a multiple of the block size of the texture.  I'm not sure what the rules are for 1-D array compressed textures (if they even exist) bit I'm pretty sure the compression doesn't cross slices.  If anything, we probably want to take the check below that looks at height and pull it into the if too.<br>
</p>