Thanks! Reviewed-by: Lionel Landwerlin <lionel.g.landwer...@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.landwer...@intel.com> Signed-off-by: Jordan Justen <jordan.l.jus...@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); }
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
