<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Aug 14, 2016 at 11:53 PM, Kenneth Graunke <span dir="ltr"><<a href="mailto:kenneth@whitecape.org" target="_blank">kenneth@whitecape.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thursday, August 11, 2016 2:14:57 PM PDT Jason Ekstrand wrote:<br>
> This patch series is the next on the way towards generalizing blorp for<br>
> usin in Vulkan.  At this point, I'd say the project is about 80% complete.<br>
> This series only applies on top of the last two blorp isl series which are<br>
> still undergoing review.<br>
><br>
> Patches 1-11 make minor genxml improvements.  Some of this is to make gen6<br>
> better and more like gen7-9.  The others are just general improvements that<br>
> made converting blorp easier.<br>
><br>
> Patches 12-14 make a few minor changes to i965 so that we can include<br>
> brw_state.h and brw_context.h without brw_defines.h.<br>
><br>
> Patches 15-23 re-arrange the state setup code in blorp to make the<br>
> different gens emit the packets more-or-less in the same order.  This was<br>
> very helpful when trying to debug problems when switching to genxml because<br>
> there should not be substantial changes in the batch between patch 23 and<br>
> the end of the series.<br>
><br>
> Patches 24-32 convert all of the blorp state setup to using genxml in a<br>
> single file that gets recompiled 5 times, one for each gen.  I'm extremely<br>
> happy with the end result where almost all of the code is shared across<br>
> gens.  The one exception is WM and SF setup which is so different that we<br>
> really need different code for gen6, gen7, and gen8.<br>
><br>
> This series can be found here:<br>
><br>
> <a href="https://cgit.freedesktop.org/~jekstrand/mesa/log/?h=review/blorp-genxml" rel="noreferrer" target="_blank">https://cgit.freedesktop.org/~<wbr>jekstrand/mesa/log/?h=review/<wbr>blorp-genxml</a><br>
><br>
> Cc: Topi Pohjolainen <<a href="mailto:topi.pohjolainen@intel.com">topi.pohjolainen@intel.com</a>><br>
><br>
> Jason Ekstrand (32):<br>
>   genxml/gen6: Add uint MOCS fields for most things<br>
>   genxml/gen6: Add a Surface Base Address field to HIER_DEPTH_BUFFER<br>
>   genxml/gen6: Fix the length of 3DSTATE_WM<br>
>   genxml/gen6: Add the 3D_Prim_Topo_Type enum<br>
>   genxml/gen6: Make "Depth Clear Value" a uint<br>
>   genxml: Add a uint MOCS field to DEPTH_BUFFER packets<br>
>   genxml/gen6: Make SAMPLER_STATE look a bit more like gen7<br>
>   genxml: Make VERTEX_ELEMENT_STATE::Valid a bool<br>
>   genxml: Make a couple of VERTEX_BUFFER_STATE fields boolean<br>
>   genxml: Add a uint MOCS field to VERTEX_BUFFER_STATE<br>
>   genxml/gen9: Make 3DSTATE_SBE::<wbr>AttributeActiveComponentFormat an array<br>
>   i965/state: Move is_drawing_lines/points to gen6_clip_state.c<br>
>   i965: Stop including brw_defines.h in brw_state.h<br>
>   i965: Roll intel_reg.h into brw_defines.h<br>
>   i965/blorp: Make gen6 VS and GS disable helpers static<br>
<br>
</div></div>Patches 1-15 (the above) are:<br>
Reviewed-by: Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>><br>
</blockquote></div><br></div><div class="gmail_extra">Thanks!<br></div><div class="gmail_extra"><br></div></div>