From e92959c4e03cf17b7cd829e2e56162604069abe6 Mon Sep 17 00:00:00 2001 From: Eric Anholt Date: Mon, 25 Jun 2018 11:01:44 -0700 Subject: [PATCH] v3d: Pass the whole clif_dump structure to v3d_print_group(). To generate CLIF files that the v3dv3 simulator can parse, we're going to need to decode addresses, and for that we'll need the vaddr lookup function from the clif structure from within v3d_decoder. --- src/broadcom/cle/v3d_decoder.c | 7 ++++--- src/broadcom/cle/v3d_decoder.h | 3 ++- src/broadcom/clif/clif_dump.c | 4 ++-- src/broadcom/clif/v3dx_dump.c | 8 ++++---- src/gallium/drivers/vc4/vc4_cl_dump.c | 7 ++++++- 5 files changed, 18 insertions(+), 11 deletions(-) diff --git a/src/broadcom/cle/v3d_decoder.c b/src/broadcom/cle/v3d_decoder.c index a540b1cf559..de57e5f5acb 100644 --- a/src/broadcom/cle/v3d_decoder.c +++ b/src/broadcom/cle/v3d_decoder.c @@ -37,6 +37,7 @@ #include "v3d_decoder.h" #include "v3d_packet_helpers.h" #include "v3d_xml.h" +#include "broadcom/clif/clif_private.h" struct v3d_spec { uint32_t ver; @@ -924,17 +925,17 @@ v3d_field_iterator_next(struct v3d_field_iterator *iter) } void -v3d_print_group(FILE *outfile, struct v3d_group *group, +v3d_print_group(struct clif_dump *clif, struct v3d_group *group, uint64_t offset, const uint8_t *p, bool color) { struct v3d_field_iterator iter; v3d_field_iterator_init(&iter, group, p, color); while (v3d_field_iterator_next(&iter)) { - fprintf(outfile, " %s: %s\n", iter.name, iter.value); + fprintf(clif->out, " %s: %s\n", iter.name, iter.value); if (iter.struct_desc) { uint64_t struct_offset = offset + iter.offset; - v3d_print_group(outfile, iter.struct_desc, + v3d_print_group(clif, iter.struct_desc, struct_offset, &p[iter.offset], color); } diff --git a/src/broadcom/cle/v3d_decoder.h b/src/broadcom/cle/v3d_decoder.h index cca1d6fa64c..2987f3920a1 100644 --- a/src/broadcom/cle/v3d_decoder.h +++ b/src/broadcom/cle/v3d_decoder.h @@ -34,6 +34,7 @@ struct v3d_spec; struct v3d_group; struct v3d_field; +struct clif_dump; struct v3d_group *v3d_spec_find_struct(struct v3d_spec *spec, const char *name); struct v3d_spec *v3d_spec_load(const struct v3d_device_info *devinfo); @@ -139,7 +140,7 @@ void v3d_field_iterator_init(struct v3d_field_iterator *iter, bool v3d_field_iterator_next(struct v3d_field_iterator *iter); -void v3d_print_group(FILE *out, +void v3d_print_group(struct clif_dump *clif, struct v3d_group *group, uint64_t offset, const uint8_t *p, bool color); diff --git a/src/broadcom/clif/clif_dump.c b/src/broadcom/clif/clif_dump.c index ea9cee00e93..ca8d6d3e6fd 100644 --- a/src/broadcom/clif/clif_dump.c +++ b/src/broadcom/clif/clif_dump.c @@ -131,12 +131,12 @@ clif_dump_gl_shader_state_record(struct clif_dump *clif, assert(attr); out(clif, "GL Shader State Record at 0x%08x\n", reloc->addr); - v3d_print_group(clif->out, state, 0, vaddr, ""); + v3d_print_group(clif, state, 0, vaddr, ""); vaddr += v3d_group_get_length(state); for (int i = 0; i < reloc->shader_state.num_attrs; i++) { out(clif, " Attribute %d\n", i); - v3d_print_group(clif->out, attr, 0, vaddr, ""); + v3d_print_group(clif, attr, 0, vaddr, ""); vaddr += v3d_group_get_length(attr); } } diff --git a/src/broadcom/clif/v3dx_dump.c b/src/broadcom/clif/v3dx_dump.c index 72c0d444b7a..2229fe9d25c 100644 --- a/src/broadcom/clif/v3dx_dump.c +++ b/src/broadcom/clif/v3dx_dump.c @@ -47,7 +47,7 @@ v3dX(clif_dump_packet)(struct clif_dump *clif, uint32_t offset, *size = v3d_group_get_length(inst); out(clif, "%s\n", v3d_group_get_name(inst)); - v3d_print_group(clif->out, inst, 0, cl, ""); + v3d_print_group(clif, inst, 0, cl, ""); switch (*cl) { case V3DX(GL_SHADER_STATE_opcode): { @@ -87,7 +87,7 @@ v3dX(clif_dump_packet)(struct clif_dump *clif, uint32_t offset, cl += *size; for (int i = 0; i < values.number_of_16_bit_output_data_specs_following; i++) { - v3d_print_group(clif->out, spec, 0, cl, ""); + v3d_print_group(clif, spec, 0, cl, ""); cl += v3d_group_get_length(spec); *size += v3d_group_get_length(spec); } @@ -107,13 +107,13 @@ v3dX(clif_dump_packet)(struct clif_dump *clif, uint32_t offset, cl += *size; for (int i = 0; i < values.number_of_16_bit_output_data_specs_following; i++) { - v3d_print_group(clif->out, spec, 0, cl, ""); + v3d_print_group(clif, spec, 0, cl, ""); cl += v3d_group_get_length(spec); *size += v3d_group_get_length(spec); } for (int i = 0; i < values.number_of_32_bit_output_buffer_address_following; i++) { - v3d_print_group(clif->out, addr, 0, cl, ""); + v3d_print_group(clif, addr, 0, cl, ""); cl += v3d_group_get_length(addr); *size += v3d_group_get_length(addr); } diff --git a/src/gallium/drivers/vc4/vc4_cl_dump.c b/src/gallium/drivers/vc4/vc4_cl_dump.c index ca1b9a31528..c5be426d132 100644 --- a/src/gallium/drivers/vc4/vc4_cl_dump.c +++ b/src/gallium/drivers/vc4/vc4_cl_dump.c @@ -28,6 +28,7 @@ #include "kernel/vc4_packet.h" #include "broadcom/cle/v3d_decoder.h" +#include "broadcom/clif/clif_dump.h" void vc4_dump_cl(void *cl, uint32_t size, bool is_render) @@ -41,6 +42,8 @@ vc4_dump_cl(void *cl, uint32_t size, bool is_render) }; struct v3d_spec *spec = v3d_spec_load(&devinfo); + struct clif_dump *clif = clif_dump_init(&devinfo, stderr, NULL, NULL); + uint32_t offset = 0, hw_offset = 0; uint8_t *p = cl; @@ -60,7 +63,7 @@ vc4_dump_cl(void *cl, uint32_t size, bool is_render) fprintf(stderr, "0x%08x 0x%08x: 0x%02x %s\n", offset, hw_offset, header, v3d_group_get_name(inst)); - v3d_print_group(stderr, inst, offset, p, ""); + v3d_print_group(clif, inst, offset, p, ""); switch (header) { case VC4_PACKET_HALT: @@ -75,5 +78,7 @@ vc4_dump_cl(void *cl, uint32_t size, bool is_render) hw_offset += length; p += length; } + + clif_dump_destroy(clif); } -- 2.30.2