<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">On Oct 10, 2017 4:46 AM, "Timothy Arceri" <<a href="mailto:tarceri@itsqueeze.com">tarceri@itsqueeze.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">After a recent discussion about this code from 2015 I was curious<br>
to give it a try. The outstanding review item was that we shouldn't<br>
be replacing the C11 mtx type/functions with our own, so I've renamed<br>
the fast path to simple_mtx* and added a couple of patches to make use<br>
of it.<br>
<br>
The idea is this fast mtx can be used in place of the full mtx<br>
implementation when its of type mtx_plain.<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Thanks for picking this up!</div><div dir="auto"><br></div><div dir="auto">I think all our mutexes are mtx_plain, but the simple mutexes can't be used with cond vars.</div><div dir="auto"><br></div><div dir="auto">Marek</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I though if anywhere we might see a change in the drawoverhead piglit<br>
test but I didn't see any real change.<br>
<br>
Anyway since I've made the updates I thought I'd send it out. Maybe<br>
someone else might find some better results. Kristian reported a 10%<br>
increase in some internal Intel benchmarks, I wonder if thats still the<br>
case.<br>
<br>
______________________________<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></div>