<div dir="ltr">On 6 December 2013 02:54, 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">Paul Berry <<a href="mailto:stereotype441@gmail.com" target="_blank">stereotype441@gmail.com</a>> writes:<br>
<br>
>[...]<br>
<div>> This order of patches introduces temporary build breakages, which I'd<br>
> prefer to avoid if possible.  Here's one order which I've verified works<br>
> (there may be other possibilities):<br>
><br>
> [PATCH 03/11] mesa: Add state data structures requried for<br>
> ARB_shader_image_load_store.<br>
> [PATCH 04/11] mesa: Add driver interface for ARB_shader_image_load_store.<br>
> [PATCH 06/11] mesa: Add MESA_FORMAT_ABGR2101010.<br>
> [PATCH 08/11] mesa: Implement the GL entry points defined by<br>
> ARB_shader_image_load_store.<br>
> [PATCH 01/11] glapi: Update dispatch XML files for<br>
> ARB_shader_image_load_store.<br>
> [PATCH 02/11] mesa: Add ARB_shader_image_load_store to the extension table.<br>
> [PATCH 05/11] mesa: Add image parameter queries for<br>
> ARB_shader_image_load_store.<br>
> [PATCH 07/11] mesa: Add MESA_FORMAT_SIGNED_RG88 and _RG1616.<br>
> [PATCH 09/11] mesa: Unbind deleted textures from the shader image units.<br>
> [PATCH 10/11] mesa: Validate image units when the texture state changes.<br>
> [PATCH 11/11] docs: Mark ARB_shader_image_load_store as work in progress.<br>
<br>
</div>Thanks Paul, I've adopted your patch ordering in my branch, it should be<br>
fixed now.<br>
</blockquote></div><br></div><div class="gmail_extra">Cool, thanks.<br><br></div><div class="gmail_extra">I sent comments on patches 1, 3, 6, and 8 (relative to the original ordering).  The remainder of this series is:<br>
<br>Reviewed-by: Paul Berry <<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>><br><br>I'll try to get to the other ARB_image_shader_load_store series next week.<br></div></div>