<div dir="ltr">On 15 September 2013 00:10, Francisco Jerez <span dir="ltr"><<a href="mailto:currojerez@riseup.net" target="_blank">currojerez@riseup.net</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">This patch series implements support for the<br>
ARB_shader_atomic_counters extension, which is part of GL core since<br>
GL 4.2.  It includes patches adding support for the new APIs and GLSL<br>
language features, and working back-end code for Intel Gen7 hardware<br>
-- Ivy Bridge should work with these patches alone, Haswell is going<br>
to need a small kernel change I'll probably submit for review during<br>
the next week.<br>
<br>
The series depends on Ken's surface state tidying patches [1] and<br>
on patches 1-4, which are seemingly unrelated fixes.<br>
<br>
There's also a series of ~30 unit tests for this extension I will send<br>
to the piglit mailing list soon.<br>
<br>
Thanks.<br>
<br>
[1] <a href="http://lists.freedesktop.org/archives/mesa-dev/2013-September/044691.html" target="_blank">http://lists.freedesktop.org/archives/mesa-dev/2013-September/044691.html</a><br>
<br>
[PATCH 01/24] mesa: Fix misplaced includes of "main/uniforms.h".<br>
[PATCH 02/24] glsl: Initialize all member variables of _mesa_glsl_parse_state on construction.<br>
[PATCH 03/24] i965: Initialize all member variables of vec4_instruction on construction.<br>
[PATCH 04/24] ralloc: Unify overloads of the new operator and guarantee object destruction.<br>
[PATCH 05/24] glapi: Add support for ARB_shader_atomic_counters.<br>
[PATCH 06/24] mesa: Add support for ARB_shader_atomic_counters.<br>
[PATCH 07/24] glsl: Add extension enables for ARB_shader_atomic_counters.<br>
[PATCH 08/24] glsl: Add new atomic_uint built-in GLSL type.<br>
[PATCH 09/24] glsl: Add IR node for atomic operations.<br>
[PATCH 10/24] glsl: Implement parser support for atomic counters.<br>
[PATCH 11/24] glsl: Add built-in functions and constants required for ARB_shader_atomic_counters.<br>
[PATCH 12/24] glsl: Add predicate to determine if an IR node has side effects.<br>
[PATCH 13/24] glsl: Linker support for ARB_shader_atomic_counters.<br>
[PATCH 14/24] i965: Define vtbl method that initializes an untyped R/W surface.<br>
[PATCH 15/24] i965: Implement ABO surface state emission.<br>
[PATCH 16/24] i965/gen7: Implement code generation for untyped atomic instructions.<br>
[PATCH 17/24] i965/gen7: Implement code generation for untyped surface read instructions.<br>
[PATCH 18/24] i965: Add a 'has_side_effects' back-end instruction predicate.<br>
[PATCH 19/24] i965: Handle the 'atomic_uint' GLSL type.<br>
[PATCH 20/24] i965: Add brw_reg constructors taking a dynamically determined vector width.<br>
[PATCH 21/24] i965/gen7: Handle atomic instructions from the FS back-end.<br>
[PATCH 22/24] i965/gen7: Handle atomic instructions from the VEC4 back-end.<br>
[PATCH 23/24] i965/gen7: Expose ARB_shader_atomic_counters.<br>
[PATCH 24/24] i965: Simplify the shader time code by using atomic counter helpers.<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">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 class="gmail_extra">I'm halfway through the series and I'm out of time for code review today.  I sent comments on patches 3, 4, 8, 9, 10, and 12.  Patches 1, 2, 5, 7, and 11 are:<br>
<br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br><br></div><div class="gmail_extra">Patch 6 deserves the attention of someone who is more familiar with the Mesa front end than I am, so on that one I'm just going to say:<br>
<br>Acked-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br><br></div><div class="gmail_extra">I'll try to get to patches 13-24 as soon as I can.<br></div></div>