<div dir="ltr">On 17 May 2013 10:17, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello!<br>
<br>
This patch series bumps the kernel requirement to 3.6 for Gen6+,<br>
meaning that we actually get to rely on hardware context support.<br>
That's a little painful, but even Debian ships 3.8 now, and this<br>
isn't going to make it into an actual release for several more<br>
months.<br>
<br>
It then splits our query code into Gen4-5 and Gen6+ versions.  The new<br>
Gen6+ version is a lot simpler since hardware contexts guarantee that<br>
our statistics registers don't get polluted with data from other<br>
programs running on the system.  It should be more efficient, which<br>
may help games like Minecraft (though I haven't measured).<br>
<br>
Finally, it implements the GL_PRIMITIVES_GENERATED and<br>
GL_TRANSFORM_FEEDBACK_PRIMITIVES_WRITTEN queries via hardware counters<br>
rather than manually counting on the CPU.  This paves the way for<br>
geometry shader support (which can output multiple primitives, breaking<br>
our CPU-side tracking), and should allow us to enable hardware primitive<br>
restart in a few more cases once a few more things are tidied.<br>
<br>
The next step is to eliminate the use of SOL reset and save/restore the<br>
transform feedback offsets directly.  Then we can turn on hardware<br>
primitive restart more aggressively and implement a few more transform<br>
feedback extensions.<br>
<br>
Thanks in advance for the review.<br>
<br>
--Ken<br></blockquote><div><br></div><div>I made comments on patches 1, 2, 5, 9, and 10.  I agree with Eric's comments on patch 7.  The remaining patches are:<br><br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br>
</div><div><br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>