<div dir="ltr">On 2 January 2013 10:55, Ian Romanick <span dir="ltr"><<a href="mailto:idr@freedesktop.org" target="_blank">idr@freedesktop.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class="im">On 12/29/2012 11:50 AM, Paul Berry wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
This patch fixes intel_miptree_unmap_etc() (which decompresses ETC<br>
textures to linear) to pay attention to map->x and map->y when writing<br>
to the destination image.  Previously these values were ignored,<br>
causing the xoffset and yoffset parameters passed to<br>
glCompressedTexSubImage2D() to be ignored.<br>
</blockquote>
<br></div>
I recall that ETC has additional restrictions on subimage updates.  Do we correctly respect those?</blockquote><div><br></div><div>Yes, I believe what Mesa implements is correct:<br></div><div><br>For ETC1, subimage updates are prohibited, and we respect that by returning true from compressedteximage_only_format().<br>
<br></div><div>For ETC2, subimage updates are allowed provided that xoffset, yoffset, height, and width are aligned to 4x4 block boundaries, and we check that properly in error_check_subtexture_dimensions().<br></div></div>
</div></div>