<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 17, 2015 at 11:54 AM, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@intel.com" target="_blank">chad.versace@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>On Tue 28 Jul 2015, Nanley Chery wrote:<br>
> From: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>><br>
><br>
> According to the spec, Tex*Storage* commands are supported if the GL<br>
> implementation has support for immutable textures.<br>
><br>
> Signed-off-by: Nanley Chery <<a href="mailto:nanley.g.chery@intel.com" target="_blank">nanley.g.chery@intel.com</a>><br>
> ---<br>
>  src/mesa/main/texformat.c | 56 +++++++++++++++++++++++++++++++++++++++++++++++<br>
>  1 file changed, 56 insertions(+)<br>
<br>
</span>This patch's code looks correct to me, but I'm confused by the commit<br>
message. The commit message specifically calls out the glTexStorage*<br>
functions, but _mesa_choose_tex_format() is also used by many other<br>
non-storage glTex* functions.<br>
<br>
Why specifically mention the glTexStorage* function? Is there some<br>
larger context I'm missing?<br>
</blockquote></div><br></div><div class="gmail_extra">I called out the glTex*Storage* functions because they were the only commands that wouldn't work correctly during testing. Perhaps I missed some cases (e.g., TextureViews). This patch is superseded by the following patch (which needs review): "mesa/texformat: Use format conversion function in _mesa_choose_tex_format" <a href="http://lists.freedesktop.org/archives/mesa-dev/2015-August/091564.html" target="_blank">http://lists.freedesktop.org/archives/mesa-dev/2015-August/091564.html</a></div><div class="gmail_extra"><br></div><div class="gmail_extra">Regards,</div><div class="gmail_extra">Nanley</div></div>