<div dir="ltr">Re-posted patches with updated commit messages.<div><br></div><div>Thanks,</div><div>Courtney</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Nov 7, 2013 at 2:34 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Nov 7, 2013 at 1:22 PM, Courtney Goeltzenleuchter<br>
<<a href="mailto:courtney@lunarg.com">courtney@lunarg.com</a>> wrote:<br>
> This series builds on work from Frank Henigman to optimize the<br>
> process of uploading a texture to the GPU. This series adds support for<br>
> MESA_XRGB_8888 and full miptrees where were found to be common activities<br>
> in the Smokin' Guns game. The issue was found while profiling the app<br>
> but that part is not benchmarked. Smokin-Guns uses mipmap textures with<br>
> an internal format of GL_RGB (MESA_XRGB_8888 in the driver).<br>
><br>
> These changes need a performance tool to run against to show how they<br>
> improve execution performance for specific texture formats. Using this<br>
> benchmark I've measured the following improvement on my Ivybridge<br>
> Intel(R) Xeon(R) CPU E3-1225 V2 @ 3.20GHz.<br>
><br>
> Using 1024x1024, RGBA 8888 source, mipmap<br>
> internal-format Before (MB/sec)     After XRGB (MB/sec) After mip (MB/sec)<br>
> GL_RGBA         628.15              627.15              615.90<br>
> GL_RGB          265.95              456.35              611.53<br>
<br>
</div>Put these results into the individual commit messages.<br>
<div class="im"><br>
> Test shows similar pattern for 512x512 and 256x256.<br>
<br>
</div>Wouldn't hurt to include these as well.<br>
<br>
Patch 1 (with benchmarks in the commit message) is:<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
<br>
I don't know anything about the miptree layout, so I don't feel able<br>
to review the second patch, but it seems plausible.<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr">Courtney Goeltzenleuchter<br><div>LunarG</div><div><img src="http://media.lunarg.com/wp-content/themes/LunarG/images/logo.png" width="96" height="65"><br>
</div></div>
</div>