Re: [Mesa-dev] [PATCH] intel/genxml: Fix gen10 BLEND_STATE variable length packing

2017-08-07 Thread Anuj Phogat
On Fri, Aug 4, 2017 at 3:21 PM, 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 >

Re: [Mesa-dev] [PATCH] intel/genxml: Fix gen10 BLEND_STATE variable length packing

2017-08-04 Thread Rafael Antognolli
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 On Fri, Aug 04, 2017 at 10:21:43PM +, Scott D Phillips wrote: > BLEND_STATE packing was modified to be variable-length in: > >

[Mesa-dev] [PATCH] intel/genxml: Fix gen10 BLEND_STATE variable length packing

2017-08-04 Thread Scott D Phillips
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