<div dir="auto">We also have asynchronous compilation for optimized shaders (linking separate shaders etc.) using the same threaded queue as the initial compilations, and we can't let apps suppress the optimized ones.<div dir="auto"><br></div><div dir="auto">Marek</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Apr 18, 2017 1:08 PM, "Nicolai Hähnle" <<a href="mailto:nhaehnle@gmail.com">nhaehnle@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 17.04.2017 14:07, Edward O'Callaghan wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi folks,<br>
<br>
This is my early draft on the threaded shader compile "suggestion"<br>
thread. The requirements according the spec are rather weak as<br>
it is up to the implementation, which frankly can even just ignore<br>
the requested thread count. However, afaik in the case of radeonsi<br>
it is already threaded. Anyhoo I just wanted to get some feedback..<br>
</blockquote>
<br>
Thanks for working on this. As others said, 90% of the work is elsewhere. Some thoughts on that:<br>
<br>
Since radeonsi already does parallelization of the backend compile, this extension should control the sum of threads used for frontend *and* backend compiles. In practice, that should mean having a single thread pool which services both types of compiles (and links).<br>
<br>
My first thought would be to have a driver-allocated util_queue which gets passed across the Gallium and Mesa interfaces -- or perhaps have a driver function table entry for kicking off asynchronous compiles and links.<br>
<br>
Cheers,<br>
Nicolai<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Please kindly review,<br>
<br>
Edward O'Callaghan (3):<br>
<br>
 [PATCH 1/3] mesa: Add ARB_parallel_shader_compile boilerplate<br>
 [PATCH 2/3] mesa: implement ARB_parallel_shader_compile<br>
 [PATCH 3/3] st/mesa: enable ARB_parallel_shader_compile<br>
______________________________<wbr>_________________<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="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/<wbr>mailman/listinfo/mesa-dev</a><br>
<br>
</blockquote>
<br>
<br>
-- <br>
Lerne, wie die Welt wirklich ist,<br>
Aber vergiss niemals, wie sie sein sollte.<br>
______________________________<wbr>_________________<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="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></div>