<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Oct 12, 2017 at 5:52 PM, Jordan Justen <span dir="ltr"><<a href="mailto:jordan.l.justen@intel.com" target="_blank">jordan.l.justen@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I think my requests to split 2 patches should be pretty easy, so even<br>
with those changes:<br>
<br>
Series Reviewed-by: Jordan Justen <<a href="mailto:jordan.l.justen@intel.com">jordan.l.justen@intel.com</a>><br><div><div class="h5"></div></div></blockquote><div><br></div><div>Thanks!<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
On 2017-10-11 13:38:40, Jason Ekstrand wrote:<br>
> I started trying to debug a random segfault in the pipeline cache that I<br>
> was seeing in some of the synchronization tests.  After taking a longer<br>
> look, It became obvious that doing so with the current implementation is<br>
> nuts.  Instead, we're much better off if we take advantage of the blob<br>
> structure and all of it's nice safety guarantees.<br>
><br>
> Connor Abbott (1):<br>
>   compiler/blob: make blob_reserve_bytes() more useful<br>
><br>
> Jason Ekstrand (10):<br>
>   glsl/blob: Return false from ensure_can_read on overrun<br>
>   glsl/blob: Return false from grow_to_fit if we've ever failed<br>
>   compiler: Move blob up a level<br>
>   compiler/blob: Switch to init/finsih instead of create/destroy<br>
>   compiler/blob: Add a concept of a fixed-allocation blob<br>
>   compiler/blob: Constify the reader<br>
>   compiler/blob: Make some parameters void instead of uint8_t<br>
>   anv/multialloc: Add new add_size helper<br>
>   anv/pipeline: Declare bind maps closer to their use<br>
>   anv/pipeline_cache: Rework to use multialloc and blob<br>
><br>
>  src/compiler/Makefile.sources            |   4 +-<br>
>  src/compiler/{glsl => }/blob.c           |  91 +++++++---<br>
>  src/compiler/{glsl => }/blob.h           | 104 ++++++++---<br>
>  src/compiler/glsl/shader_<wbr>cache.cpp       |  39 ++--<br>
>  src/compiler/glsl/tests/blob_<wbr>test.c      |   4 +-<br>
>  src/intel/vulkan/anv_pipeline.<wbr>c          |  18 +-<br>
>  src/intel/vulkan/anv_pipeline_<wbr>cache.c    | 298 +++++++++++++++---------------<wbr>-<br>
>  src/intel/vulkan/anv_private.h           |   8 +-<br>
>  src/mesa/state_tracker/st_<wbr>shader_cache.c |  23 +--<br>
>  src/mesa/state_tracker/st_<wbr>shader_cache.h |   2 +-<br>
>  10 files changed, 333 insertions(+), 258 deletions(-)<br>
>  rename src/compiler/{glsl => }/blob.c (79%)<br>
>  rename src/compiler/{glsl => }/blob.h (76%)<br>
><br>
> --<br>
> 2.5.0.400.gff86faf<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> mesa-dev mailing list<br>
> <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> <a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
</blockquote></div><br></div></div>