Re: [Mesa-dev] [PATCH] intel: decoder: handle 0 sized structs

2018-08-28 Thread Lionel Landwerlin
Thanks Andres, Please see my response to Re: [Mesa-dev] [PATCH] intel: decoder: unify MI_BB_START field naming On 27/08/2018 22:20, Andres Gomez wrote: Lionel, should we also include this in the stable queues ? On Sat, 2018-08-25 at 18:23 +0100, Lionel Landwerlin wrote: Gen7.5 has a

Re: [Mesa-dev] [PATCH] intel: decoder: handle 0 sized structs

2018-08-27 Thread Andres Gomez
Lionel, should we also include this in the stable queues ? On Sat, 2018-08-25 at 18:23 +0100, Lionel Landwerlin wrote: > Gen7.5 has a BLEND_STATE of size 0 which includes a variable length > group. We did not deal with that very well, leading to an endless > loop. > > Signed-off-by: Lionel

Re: [Mesa-dev] [PATCH] intel: decoder: handle 0 sized structs

2018-08-25 Thread Jason Ekstrand
Reviewed-by: Jason Ekstrand On Sat, Aug 25, 2018 at 12:23 PM Lionel Landwerlin < lionel.g.landwer...@intel.com> wrote: > Gen7.5 has a BLEND_STATE of size 0 which includes a variable length > group. We did not deal with that very well, leading to an endless > loop. > > Signed-off-by: Lionel

[Mesa-dev] [PATCH] intel: decoder: handle 0 sized structs

2018-08-25 Thread Lionel Landwerlin
Gen7.5 has a BLEND_STATE of size 0 which includes a variable length group. We did not deal with that very well, leading to an endless loop. Signed-off-by: Lionel Landwerlin Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107544 --- src/intel/common/gen_decoder.c | 12 1 file