<div id="geary-body" dir="auto"></div><div id="geary-quote" dir="auto"><br>On ma., aug. 27, 2018 at 9:49 AM, Gert Wollny <gert.wollny@collabora.com> wrote:<br><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Am Montag, den 20.08.2018, 14:10 +0200 schrieb Erik Faye-Lund:
<blockquote> This is quite useful for debugging shader-transpiling issues in
 virglrenderer.
</blockquote>Isn't this coverted by ST_DEBUG=tgsi? 
<br></div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">Ah, I wasn't even aware of this. </span></div><div><span style="white-space: pre-wrap;"><br></span></div>There's one minor difference here, though: virgl does some transformations on the TGSI before passing it to the host. It can sometimes be useful to see the difference.</span><div><span style="white-space: pre-wrap;"><br></span><blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">Also, virglrenderer has a variable vrend_dump_shaders in
vrend_renderer.c that enables dumping all the TGSI + the created GLSL
shaders. </div></blockquote><span style="white-space: pre-wrap;"><div><span style="white-space: pre-wrap;"><br></span></div><div><span style="white-space: pre-wrap;">Yeah, and that's also useful. But there's two differences here:</span></div><div><span style="white-space: pre-wrap;">- </span>vrend_dump_shaders dumps the TGSI *after* parsing, comparing the over-the-wire TGSI and the parsed TGSI has helped me in the past to find flags being culled during parsing etc in the past.</div><div>- vrend_dump_shaders needs a recompile of virglrenderer, which makes it a lot more inconvenient from a turn-around point of view if running on qemu for instance.</div><div><br></div><div>I don't know, perhaps this is a bit too many similar features? I can certainly drop this patch for now.</div><div><br></div><div>Another (maybe more useful) option could be to allow a per-client override of the vrend_dump_shaders-functionality that could be enabled from the host?</div></span></div></div>