Oh, I saw it had the old xml and was assuming it didn't cause any errors,
but clearly I was wrong.
Reviewed-by: Rafael Antognolli <rafael.antogno...@intel.com>
On Fri, Aug 04, 2017 at 10:21:43PM +0000, Scott D Phillips wrote:
> BLEND_STATE packing was modified to be variable-length in:
>
> 9670124e31 genxml: Make BLEND_STATE command support variable length array.
>
> The initial gen10.xml still had the old, fixed-length style
> definition for BLEND_STATE. So gen10_upload_blend_state would
> overwrite the packed BLEND_STATE_ENTRYs with its own fixed array
> of all-zero entries when packing BLEND_STATE. This caused
> BLEND_STATE upload to not work at all.
>
> Fixes: aa416f515a ("i965/genxml: Add gen10.xml")
> ---
> src/intel/genxml/gen10.xml | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/genxml/gen10.xml b/src/intel/genxml/gen10.xml
> index 23c2adb995..a7ae49ae65 100644
> --- a/src/intel/genxml/gen10.xml
> +++ b/src/intel/genxml/gen10.xml
> @@ -554,7 +554,7 @@
> <field name="Write Disable Blue" start="0" end="0" type="bool"/>
> </struct>
>
> - <struct name="BLEND_STATE" length="17">
> + <struct name="BLEND_STATE" length="1">
> <field name="Alpha To Coverage Enable" start="31" end="31" type="bool"/>
> <field name="Independent Alpha Blend Enable" start="30" end="30"
> type="bool"/>
> <field name="Alpha To One Enable" start="29" end="29" type="bool"/>
> @@ -564,7 +564,7 @@
> <field name="Color Dither Enable" start="23" end="23" type="bool"/>
> <field name="X Dither Offset" start="21" end="22" type="uint"/>
> <field name="Y Dither Offset" start="19" end="20" type="uint"/>
> - <group count="8" start="32" size="64">
> + <group count="0" start="32" size="64">
> <field name="Entry" start="0" end="63" type="BLEND_STATE_ENTRY"/>
> </group>
> </struct>
> --
> 2.11.0
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
