<div dir="ltr"><div>Also, you can find most of the gen4-5 XML here:<br><br><a href="https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/blorp-gen4">https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=wip/blorp-gen4</a><br><br></div>That should save you some typing. :-)<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Mar 3, 2017 at 8:57 AM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Lois-Francis,<br>
<br>
Thanks for taking a crack at this!  There are a couple other people (I put them in the Cc) who have talked about doing this so thank you for sending the announcement!<br>
<br>
My primary request as you dive into this would be that you do things in such a way that we can have a fairly tight feedback loop.  I think both Ken and myself have a pretty clear idea of what things should look like when you're done and I'd like to avoid any chance that you send a 50 patch series and then get told to go back through the whole thing and make some mechanical change.<br>
<br>
One other comment: look at blorp.  It's already using genxml with the i965 driver although there's a but of an abstraction layer there.  It should give you a decent idea about how to make a brw_emit() macro by combining things and removing the abstraction.<br>
<br>
--Jason<br>
<br>
Sent with AquaMail for Android<br>
<a href="http://www.aqua-mail.com" rel="noreferrer" target="_blank">http://www.aqua-mail.com</a><div class="HOEnZb"><div class="h5"><br>
<br>
<br>
On March 3, 2017 6:41:59 AM Louis-Francis Ratté-Boulianne <<a href="mailto:lfrb@collabora.com" target="_blank">lfrb@collabora.com</a>> wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
As to avoid any duplicate work, I want to inform everyone that I'm in<br>
the process of modifying the i965 driver so that it uses the same<br>
genxml infrastructure that the one used by the Intel Vulkan driver. The<br>
task has been proposed by Jason Ekstrand. <br>
<br>
I will probably post my first patchset today that will contain the<br>
basic call to replace the BEGIN_BATCH/ADVANCE_BATCH macros and the XML<br>
specification files for generations 4, 4.5 and 5.<br>
<br>
Let me know if you have any question or suggestion related to this<br>
work.<br>
<br>
--<br>
Louis-Francis Ratté-Boulianne<br>
</blockquote>
<br>
<br>
</div></div></blockquote></div><br></div>