[Mesa-dev] [PATCH v2 1/4] intel/aubinator_error_decode: Fix structure decode data

[Lionel Landwerlin]

Thanks!

Reviewed-by: Lionel Landwerlin <lionel.g.landwerlin at intel.com>

On 06/04/17 06:51, Jordan Justen wrote:
> The call to gen_print_group should provide a pointer to the beginning
> of the the structure data, not the start of the batch data.
>
> Cc: Lionel Landwerlin <lionel.g.landwerlin at intel.com>
> Signed-off-by: Jordan Justen <jordan.l.justen at intel.com>
> ---
>   src/intel/tools/aubinator_error_decode.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/intel/tools/aubinator_error_decode.c b/src/intel/tools/aubinator_error_decode.c
> index 8a67f4f1698..1bdab00a663 100644
> --- a/src/intel/tools/aubinator_error_decode.c
> +++ b/src/intel/tools/aubinator_error_decode.c
> @@ -239,7 +239,7 @@ static void decode(struct gen_spec *spec,
>         printf("%s0x%08"PRIx64":  0x%08x:  %-80s%s\n",
>                color, offset, p[0], gen_group_get_name(inst), reset_color);
>   
> -      gen_print_group(stdout, inst, offset, data,
> +      gen_print_group(stdout, inst, offset, p,
>                         option_color == COLOR_ALWAYS);
>         length = gen_group_get_length(inst, p);
>      }