<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 26, 2015 at 11:15 AM, Neil Roberts <span dir="ltr"><<a href="mailto:neil@linux.intel.com" target="_blank">neil@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>Nanley Chery <<a href="mailto:nanleychery@gmail.com" target="_blank">nanleychery@gmail.com</a>> writes:<br>
<br>
> +      /* Throw an INVALID_OPERATION error if the target is<br>
> +       * TEXTURE_CUBE_MAP_ARRAY and the format is not ASTC.<br>
> +       */<br>
> +      if (target_can_be_compresed &&<br>
> +          ctx->Extensions.KHR_texture_compression_astc_ldr &&<br>
> +          layout != MESA_FORMAT_LAYOUT_ASTC)<br>
>           return write_error(error, GL_INVALID_OPERATION);<br>
<br>
</span>This seems to cause regressions in the following Piglit tests for Gen9:<br>
<br>
piglit.spec.ext_texture_compression_s3tc.getteximage-targets cube_array s3tc<br>
piglit.spec.arb_texture_cube_map_array.fbo-generatemipmap-cubemap array s3tc_dxt1<br>
<br>
I think the spec for the extension is based on the GLES spec. Perhaps<br>
the intention was to add ASTC support for cube-map array textures<br>
whereas previously in GLES no compressed formats were supported for this<br>
target. However in regular GL presumably S3TC is supposed to be<br>
supported. As it stands this patch disables cube-map array textures for<br>
any formats other than ASTC whenever the ASTC extension is available, so<br>
it disables S3TC on Gen9.<br>
<br>
Maybe this section should be limited to GLES?<br>
<br></blockquote><div><br></div><div>Limiting this to GLES sounds like a good solution. Note that there is similar logic for the<br></div><div>TEXTURE_3D target further down.<br><br></div><div>Nanley<br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Regards,<br>
- Neil<br>
</blockquote></div><br></div></div>