[Mesa-dev] [PATCH] intel/aubinator: Use the correct length for MEDIA commands

Lionel Landwerlin lionel.g.landwerlin at intel.com
Tue Nov 22 09:53:15 UTC 2016


On 22/11/16 02:58, Jason Ekstrand wrote:
> ---
>   src/intel/tools/decoder.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/intel/tools/decoder.c b/src/intel/tools/decoder.c
> index 6bd02bf..55488eb 100644
> --- a/src/intel/tools/decoder.c
> +++ b/src/intel/tools/decoder.c
> @@ -612,8 +612,8 @@ gen_group_get_length(struct gen_group *group, const uint32_t *p)
>            return field(h, 0, 7) + 2;
>         case 1:
>            return 1;
> -      case 2:
> -         return 2;
> +      case 2: /* MEDIA */
> +         return field(h, 0, 7) + 2;

The documentation seems to indicate 0..15 as bits for DWord Length.
Am I missing something?

>         case 3:
>            return field(h, 0, 7) + 2;
>         }




More information about the mesa-dev mailing list